-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
feat: upgrade prometheus #13671
Merged
Merged
feat: upgrade prometheus #13671
Changes from 16 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
034b7a7
Upgrade prometheus
pracucci 15067f4
Fix compilation
pracucci f42274c
fix
pracucci c3e9c6b
Fixed more compilation issues
pracucci 80406a6
Fix Test_match
pracucci 0e8a115
Fix TestParse
pracucci 3b5699c
Fixed Test_LoadRetentionRules
pracucci 5f03d98
Fixed Test_store_decodeReq_Matchers
pracucci dff595c
Add nolint rule for grpc.Dial()
pracucci e978d24
Vendor prometheus from a branch to fix the panic
pracucci 1d86aec
Fixed more tests doing matchers comparison
pracucci fc2adf7
Re-vendor prometheus from main
pracucci 641aff2
Added more nolint rules
pracucci c3e6144
Fix TestParse but lower confidence
pracucci dde2410
Fixed deprecation
pracucci cb9089f
Remove dead code
pracucci 6b239f7
fixed test by asserting the matchers without FastRegexMatcher
vlad-diachenko 88f73d0
Merge branch 'refs/heads/main' into fix-last-prometheus-vendoring
vlad-diachenko File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note to reviewers:
types.NetworkResource
is deprecated, butnetwork.Inspect
is expected to be the same exact type.