Skip to content

Remove ECR repository support#103

Merged
ian-flores merged 1 commit intomainfrom
remove-ecr-support
Feb 3, 2026
Merged

Remove ECR repository support#103
ian-flores merged 1 commit intomainfrom
remove-ecr-support

Conversation

@ian-flores
Copy link
Contributor

Summary

  • Remove ECR repository creation from workload and control room persistent steps
  • Remove ECR VPC endpoints (ecr.api, ecr.dkr) from standard VPC endpoint services
  • Delete lib/aws/ecr.go and stub Registry methods with ErrECRDeprecated
  • Remove manage_ecr_repositories config option
  • Remove ecr-login, skopeo-login, latest-images Justfile recipes

Context

ECR is no longer used - images are now pulled from public registries. This removes all ECR-related code and VPC endpoints.

Test plan

  • All Python tests pass (108 tests)
  • All Go tests pass
  • Dry-run on ganso01-staging shows expected ECR VPC endpoint deletions
  • Applied to ganso01-staging successfully

ECR is no longer used - images are now pulled from public registries.

- Remove ECR repository creation from workload and control room persistent steps
- Remove ECR VPC endpoints (ecr.api, ecr.dkr) from standard VPC endpoint services
- Delete lib/aws/ecr.go and stub Registry methods with ErrECRDeprecated
- Remove manage_ecr_repositories config option
- Remove ecr-login, skopeo-login, latest-images Justfile recipes
@ian-flores ian-flores linked an issue Feb 3, 2026 that may be closed by this pull request
7 tasks
@ian-flores ian-flores marked this pull request as ready for review February 3, 2026 01:05
@ian-flores ian-flores requested a review from a team as a code owner February 3, 2026 01:05
@ian-flores ian-flores requested a review from stevenolen February 3, 2026 01:05
Copy link
Contributor

@timtalbot timtalbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Can we just outright delete registry.go and registry_test.go too?

@ian-flores
Copy link
Contributor Author

Nice! Can we just outright delete registry.go and registry_test.go too?

I'm going to leave them to document the deprecation in case existing users of the CLI try to leverage those features. I can remove it one or two releases from now.

@ian-flores ian-flores merged commit 079fde2 into main Feb 3, 2026
7 of 14 checks passed
@ian-flores ian-flores deleted the remove-ecr-support branch February 3, 2026 14:45
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.

Remove ECR/ACR resource definitions after Docker Hub migration

2 participants