-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
show default zone and default group #2852
Comments
@critical27 @darionyaphet pls discuss this requirement, is it need be resolved in v2.6.0? |
more information |
You mean show groups return a default group by default? But I can't figure out how to fill the |
Yes, that's also my question... If we figure out the zone with We could have a discussion. |
We discuss on a meeting, by default we return: (suppose we have n hosts before)
WDYT? |
Show default group may be a good idea. I agree with you |
Talked offline with @darionyaphet , we'll print default group when exec But I don't know whether we need to print the host in |
Talked offline, will not support this interface until 3.0. Because a host could both belongs to a default zone and a user create zone, which breaks the invariant. |
agree and then close issue |
there are some concerns about default zone.
if user upgrade from 2.0.1
balance remove
if user creates space without group.
it's better to show the default zone and default group, and explain the logic how we consider a storage in default zone or not.
The text was updated successfully, but these errors were encountered: