Skip to content
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

Read version from file and tag docker image in GitHub workflow #66

Merged
merged 2 commits into from
Oct 31, 2024

Conversation

ll-nick
Copy link
Collaborator

@ll-nick ll-nick commented Oct 31, 2024

I just wanted to try the demo since it finally made it into master 🚀

I realized, that we never created the version tag for the docker images. I'm not sure if that was planned in #61 but I figured we should fix this one quickly because the current master won't work (without adjustments at least).

So this PR just adds to the workflow by reading the version from the version file and tagging the docker image we are building already. [I tested the action on this branch already])https://github.com/KIT-MRT/arbitration_graphs/actions/runs/11611034651) so we should be good to go.

@ll-nick ll-nick requested a review from orzechow October 31, 2024 11:33
@ll-nick ll-nick self-assigned this Oct 31, 2024
Copy link
Member

@orzechow orzechow left a comment

Choose a reason for hiding this comment

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

Awesome, I didn't realize the docker image CD is already setup 🤩

@ll-nick ll-nick merged commit 03e8f88 into main Oct 31, 2024
@ll-nick ll-nick deleted the add_version_tag_to_docker_image branch October 31, 2024 12:30
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