Skip to content

feat(objectstorage): add default retention for buckets#1552

Merged
SerseusWasTaken merged 3 commits into
mainfrom
feat/object-storage-default-retention
Jul 9, 2026
Merged

feat(objectstorage): add default retention for buckets#1552
SerseusWasTaken merged 3 commits into
mainfrom
feat/object-storage-default-retention

Conversation

@SerseusWasTaken

Copy link
Copy Markdown
Contributor

Relates to STACKITTPR-727

Description

Adds a default retention ressource for buckets

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)

@SerseusWasTaken SerseusWasTaken requested a review from a team as a code owner July 1, 2026 07:09
@SerseusWasTaken SerseusWasTaken force-pushed the feat/object-storage-default-retention branch 2 times, most recently from cd91879 to 8f849fb Compare July 8, 2026 06:12
Comment thread docs/resources/objectstorage_default_retention.md
Comment thread stackit/internal/services/objectstorage/default-retention/datasource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/datasource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/resource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/resource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/resource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/resource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/datasource.go Outdated
Comment thread stackit/internal/services/objectstorage/default-retention/resource_test.go Outdated
@SerseusWasTaken SerseusWasTaken force-pushed the feat/object-storage-default-retention branch from 8f849fb to d1d90e6 Compare July 8, 2026 12:59
@SerseusWasTaken SerseusWasTaken force-pushed the feat/object-storage-default-retention branch from d1d90e6 to 1d9b620 Compare July 9, 2026 09:09
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/stackitcloud/terraform-provider-stackit/stackit 1.18% (ø)
github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/objectstorage 0.00% (ø)
github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/objectstorage/default-retention 4.37% (+4.37%) 👍

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/objectstorage/default-retention/datasource.go 0.00% (ø) 42 (+42) 0 42 (+42)
github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/objectstorage/default-retention/resource.go 5.49% (+5.49%) 164 (+164) 9 (+9) 155 (+155) 👍
github.com/stackitcloud/terraform-provider-stackit/stackit/provider.go 1.18% (ø) 169 2 167

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/objectstorage/default-retention/resource_test.go
  • github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/objectstorage/objectstorage_acc_test.go

@SerseusWasTaken SerseusWasTaken merged commit 131e627 into main Jul 9, 2026
3 checks passed
@SerseusWasTaken SerseusWasTaken deleted the feat/object-storage-default-retention branch July 9, 2026 09:40
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.

2 participants