Skip to content

Conversation

@l0wl3vel
Copy link
Contributor

@l0wl3vel l0wl3vel commented Dec 9, 2025

Description

Add folder role assignment resource

relates to #1089

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@l0wl3vel l0wl3vel marked this pull request as ready for review December 9, 2025 09:27
@l0wl3vel l0wl3vel requested a review from a team as a code owner December 9, 2025 09:27
Signed-off-by: Benjamin Ritter <benjamin.ritter@stackit.cloud>
@l0wl3vel l0wl3vel force-pushed the feat/folder-role-assignment branch from 003c89c to 4b91d99 Compare December 9, 2025 10:31
@@ -0,0 +1,6 @@

resource "stackit_authorization_folder_role_assignment" "serviceaccount" {
resource_id = var.organization_id
Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, done

@l0wl3vel l0wl3vel force-pushed the feat/folder-role-assignment branch from 4b91d99 to 9df87a6 Compare December 9, 2025 10:40

var testConfigVars = config.Variables{
"project_id": config.StringVariable(testutil.ProjectId),
"test_service_account": config.StringVariable(testutil.TestProjectServiceAccountEmail),
Copy link
Contributor

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.

Copy link
Contributor Author

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

@rubenhoenle
Copy link
Member

Duplicate of #1058

@rubenhoenle rubenhoenle marked this as a duplicate of #1058 Dec 9, 2025
@rubenhoenle
Copy link
Member

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>
@l0wl3vel l0wl3vel force-pushed the feat/folder-role-assignment branch from 9df87a6 to 194cb90 Compare December 9, 2025 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants