Skip to content
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

Backport external prometheus fixes #531

Closed
wants to merge 2 commits into from

Conversation

scrungus
Copy link

No description provided.

This avoids the need to use a proxy, or some other means, to connect to
Prometheus. This is disabled by default and can be enabled by setting
enable_prometheus_server_external to true.

Change-Id: Ia0af044ff436c2a204b357750a16ff49fcdfec45
(cherry picked from commit 37c2ab2)
Closes-Bug: #2043831
Change-Id: I010fabd255d93d5329de82af2b5d21c8fa7d93c4
(cherry picked from commit 2c9dc5d)
@scrungus scrungus requested a review from a team as a code owner February 15, 2024 15:21
@markgoddard
Copy link

This is quite a large backport for yoga. There may be a reason we didn't do this before. Who needs/wants it?

@scrungus
Copy link
Author

This was for cambridge, but we've decided to do this via overrides instead. closing

@scrungus scrungus closed this Feb 16, 2024
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