-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
Add better verbosity to the etcdctl endpoint status output that includes percentage of DB used (to illustrate Defrag rate) #17872
Comments
@Scotchman0, it may be helpful if you can add the current output to compare to your suggestion. /assign @Scotchman0 |
I like the idea, but after seeing the implementation, I wonder if it would be better to add more columns rather than reusing/multi-purposing the current column. I defer to @ahrtr / @serathius with their point of view. |
updated the initial with the comparative of what we've got now versus proposed change - I'm open to separating by column; fairly easy to make those a separate block instead of joining. |
alternative implementation with separate columns for comparison might look like the following if it's desirable to segment instead of conjoin an output block:
|
Would it be easier to put the storage information as usage I would be also interested if we could quota somewhere, not sure it's accessible via API. |
I agree it's a minor enhancement, but on other hand, it's minor breaking on the user experience. I like the idea of adding a separate new column.
Makes sense to me.
Not sure I got the point. Do you mean that etcdserver also returns storage quota (e.g. 2GiB) to client when users run |
I got a few minutes today to just pull the config through: will add more tests in a bit, you guys can have a look in the meantime 👍 |
I went ahead and modified the PR #17871 to include separated columns for each output + a new block for Quota pending #17877 approval/merge to reference the new quota block: might look like the below:
|
Human readable table is not an API. We don't guarantee column names or contents. It should be different to other formats like json, but table is not parsable and it should not be so. |
|
Just an update to indicate the PR is cleaned/approved for final eyes/checks/thoughts here.
|
Done in #17871 |
What would you like to be added?
I have opened a PR with the specific changes: #17871
Why is this needed?
Adding the output from
OMC
which grants visibility of the size of the DB, how much is in use, and then providing the difference as a percentage for quick visual on storage impact on the cluster. A minor change but one that makes it much easier to illustrate where storage is going.Current release:
$ etcdctl endpoint status -w table
proposed change:
$ etcdctl endpoint status -w table
This output is available in OMC (used for analysis of existing etcd databases running on openshift presently) and visual representation is not available to customers as a default, when it could be.
The text was updated successfully, but these errors were encountered: