Skip to content

Add support for ECR public repository (public.ecr.aws)#379

Open
analytically wants to merge 1 commit into
concourse:masterfrom
analytically:publicecr
Open

Add support for ECR public repository (public.ecr.aws)#379
analytically wants to merge 1 commit into
concourse:masterfrom
analytically:publicecr

Conversation

@analytically

Copy link
Copy Markdown
Contributor

This commit adds support for Amazon ECR Public repositories to the registry-image-resource. Key changes include:

  • Skip authentication for public ECR repositories in check and in operations since they are publicly accessible
  • For out operations on public ECR, use AWS credentials but skip ECR token retrieval

The implementation detects public ECR repositories by looking for "public.ecr.aws" in the repository string and applies different authentication logic based on the operation type.

This enables Concourse pipelines to both pull from public ECR repositories without authentication and push to public ECR repositories with proper AWS credentials.

STILL TO BE TESTED

This commit adds support for Amazon ECR Public repositories to the
registry-image-resource. Key changes include:

- Skip authentication for public ECR repositories in check and in operations
  since they are publicly accessible
- For out operations on public ECR, use AWS credentials but skip ECR token
  retrieval

The implementation detects public ECR repositories by looking for
"public.ecr.aws" in the repository string and applies different
authentication logic based on the operation type.

This enables Concourse pipelines to both pull from public ECR
repositories without authentication and push to public ECR repositories
with proper AWS credentials.

Signed-off-by: Mathias Bogaert <mathias.bogaert@gmail.com>
@taylorsilva taylorsilva moved this from Todo to Waiting on Contributor in Pull Requests Nov 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting on Contributor

Development

Successfully merging this pull request may close these issues.

2 participants