Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

workflow: fix the tagging docker images build from new tags #145

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

fmuyassarov
Copy link
Collaborator

When docker image build workflow is triggered based on the new tag push which usually includes v as prefix, the workflow drops the v prefix. As such, when v0.2.0 tag is pushed to GitHub, we end up with 0.2.0 tags in the docker images. This commit fixes the workflow to keep the v prefix. Currently, our deployments are tagging v0.2.0 tags for the plugins while published images don't include the v and deployment are failing.

When docker image workflow is triggered based on the new tag push
which usually includes v as prefix, the workflow drops the v prefix.
As such, when v0.2.0 tag is pushed to GitHub, we end up with 0.2.0
tags in the docker images. This commit fixes the workflow to keep
the v prefix.

Signed-off-by: Feruzjon Muyassarov <feruzjon.muyassarov@intel.com>
@fmuyassarov fmuyassarov requested review from klihub and marquiz October 11, 2023 21:21
Copy link
Collaborator

@marquiz marquiz left a comment

Choose a reason for hiding this comment

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

I think this makes sense. Just use the same vX.Y.Z version number everywhere (git tag, documentation, helm chart, image tags...). I don't know why v was dropped off(?)

ping @klihub @kad @askervin

Copy link
Collaborator

@marquiz marquiz left a comment

Choose a reason for hiding this comment

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

Actually, even make images creates images with the v0.1.2 tag

@marquiz marquiz merged commit ea5ce41 into containers:main Oct 12, 2023
@fmuyassarov fmuyassarov deleted the fix-versioning-images branch October 12, 2023 08:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants