-
Notifications
You must be signed in to change notification settings - Fork 54
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add org unit resource and datasource #63
Conversation
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.
Just curious on the datasource, would it make more sense to lookup by name?
} | ||
|
||
func dataSourceOrgUnitRead(ctx context.Context, d *schema.ResourceData, meta interface{}) diag.Diagnostics { | ||
if d.Get("org_unit_id") != "" { |
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.
Is it normal to have a data source that is essentially looked up by its ID?
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.
oh, you know what, let me take another look at this, i might have messed something up when i was testing it out. i think we can maybe look it up by org_unit_path
, but I don't think we can look it up by name (although, org_unit_path
is essentially <parent_org_unit_path>/<name>
which would definitely make more sense.
* add org unit resource and data source * remove datasource ability to get by orgunit path * adding docs * add ability go get org unit datasource with org_unit_path * add new docs with org_unit_path change
* Add org unit resource and datasource (#63) * add org unit resource and data source * remove datasource ability to get by orgunit path * adding docs * add ability go get org unit datasource with org_unit_path * add new docs with org_unit_path change * Implement RoleAssignment Resource and Role datasource/resource (#66) * Implement role_assignment resource and role datasource * Implement Role resource * update changelogs, merge in main Co-authored-by: appilon <apilon@hashicorp.com>
Fixes #54
Added org unit resource and datasource.