CLOUDSTACK-9756: Configure to ignore the ipassoc failure#1917
Conversation
|
Travis CI test_outofbandmanagement is failing which is not related to this PR changes. It seems there is problem with the test case. Traceback (most recent call last): File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 596, in process_request_thread File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 331, in finish_request File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 652, in init File "/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 436, in handle File "/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 112, in handle File "/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 143, in initiate_session error: unpack requires a string argument of length 2 Traceback (most recent call last): File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 596, in process_request_thread File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 331, in finish_request File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 652, in init File "/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 436, in handle File "/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 112, in handle File "/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 143, in initiate_session error: unpack requires a string argument of length 2 |
|
@jayapalu thanks, yes it seems an intermittent issue with |
| "If true, when account has dedicated public ip range(s), once the ips dedicated to the account have been consumed ips will be acquired from the system pool", | ||
| true, ConfigKey.Scope.Account); | ||
|
|
||
| static final ConfigKey<Boolean> RulesContinueOnError = new ConfigKey<Boolean>("Advanced", Boolean.class, "network.rule.delete.ignoreerror", "true", |
There was a problem hiding this comment.
The default for RulesContinueOnError is set to true, means the errors will be ignored.
Shouldn't the default be set to false ?
There was a problem hiding this comment.
To continue with the existing behavior it is set to true. If some one want to enable they can set to false.
ACS CI BVT RunSumarry: Link to logs Folder (search by build_no): https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0 Failed tests: Skipped tests: Passed test suits: |
|
Code LGTM |
|
There are two LGTMs and there are not test failures in the test results. So marking tag:mergeready |
|
@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-667 |
|
@blueorangutan test |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-1024)
|
On failure of ip assoc in back end (In VR), added config option to ignore or not to ignore failure. Using this failures won't be ignored and ip address will not set to free in DB.