Uh oh!
There was an error while loading. Please reload this page.
CLOUDSTACK-9502: DS template copies don’t get deleted in VMware ESXi with multiple clusters and zone wide storage (include CLOUDSTACK-9386 into 4.9 release branch) - #1676
Conversation
69854e9 to
d7c10fcCompare…with multiple clusters and zone wide storage
d7c10fc to
4104ceaCompare@jburwell We just re-run integration tests. The same environment as 1560. |
yadvr
commented
Sep 20, 2016
@blueorangutan package |
blueorangutan
commented
Sep 20, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Sep 20, 2016
Packaging result: ✔centos6 ✖centos7 ✖debian. JID-6 |
blueorangutan
commented
Sep 21, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-14 |
jburwell
commented
Sep 28, 2016
LGTM for code review. We are awaiting the results of failures in #1692 to merge smoke tests fixes. When that is completed, we will rebase this PR and execute smoke tests. Upon passage, we will merge this PR. |
serg38
commented
Sep 29, 2016
yadvr
commented
Oct 21, 2016
@blueorangutan package |
blueorangutan
commented
Oct 21, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Oct 21, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-82 |
nvazquez
commented
Oct 31, 2016
Hi @rhtyd @jburwell could this be merged? |
jburwell
commented
Oct 31, 2016
@blueorangutan test centos6 vmware55u3 |
blueorangutan
commented
Oct 31, 2016
@jburwell a Trillian-Jenkins test job (centos6 mgmt + vmware55u3) has been kicked to run smoke tests |
jburwell
commented
Oct 31, 2016
@nvazquez kicking regression tests now. If they pass, then I will add my LGTM and merge. |
nvazquez
commented
Oct 31, 2016
Thanks! @jburwell |
jburwell
commented
Nov 1, 2016
@rhtyd can you check into the status of the test run? |
yadvr
commented
Nov 1, 2016
@blueorangutan help @jburwell incorrect hypervisor name provided, as replied earlier bo does not check the passed arguments. I'll fix this. |
blueorangutan
commented
Nov 1, 2016
@rhtyd I understand these words: "help", "hello", "thanks", "package", "test" |
yadvr
commented
Nov 1, 2016
Tests have been manually kicked on this now. |
blueorangutan
commented
Nov 1, 2016
Trillian test result (tid-234)
|
jburwell
commented
Nov 1, 2016
@nvazquez could you please investigate the snapshot test failures? @murali-reddy could you please investigate the VR test failures? These failures appear similar to failures seen on #1681 and #1680? |
serg38
commented
Nov 1, 2016
@jburwell This is most likely blueorangutan issue. This code is already merged into Master via PR1560. We can even cancel this PR and leave it in 4.10 only. |
jburwell
commented
Nov 1, 2016
@serg38 it may be environmental, but those tests are passing on other PRs in the same environment configuration. Therefore, I would like to determine why it is breaking here sand not elsewhere. |
jburwell
commented
Nov 1, 2016
@serg38 on what PR was this code merged to master? When we merge this PR, we need to revert and properly forward merge this fix in order to maintain traceability across release branches. |
serg38
commented
Nov 1, 2016
@blueorangutan test vmware-55u3 |
serg38
commented
Nov 1, 2016
@jburwell PR1560. At this point it might be make sense to leave it for 4.10 and not to forward merge it. I checked blueorangutan log and without management server log it is impossible to determine exact reason for the exception. Most likely you had a hypervisor in a failed/disconnected state during the test "Exception: Job failed: {jobprocstatus : 0, created : u'2016-11-01T16:12:54+0000', jobresult : {errorcode : 530, errortext : u'Resource [Host:2] is unreachable: Host 2: Unable to start instance due to Unable to start VM:529663de-7158-4303-9948-5c3ea9f939cc due to error in finalizeStart, not retrying'}, cmd : u'org.apache.cloudstack.api.command.admin.router.RebootRouterCmd', userid : u'896fea80-a00e-11e6-a5b5-0629ca010832', jobstatus : 2, jobid : u'ff89481d-9894-480f-8b15-b678b91baf27', jobresultcode : 530, jobinstanceid : Can we kick off test again? |
murali-reddy
commented
Nov 3, 2016
@jburwell VR failures seen are intermittent failures we are seeing w.r.t VPC static routes, RVPC VR in other PR's as well. Not related to the patch. |
jburwell
commented
Nov 7, 2016
@murali-reddy only the following failures are expected:
The test failures, particularly around storage, are a bit concerning and need to be explained/fixed before merging. I am kicking off another round of tests to help determine whether or not these issues are regressions in the management server (potentially from another commit) or environmental. @blueorangutan test centos6 vmware-55u3 |
blueorangutan
commented
Nov 7, 2016
@jburwell a Trillian-Jenkins test job (centos6 mgmt + vmware-55u3) has been kicked to run smoke tests |
blueorangutan
commented
Nov 8, 2016
Trillian test result (tid-299)
|
serg38
commented
Nov 8, 2016
@jburwell Non-VPC tests on VmWare passed this time. This one seems to be ready for merging. |
serg38
commented
Nov 17, 2016
CLOUDSTACK-9502: DS template copies dont get deleted in VMware ESXi with multiple clusters and zone wide storage (include CLOUDSTACK-9386 into 4.9 release branch)Include #1560 into 4.9 release branch * pr/1676: CLOUDSTACK-9502: DS template copies don’t get deleted in VMware ESXi with multiple clusters and zone wide storage Signed-off-by: John Burwell <meaux@cockamamy.net>
Include #1560 into 4.9 release branch