-
Notifications
You must be signed in to change notification settings - Fork 411
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
adds vpa daemonset handling #1779
base: main
Are you sure you want to change the base?
Conversation
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.
@KitSutliff were you able to do any manual testing and observe the vertical scaling working as expected?
Ah no. haven't reproduced it on a working cluster, developing an end-to-end based on the hpa tests |
…r is enabled (open-telemetry#1712) * Code fix for issues open-telemetry#1623, open-telemetry#1622, open-telemetry#958 * updated the PR with an alternative logic to fi the relabel config issue * fixed codelint issues * reverted the logic to update scrape_configs * fixed unit test case failures * fixed code lint issues * changes to replace_config to handle unamrshalling issue when tarallocatorewrite flag is enabled * fixed unit test case failure * updated e2e test to cover labelkeep usecase * updated unit test to escape $ signs * updated unit test to cover missing and invalig configs * resolved merge conflicts * updated tests, refactored logic based on the new commits * uncommented labeldrop e2e test * added back target allocator attribute * comment for unescaping TA config * updated the README document to reflect the traslations * updated the README document to reflect the traslations * updated the TA config section in the README document * addressed the review comment on README doc --------- Co-authored-by: Jacob Aronoff <[email protected]>
* Bump python to 1.18.0/0.39b0 * changelog --------- Co-authored-by: Jacob Aronoff <[email protected]>
* release the operator * update changelog and versions * remove last compat * Fix changelog * more chlog
* Bump all the deps * linter
@KitSutliff looks like a bad rebase, you may want to reattempt your rebase on main. |
Tooled around with this and think I've got a fix. Did not implement sidecar for now. Any thoughts?
fixes #1667