CLOUDSTACK-9821: Fixed issue in deploying vm in basic zone#1991
Conversation
| tmpname = 'ipsetqzvxtmp' | ||
| try: | ||
| util.pread2(['/bin/bash', '-c', 'ipset -N ' + tmpname + type]) | ||
| util.pread2(['/bin/bash', '-c', 'ipset', '-N ', tmpname , type]) |
There was a problem hiding this comment.
is the space after "-N". "-F" . "-X" needed ?
|
code LGTM. not tested. |
|
I am testing it again. Please wait till I update |
|
Updated the patch. Tested it no issues found. |
|
@borisstoyanov can you start the tests on this? |
|
basic zone is not supported currently with BO @karuturi |
|
I manually verified this. merging this now. |
CLOUDSTACK-9821: Fixed issue in deploying vm in basic zoneFixed issue in deploying vm in basic zone. There is issue in ipset command with xenserver 6.5. In util.pread2 ipset and -N is passed as single string and it caused the issue in command failure. util.pread2(['/bin/bash', '-c', 'ipset', '-N ', tmpname , type]) * pr/1991: CLOUDSTACK-9821: Fixed issue in deploying vm in basic zone Signed-off-by: Rajani Karuturi <rajani.karuturi@accelerite.com>
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: |
6 participants
Fixed issue in deploying vm in basic zone.
There is issue in ipset command with xenserver 6.5. In util.pread2 ipset and -N is passed as single string and it caused the issue in command failure.
util.pread2(['/bin/bash', '-c', 'ipset', '-N ', tmpname , type])