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

roachprod: add custom tags for prometheus config #124409

Merged

Conversation

nameisbhaskar
Copy link
Contributor

few of the labels are missing in prometheus with the change from "gce_sd_configs" to "file_sd_configs". This is because these were extracted from the VM metadata which is not available now. So, the same values are set from roachprod now.

Informs: #124319
Epic: none

@nameisbhaskar nameisbhaskar requested a review from a team as a code owner May 20, 2024 06:22
@nameisbhaskar nameisbhaskar requested review from renatolabs and vidit-bhat and removed request for a team May 20, 2024 06:22
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@nameisbhaskar nameisbhaskar force-pushed the user/bhaskar/add-custom-labels branch 2 times, most recently from 2011803 to 0fc302a Compare May 20, 2024 12:53
few of the labels are missing in prometheus with the change
from "gce_sd_configs" to "file_sd_configs". This is because
these were extracted from the VM metadata which is not
available now. So, the same values are set from roachprod now.

Informs: cockroachdb#124319
Epic: none
@nameisbhaskar nameisbhaskar force-pushed the user/bhaskar/add-custom-labels branch from 0fc302a to 9e30f6f Compare May 21, 2024 03:14
@nameisbhaskar
Copy link
Contributor Author

Thanks @srosenberg for reviewing this!!

@nameisbhaskar
Copy link
Contributor Author

bors r=@srosenberg

@craig craig bot merged commit d146ecf into cockroachdb:master May 21, 2024
19 checks passed
@nameisbhaskar nameisbhaskar deleted the user/bhaskar/add-custom-labels branch May 21, 2024 03:47
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.

None yet

3 participants