Skip to content

fix: use eu01 region for AWS provider to fix compatibility with hashicorp/aws v6#202

Open
nroi wants to merge 1 commit into
mainfrom
feature/fix-stackit-s3
Open

fix: use eu01 region for AWS provider to fix compatibility with hashicorp/aws v6#202
nroi wants to merge 1 commit into
mainfrom
feature/fix-stackit-s3

Conversation

@nroi

@nroi nroi commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

AWS provider v6 introduced enhanced region support which changed CreateBucket
to always include LocationConstraint, even for us-east-1. STACKIT Object
Storage (StorageGRID) rejects us-east-1 as an invalid location constraint.

Set region to eu01 (the STACKIT Object Storage region) so the LocationConstraint
sent matches what StorageGRID is configured to accept. Add skip_region_validation
as eu01 is not a standard AWS region name.

https://github.com/stackitcloud/terraform-provider-stackit/blob/main/docs/guides/aws_provider_s3_stackit.md

BD-2505

@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Scorecard Check

Scorecard run on commit 4c0723aeeae38285932125ed634cd062080eb663 relative to origin/main

📊 meshstack-hub Module Scorecard

Generated: 2026-06-10 | Modules scanned: 1 | Categories: 5

📋 Per-Module Category Summary

Score per category per building block. n/a = category does not apply to this module.

Module Overall Core Structure Integration Azure Backplane STACKIT Backplane Testing
stackit/storage-bucket 🟢 100% 🟢 100% 🟢 100% n/a 🟢 100% 🟢 100%

All checks passing! This module meets all scorecard criteria.

Core Structure — ✅ all passing

Basic module file structure and documentation — applies to 1 modules

Module Score 📦 🔗 📋 📝 🖼️ 📌 🔒
stackit/storage-bucket 🟢 100%

Core Structure — Summary

Emoji Criterion Coverage Status
📦 buildingblock/ directory exists 1/1 🟢 100%
🔗 meshstack_integration.tf present 1/1 🟢 100%
📋 buildingblock/APP_TEAM_README.md present (no-integration fallback) n/a
📝 buildingblock/README.md with YAML front-matter 1/1 🟢 100%
🖼️ buildingblock/logo.png included 1/1 🟢 100%
📌 buildingblock/versions.tf present 1/1 🟢 100%
🔒 Provider versions use minimum constraint (>=) 1/1 🟢 100%
Integration — ✅ all passing

meshstack_integration.tf conventions — applies to 1 modules

Module Score 🏷️ 🏢 📤 🔌 📎 🔀 📋 🏷️ 📖 📝 📊 🚫
stackit/storage-bucket 🟢 100%

Integration — Summary

Emoji Criterion Coverage Status
🏷️ variable "hub" in integration 1/1 🟢 100%
🏢 variable "meshstack" in integration 1/1 🟢 100%
📤 building_block_definition output exposed 1/1 🟢 100%
🔌 meshcloud/meshstack in required_providers 1/1 🟢 100%
📎 backplane source uses var.hub.git_ref 1/1 🟢 100%
🔀 ref_name uses var.hub.git_ref 1/1 🟢 100%
📋 version_spec.draft uses var.hub.bbd_draft 1/1 🟢 100%
🏷️ BBD metadata.tags forwards var.meshstack.tags 1/1 🟢 100%
📖 BBD readme field present 1/1 🟢 100%
📝 BBD readme starts with plain-text description (no heading) 1/1 🟢 100%
📊 BBD readme has shared responsibility table (✅/❌) 1/1 🟢 100%
🚫 No documentation_md output in backplane 1/1 🟢 100%
Azure Backplane — not applicable

Azure UAMI-based automation principal conventions — applies to 0 modules

No applicable modules.

STACKIT Backplane — ✅ all passing

STACKIT WIF-based automation principal conventions — applies to 1 modules

Module Score 🔐 🚫 📤
stackit/storage-bucket 🟢 100%

STACKIT Backplane — Summary

Emoji Criterion Coverage Status
🔐 Uses stackit_service_account_federated_identity_provider 1/1 🟢 100%
🚫 No stackit_service_account_key resource 1/1 🟢 100%
📤 Outputs service_account_email (not key) 1/1 🟢 100%
Buildingblock provider uses use_oidc = true 1/1 🟢 100%
Testing — ✅ all passing

End-to-end test coverage — applies to 1 modules

Module Score ⚙️ 🧪
stackit/storage-bucket 🟢 100%

Testing — Summary

Emoji Criterion Coverage Status
⚙️ backplane/ directory (optional tier) 1/1 🟢 100%
🧪 e2e/ test directory exists 1/1 🟢 100%
e2e/ contains .tftest.hcl files 1/1 🟢 100%

@aws-amplify-eu-central-1

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-202.d1o16zfeoh2slu.amplifyapp.com

…corp/aws v6

AWS provider v6 introduced enhanced region support which changed CreateBucket
to always include LocationConstraint, even for us-east-1. STACKIT Object
Storage (StorageGRID) rejects us-east-1 as an invalid location constraint.

Set region to eu01 (the STACKIT Object Storage region) so the LocationConstraint
sent matches what StorageGRID is configured to accept. Add skip_region_validation
as eu01 is not a standard AWS region name.

BD-2505
@nroi nroi force-pushed the feature/fix-stackit-s3 branch from a7c5c34 to 0e808c1 Compare June 10, 2026 14:25
@nroi nroi marked this pull request as ready for review June 10, 2026 14:29
@nroi nroi requested a review from henryde June 10, 2026 14:29
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.

1 participant