-
Notifications
You must be signed in to change notification settings - Fork 18.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
Carry 24492: Remove swarm inspect and use info instead #25042
Conversation
Signed-off-by: Tibor Vass <tibor@docker.com>
Remove the swarm inspect command and use docker info instead to display swarm information if the current node is a manager. Signed-off-by: Vincent Demeester <vincent@sbr.pm>
LGTM, but docker info output is becoming quite large, for readability we may want to consider introducing subcommand to info later on (e,g |
fmt.Fprintf(dockerCli.Out(), " Managers: %d\n", info.Swarm.Managers) | ||
fmt.Fprintf(dockerCli.Out(), " Nodes: %d\n", info.Swarm.Nodes) | ||
fmt.Fprintf(dockerCli.Out(), " Name: %s\n", info.Swarm.Cluster.Spec.Annotations.Name) | ||
fmt.Fprintf(dockerCli.Out(), " Orchestration:\n") | ||
fmt.Fprintf(dockerCli.Out(), " Task History Retention: %d\n", info.Swarm.Cluster.Spec.Orchestration.TaskHistoryRetentionLimit) |
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.
either "Task History Retention Limit: %d" or "Task History Retention: %d tasks"
Some minor suggestions, otherwise LGTM. |
LGTM |
LGTM, I agree with @aaronlehmann's suggestions |
@aaronlehmann fixed |
Merge away when CI is ready. |
Signed-off-by: Tibor Vass <tibor@docker.com>
Win2lin has issues, it's unrelated |
Carrying #24492
Vendored docker/engine-api#334
Closes #24492
Closes #24148