Skip to content

latest tag not updated #73

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

Closed
codeboten opened this issue Feb 17, 2022 · 8 comments · Fixed by #220
Closed

latest tag not updated #73

codeboten opened this issue Feb 17, 2022 · 8 comments · Fixed by #220
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@codeboten
Copy link
Contributor

The latest tag in dockerhub isn't getting updated during the release process. This is causing user confusion.

@Aneurysm9
Copy link
Member

Can we remove it? I've never been a fan of using mutable tags in this manner.

@jpkrohling
Copy link
Member

I think we should not remove tags that were published, this could break people currently using it.

Related to #54

@codeboten
Copy link
Contributor Author

Can we remove it? I've never been a fan of using mutable tags in this manner.

I'm not a fan of it either, but people (myself included) do use them and we've already been publishing it. I would prefer we keep it updated moving forward.

@mhellsten-ml
Copy link

This is still an issue. Having the "latest" tag point the latest stable release is Docker community standard, so I don't think this is just a matter of preference or style. People who don't want to pull the latest when they relaunch would/should use a version-specific tag. Thanks.

codeboten pushed a commit to codeboten/opentelemetry-collector-releases that referenced this issue Jun 3, 2022
This is an alternative implementation to open-telemetry#76. I'll leave it up to reviewers on whether the bash script is easier to maintain than a github action, I don't have a strong preference here, but am not very familiar with typescript which is why i put together this alternative PR.

Fix open-telemetry#73
@delanym
Copy link

delanym commented Jul 13, 2022

Delete it, let the 2 people (or was it 3?) who still rely on this old tag for their mission critical telemetry systems (is there such a thing?) quickly fix the issue, and we can all get on with our lives.
Having an invalid tag hanging around confusing everyone is worse.

@mhellsten-ml
Copy link

Delete it, let the 2 people (or was it 3?) who still rely on this old tag for their mission critical telemetry systems (is there such a thing?) quickly fix the issue, and we can all get on with our lives. Having an invalid tag hanging around confusing everyone is worse.

This isn't for production so much as development: People who throw up a local stack of supporting services typically expect to get the latest stable release on the latest tag. When the behavior of the container that comes up doesn't match the documentation it's very confusing. 2 or 3 people commenting on this issue doesn't mean it's the entire universe of people this affects.

@dyladan
Copy link
Member

dyladan commented Jul 26, 2022

This is causing active confusion with users. Either fix the tag or remove it but leaving it broken is misleading and the worst of both worlds. latest tag is a docker community idiom that many (most?) users not only expect to work but are frustrated when it doesn't. I very often use the latest tag of docker images as a branch in integration tests so that we can catch issues without having to manually modify the version matrix.

@TylerHelmuth TylerHelmuth added the help wanted Extra attention is needed label Jul 26, 2022
@bogdandrutu
Copy link
Member

bogdandrutu commented Jul 27, 2022

Updated the latest tag to 0.56.0, using manifest and image group as the base tag.

Commands for contrib (same for core image):

docker pull otel/opentelemetry-collector-contrib:0.56.0-386

docker pull otel/opentelemetry-collector-contrib:0.56.0-amd64

docker pull otel/opentelemetry-collector-contrib:0.56.0-arm64

docker tag otel/opentelemetry-collector-contrib:0.56.0-386 otel/opentelemetry-collector-contrib:latest-386

docker tag otel/opentelemetry-collector-contrib:0.56.0-amd64 otel/opentelemetry-collector-contrib:latest-amd64

docker tag otel/opentelemetry-collector-contrib:0.56.0-arm64 otel/opentelemetry-collector-contrib:latest-arm64

docker push otel/opentelemetry-collector-contrib:latest-386

docker push otel/opentelemetry-collector-contrib:latest-amd64

docker push otel/opentelemetry-collector-contrib:latest-arm64

docker manifest create otel/opentelemetry-collector-contrib:latest --amend otel/opentelemetry-collector-contrib:latest-386 --amend otel/opentelemetry-collector-contrib:latest-amd64 --amend otel/opentelemetry-collector-contrib:latest-arm64

docker manifest push docker.io/otel/opentelemetry-collector-contrib:latest

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