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

Support scraping config-reloader sidecar for Thanos Ruler #245

Merged
merged 1 commit into from
Sep 15, 2021

Conversation

philipgough
Copy link
Contributor

@philipgough philipgough commented Sep 2, 2021

  • I added CHANGELOG entry for this change.
  • [] Change is not relevant to the end user.

Changes

Adds config to allow scraping of the Thanos Ruler config reloader sidecar.

Verification

Deployed on minikube, look at the targets page and see thanos ruler with the desired endpoint is up.

Screenshot 2021-09-07 at 12 26 43

@philipgough philipgough force-pushed the mon-1085 branch 3 times, most recently from db7ac2d to 338af9c Compare September 2, 2021 15:17
@philipgough philipgough marked this pull request as draft September 2, 2021 17:41
@philipgough philipgough force-pushed the mon-1085 branch 2 times, most recently from d9de62a to 5add2b9 Compare September 7, 2021 11:32
Signed-off-by: Philip Gough <philip.p.gough@gmail.com>
@philipgough philipgough changed the title WIP: Support scraping config-reloader sidecar for Thanos Ruler Support scraping config-reloader sidecar for Thanos Ruler Sep 7, 2021
@philipgough philipgough marked this pull request as ready for review September 7, 2021 11:34
Copy link
Member

@kakkoyun kakkoyun left a comment

Choose a reason for hiding this comment

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

LGTM

@kakkoyun kakkoyun merged commit 24559b5 into thanos-io:main Sep 15, 2021
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