-
Notifications
You must be signed in to change notification settings - Fork 20
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
DPE-2617 autotune production profile #231
Conversation
Codecov Report
@@ Coverage Diff @@
## main #231 +/- ##
==========================================
+ Coverage 80.71% 80.73% +0.01%
==========================================
Files 9 9
Lines 1862 1869 +7
Branches 304 307 +3
==========================================
+ Hits 1503 1509 +6
Misses 309 309
- Partials 50 51 +1
|
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.
Please add basic test here to make Codecov happy. Tnx!
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.
We should increment the lib version. I've created canonical/postgresql-k8s-operator#264 for k8s
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.
Ufff..... we need an automated protection here on CI/CD. Good ideas are welcome!
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.
Thanks Paulo! LGTM.
I'm merging without tests end because the only change was a |
* auto tune based on memory * increased coverage * bump lib
Port of pg-k8s pr#260