Skip to content

fix(opencode): remove AWS from GitOps updater - #13

Merged
xnoto merged 1 commit into
mainfrom
fix/remove-chart-updater-aws
Aug 28, 2026
Merged

fix(opencode): remove AWS from GitOps updater#13
xnoto merged 1 commit into
mainfrom
fix/remove-chart-updater-aws

Conversation

@xnoto

@xnoto xnoto commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • consume the SOPS-backed CHART_UPDATER_GITHUB_APP_PRIVATE_KEY repository secret
  • remove AWS OIDC credential configuration and Secrets Manager reads
  • remove the now-unneeded id-token: write permission

Validation

  • make test
  • actionlint .github/workflows/helm.yml
  • workflow YAML parse and diff hygiene checks

Rollout dependency

Merge only after makeitworkcloud/tfroot-github#42 has applied the repository secret.

@github-actions

Copy link
Copy Markdown

Chart CI passed

Repository hygiene and Helm validation passed. View the workflow run.

@xnoto
xnoto merged commit 7edddf2 into main Aug 28, 2026
4 checks passed
@xnoto
xnoto deleted the fix/remove-chart-updater-aws branch August 28, 2026 16:35
xnoto added a commit to makeitworkcloud/tfroot-aws that referenced this pull request Aug 28, 2026
## Summary
- remove the chart updater GitHub Actions OIDC role
- remove its inline Secrets Manager read policy
- update generated resource documentation

## Validation
- repository pre-commit hooks passed after removing a stale local
`.terraform` cache
- Terraform validate, tflint, Checkov, fmt, docs, and secret-detection
hooks passed

## Rollout dependency
Merge only after makeitworkcloud/tfroot-github#42 has applied and
makeitworkcloud/charts#13 is merged. This ensures the updater no longer
references the role before Terraform destroys it.

The manually created Secrets Manager secret is intentionally not deleted
by this PR.
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