Skip to content

set value to return as docker-image-name-with-tag #107

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

Merged
merged 2 commits into from
Oct 29, 2024
Merged

Conversation

elrayle
Copy link
Collaborator

@elrayle elrayle commented Oct 29, 2024

Description

The build failed to deploy for service-api. The ultimate cause was that the output docker-image-name-with-tag was not getting set. The value of this output needs to be the docker image that will be used for the deployment.

For service-api this is the ghcr image for both dev and prod.
For the crawler, it will use ghcr image for dev and Docker Hub image for prod. (at least for now)

This updates the version number to 3.1.2.

Follow on work

  • Once this is merged, a new release of the operations repo needs to be cut at 3.1.2
  • Service needs workflows to be updated to use operations/v3.1.2
  • Crawler needs workflows to be updated to use operations/v3.1.2
  • Website will need to be updated too. Needs more exploration.

CC: @ljones140 for review

@elrayle elrayle force-pushed the elr/workflow branch 2 times, most recently from 4829e4a to 59113f3 Compare October 29, 2024 03:40
Copy link
Contributor

@ljones140 ljones140 left a comment

Choose a reason for hiding this comment

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

Looks great

docker_image_tags="${{ inputs.docker-hub-username }}/${{ env.IMAGE_NAME }}"
echo "DOCKER_IMAGE_TAGS=${{ env.DOCKER_IMAGE_TAGS }}, $docker_image_tags" >> $GITHUB_ENV
# use Docker Hub image if it is created by overwriting DOCKER_IMAGE_TO_USE
echo "DOCKER_IMAGE_TO_USE=$docker_image_tags" >> $GITHUB_ENV
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice so the explicit DOCKER_IMAGE_TO_USE fixes the bug I introduced?

@elrayle elrayle merged commit d758867 into main Oct 29, 2024
2 checks passed
@elrayle elrayle deleted the elr/workflow branch October 29, 2024 11:22
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