Skip to content

Makefile.common: Push major version tags to registry#17892

Merged
roidelapluie merged 1 commit intoprometheus:mainfrom
roidelapluie:roidelapluie/pushver
Jan 22, 2026
Merged

Makefile.common: Push major version tags to registry#17892
roidelapluie merged 1 commit intoprometheus:mainfrom
roidelapluie:roidelapluie/pushver

Conversation

@roidelapluie
Copy link
Copy Markdown
Member

In commit 74775d7 "Add major version tag (#8026)" from 2020, the docker-tag-latest target was updated to create major version tags (v2, v3, etc.) but these tags were never actually pushed to the registry. They existed locally only after tagging but were never published.

This commit fixes the issue by:

  • Adding logic to docker-publish to push major version tags when DOCKER_IMAGE_TAG="latest" (triggered by promci during releases)
  • Adding logic to docker-manifest to create major version manifests when DOCKER_IMAGE_TAG="latest"

Pre-release filtering is handled at the promci level, where the regex check ^v[0-9]+(.[0-9]+){2}$ already ensures only stable releases trigger the "latest" tagging workflow.

Which issue(s) does the PR fix:

Does this PR introduce a user-facing change?

NONE

In commit 74775d7 "Add major version tag (prometheus#8026)" from 2020, the
docker-tag-latest target was updated to create major version tags
(v2, v3, etc.) but these tags were never actually pushed to the
registry. They existed locally only after tagging but were never
published.

This commit fixes the issue by:
- Adding logic to docker-publish to push major version tags when
  DOCKER_IMAGE_TAG="latest" (triggered by promci during releases)
- Adding logic to docker-manifest to create major version manifests
  when DOCKER_IMAGE_TAG="latest"

Pre-release filtering is handled at the promci level, where the regex
check ^v[0-9]+(\.[0-9]+){2}$ already ensures only stable releases
trigger the "latest" tagging workflow.

Signed-off-by: Julien Pivotto <291750+roidelapluie@users.noreply.github.com>
@roidelapluie roidelapluie requested a review from SuperQ January 19, 2026 14:48
@roidelapluie roidelapluie merged commit 8e6b097 into prometheus:main Jan 22, 2026
53 of 54 checks passed
@bitfehler
Copy link
Copy Markdown

@roidelapluie @SuperQ Hey folks, after merging the latest "Sync common files" in ipmi-exporter, I get an error publishing artifacts that seems related to this: https://github.com/prometheus-community/ipmi_exporter/actions/runs/22058914370/job/63734029249#step:4:891

Do you happen to know what's going on there? Might well be me holding it wrong, but e/t has worked fine up until now, so not sure what the issue could be...

@roidelapluie
Copy link
Copy Markdown
Member Author

This is not related to this change: docker/bake-action#322. The issue is the --platform.

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.

3 participants