Uh oh!
There was an error while loading. Please reload this page.
[4.9] CLOUDSTACK-9692: Fix password server issue in redundant VRs - #1871
Conversation
yadvr
commented
Dec 28, 2016
@ustcweizhou is there an existing marvin test to validate this? |
blueorangutan
commented
Dec 28, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
luhaijiao
commented
Dec 28, 2016
This function seems broken since 4.7. That's also one of the major reasons we are reluctant to upgrade to 4.9.x since we have quite a few clients using RVR now. I would suggest to review this and include this in 4.9.2 or 4.9.3 asap. Thanks ! |
blueorangutan
commented
Dec 28, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-432 |
yadvr
commented
Dec 28, 2016
@blueorangutan test |
blueorangutan
commented
Dec 28, 2016
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
blueorangutan
commented
Dec 28, 2016
Trillian test result (tid-744)
|
The password server in RVRs has wrong parameters as the gateway of guest nics is None. In this case, we should get the gateway from /var/cache/cloud/cmdline.
e3b815f to
066b374Compareustcweizhou
commented
Dec 30, 2016
@rhtyd Could you please run the test again ? |
yadvr
commented
Jan 3, 2017
@blueorangutan package |
blueorangutan
commented
Jan 3, 2017
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Jan 3, 2017
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-441 |
yadvr
commented
Jan 3, 2017
@blueorangutan test |
blueorangutan
commented
Jan 3, 2017
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
luhaijiao
commented
Jan 14, 2017
is this ready to be merged ? |
karuturi
commented
Feb 2, 2017
@rhtyd can you post the results of the previous test run? |
yadvr
commented
Feb 2, 2017
@karuturi looks like it failed, I'll re-kick it |
blueorangutan
commented
Feb 2, 2017
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Feb 2, 2017
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-464 |
yadvr
commented
Feb 2, 2017
@blueorangutan test |
blueorangutan
commented
Feb 2, 2017
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
karuturi
commented
Feb 2, 2017
@rhtyd thanks |
blueorangutan
commented
Feb 2, 2017
Trillian test result (tid-790)
|
borisstoyanov
commented
Feb 2, 2017
I think tests look good now. LGTM |
karuturi
commented
Feb 3, 2017
merging this now. |
[4.9] CLOUDSTACK-9692: Fix password server issue in redundant VRsThe password server in RVRs has wrong parameters as the gateway of guest nics is None. In this case, we should get the gateway from /var/cache/cloud/cmdline. This issue is caused by commit 45642b8 * pr/1871: CLOUDSTACK-9692: Fix password server issue in redundant VRs Signed-off-by: Rajani Karuturi <rajani.karuturi@accelerite.com>
The password server in RVRs has wrong parameters as the gateway of guest nics is None.
In this case, we should get the gateway from /var/cache/cloud/cmdline.
This issue is caused by commit 45642b8