[GitHub] cloudstack issue #1666: CLOUDSTACK-9480, CLOUDSTACK-9495 fix egress rule inc...

2016-09-21 Thread murali-reddy
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...

2016-09-21 Thread cloudmonger
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...

2016-09-20 Thread blueorangutan
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...

2016-09-19 Thread karuturi
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...

2016-09-19 Thread karuturi
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...

2016-09-19 Thread murali-reddy
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.
---