-
Notifications
You must be signed in to change notification settings - Fork 23
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
fix image build on main branch #179
Conversation
I actually had removed that… Cause I probably misunderstood the intent... |
@@ -24,7 +24,7 @@ jobs: | |||
if: ${{ github.ref_name == env.MAIN_BRANCH_NAME }} | |||
id: add-latest-tag | |||
run: | | |||
echo "IMG_TAGS=latest" >> $GITHUB_ENV | |||
echo "IMG_TAGS=${{ env.IMG_TAGS }} latest" >> $GITHUB_ENV |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've seen that missing and thought it was intentional. Sorry.
The actual intention was to generate the tag with the git sha, which is being used in the smoke test. I agree that |
I think we want to, yes… I'm just pondering the utility of |
ok, let me change a bit to have only |
@@ -24,7 +24,7 @@ jobs: | |||
if: ${{ github.ref_name == env.MAIN_BRANCH_NAME }} | |||
id: add-latest-tag | |||
run: | | |||
echo "IMG_TAGS=latest" >> $GITHUB_ENV | |||
echo "IMG_TAGS=${{ env.IMG_TAGS }} latest" >> $GITHUB_ENV |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we have this just be: github.sha
for this step, where ref_name
is main
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ugh nevermind that comment, I somehow never submitted this 🤔
The
main
docker image tag has not been built for a while. Same for image builds with git sha tags from themain
branch.