[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/636 --- 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

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126288404 @sspans Since Test case is updated to cover VXLAN also, LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126258728 ok --- 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 en

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user sspans commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126257887 The plugin would fail, similar to how it fails now when you ask ik to do vxlan when it only supports STT. The encapsulation type is something that you first config

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126250367 @sspans yes but they might be tempted to do some vxlanning after upgrading ACS not upgrading NSX . what would happen? --- If your project is set up for it, you

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user sspans commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126250153 Testing and a unit-test will still be added. We need this feature because we're planning to roll out new zones with VXLAN, which if the changes remain sma

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user sspans commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126247958 Daan: any supported release of vmware nsx will support VXLAN. People running older releases probably already have an STT setup running, and should not encounter pr

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-126047538 does every version of nvp support vxlan? --- 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

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/636#issuecomment-125961987 @sspans any testing done? --- 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 n

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

GitHub user sspans opened a pull request: https://github.com/apache/cloudstack/pull/636 Update NiciraNvpGuestNetworkGuru.java Add support for VXLAN as isolation method You can merge this pull request into a Git repository by running: $ git pull https://github.com/sspans/cloudst