-
Notifications
You must be signed in to change notification settings - Fork 42
feat: add folder role assignment resource #1090
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Benjamin Ritter <benjamin.ritter@stackit.cloud>
003c89c to
4b91d99
Compare
| @@ -0,0 +1,6 @@ | |||
|
|
|||
| resource "stackit_authorization_folder_role_assignment" "serviceaccount" { | |||
| resource_id = var.organization_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.
This should be a folder id, right?
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.
yeah, done
4b91d99 to
9df87a6
Compare
|
|
||
| var testConfigVars = config.Variables{ | ||
| "project_id": config.StringVariable(testutil.ProjectId), | ||
| "test_service_account": config.StringVariable(testutil.TestProjectServiceAccountEmail), |
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.
Here is a folder_id missing as well.
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.
added the test folder to prerequisites.tf
|
Duplicate of #1058 |
|
There's also a state drift problem within this resource reported which might should be fixed first: #1067 |
Signed-off-by: Benjamin Ritter <benjamin.ritter@stackit.cloud>
9df87a6 to
194cb90
Compare
Description
Add folder role assignment resource
relates to #1089
Checklist
make fmtexamples/directory)make generate-docs(will be checked by CI)make test(will be checked by CI)make lint(will be checked by CI)