aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNicholas Verne <nverne@chromium.org>2018-09-18 11:04:33 +1000
committerchrome-bot <chrome-bot@chromium.org>2018-09-19 08:59:49 -0700
commit137cb973fb0eb203f47e50adce03baa4ff06a21b (patch)
treef11f275c9000ef40116dd190708828879d8686b9
parent48093e2673c64d27062da7df353d7f7cf2aa398f (diff)
downloadsystem_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.proto18
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.