Skip to content

Conversation

@puehringer
Copy link
Contributor

@puehringer puehringer commented Dec 23, 2025

@puehringer puehringer changed the title Feat add registry type feat: add registry type Dec 23, 2025
@puehringer puehringer requested a review from Copilot December 23, 2025 11:42
@puehringer puehringer marked this pull request as ready for review December 23, 2025 11:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for configuring the ECR registry type (public or private) across GitHub Actions workflows and reusable actions. The changes enable the actions to work with both AWS ECR public and private registries by passing the registry type to the aws-actions/amazon-ecr-login action.

  • Adds ecr_registry_type input parameter to three custom actions with a default value of "private" for backward compatibility
  • Passes the ecr_registry_type parameter from the workflow to the pull-push-image action via the destination configuration
  • Configures the aws-actions/amazon-ecr-login action to use the specified registry type

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/build-docker-artifacts-push.yml Adds destination_ecr_registry_type parameter to pass registry type configuration from AWS config to the pull-push-image action
.github/actions/retag-image/action.yml Adds ecr_registry_type input with default "private" and passes it to amazon-ecr-login action
.github/actions/get-ecr-scan-result/action.yml Adds ecr_registry_type input with default "private" and passes it to amazon-ecr-login action
.github/actions/build-push-image/action.yml Adds ecr_registry_type input with default "private" and passes it to amazon-ecr-login action

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 57 to 58
with:
registry-type: ${{ inputs.ecr_registry_type }}
Copy link

Copilot AI Dec 23, 2025

Choose a reason for hiding this comment

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

The AWS CLI commands used later in this action (lines 67, 74) are only compatible with private ECR registries. When ecr_registry_type is set to "public", these commands will fail as they need to use "aws ecr-public" instead of "aws ecr". Consider adding conditional logic in the scan results step to handle both registry types appropriately.

Copilot uses AI. Check for mistakes.
@puehringer puehringer merged commit 5a27f0f into main Dec 23, 2025
1 check passed
@puehringer puehringer deleted the feat-add-registry-type branch December 23, 2025 11:46
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.

3 participants