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

Revert "[chore] Replace usage of metadatatest.SetupTelemetry with new componentest" #37626

Closed
wants to merge 1 commit into from

Conversation

songy23
Copy link
Member

@songy23 songy23 commented Jan 31, 2025

Reverts #37613

@ArthurSens
Copy link
Member

ArthurSens commented Jan 31, 2025

I'm not sure it's related that PR. I've just made a change to the compliance test that I shouldn't have made. At least not yet

prometheus/compliance#134

@songy23
Copy link
Member Author

songy23 commented Jan 31, 2025

@ArthurSens you're right probably, the prometheus-compliance-tests still failed in this PR

@songy23 songy23 closed this Jan 31, 2025
@songy23 songy23 deleted the revert-37613-rm-more-usage-metadatatest branch January 31, 2025 18:19
@ArthurSens
Copy link
Member

I'm reverting the PR over there and merging again only once #36873 is ready

@ArthurSens
Copy link
Member

Reverted!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants