Uh oh!
There was an error while loading. Please reload this page.
CLOUDSTACK-8623: CPVM fails to start after MS is restarted during its… - #571
CLOUDSTACK-8623: CPVM fails to start after MS is restarted during its…#571koushik-das wants to merge 2 commits into
Conversation
… initial start-up process If SSVM cannot be started then the existing entry is removed and a new SSVM gets created. Made similar changes for CPVM as well. Also cleaned up some log messages in SSVM and CPVM manager code.
asfbot
commented
Jul 9, 2015
cloudstack-pull-rats #25 SUCCESS |
asfbot
commented
Jul 9, 2015
cloudstack-pull-requests #720 SUCCESS |
kishankavala
commented
Jul 10, 2015
LGTM. |
karuturi
commented
Jul 10, 2015
Can you add some unit tests or marvin tests? |
yadvr
commented
Jul 10, 2015
LGTM. |
DaanHoogland
commented
Jul 10, 2015
I would like to see tests with such a change as well |
… initial start-up process Added unit tests
koushik-das
commented
Jul 13, 2015
@karuturi@DaanHoogland Added unit tests for CPVM allocation. |
asfbot
commented
Jul 13, 2015
cloudstack-pull-rats #37 ABORTED |
DaanHoogland
commented
Jul 13, 2015
test logic makes sense; LGTM waiting for travis |
asfbot
commented
Jul 13, 2015
cloudstack-pull-requests #733 UNSTABLE |
koushik-das
commented
Jul 14, 2015
Although RAT build status is showing ABORTED, the report is clean. Also the build failed due to some random test failure. Since travis is green, I am going ahead with the merge. |
… initial start-up process Added unit tests This closesapache#571
koushik-das
commented
Jul 14, 2015
Manually closing PR. I did a rebase to do away with unnecessary merge commit. |
… initial start-up process Added unit tests This closesapache#571
… initial start-up process
If SSVM cannot be started then the existing entry is removed and a new SSVM gets created. Made similar changes for CPVM as well.
Also cleaned up some log messages in SSVM and CPVM manager code.