Project

General

Profile

Bug #28506

Fix prometheus telemetry

Added by Ondřej Ezr 9 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Ondřej Ezr
Category:
Monitoring
Target version:
1.24.1
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/7271, https://github.com/theforeman/foreman/pull/7249
Fixed in Releases:
1.24.1, 2.0.0
Found in Releases:
Nightly

Description

Prometheus telemetry is broken due to an upgrade of the gem to version 1.0

Associated revisions

Revision b50324b4 (diff)
Added by Ondřej Ezr 9 months ago

Fixes #28506 - fix upgraded prometheus

Revision 6ecab45f (diff)
Added by Lukas Zapletal 9 months ago

Refs #28506 - fixed telemetry boot require

History

#1 Updated by The Foreman Bot 9 months ago

  • Assignee set to Ondřej Ezr
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7249 added

#2 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases 2.0.0 added

#3 Updated by Ondřej Ezr 9 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 9 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7271 added

#5 Updated by Lukas Zapletal 9 months ago

  • Target version set to 1.24.1

Tomer, I would like to request backport this one into 1.24. Monitoring has some issues downstream and I would like to get this opportunity to load telemetry data via Prometheus API rather than statsd. This would require prometheus rubygem version bump to 1.0.0 too if that's not a problem. Can file all the PRs.

#6 Updated by Tomer Brisker 9 months ago

  • Fixed in Releases 1.24.1 added

I cherry-picked the upgrade to 1.24 branch, please open a PR to update packaging accordingly.

Also available in: Atom PDF