Github user insom commented on the issue:

    https://github.com/apache/cloudstack/pull/1451
  
    @ustcweizhou A _side effect_ of this PR is setting the minimal timeout to 
120, but that's what the original code intended - except that because of not 
passing through the timeout to `applyConfigToVR` it wasn't taking effect.
    
    What this PR really does is make the code work as intended. I'm up for 
improvements but if this is blocking upgrades (and at least iWeb and one other 
CloudStack user were hit by it) seems like merging this and _then_ moving it to 
a per-host setting would be a good idea?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to