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

Inconsistent Docker version tags #2677

Closed
knittl opened this issue Sep 8, 2022 · 4 comments · Fixed by #2678
Closed

Inconsistent Docker version tags #2677

knittl opened this issue Sep 8, 2022 · 4 comments · Fixed by #2678

Comments

@knittl
Copy link
Contributor

knittl commented Sep 8, 2022

Brief summary

Kudos on the 0.40.0 release! 🎉

I'm not sure if this even qualifies a bug, but the Docker version tag format hat changed from grafana/k6:0.39.0 to grafana/k6:v0.40.0 (note the v).

It took me a few minutes to find out why docker pull would not fetch the latest version. Was it intentional? Could be related to #2676

Nothing serious, just wanted to mention it.

k6 version

v0.40.0

OS

n/a

Docker version and image (if applicable)

v0.40.0

Steps to reproduce the problem

  1. docker pull grafana/k6:0.39.0 works
  2. docker pull grafana/k6:0.40.0 does not work
  3. docker pull grafana/k6:v0.40.0 works

Expected behaviour

I can pull 0.40.0 the same way I pulled the previous versions :)

Actual behaviour

Docker tag of 0.40.0 is actually v0.40.0

@na--
Copy link
Member

na-- commented Sep 9, 2022

Ah... I hate CI changes 😭 Thanks for letting us know, we will fix this manually for 0.40.0 / v0.40.0 and fix the CI so it doesn't happen in the future.

@na--
Copy link
Member

na-- commented Sep 9, 2022

The manually re-tagged and pushed loadimpact/k6:v0.40.0 as loadimpact/k6:0.40.0 and the grafana/k6:0.40.0 tag was fixed with this one-off commit: c90346a

I'll make a proper fix for the bug I introduced in #2591. And it only applied for the docker container registry images, the ghcr.io image was correctly pushed as 0.40.0... 😭

I think the problem was that I didn't export this variable (or repeat the same string manipulation in the subsequent step for Docker Hub):

VERSION="${VERSION#v}"

@na-- na-- added this to the v0.41.0 milestone Sep 9, 2022
na-- added a commit that referenced this issue Sep 9, 2022
This fixes #2677. Because of a bug introduced in #2591, the CI would push `vX.Y.Z` docker images when before they were with the format `X.Y.Z`, without the "v" prefix. The v0.40.0 image was manually fixed by c90346a
@na--
Copy link
Member

na-- commented Sep 9, 2022

#2678 is the PR that hopefully fixes this... 🤞

@na-- na-- closed this as completed in #2678 Sep 9, 2022
na-- added a commit that referenced this issue Sep 9, 2022
This fixes #2677. Because of a bug introduced in #2591, the CI would push `vX.Y.Z` docker images when before they were with the format `X.Y.Z`, without the "v" prefix. The v0.40.0 image was manually fixed by c90346a
@knittl
Copy link
Contributor Author

knittl commented Sep 9, 2022

Awesome, thanks for the quick resolution! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants