Skip to content

Trim down cvd create output#2523

Merged
cjreynol merged 10 commits intogoogle:mainfrom
cjreynol:trim_cvd_create
May 8, 2026
Merged

Trim down cvd create output#2523
cjreynol merged 10 commits intogoogle:mainfrom
cjreynol:trim_cvd_create

Conversation

@cjreynol
Copy link
Copy Markdown
Collaborator

@cjreynol cjreynol commented May 6, 2026

Bug: 507537392

@cjreynol cjreynol self-assigned this May 6, 2026
@cjreynol cjreynol changed the title Push path adjustment message to verbose logging Trim down cvd create output May 7, 2026
@cjreynol cjreynol force-pushed the trim_cvd_create branch from a3af7d8 to ca5c957 Compare May 7, 2026 19:36
@cjreynol cjreynol requested a review from jemoreira May 7, 2026 19:39
@cjreynol cjreynol added kokoro:force-run Trigger a presubmit build unconditionally. kokoro:run Run e2e tests. labels May 7, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels May 7, 2026
cjreynol added 2 commits May 7, 2026 15:46
This logic is going to be revisted for metrics gathering as part of
b/492179685.

Bug: 507537392
To trim down the `cvd create` output.

Bug: 507537392
@cjreynol cjreynol force-pushed the trim_cvd_create branch from ca5c957 to f506183 Compare May 7, 2026 22:46
@cjreynol cjreynol marked this pull request as ready for review May 7, 2026 22:46
@cjreynol cjreynol added kokoro:force-run Trigger a presubmit build unconditionally. kokoro:run Run e2e tests. labels May 7, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels May 7, 2026
Comment thread base/cvd/cuttlefish/host/commands/start/validate_metrics_confirmation.cpp Outdated
@cjreynol cjreynol force-pushed the trim_cvd_create branch from f506183 to ed9ca97 Compare May 8, 2026 00:05
cjreynol added 8 commits May 7, 2026 17:12
This message is always output when starting a device, partly because of
the V2 metrics.  There is no reason to duplicate it, and the V1 metrics
confirmation dialog will be removed when V2 metrics are complete
(b/463443661).

Also, trimmed down the messages that use identical language to previous
notices.

Bug: 507537392
Trying to trim down the `cvd create` output.

Bug: 507537392
Also, trim the message to minimize output in `cvd create` noise.

Bug: 507537392
In favor of directing users towards the new `cvd logs` command.

Bug: 507537392
Trying to trim down the `cvd create` output.

Bug: 507537392
Trying to trim down `cvd create` output.

Also, I moved the printing of that message until AFTER the file
existence check is run.  If it does not exist, it is not being read.

Bug: 507537392
The remaining message is the one that is printed to the console for the
user.

Bug: 507537392
And make the failed message log at the error level to match the
reasoning for the message.

Users do not need to see that message when the "Virutal device booted
successfully" already exists.

Bug: 507537392
@cjreynol cjreynol force-pushed the trim_cvd_create branch from ed9ca97 to 4f1f61b Compare May 8, 2026 00:13
@cjreynol cjreynol added kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels May 8, 2026
@cjreynol cjreynol enabled auto-merge May 8, 2026 00:14
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels May 8, 2026
@cjreynol cjreynol added this pull request to the merge queue May 8, 2026
Merged via the queue into google:main with commit f97d360 May 8, 2026
28 checks passed
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.

3 participants