Skip to content

[patch] add support for creating required repositories in aws ecr #1791

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

terenceq
Copy link
Contributor

@terenceq terenceq commented Jun 3, 2025

AWS Elastic Container Registry mirroring support

Provide support for mirroring images to AWS Elastic Container Registry (ECR) instances.

Description

In order to select ECR instances as the target registry for mirroring the capability to create the required ECR repositories must be provided. That is each image that will be mirrored needs to have the applicable ECR repository created in advance. For example the AWS cli command must be executed before an image can be uploaded:

aws ecr create-repository --repository-name {image_repo_name} --region {registry_ecr_aws_region}"

In order to use ECR as a target registry the following env variables should be set:

export REGISTRY_PRIVATE_HOST={AWS ECR host}
export REGISTRY_PRIVATE_PORT=443
export REGISTRY_USERNAME=AWS
export REGISTRY_PASSWORD= {AWS ECR login token}
MIRROR_TARGET_IS_ECR=true
AWS_ECR_REGION={AWS region where ECR instance is located}

Before the applicable mirroring command is invoked the following should be satisfied:

  • aws cli available in the current path
  • aws credentials configured so that the aws ecr create-repository may succeed

Test Results

Successfully mirrored several catalogs/channels to various AWS ECR instances.


⚠️ Notes for Reviewers

  • Ensure you have understood the PR guidelines in the Playbook before proceeding with a review.
  • Ensure all sections in the PR template are appropriately completed.

terenceq added 23 commits May 6, 2025 12:48
@terenceq terenceq marked this pull request as ready for review July 9, 2025 16:19
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