feat(resourcemanager): add folder resource/datasource #975

h3adex · 2025-09-01T14:01:20Z

Description

Implements full support for resource and data source for folders (STACKITTPR-213). Both are marked as beta.

Acceptance tests/Docs updated for:

stackit_resourcemanager_project  
stackit_resourcemanager_folder

stackit_resourcemanager_project improvements:

Added attributes: created_at, updated_at
Removed incorrect service_account_email documentation

Example configuration:

locals {
  org_id = "xxxx"
}

resource "stackit_resourcemanager_folder" "folder01" {
  name                = "test"
  owner_email         = "<your-mail>@stackit.cloud"
  parent_container_id = local.org_id
  labels = {
    "env" = "dev"
  }
}

data "stackit_resourcemanager_folder" "folder01" {
  container_id = stackit_resourcemanager_folder.folder01.container_id
}

# Projects can be added under folders. When deleting a project,
# it may remain invisible for up to 7 days.
# Folder deletion may fail during this period.
/*resource "stackit_resourcemanager_project" "project01" {
  name                = "proj-01-updated"
  owner_email         = "<your-mail>@stackit.cloud"
  parent_container_id = stackit_resourcemanager_folder.folder01.container_id
}*/

E2E test results in the comments.

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)

h3adex · 2025-09-02T09:54:48Z

E2E Run

STACKIT_SERVICE_ACCOUNT_TOKEN=eyJ
TF_ACC=1
TF_ACC_PROJECT_ID=xxxx-xxxx-xxxx-xxxx
TF_ACC_REGION=eu01
TF_ACC_TEST_PROJECT_PARENT_UUID=xxxx-xxxx-xxxx-xxxx
TF_ACC_TEST_PROJECT_PARENT_CONTAINER_ID=test-company
[email protected]

stackit/internal/services/resourcemanager/project/resource.go

stackit/internal/services/resourcemanager/project/datasource.go

Signed-off-by: Mauritz Uphoff <[email protected]>

…sourcemanager project/folder Signed-off-by: Mauritz Uphoff <[email protected]>

Signed-off-by: Mauritz Uphoff <[email protected]>

marceljk · 2025-09-12T12:47:36Z

examples/resources/stackit_resourcemanager_folder/resource.tf

+  name                = "example-project"
+  owner_email         = "[email protected]"
+  parent_container_id = stackit_resourcemanager_folder.example.container_id
+}


Can you add an example, how to import a folder like we have it for other resources:

terraform-provider-stackit/examples/resources/stackit_resourcemanager_project/resource.tf

Lines 11 to 17 in fe258cf

# Only use the import statement, if you want to import an existing resourcemanager project

# Note: There will be a conflict which needs to be resolved manually.

# Must set a configuration value for the owner_email attribute as the provider has marked it as required.

import {

to = stackit_resourcemanager_project.import-example

id = var.container_id

}

marceljk · 2025-09-12T13:08:54Z

stackit/internal/services/resourcemanager/folder/datasource.go

+var (
+	_ datasource.DataSource = &folderDataSource{}


nitpick. to ensure the Configure method is always implemented and not changed accidentally

Suggested change

var (

_ datasource.DataSource = &folderDataSource{}

var (

_ datasource.DataSource = &folderDataSource{}

_ datasource.DataSourceWithConfigure = &folderDataSource{}

Can be adjusted for the project datasource as well
https://github.com/h3adex/terraform-provider-stackit/blob/fe258cf586e734d019fd83c8337dd2481361fe02/stackit/internal/services/resourcemanager/project/datasource.go#L27-L29

marceljk · 2025-09-12T13:25:39Z

stackit/internal/services/resourcemanager/folder/resource.go

+			ctx,
+			&resp.Diagnostics,
+			"Error deleting folder. Deletion may fail because associated projects remain hidden for up to 7 days after user deletion due to technical requirements.",
+			fmt.Sprintf("API call failed: %v", err),


Suggested change

fmt.Sprintf("API call failed: %v", err),

fmt.Sprintf("Calling API: %v", err),

marceljk · 2025-09-12T13:38:53Z

stackit/internal/services/resourcemanager/folder/resource.go

+// mapFolderCreateFields maps the Create Folder API response to the Terraform model and update the Terraform state
+func mapFolderCreateFields(ctx context.Context, resp *resourcemanager.FolderResponse, model *Model, state *tfsdk.State) error {
+	return mapFolderFields(ctx, resp.ContainerId, resp.Name, resp.FolderId, resp.Labels, resp.Parent, resp.CreationTime, resp.UpdateTime, model, state)
+}
+
+// mapFolderDetailsFields maps the GetDetails API response to the Terraform model and update the Terraform state
+func mapFolderDetailsFields(ctx context.Context, resp *resourcemanager.GetFolderDetailsResponse, model *Model, state *tfsdk.State) error {
+	return mapFolderFields(ctx, resp.ContainerId, resp.Name, resp.FolderId, resp.Labels, resp.Parent, resp.CreationTime, resp.UpdateTime, model, state)
+}


Would be probably better readable when using just one map function. For this you just need to add an additional GET call in the Create method and use the response of it for the map function instead of passing each attribute separate

marceljk · 2025-09-12T13:48:42Z