Skip to content

Cloud-SDK extraction B/C/D — PR 3: serve s3 IaCStateBackend#14

Merged
intel352 merged 4 commits into
mainfrom
feat/s3-iac-state-backend
May 15, 2026
Merged

Cloud-SDK extraction B/C/D — PR 3: serve s3 IaCStateBackend#14
intel352 merged 4 commits into
mainfrom
feat/s3-iac-state-backend

Conversation

@intel352
Copy link
Copy Markdown
Contributor

Summary

  • Ports workflow's S3-compatible IaC state store into the aws plugin (internal/statebackend/s3.go) and serves it as the s3 backend via pb.IaCStateBackendServer + the Configure RPC.
  • DO-specific behavior stripped from the aws copy: the DO_SPACES_* credential env-var fallbacks and the <region>.digitaloceanspaces.com endpoint derivation — an aws s3 backend uses an explicit endpoint config (S3-compatible) or the SDK's standard AWS S3 endpoint resolution, never a DO URL.
  • Lazy construction via Configure (the azure precedent); workflow pinned to PR 1 merge 522748f3.

Plan PR 3 (cross-repo) of the cloud-SDK-extraction B/C/D plan. PR 4 (storage.s3 + step.s3_upload + credentials) follows on the same repo.

Test plan

  • go build ./..., go test ./... green incl. host conformance
  • Configure builds store; 6 state RPCs round-trip; ListBackendNames["s3"]; state-RPC-before-ConfigureFailedPrecondition
  • spec + quality review passed (zero Critical/Important)

🤖 Generated with Claude Code

@intel352 intel352 merged commit d9a2326 into main May 15, 2026
5 checks passed
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