-
Notifications
You must be signed in to change notification settings - Fork 876
feat: integrate agentAPI with resources monitoring logic #16438
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
9d42cad
work on new agent version
defelmnq 7b2d19e
improve function for resources monitoring
defelmnq 0124d60
add missing files
defelmnq 3661e8c
work on resources monitor tests
defelmnq a5a788e
apply fmt and lint
defelmnq 91d1515
work on dbauthz tests
defelmnq 0bc7632
work on dbauthz
defelmnq 3085041
work on rbac
defelmnq 120a37b
continue to iterate
defelmnq dd8ed40
continue to iterate
defelmnq 0a8941b
continue to iterate
defelmnq f3388b4
work on tests
defelmnq 523f6fd
improve testing
defelmnq 06adbf7
improve error messages
defelmnq c7b03d0
rework architecture of resources monitor
defelmnq 2c3d171
improve resourcesmonitor struct
defelmnq 18b65e0
improve resourcesmonitor struct
defelmnq c95b05a
change proto payload for get resources monitoring config
defelmnq c79b6cb
change proto payload for get resources monitoring config
defelmnq b28d4fa
rework fetcher and tests
defelmnq 7701624
fix tests
defelmnq 5fad903
fix tests
defelmnq b611ae5
fix tests
defelmnq 3c65b8a
fix logic
defelmnq 63c5869
improve testing fetcher and rename struct
defelmnq 2d3eeb5
lint
defelmnq e17aafc
work on dbauthz
defelmnq c5a4201
improve dbauthz for fetching
defelmnq 262a672
change dbauthz permissions
defelmnq dbca96e
finalise tests
defelmnq 3145eab
fix comments from github
defelmnq 3bec324
add collectedAt
defelmnq File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
improve error messages
- Loading branch information
commit 06adbf74bc17311a1f97232ce7b0b0e603e234bc
There are no files selected for viewing
This file contains hidden or 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.
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.
self review : We have this intermediate function in order to have an extra layer, and simplicify for testing purpose.
As all arguments are injected as parameters to
PushResourcesMonitoringWithConfig
we can use the mocked versions.