Skip to content

Conversation

hellohawaii
Copy link

Original codes will result in an error Pushes should be of the form docker push HOST-NAME/PROJECT-ID/REPOSITORY/IMAGE. The correct format for the path is <region>-docker.pkg.dev/<project-id>/<repository-name>/<image-name>:<tag>. The previous version forgot to add <repository-name>. Update to fix this bug.

Also change the comments to the correct location.

@Copilot Copilot AI review requested due to automatic review settings August 18, 2025 03:37
@hellohawaii hellohawaii requested review from a team as code owners August 18, 2025 03:37
Copy link

@Copilot 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 fixes a build error in the Google Cloud Run Docker deployment workflow by correcting the Docker image path format and reorganizing comments. The main issue was a missing repository name component in the Artifact Registry path.

  • Adds the REPOSITORY environment variable for the Artifact Registry repository name
  • Fixes the Docker tag format to include the repository name in the correct path structure
  • Reorganizes comments for better code structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

- name: 'Build and Push Container'
run: |-
DOCKER_TAG="$${{ env.REGION }}-docker.pkg.dev/${{ env.PROJECT_ID }}/${{ env.SERVICE }}:${{ github.sha }}"
DOCKER_TAG="$${{ env.REGION }}-docker.pkg.dev/${{ env.PROJECT_ID }}/${{ env.REPOSITORY }}/${{ env.SERVICE }}:${{ github.sha }}"
Copy link

Copilot AI Aug 18, 2025

Choose a reason for hiding this comment

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

There's an extra dollar sign in the DOCKER_TAG assignment. It should be DOCKER_TAG="${{ env.REGION }}-docker.pkg.dev/..." not DOCKER_TAG="$${{ env.REGION }}-docker.pkg.dev/..."

Suggested change
DOCKER_TAG="$${{ env.REGION }}-docker.pkg.dev/${{ env.PROJECT_ID }}/${{ env.REPOSITORY }}/${{ env.SERVICE }}:${{ github.sha }}"
DOCKER_TAG="${{ env.REGION }}-docker.pkg.dev/${{ env.PROJECT_ID }}/${{ env.REPOSITORY }}/${{ env.SERVICE }}:${{ github.sha }}"

Copilot uses AI. Check for mistakes.

Copy link
Author

Choose a reason for hiding this comment

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

This error has been fixed in #3024

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