Skip to content
This repository has been archived by the owner on Oct 28, 2024. It is now read-only.

Support System Profiles #84

Open
j-don opened this issue Jun 11, 2020 · 1 comment
Open

Support System Profiles #84

j-don opened this issue Jun 11, 2020 · 1 comment

Comments

@j-don
Copy link

j-don commented Jun 11, 2020

Right now, the cdap_profile resource requires a namespace, and omitting it uses the "default" namespace, so it is currently impossible to configure System Compute Profiles.

@umairidris
Copy link
Member

umairidris commented Jun 11, 2020

It's been a while since I looked into this but one of the caveats is data fusion by default sets some system profile properties, so if we send an API with new ones it will overwrite the existing ones. Trying to fetch and append existing ones is unidiomatic for Terraform. You also can't create/delete system namespace so it doesn't fit super well to the Terraform model. Thus, we chose to just support profiles at the namespace level.

I would encourage using namespace level profiles as a best practice.

If this is a highly desired feature I would consider accepting a PR that added a 'type' field that defaults to 'NAMESPACE', but can also be set to 'SYSTEM'. In the case it is NAMESPACE then we check the namespace field, else it is ignored.

It would need to fit the Terraform model however, so would need answers to the existing properties and creation/deletion questions above.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants