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

Scrape GuestOS metrics-proxy and clean up issues in multiservice-discovery. #91 #93

Merged
merged 13 commits into from
Jan 25, 2024

Conversation

DFINITYManu
Copy link
Contributor

multiservice-discovery was incorrectly discovering HostOS node exporter as GuestOS node exporter.

The matter has been corrected. This is a sample diff from its prior output and the current output. Note IP address of host_node_exporter in the prior scrape.

image

@DFINITYManu DFINITYManu requested a review from a team as a code owner January 22, 2024 17:27
Copy link
Contributor

@nikola-milosa nikola-milosa left a comment

Choose a reason for hiding this comment

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

Great job, thank you a lot! If you need any help in deploying ping me

Copy link
Member

@sasa-tomic sasa-tomic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@DFINITYManu DFINITYManu added this pull request to the merge queue Jan 25, 2024
Merged via the queue into main with commit cd4b854 Jan 25, 2024
3 checks passed
@DFINITYManu DFINITYManu deleted the container-metricsproxyguestscrape branch January 25, 2024 09:53
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

4 participants