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
@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

@l0wl3vel l0wl3vel force-pushed the feat/folder-role-assignment branch from 9df87a6 to 194cb90 Compare December 9, 2025 11:39
@github-actions
Copy link

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

@github-actions github-actions bot added the Stale PR is marked as stale due to inactivity. label Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stale PR is marked as stale due to inactivity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants