vr: Fix vpc router in UNKNOWN state#3465
Conversation
|
@blueorangutan package |
|
@blueorangutan package |
borisstoyanov
left a comment
There was a problem hiding this comment.
This LGTM, but it needs to be verified with marvin smoketests
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-83 |
|
@blueorangutan test |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
@blueorangutan test |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
I've tried kicking at least 5 environments/builds now. For some reason the systemvms fail to start with this PR, can you check @ustcweizhou |
|
Let try again against master |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
@rhtyd if it still fails I will update the PR to use "grep -w eth0/eth1" |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-104 |
|
Okay @ustcweizhou let me test one more time. |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
If there are more than 10 vpc tiers or public ip subnets in a VPC, eth1X will be added in vpc router. The redundant state is UNKNOWN in this case.
940724e to
3c8655e
Compare
|
@rhtyd I have updated the code to use "grep -w" |
|
@blueorangutan package |
|
@ustcweizhou a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-108 |
|
@blueorangutan test |
|
Trillian test result (tid-127)
|
|
@ustcweizhou the previous show dev syntax LGTM, can you put that back in? |
|
Tests lgtm, I'll rekick one more time. |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-109 |
|
@rhtyd ok |
|
If the next run passes or does not have common errors from the previous run, we should merge this. |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-135)
|
This fixes similar issue with #3465. Meanwhile change log level of CsHelper.execute2 from DEBUG to INFO and fix some typo.
This fixes similar issue with apache#3465. Meanwhile change log level of CsHelper.execute2 from DEBUG to INFO and fix some typo.
Description
If there are more than 10 vpc tiers or public ip subnets in a VPC, eth1X will be added in vpc router.
The redundant state is UNKNOWN in this case.
Types of changes
How Has This Been Tested?
creaete more than 10 vpc tiers, then vpc router will be UNKNOWN
it can be fixed by this commit.