vr: fix vr in unknown state (more)#3848
Conversation
|
code lgtm but it is only an improvement, is that right @weizhouapache or does it really solve a bug? |
|
@DaanHoogland I believe there are some issues caused by this when vpc has more than 10 tiers. The change on log level can be regarded as an improvement. |
yadvr
left a comment
There was a problem hiding this comment.
LGTM, the syntax to show dev is correct
|
@blueorangutan package |
|
@blueorangutan package |
|
@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-682 |
grep eth10 or -eth11 etc, got it thanks @weizhouapache |
|
@blueorangutan test |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-843)
|
This fixes similar issue with apache#3465. Meanwhile change log level of CsHelper.execute2 from DEBUG to INFO and fix some typo.
Description
This fixes similar issue with #3465
Meanwhile change log level of CsHelper.execute2 from DEBUG to INFO and fix some typo
Types of changes
Screenshots (if appropriate):
How Has This Been Tested?