diff options
author | Nicholas Verne <nverne@chromium.org> | 2018-09-18 11:04:33 +1000 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-09-19 08:59:49 -0700 |
commit | 137cb973fb0eb203f47e50adce03baa4ff06a21b (patch) | |
tree | f11f275c9000ef40116dd190708828879d8686b9 | |
parent | 48093e2673c64d27062da7df353d7f7cf2aa398f (diff) | |
download | system_api-137cb973fb0eb203f47e50adce03baa4ff06a21b.tar.gz |
Add VmStatus to StartVmResponse.
When this status indicates the VM is already running, CrostiniManager can
avoid waiting forever for a signal that Tremplin has started in the VM.
Bug: crbug:884576
Change-Id: I7c4577a4fa5d0bcba2c158a046ad31e81236b421
Reviewed-on: https://chromium-review.googlesource.com/1226733
Commit-Ready: Nicholas Verne <nverne@chromium.org>
Tested-by: Nicholas Verne <nverne@chromium.org>
Reviewed-by: Jeffrey Kardatzke <jkardatzke@google.com>
-rw-r--r-- | dbus/vm_concierge/service.proto | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/dbus/vm_concierge/service.proto b/dbus/vm_concierge/service.proto index 6d3aac1..c3dc590 100644 --- a/dbus/vm_concierge/service.proto +++ b/dbus/vm_concierge/service.proto @@ -103,6 +103,20 @@ message StartVmRequest { string owner_id = 7; } +enum VmStatus { + // Unknown status. + VM_STATUS_UNKNOWN = 0; + + // The VM is already running. + VM_STATUS_RUNNING = 1; + + // The VM is currently starting up. + VM_STATUS_STARTING = 2; + + // The VM failed to startup. + VM_STATUS_FAILURE = 3; +} + // Information sent back by vm_concierge in response to a StartVm dbus message. message StartVmResponse { // If true, the VM was started successfully. |vm_info| will have non-default @@ -114,6 +128,10 @@ message StartVmResponse { // Information about the VM that was started, if successful. VmInfo vm_info = 3; + + // Status of the VM. If VM_STATUS_RUNNING, it is not necessary to wait for a + // TremplinStartedSignal before starting a container in the VM. + VmStatus status = 4; } // Information that must be included with every StopVm dbus message. |