Skip to content

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 7, 2024

Bumps io.prometheus:prometheus-metrics-bom from 1.3.1 to 1.3.3.

Release notes

Sourced from io.prometheus:prometheus-metrics-bom's releases.

v1.3.3

What's Changed

Full Changelog: prometheus/client_java@v1.3.2...v1.3.3

v1.3.2

What's Changed

New Contributors

Full Changelog: prometheus/client_java@v1.3.1...v1.3.2

Commits
  • 2c022be create release (#1194)
  • da91960 revert incompatible api change (#1192)
  • 7bc442f avoid ConcurrentModificationException (#1191)
  • 76cb93e Update RELEASING.md (#1188)
  • fc583c2 improve release (#1187)
  • 05c6862 Bump org.apache.maven.plugins:maven-javadoc-plugin from 3.10.1 to 3.11.1 (#1185)
  • 036570f Bump org.apache.maven.plugins:maven-failsafe-plugin from 3.5.1 to 3.5.2 (#1184)
  • 048cbd5 Bump org.apache.maven.plugins:maven-surefire-plugin from 3.5.1 to 3.5.2 (#1183)
  • 1b4d99d Bump org.codehaus.mojo:exec-maven-plugin from 3.4.1 to 3.5.0 (#1181)
  • 5c81cc1 Bump io.opentelemetry.instrumentation:opentelemetry-instrumentation-bom-alpha...
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [io.prometheus:prometheus-metrics-bom](https://github.com/prometheus/client_java) from 1.3.1 to 1.3.3.
- [Release notes](https://github.com/prometheus/client_java/releases)
- [Commits](prometheus/client_java@v1.3.1...v1.3.3)

---
updated-dependencies:
- dependency-name: io.prometheus:prometheus-metrics-bom
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependency-upgrade A dependency upgrade java Pull requests that update Java code labels Nov 7, 2024
@dependabot dependabot bot added this to the 1.next milestone Nov 7, 2024
@jonatan-ivanov jonatan-ivanov modified the milestones: 1.next, 1.14.0 Nov 7, 2024
@jonatan-ivanov
Copy link
Member

jonatan-ivanov commented Nov 7, 2024

The build is failing because the Prometheus Java Client used to let users to use colons (:) in the metric names. Apparently, this should have not been allowed.

See the docs: https://prometheus.io/docs/concepts/data_model/#metric-names-and-labels

Metric names may contain ASCII letters, digits, underscores, and colons.
It must match the regex [a-zA-Z_:][a-zA-Z0-9_:]*.
Note: The colons are reserved for user defined recording rules. They should not be used by exporters or direct instrumentation."

So even if metric names can contain colons (:), exporters (like the Prometheus Java Client or Micrometer) should not use it. The Prometheus Java Client replaces colons (:) to underscores (_) starting from 1.3.3.

See

@jonatan-ivanov jonatan-ivanov force-pushed the dependabot/gradle/main/io.prometheus-prometheus-metrics-bom-1.3.3 branch from 0fdc4c0 to 5b5bdf0 Compare November 7, 2024 22:20
The Prometheus Java Client used to let users to use colons (":")
in the metric names. Apparently, this should have not been allowed.

See the docs: https://prometheus.io/docs/concepts/data_model/#metric-names-and-labels
"Metric names may contain ASCII letters, digits, underscores,
and colons. It must match the regex [a-zA-Z_:][a-zA-Z0-9_:]*.
Note: The colons are reserved for user defined recording rules.
They should not be used by exporters or direct instrumentation."

So even if metric names can contain colons (":"),
exporters (like the Prometheus Java Client or Micrometer)
should not use it. The Prometheus Java Client replaces colons (":")
to underscores ("_") starting from 1.3.3.

See prometheus/client_java#974
See prometheus/client_java#975
See #5649
@jonatan-ivanov jonatan-ivanov force-pushed the dependabot/gradle/main/io.prometheus-prometheus-metrics-bom-1.3.3 branch from 5b5bdf0 to ca0750e Compare November 7, 2024 22:24
@jonatan-ivanov jonatan-ivanov merged commit 269fab3 into main Nov 7, 2024
8 checks passed
@jonatan-ivanov jonatan-ivanov deleted the dependabot/gradle/main/io.prometheus-prometheus-metrics-bom-1.3.3 branch November 7, 2024 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency-upgrade A dependency upgrade java Pull requests that update Java code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant