-
Notifications
You must be signed in to change notification settings - Fork 49
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
Compact cluster status #2714
base: main
Are you sure you want to change the base?
Compact cluster status #2714
Conversation
9f03df3
to
3fe3dd3
Compare
Not a fan of the letter-per-role approach. Perhaps the proposal is too compact? I can't quickly understand most of the columns. |
@AhmedSoliman I tried to use the full form but to me it looks like a lot of data in one column. But maybe you are right about being too compact. Probably better to split the PARTITIONS into |
0555aaa
to
78508f6
Compare
f3458e2
to
58df8bf
Compare
This is definitely more readable; did we give up on trying to convey the sequencer-partition leader colocation status? I would consider using singular 'LEADER |
Summary: - Support parsing an "range" input. - Handle id range in some common operations - `logs describe` - `logs find-tail` - `logs reconfigure` - `snapshots create-snapshot`
Summary: Print out a compact cluster status by default Extended view of the cluster status can still be accessed via the `--extra` flag > There are also some minor fixes to the restatectl because of conflicting short flags across different commands.
@pcholakov the sequence/partition colocation status is only denoted by the colour so far. The green on the sequencer status means it's optimal colocation, if yellow then it's not. But this might be confusing if colours are not enabled |
That's perfect, I think colour is okay as this is a relatively advanced concept. |
Compact cluster status
Summary:
Print out a compact cluster status by default
Extended view of the cluster status can still be accessed
via the
--extra
flagStack created with Sapling. Best reviewed with ReviewStack.
range input
for some ids #2723