- Notifications
You must be signed in to change notification settings - Fork3
feat: addgroup_sync
androle_sync
forcoderd_organization_resource
#147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
return | ||
} | ||
// data.GroupSync = ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I don't recall if you can write to a config block (blocks can't be computed) like this. I suspect you can't.
If you can, I assume it would just be to error if the post-apply plan isn't empty.
For that you'd need to usetypes.ObjectValueFrom
and provide it the type of the object using amap[string]attr.Type
. I've got an example on the template resource & data source.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I think I figured something out that works 😄
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
few comments but otherwise nice, lgtm!
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
ethanndicksonDec 3, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I just realised we're missing enterprise/premium entitlement checks on this and the provisioner key resource. We have them on the workspace proxy and the enterprise features of template resources, but they run duringapply
, when they should really run duringplan
. Let's not worry about it for now, and at some point I'll see if I can get them working at plan-time using a plan modifier (validators don't have the provider config we need).
If that doesn't work we can just copy the ones at apply-time to these two new resources.
stringvalidator.LengthAtLeast(1), | ||
}, | ||
}, | ||
"auto_create_missing": schema.BoolAttribute{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This defaults tofalse
on coder right? Should just have adefault
set in the schema, and then you can remove the null checks on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
would that preventgroup_sync
from beingnull
tho?
ethanndicksonDec 4, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Nope, the default only gets computed if the parent is known, so:
group_sync {}
would default any values within it, but omitting the block altogether would not.
If you wanted to default the block to known, and the bool to false, then you'd also add aDefault
to the block itself.
283e3cc
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Group and role sync settings can now be configured as such: