Skip to content
This repository was archived by the owner on Jun 23, 2026. It is now read-only.

chore(ci): publish to Velocity ECR via OIDC, drop kubectl rollout - #45

Merged
ChewingGlass merged 4 commits into
masterfrom
chore/velocity-ecr-publish-role
Jun 12, 2026
Merged

chore(ci): publish to Velocity ECR via OIDC, drop kubectl rollout#45
ChewingGlass merged 4 commits into
masterfrom
chore/velocity-ecr-publish-role

Conversation

@jt-lumen

Copy link
Copy Markdown
Contributor

What

Switches this repo's image-publish CI to the Velocity account's ECR publish role (assumed via GitHub OIDC) and removes the in-CI kubectl rollout restart deploy step.

Changes (master.yml + mainnet-beta.yml)

  • ECR auth → Velocity publish role via OIDC:
    • master.ymlrole-to-assume: ${{ vars.VELOCITY_NON_PROD_ECR_PUBLISH_ROLE }}
    • mainnet-beta.ymlrole-to-assume: ${{ vars.VELOCITY_PROD_ECR_PUBLISH_ROLE }}
  • Removed the deploy job (kubectl rollout restart). Rollouts are now handled by ArgoCD Image Updater, which watches the new image tags.

Why

Part of the AWS account migration to the Velocity org. The new accounts expose a dedicated VelocityEcrPublishRole (scoped to image push only) that CI assumes directly via OIDC. Deployment is GitOps now, so CI no longer needs cluster (kubectl) access.

Required org/repo Actions variables

  • VELOCITY_NON_PROD_ECR_PUBLISH_ROLE = arn:aws:iam::725652782496:role/VelocityEcrPublishRole
  • VELOCITY_PROD_ECR_PUBLISH_ROLE = arn:aws:iam::039438368040:role/VelocityEcrPublishRole

🤖 Generated with Claude Code

jt-lumen and others added 4 commits June 12, 2026 12:06
- master.yml / mainnet-beta.yml restored to the old drift deploy flow
  (old infra keeps deploying until decommission)
- new velocity-publish.yml: pushing a vX.Y.Z tag publishes that exact,
  immutable version to Velocity ECR (prod copy gated until the prod
  account exists). Deploy = bump the gitops pin in infrastructure-v3.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ChewingGlass
ChewingGlass merged commit 969e5d5 into master Jun 12, 2026
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants