[
https://issues.apache.org/jira/browse/CLOUDSTACK-9503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15513584#comment-15513584
]
ASF GitHub Bot commented on CLOUDSTACK-9503:
--------------------------------------------
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1678#discussion_r80064873
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtPvlanSetupCommandWrapper.java
---
@@ -58,11 +59,11 @@ public Answer execute(final PvlanSetupCommand command,
final LibvirtComputingRes
String result = null;
try {
final String guestBridgeName =
libvirtComputingResource.getGuestBridgeName();
- final int timeout = libvirtComputingResource.getTimeout();
+ final Duration timeout = libvirtComputingResource.getTimeout();
if (command.getType() == PvlanSetupCommand.Type.DHCP) {
final String ovsPvlanDhcpHostPath =
libvirtComputingResource.getOvsPvlanDhcpHostPath();
- final Script script = new Script(ovsPvlanDhcpHostPath,
timeout, s_logger);
+ final Script script = new Script(ovsPvlanDhcpHostPath,
timeout.getMillis(), s_logger);
--- End diff --
Please consider adding an overridden version of the `Script(String, int,
Logger)` constructor that accepts `Duration` to encapsulate this type
conversion.
> The router script times out resulting in failure of deployment
> --------------------------------------------------------------
>
> Key: CLOUDSTACK-9503
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9503
> Project: CloudStack
> Issue Type: Bug
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Components: Virtual Router
> Affects Versions: 4.9.0
> Environment: KVM, Xen
> Reporter: Abhinandan Prateek
> Assignee: Abhinandan Prateek
> Fix For: 4.9.1.0
>
>
> When starting the virtual router in a shared network in advance zone the
> scripts on router time out. This happen as there are several sub-commands
> that are consolidated in a single command. The default timeout of 2 minutes
> is short.
> 2016-09-09 00:06:25,016 ERROR [c.c.n.r.VirtualNetworkApplianceManagerImpl]
> (Work-Job-Executor-110:ctx-e8089ec7 job-5135/job-5137 ctx-c3a8da18)
> (logid:8aedea66) process hasn't exited
> 2016-09-09 00:06:25,016 WARN [c.c.n.r.VirtualNetworkApplianceManagerImpl]
> (Work-Job-Executor-110:ctx-e8089ec7 job-5135/job-5137 ctx-c3a8da18)
> (logid:8aedea66) Command: com.cloud.agent.api.Command failed while starting
> virtual router
> 2016-09-09 00:06:25,016 INFO [c.c.v.VirtualMachineManagerImpl]
> (Work-Job-Executor-110:ctx-e8089ec7 job-5135/job-5137 ctx-c3a8da18)
> (logid:8aedea66) The guru did not like the answers so stopping
> VM[DomainRouter|r-3445-VM]
> —
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
