[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1666 @cloudmonger thanks test_extendPhysicalNetworkVlan failed due to 'physicalnetwork 200 has allocated vnets in the range 734-735'. Possibly due to stale network that was not cleaned by one of the tests. Ran test manually, it passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1666 ### ACS CI BVT Run **Sumarry:** Build Number 101 Hypervisor xenserver NetworkType Advanced Passed=102 Failed=1 Skipped=4 _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0 **Failed tests:** * test_non_contigiousvlan.py * test_extendPhysicalNetworkVlan Failing since 2 runs **Skipped tests:** test_vm_nic_adapter_vmxnet3 test_static_role_account_acls test_3d_gpu_support test_deploy_vgpu_enabled_vm **Passed test suits:** test_deploy_vm_with_userdata.py test_affinity_groups_projects.py test_portable_publicip.py test_over_provisioning.py test_global_settings.py test_scale_vm.py test_service_offerings.py test_routers_iptables_default_policy.py test_loadbalance.py test_routers.py test_reset_vm_on_reboot.py test_snapshots.py test_deploy_vms_with_varied_deploymentplanners.py test_network.py test_router_dns.py test_login.py test_deploy_vm_iso.py test_list_ids_parameter.py test_public_ip_range.py test_multipleips_per_nic.py test_regions.py test_affinity_groups.py test_network_acl.py test_pvlan.py test_volumes.py test_ssvm.py test_nic.py test_deploy_vm_root_resize.py test_resource_detail.py test_secondary_storage.py test_vm_life_cycle.py test_disk_offerings.py --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1666 Packaging result: âcentos6 âcentos7 âdebian. JID-16 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1666 I think in the latest code change, `fwr` is not defined before and hence `fwr +=` operation will fail. Also, the ICMP rule is still appended and not Inserted in the beginning. Is that ok? (I dont have enough iptables knowledge.) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1666 @murali-reddy recurring failures will be listed as failing since last n runs. The router related tests(lb, network ops) are failing due to this PR. Are the tests running(manual or BVT) at your end after the new changes? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1666 @jburwell Jenkins was failing unrelated unit-test failure. I have triggered new Jenkins build which has passed. Changed the title of the PR to reflect that its fixing issues CLOUDSTACK-9480, CLOUDSTACK-9495. Will send a new PR for 4.8. @karuturi @cloudmonger failed tests seems to have failed in build #99 as well. Is it environment issue? Failure in test_vm_snapshots.py are totally unrelated to changes in the fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---