[GitHub] cloudstack pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-25 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/1452


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-25 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-214211132
  
@swill I think this one has enough LGTM and testing and the only question I 
see remaining has been answerred. @pedro-martins is not responded anymore so 
let's merge.


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-25 Thread prashanthvarma
Github user prashanthvarma commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-214209793
  
@DaanHoogland, We haven't heard from @pedro-martins in a while. Let's hope, 
he responds soon (or) someone form the community reviews our code.


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211453267
  
@pedro-martins can you live with the present state?


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-18 Thread swill
Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211370314
  
I think this is ready pending 1 more LGTM code review...


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread prashanthvarma
Github user prashanthvarma commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-26802
  
@DaanHoogland That's a good suggestion. We will discuss on the optimal 
location in the plugin's project directory, and incorporate them in the next PR.

Thank you for the review and suggestions !!


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211099345
  
@swill LGTM based on testing and code walkthrough


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211099195
  
@prashanthvarma maybe you can incorporate it in the project dir for the 
plugin somehow. It seems to me to be the best place for it. maybe accompanied 
with a readme on how to install/test?


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread prashanthvarma
Github user prashanthvarma commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211092880
  
@DaanHoogland you are right, that is the main motive :). Let me know, if 
you want me to publish it elsewhere as well for easier user access.

Thank you !!


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211060040
  
thanks @prashanthvarma , mostly usefull for users of your sdn solution. 
Hopefully some of them will test ;)


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread prashanthvarma
Github user prashanthvarma commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211044426
  
@DaanHoogland Sure, here is an example Nuage VSP SDN plugin specific Marvin 
tests config file contents:

{
"zones": [
{
"name": "MyZone",
"guestcidraddress": "10.1.1.0/24", 
"dns2": "8.8.6.6", 
"dns1": "10.10.0.10", 
"physical_networks": [
{
"broadcastdomainrange": "Zone", 
"isolationmethods": [
"VLAN"
], 
"name": "Management Network", 
"traffictypes": [
{
"kvm": "mgmtbr0", 
"typ": "Management"
}, 
{
"kvm": "mgmtbr0", 
"typ": "Public"
}, 
{
"kvm": "mgmtbr0", 
"typ": "Storage"
}
], 
"providers": [
{
"broadcastdomainrange": "ZONE", 
"name": "VirtualRouter"
}
]
}, 
{
"broadcastdomainrange": "Zone", 
"isolationmethods": [
"VSP"
], 
"name": "Nuage Network", 
"traffictypes": [
{
"kvm": "alubr0", 
"typ": "Guest"
}
], 
"providers": [
{
"broadcastdomainrange": "ZONE", 
"name": "VirtualRouter"
}, 
{
"broadcastdomainrange": "ZONE", 
"name": "NuageVsp", 
"devices": [
{
"username": "csproot", 
"retryinterval": "60", 
"hostname": "10.30.35.129",
"apiversion": "v3_2", 
"retrycount": "4", 
"password": "csproot", 
"port": 8443
}
]
}, 
{
"broadcastdomainrange": "ZONE", 
"name": "VpcVirtualRouter"
}, 
{
"broadcastdomainrange": "ZONE", 
"name": "InternalLbVm"
}
]
}
], 
"securitygroupenabled": "false", 
"ipranges": [
{
"startip": "10.29.1.1", 
"endip": "10.29.9.255", 
"netmask": "255.255.0.0", 
"gateway": "10.29.0.1"
}
], 
"networktype": "Advanced", 
"pods": [
{
"endip": "10.20.15.255", 
"name": "P0", 
"startip": "10.20.8.0", 
"netmask": "255.255.0.0", 
"clusters": [
{
"clustername": "P0C0", 
"hypervisor": "kvm", 
"hosts": [
{
"username": "root", 
"url": "http://test-kvm";, 
"password": "password"
}
], 
"clustertype": "CloudManaged", 
"primaryStorages": [
{
"url": "nfs://10.20.128.14/primary", 
"name": "P0C0-primaryStorage"
}
]
}

[GitHub] cloudstack pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211038147
  
@prashanthvarma  can you publish nuage_ant.cfg?


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-211032986
  
did the tests and again two failures, replay succeeded


[1452.results.network.txt](https://github.com/apache/cloudstack/files/222681/1452.results.network.txt)

[1452.results.vpc_routers.txt](https://github.com/apache/cloudstack/files/222683/1452.results.vpc_routers.txt)


[1452.results.routers_network_ops.txt](https://github.com/apache/cloudstack/files/222682/1452.results.routers_network_ops.txt)

```
# ssh 192.168.23.5
The authenticity of host '192.168.23.5 (192.168.23.5)' can't be established.
ECDSA key fingerprint is a6:38:aa:5b:07:f9:53:8c:c0:57:19:c7:8f:e4:f1:a7.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added '192.168.23.5' (ECDSA) to the list of known 
hosts.
root@192.168.23.5's password: 
# ping -c 3 8.8.8.8
PING 8.8.8.8 (8.8.8.8): 56 data bytes
64 bytes from 8.8.8.8: seq=0 ttl=46 time=24.347 ms
64 bytes from 8.8.8.8: seq=1 ttl=46 time=23.389 ms
64 bytes from 8.8.8.8: seq=2 ttl=46 time=26.818 ms

--- 8.8.8.8 ping statistics ---
3 packets transmitted, 3 packets received, 0% packet loss
round-trip min/avg/max = 23.389/24.851/26.818 ms
```


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-16 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-210849171
  
started the suite now but keep in mind that no sdn specific tests are done 
in it, @swill  


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-15 Thread swill
Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-210646612
  
It seems to be in pretty good order.  I would like some code reviews and 
since there are changes to logic outside of the tests, I would like a full CI 
run against it.  @DaanHoogland are you setup to run at least the networking 
stuff against this?


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-15 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-210643703
  
@KrisSterckx I think so but not sure. @swill can you put this in?


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-04-15 Thread KrisSterckx
Github user KrisSterckx commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-210464551
  
Hi @DaanHoogland  is this PR still in the candidate list of 4.9.0 ?

If there is anything that I & the team can facilitate in doing, pls let me 
know.

Thanks, Kris



---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-03-31 Thread nlivens
Github user nlivens commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-203794406
  
@pedro-martins, I do like it, but we didn't change that code, we just 
restructured it a bit so I didn't want to touch the original code, I can still 
make that change though, that's not an issue :)


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-03-30 Thread pedro-martins
Github user pedro-martins commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-203581569
  
@nlivens great! And sorry for the delay =)
I saw that you used the String.format as I suggested, but and about the 
using of CollectionUtils.isEmpty() in the class 
org.apache.cloudstack.network.element.InternalLoadBalancerElement.java at line 
344? Did you don't like the suggestion or you just has forgotten to change?

Thanks !


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-03-29 Thread nlivens
Github user nlivens commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-202859071
  
@pedro-martins Updated the PR


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-03-27 Thread pedro-martins
Github user pedro-martins commented on the pull request:

https://github.com/apache/cloudstack/pull/1452#issuecomment-202163598
  
Hi @prashanthvarma.

How about to use String.format() to create the strings in the loggers? The 
use of String format will turn the strings in the logs more legible.

Also, at line 344, could you use Collection.isEmpty(internalLbVms) instead 
of "internalLbVms == null || internalLbVms.isEmpty()" ?

Ty. 


---
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 pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-03-23 Thread prashanthvarma
GitHub user prashanthvarma opened a pull request:

https://github.com/apache/cloudstack/pull/1452

CLOUDSTACK-9322: Support for Internal LB fuctionality with Nuage VSP SDN 
Plugin including Marvin tests

Task: https://issues.apache.org/jira/browse/CLOUDSTACK-9322

PR contents:
1) UI changes to support LB provider “InternalLbVm” during VPC offering 
creation.
2) Bug fix for CLOUDSTACK-9320.
3) Nuage VSP SDN Plugin related enhancements for VPC network functionality.
4) Marvin test coverage for Internal LB support on master with Nuage VSP 
SDN Plugin.
5) Enhancements on our exiting Marvin test code (nuagevsp plugins 
directory).
6) PyFlakes & PEP8 compliance with our Marvin test code.

Test run:
CloudStack$ nosetests --with-marvin --marvin-config=nuage_ant.cfg 
test/integration/plugins/nuagevsp/ -a tags=nuagevsp

Test results:
Test user data and password reset functionality with Nuage VSP SDN plugin 
... === TestName: test_nuage_UserDataPasswordReset | Status : SUCCESS ===
ok
Test Nuage VSP VPC Offering with different combinations of LB service 
providers ... === TestName: test_01_nuage_internallb_vpc_Offering | Status : 
SUCCESS ===
ok
Test Nuage VSP VPC Network Offering with and without Internal LB service 
... === TestName: test_02_nuage_internallb_vpc_network_offering | Status : 
SUCCESS ===
ok
Test Nuage VSP VPC Networks with and without Internal LB service ... === 
TestName: test_03_nuage_internallb_vpc_networks | Status : SUCCESS ===
ok
Test Nuage VSP VPC Internal LB functionality with different combinations of 
Internal LB rules ... === TestName: test_04_nuage_internallb_rules | Status : 
SUCCESS ===
ok
Test Nuage VSP VPC Internal LB functionality by performing (wget) traffic 
tests within a VPC ... === TestName: test_05_nuage_internallb_traffic | Status 
: SUCCESS ===
ok
Test Nuage VSP VPC Internal LB functionality with different LB algorithms 
by performing (wget) traffic tests ... === TestName: 
test_06_nuage_internallb_algorithms_traffic | Status : SUCCESS ===
ok
Test Nuage VSP VPC Internal LB functionality with restarts of VPC network 
components by performing (wget) ... === TestName: 
test_07_nuage_internallb_vpc_network_restarts_traffic | Status : SUCCESS ===
ok
Test Nuage VSP VPC Internal LB functionality with InternalLbVm appliance 
operations by performing (wget) ... === TestName: 
test_08_nuage_internallb_appliance_operations_traffic | Status : SUCCESS ===
ok
Test Basic VPC Network Functionality with Nuage VSP SDN plugin ... === 
TestName: test_nuage_vpc_network | Status : SUCCESS ===
ok
Test Nuage VSP SDN plugin with basic Isolated Network functionality ... === 
TestName: test_nuage_vsp | Status : SUCCESS ===
ok

--
Ran 11 tests in 12094.705s

OK

Test run logs:



PEP8 & PyFlakes Compliance:
CloudStack$ pep8 --max-line-length=150 
test/integration/plugins/nuagevsp/*.py
CloudStack$ pyflakes test/integration/plugins/nuagevsp/nuageTestCase.py
CloudStack$ pyflakes 
test/integration/plugins/nuagevsp/test_nuage_password_reset.py
CloudStack$ pyflakes 
test/integration/plugins/nuagevsp/test_nuage_vpc_internal_lb.py
CloudStack$ pyflakes 
test/integration/plugins/nuagevsp/test_nuage_vpc_network.py
CloudStack$ pyflakes test/integration/plugins/nuagevsp/test_nuage_vsp.py
CloudStack$ pyflakes test/integration/plugins/nuagevsp/*.py

#CLOUDSTACK-9322


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/prashanthvarma/cloudstack master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1452.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1452


commit be67e5cf0649e906291883d974873fb9f49aaff2
Author: Nick Livens 
Date:   2016-02-18T10:03:34Z

CLOUDSTACK-9322 : Changes to support InternalLbVm with Nuage VSP plugin

commit 7f811d9962e4112ac789129c3e73c4652db00c54
Author: Nick Livens 
Date:   2016-03-21T13:34:18Z

CLOUDSTACK-9320 : InternalLBVM is not getting destroyed when the last 
Internal Load Balancer rule is removed for the corresponding source IP address

commit 12085aae2caa4562dd7740d857e21f746d5a7748
Author: Prashanth Manthena 
Date:   2016-03-23T14:59:41Z

CLOUDSTACK-9322 : Marvin tests for Internal Lb with Nuage VSP




---
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.
---