[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-16 Thread sspans
Github user sspans commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1249#discussion_r47763613 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/checkrouter.sh --- @@ -17,21 +17,20 @@ # under the License. STATUS=UNKNOWN

[GitHub] cloudstack pull request: CLOUDSTACK-8906: /var/log/cloud/ doesn't ...

2015-10-27 Thread sspans
Github user sspans commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/883#discussion_r43131260 --- Diff: scripts/vm/hypervisor/xenserver/logrotate --- @@ -0,0 +1,29 @@ +#!/bin/bash +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-8715: qemu-guest-agent s...

2015-11-04 Thread sspans
Github user sspans commented on the pull request: https://github.com/apache/cloudstack/pull/1027#issuecomment-153667126 Consistent Snapshots are not possible without freeze/thaw via the guest-agent. I think most people want good snapshots of their virtual machines. That

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

2015-07-29 Thread sspans
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

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

2015-07-30 Thread sspans
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

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

2015-07-30 Thread sspans
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 r

[GitHub] cloudstack pull request: Update NiciraNvpGuestNetworkGuru.java

2015-07-30 Thread sspans
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