[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-301980112 @rhtyd PR changes are related VR scripts. test_01_create_volume failure may be intermittent. Anyway in latest results test_01_create_volume is passed. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-301671593 The test cases failures are not only seen this PR in other PRs also these test failures are seen. Marking this PR for merge ready This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-301431379 @borisstoyanov Can you please comment on the test failures, I feel the test failures are because of the setup or test issues or intermittent failures. If you look at 'test_01_create_volume' I haven't done any changes related to that. 'test_04_rvpc_privategw_static_routes' it is failing in other PRs as well PR #1995. Can you look at failure tests so that I can mark merge ready. test_01_create_volumeFailure 189.58 test_volumes.py test_04_rvpc_privategw_static_routes Failure 915.18 test_privategw_acl.py test_03_vpc_privategw_restart_vpc_cleanupFailure 222.43 test_privategw_acl.py test_01_vpc_privategw_aclFailure 121.93 test_privategw_acl.py test_04_rvpc_network_garbage_collector_nics Error 136.77 test_vpc_redundant.py test_01_primary_storage_nfs Error 0.19test_primary_storage.py ContextSuite context=TestStorageTags>:setup Error 0.37 test_primary_storage.py ContextSuite context=TestDeployVmRootSize>:setup Error 0.00 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-301426965 @rhtyd Currently In this PR I have removed few incorrect/error iptables rules and logging the exit status of the commands. This will gives the errors in the VR logs. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-301382518 test_public_ip_admin_account is failed because of test setup issue. I have run the test locally it is passed. So marking merge-ready label. $ cat /tmp//MarvinLogs/test_network_WK9OKJ/results.txt Test for Associate/Disassociate public IP address for admin account ... === TestName: test_public_ip_admin_account | Status : SUCCESS === ok -- Ran 1 test in 72.100s OK This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-300395989 In my local test test_02_redundant_VPC_default_routes is passed. Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS === ok -- Ran 1 test in 795.766s OK ===In Trillian it is failred because the VPC creation failed.= Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : EXCEPTION === ERROR Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : FAILED === FAIL Create a redundant VPC with 1 Tier, 1 VM, 1 ACL, 1 PF and test Network GC Nics ... === TestName: test_04_rvpc_network_garbage_collector_nics | Status : FAILED === FAIL Create a redundant VPC with 1 Tier, 1 VM, 1 ACL, 1 PF and test Network GC Nics ... === TestName: test_05_rvpc_multi_tiers | Status : FAILED === FAIL === TestName: test_05_rvpc_multi_tiers | Status : EXCEPTION === ERROR == ERROR: Create a redundant VPC with two networks with two VMs in each network -- Traceback (most recent call last): File "/marvin/tests/smoke/test_vpc_redundant.py", line 318, in setUp domainid=self.account.domainid) File "/usr/lib/python2.7/site-packages/marvin/lib/base.py", line 4344, in create return VPC(apiclient.createVPC(cmd).__dict__) File "/usr/lib/python2.7/site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py", line 2048, in createVPC response = self.connection.marvinRequest(command, response_type=response, method=method) File "/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 379, in marvinRequest raise e Exception: Job failed: {jobprocstatus : 0, created : u'2017-05-04T17:28:42+', jobresult : {errorcode : 530, errortext : u'**Failed to create VPC**'}, cmd : u'org.apache.cloudstack.api.command.admin.vpc.CreateVPCCmdByAdmin', userid : u'42cfe558-308f-11e7-9381-06dcb4010722', jobstatus : 2, jobid : u'd2bf42c3-76f1-403d-81cc-a5f660859faa', jobresultcode : 530, jobresulttype : u'object', jobinstancetype : u'None', accountid : u'42cfd80c-308f-11e7-9381-06dcb4010722'} >> begin captured stdout << - === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : EXCEPTION === This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-300371558 I think few of the failures and errors are because of the test issues not because of changes. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-300144222 @borisstoyanov I have updated the PR. Can you please trigger the Trillian This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-299850524 Most of the test failures for because of the hidden bugs in iptables configuration. Please see the below logs from the VR, ACL_INBOUND_eth1 chain is not created. Added rule into this chain got failed. 2017-05-08 11:11:13,546 CsHelper.py execute2:209 Executing: iptables -t filter -I FORWARD -o eth1 -d 10.147.46.0/24 -j ACL_INBOUND_eth1 2017-05-08 11:11:13,551 configure.py main:1031 Exception while configuring router Traceback (most recent call last): File "/opt/cloud/bin/configure.py", line 1015, in main nf.compare(config.get_fw()) File "/opt/cloud/bin/cs/CsNetfilter.py", line 180, in compare raise Exception("iptables command got failed with error: %s " %error) Exception: iptables command got failed with error: == # cat /var/log/cloud.log | grep 'ACL_INBOUND_eth1' 2017-05-08 11:11:13,546 CsNetfilter.py compare:159 Checking if the rule already exists: rule={u'-d': u'10.147.46.0/24', u'-o': u'eth1', u'-A': u'FORWARD', u'-j': u'ACL_INBOUND_eth1'} table=filter chain=FORWARD 2017-05-08 11:11:13,546 CsNetfilter.py compare:164 Add: rule=-A FORWARD -o eth1 -d 10.147.46.0/24 -j ACL_INBOUND_eth1 table=filter 2017-05-08 11:11:13,546 CsHelper.py execute2:209 Executing: iptables -t filter -I FORWARD -o eth1 -d 10.147.46.0/24 -j ACL_INBOUND_eth1 root@r-36-QA:~# This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands
jayapalu commented on issue #2018: CLOUDSTACK-9848: Added exit status checking for the iptables commands URL: https://github.com/apache/cloudstack/pull/2018#issuecomment-298873490 few of the cleanup of test cases are failing and these are not related my changes. @borisstoyanov Can you trigger the tests again. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services