Skip to content

cd: push to shared-services ECR, drop environment#40

Merged
alexhui01 merged 1 commit into
mainfrom
chore/cd-push-shared-services-drop-environment
Jun 5, 2026
Merged

cd: push to shared-services ECR, drop environment#40
alexhui01 merged 1 commit into
mainfrom
chore/cd-push-shared-services-drop-environment

Conversation

@alexhui01

Copy link
Copy Markdown
Collaborator

Simplifies the build/push workflow to a single CI flow and points it at the shared-services ECR.

Changes (.github/workflows/cd.yaml)

  • Drop environment usage (per "no need environment, we just have ci"): removed the workflow_dispatch environment input, the detect-changes ENVIRONMENT env + environment output, and the build-and-push environment: key.
  • Shared-services repo path: build ECR_REPOSITORY as \${ECR_REPOSITORY_PREFIX}-\${SERVICE} (hyphen) so it targets strata/batch-exp-backend / strata/batch-exp-frontend — the previous \$PREFIX/\${SERVICE} (slash) produced …/batch-exp/frontend, which doesn't exist in shared-services.

Required repo settings (coordinated with this change)

  • Variable ECR_REPOSITORY_PREFIX = strata/batch-exp
  • Secret AWS_ECR_ROLE = arn:aws:iam::496607027995:role/github-actions-checkpoint-explorer-push (the dedicated least-privilege role, scoped to those two repos)

With both set, CI builds push directly to shared-services and the manual master→shared mirror is no longer needed.

Note

Actions are still tag-pinned (@v4/@v2/@v3); SHA-pinning can be a follow-up per CI hardening standards.

Simplify the build/push workflow to a single CI flow:
- Remove the workflow_dispatch environment input, the detect-changes
  ENVIRONMENT env + environment output, and the build-and-push environment key.
- Build the ECR repo path as ${ECR_REPOSITORY_PREFIX}-${SERVICE} (hyphen) to
  match the shared-services repos strata/batch-exp-{backend,frontend}.

Requires repo settings: ECR_REPOSITORY_PREFIX=strata/batch-exp and AWS_ECR_ROLE
set to the shared-services push role (github-actions-checkpoint-explorer-push).
@alexhui01 alexhui01 added this pull request to the merge queue Jun 5, 2026
Merged via the queue into main with commit 68f6473 Jun 5, 2026
3 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.

2 participants