Commit 8beb296c authored by Nicholas Verne's avatar Nicholas Verne Committed by Commit Bot

StartVm dbus timeout increased

This increase is temporary and should be reverted once LXD startup code
has been edited to remove unnecessary crypto calculations. These take much
longer on Kevin than Eve at present.

Bug: 863190
Change-Id: Ib2a004c46ecbf45c097bf3d6843e930b5edec7f9
Reviewed-on: https://chromium-review.googlesource.com/1152334Reviewed-by: default avatarRyo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Nicholas Verne <nverne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578536}
parent 9aa7c545
...@@ -112,8 +112,11 @@ class ConciergeClientImpl : public ConciergeClient { ...@@ -112,8 +112,11 @@ class ConciergeClientImpl : public ConciergeClient {
return; return;
} }
// TODO(nverne): revert to TIMEOUT_USE_DEFAULT when StartVm no longer
// requires unnecessary long running crypto calculations.
constexpr int kStartVmTimeoutMs = 160 * 1000;
concierge_proxy_->CallMethod( concierge_proxy_->CallMethod(
&method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT, &method_call, kStartVmTimeoutMs,
base::BindOnce(&ConciergeClientImpl::OnDBusProtoResponse< base::BindOnce(&ConciergeClientImpl::OnDBusProtoResponse<
vm_tools::concierge::StartVmResponse>, vm_tools::concierge::StartVmResponse>,
weak_ptr_factory_.GetWeakPtr(), std::move(callback))); weak_ptr_factory_.GetWeakPtr(), std::move(callback)));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment