Uh oh!
There was an error while loading. Please reload this page.
CLOUDSTACK-9498: VR CsFile search utility methods fail when search st… - #1680
Conversation
jburwell
commented
Sep 22, 2016
LGTM for code |
abhinandanprateek
left a comment
There was a problem hiding this comment.
@murali-reddy the changes look good
LGTM for PR
cloudmonger
commented
Sep 23, 2016
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: |
murali-reddy
commented
Sep 23, 2016
@cloudmonger error seems to be due to 'physicalnetwork 200 has allocated vnets in the range 731-732' This is same error seen in test run. After test run in done, please see if there is network that was not cleaned and reason for that. This patch is contained to VPN user add/delete functionality, unrelated to the error. |
jburwell
commented
Sep 26, 2016
@blueorangutan package |
blueorangutan
commented
Sep 26, 2016
@jburwell a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
jburwell
commented
Sep 26, 2016
@murali-reddy could you please add and/or update Marvin test cases to verify this fix? |
blueorangutan
commented
Sep 26, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-26 |
murali-reddy
commented
Sep 27, 2016
@jburwell changed the existing marvin test to use password as "p@a#s=s.w_o-r+d", which will cover the all chars permitted for VPN user password |
yadvr
commented
Oct 21, 2016
@murali-reddy can you check if changes to the test_data will work in general |
blueorangutan
commented
Oct 21, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Oct 21, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-85 |
…ring has 're' meta chars, and causing VPN user add/deelte to fail -there is no real use of python 're' in CsFile.py utility methods searchString, deleteLine Replacing with regular string search instead. -modifying the smoke test for VPN user add/delete to have all permissable chars
murali-reddy
commented
Oct 28, 2016
@blueorangutan package |
blueorangutan
commented
Oct 28, 2016
@murali-reddy a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Oct 28, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-113 |
murali-reddy
commented
Oct 28, 2016
@blueorangutan test |
blueorangutan
commented
Oct 28, 2016
@murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
murali-reddy
commented
Oct 31, 2016
@blueorangutan test |
blueorangutan
commented
Oct 31, 2016
@murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
blueorangutan
commented
Oct 31, 2016
Trillian test result (tid-228)
|
jburwell
commented
Nov 1, 2016
@blueorangutan test centos6 vmware55u3 |
blueorangutan
commented
Nov 1, 2016
@jburwell a Trillian-Jenkins test job (centos6 mgmt + vmware55u3) has been kicked to run smoke tests |
yadvr
commented
Nov 1, 2016
The job failed due to systemvms failed to come up, I'll re-kick this. |
blueorangutan
commented
Nov 1, 2016
Trillian test result (tid-233)
|
@murali-reddy the ( |
yadvr
commented
Nov 20, 2016
@blueorangutan package |
blueorangutan
commented
Nov 20, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Nov 20, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-197 |
yadvr
commented
Nov 23, 2016
LGTM. Merging this based on review and tests. |
CLOUDSTACK-9498: VR CsFile search utility methods fail when search stThere is no real use of python 're' module in CsFile.py utility methods searchString, deleteLine. Regular string search is sufficient. These methods are used only for VPN user add/delete. Since VPN user password can have python 're' module meta characters, it interfere with search functionality. Replacing re.search() with regular string search instead. Change is confined to VPN add/delete users. Have run the test/integration/component/test_vpn_users.py VPN remote access user limit tests ... === TestName: test_01_VPN_user_limit | Status : SUCCESS === ok Test create VPN when L2TP port in use ... === TestName: test_02_use_vpn_port | Status : SUCCESS === ok Test create NAT rule when VPN when L2TP enabled ... === TestName: test_03_enable_vpn_use_port | Status : SUCCESS === ok Test add new users to existing VPN ... === TestName: test_04_add_new_users | Status : SUCCESS === ok Test add duplicate user to existing VPN ... === TestName: test_05_add_duplicate_user | Status : SUCCESS === ok Test as global admin, add a new VPN user to an existing VPN entry ... === TestName: test_06_add_VPN_user_global_admin | Status : SUCCESS === ok Test as domain admin, add a new VPN user to an existing VPN entry ... === TestName: test_07_add_VPN_user_domain_admin | Status : SUCCESS === ok * pr/1680: CLOUDSTACK-9498: VR CsFile search utility methods fail when search string has 're' meta chars, and causing VPN user add/deelte to fail Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
There is no real use of python 're' module in CsFile.py utility methods searchString, deleteLine. Regular string search is sufficient. These methods are used only for VPN user add/delete. Since VPN user password can have python 're' module meta characters, it interfere with search functionality.
Replacing re.search() with regular string search instead.
Change is confined to VPN add/delete users. Have run the test/integration/component/test_vpn_users.py
VPN remote access user limit tests ... === TestName: test_01_VPN_user_limit | Status : SUCCESS ===
ok
Test create VPN when L2TP port in use ... === TestName: test_02_use_vpn_port | Status : SUCCESS ===
ok
Test create NAT rule when VPN when L2TP enabled ... === TestName: test_03_enable_vpn_use_port | Status : SUCCESS ===
ok
Test add new users to existing VPN ... === TestName: test_04_add_new_users | Status : SUCCESS ===
ok
Test add duplicate user to existing VPN ... === TestName: test_05_add_duplicate_user | Status : SUCCESS ===
ok
Test as global admin, add a new VPN user to an existing VPN entry ... === TestName: test_06_add_VPN_user_global_admin | Status : SUCCESS ===
ok
Test as domain admin, add a new VPN user to an existing VPN entry ... === TestName: test_07_add_VPN_user_domain_admin | Status : SUCCESS ===
ok