The tests that failed for this PR do not seem to be likely because of this PR.
That being the case and with all XenServer-oriented managed storage-related integration tests having passed (per above comments), this PR LGTM. We have two LGTMs, CI passed, and integration tests look reasonable (per previous comments). Merging this PR. [ Full content available at: https://github.com/apache/cloudstack/pull/2585 ] This message was relayed via gitbox.apache.org for [email protected]
