Skip to content

fix(renovate): correct ama-metrics version parsing#8920

Merged
Devinwong merged 1 commit into
mainfrom
devinwong/fix-ama-metrics-versioning
Jul 13, 2026
Merged

fix(renovate): correct ama-metrics version parsing#8920
Devinwong merged 1 commit into
mainfrom
devinwong/fix-ama-metrics-versioning

Conversation

@Devinwong

@Devinwong Devinwong commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

What this PR does / why we need it

Two-commit fix for the ama-metrics Renovate rule:

Commit 1 — added named major group to fix the 6.27.0 > 7.0.0 mis-ordering that caused a downgrade PR (#8897).

Commit 2 — removed the named prerelease group introduced in commit 1. Having a prerelease group causes Renovate to mark all ama-metrics tags as unstable and skip them entirely (no updates detected).

The final regex:

  • Named major/minor/patch → correct version ordering
  • Date/SHA segment left as a non-capturing group (?:...) → not treated as prerelease, so tags are considered stable
  • Named compatibility → differentiates cfg, targetallocator, ccp, win variants

Related

Renovate's ama-metrics regex did not capture a named major group, which caused incorrect ordering (e.g., 6.27.0 ranked above 7.x). Update the rule to parse ama-metrics tags explicitly, including optional variant suffixes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Renovate configuration to correctly parse and compare ama-metrics image tags so Renovate stops proposing “downgrade” PRs when moving between major versions (e.g., 6.x → 7.x).

Changes:

  • Updates the versioning regex for azuremonitor/containerinsights/ciprod/prometheus-collector/images to include a named major group.
  • Tightens the parsing to the expected tag format (<major>.<minor>.<patch>-main-<dd-mm-yyyy>-<sha>) with an optional known suffix (cfg|targetallocator|ccp|win).

@Devinwong Devinwong merged commit 2172f1e into main Jul 13, 2026
22 checks passed
@Devinwong Devinwong deleted the devinwong/fix-ama-metrics-versioning branch July 13, 2026 21:34
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