-
Notifications
You must be signed in to change notification settings - Fork 561
kubernetes_management: add benchmark + base cluster management plane methods #6746
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
base: master
Are you sure you want to change the base?
Changes from 12 commits
c30c6ba
6bdc03e
c959389
bf8c1d1
b0c04ec
0e1dba3
620fd56
e2a106f
17df7f8
f431c3f
6e38531
9562498
5662918
2957606
1f36225
15ee9b3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,7 @@ | ||
| ### New features: | ||
| - Add kubernetes_management benchmark for measuring GKE/EKS/AKS management | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. there's a new features section below which is where this should go.
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done — turns out the entry was under a stale "New features" header at the top of the file; moved it into the actual current section further down (with the Ubuntu 26.04 / MySQL sysbench entries). |
||
| plane API responsiveness. | ||
|
|
||
| ### Breaking changes: | ||
|
|
||
| - Added --accept_licenses flag. User have to turn this flag on to acknowledge | ||
|
|
||
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.
there's an existing new features section below. Add to there.
Other comments:
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.
Done — moved it into the existing new-features section, collapsed to a single line, and dropped both the author credit and the abstract-methods bullet.