Skip to content

Better cvd help stop message#2713

Open
jemoreira wants to merge 1 commit into
google:mainfrom
jemoreira:stop
Open

Better cvd help stop message#2713
jemoreira wants to merge 1 commit into
google:mainfrom
jemoreira:stop

Conversation

@jemoreira

@jemoreira jemoreira commented Jun 13, 2026

Copy link
Copy Markdown
Member

Updated to document stopping a subset of instances in the group. More details on how the stop operation works.
Document selector flags too.
Adds the wait_for_launcher_seconds flag alias to make the time unit more obvious.

Bug: b/523416547

Updated to document stopping a subset of instances in the group.
More details on how the stop operation works.
Document selector flags too.
Adds the wait_for_launcher_seconds flag alias to make the time unit more
obvious.
@jemoreira jemoreira requested a review from Databean June 13, 2026 00:22
@jemoreira jemoreira enabled auto-merge June 13, 2026 00:27
description.emplace_back(
HelpParagraph::Raw("Usage:\n cvd [selectors] stop [args]"));
description.emplace_back(
"Stop a subset of instances from a group. A single instance, severall or "

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: several

Comment on lines +119 to +120
"The stop operation attemps to graciously stop the instances by sending "
"an ACPI shutdown. If the shutdown doesn't complete within the "

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no "ACPI shutdown", was this hallucinated? The distinction is whether processes are SIGKILLed or not. The "nice" shutdown still uses crosvm stop, which immediately stops all KVM threads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants