[GitHub] cloudstack pull request #1689: Switched to the official SolidFire SDK for Py...

2016-09-23 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/1689 Switched to the official SolidFire SDK for Python SolidFire has recently released an official SDK for Python. I have converted all of the SolidFire integration tests over to maki

[GitHub] cloudstack issue #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-09-23 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1451 @insom do not worry, actually I am +1 with this PR. I agree with what you said. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] cloudstack issue #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-09-23 Thread insom
Github user insom commented on the issue: https://github.com/apache/cloudstack/pull/1451 @ustcweizhou A _side effect_ of this PR is setting the minimal timeout to 120, but that's what the original code intended - except that because of not passing through the timeout to `applyConfigTo

[GitHub] cloudstack issue #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-09-23 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1451 many people thought that changing router.aggregation.command.each.timeout in global setting will increase the timeout. Actually it is not, at least for KVM. for KVM hypervisors, we

Re: vmware macchinina template

2016-09-23 Thread Nux!
Hi, I am away for a few days, won't have a chance to look at this till next week. There should be an 'old' dir on the same location, can you give that a try, see if it has the driver? -- Sent from my mobile device. Please excuse my brevity. On 23 September 2016 13:05:26 CEST, Paul Angus wrot

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-23 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 I got it to build, will be testing it soon... --- 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 feat

[GitHub] cloudstack pull request #1688: Marvin Tests: Fix VPC network offering select...

2016-09-23 Thread PaulAngus
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1688 Marvin Tests: Fix VPC network offering selection It appears that original code did not expect there to be an alternate default VPC offering (Nuage) . VPC selection uses isdefault=True whi

[GitHub] cloudstack pull request #1687: Marvintests vpcofferingfix

2016-09-23 Thread PaulAngus
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1687 --- 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

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Trillian test result (tid-47) Environment: vmware-55u3 (x2), Advanced Networking Total time taken: 31689s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1685 /cc @jburwell @karuturi this is an example PR to show various combinations of Trillian environments (hypervisors and mgmt server os). --- If your project is set up for it, you can reply to this e

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1684 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-24 --- 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 do

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Trillian test result (tid-45) Environment: vmware-60u2 (x2), Advanced Networking Total time taken: 29784s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Trillian test result (tid-46) Environment: kvm-centos6 (x2), Advanced Networking Total time taken: 28905s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 @abhinandanprateek this issue also affects 4.8. Would it be possible to [change the base branch](https://github.com/blog/2224-change-the-base-branch-of-a-pull-request) of this PR to 4.8? --

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1684 @jburwell a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 @blueorangutan package --- 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 wis

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 LGTM for code. @pdion891 could you re-execute your upgrade scenario to verify that CLOUDSTACK-9489 is fixed? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1687: Marvintests vpcofferingfix

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1687 @PaulAngus it looks like a bunch of commits got picked up inadvertently triggering a large number of unrelated changes. Ping me, and I will help reduce down to the one expected change. --- I

[GitHub] cloudstack issue #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-09-23 Thread insom
Github user insom commented on the issue: https://github.com/apache/cloudstack/pull/1451 I got contacted directly by someone else hit by this bug and looking for a fix. They saw I had commits relating to their problem. What needs to happen to get it merged? --- If your project is se

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Trillian test result (tid-43) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 26861s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1669 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-23 --- 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 do

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1669 @jburwell a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1669 @blueorangutan package --- 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 wis

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1669 LGTM based on 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

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-23 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1669 @milamberspace to pile onto @rhtyd's comment, I would like to get this change into 4.10.0.0 which freezes Sunday (25 Sept 2016). --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Trillian test result (tid-39) Environment: xenserver-65sp1 (x2), Advanced Networking Total time taken: 31973s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/down

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-23 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 @milamberspace since this changes the way translations are consumed, would you help review/test this? Thanks. FYI - this does not change how translation workflow works. --- If your project is se

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Migrate master to Spring 4.x

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Trillian test result (tid-41) Environment: kvm-centos6 (x2), Advanced Networking Total time taken: 28228s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Trillian test result (tid-44) Environment: xenserver-65sp1 (x2), Advanced Networking Total time taken: 18399s Marvin logs: https://github.com/blueorangutan/acs-prs/releases/down

[GitHub] cloudstack issue #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-23 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1686 @rhtyd I think you meant guest_os_type not guest_os_id. In this case the issue is not only that it returns ID instead of UUID but also that it doesn't return ID of os type. In the sample output o

[GitHub] cloudstack issue #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-23 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1686 @rhtyd Somehow it works fine without this patch curl "http://localhost:8096//client/api?command=listVirtualMachines&listall=true&response=json"; {"listvirtualmachinesresponse":{"count":1,

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-09-23 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1680 @cloudmonger error seems to be due to 'physicalnetwork 200 has allocated vnets in the range 731-732' This is same error seen in test run. After test run in done, please see if there is netw

[GitHub] cloudstack issue #1680: CLOUDSTACK-9498: VR CsFile search utility methods fa...

2016-09-23 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1680 ### ACS CI BVT Run **Sumarry:** Build Number 103 Hypervisor xenserver NetworkType Advanced Passed=102 Failed=1 Skipped=4 _Link to logs Folder (searc

vmware macchinina template

2016-09-23 Thread Paul Angus
Hi Nux, How are you doing? I'm fixing the marvin tests to point to http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-vmware.ova But Serg points out that the vmware template doesn't have an E1000 (intel pro/1000) driver in it so can't enable interfaces. Could you fix or let me know ho

[GitHub] cloudstack pull request #1687: Marvintests vpcofferingfix

2016-09-23 Thread PaulAngus
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1687 Marvintests vpcofferingfix It appears that original code did not expect there to be an alternate default VPC offering (Nuage) . VPC selection uses isdefault=True which selects the Nuage N

Re: Request for comments: Moving to Spring 4 and Java 8

2016-09-23 Thread John Burwell
All, Reviving this thread as it would be a Good Thing(tm) to get Spring 4 merged for 4.10.0.0. Based on Rohit’s original question, it appears that the only issue is F5 on Java8 (not on Java7). Is there someone available with F5 knowledge who can investigate it? If not, would it be acceptable

[GitHub] cloudstack issue #1663: [LTS/blocker] CLOUDSTACK-6432: Prevent DNS reflectio...

2016-09-23 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1663 @s-seitz the fix should work for newly deployed VRs, for existing VR there may be existing rules or chains blocking the intended fix. Are you getting issues or seeing the same behaviour with new V

[GitHub] cloudstack issue #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1686 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-22 --- 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 do

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-21 --- 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 do

[GitHub] cloudstack pull request #1686: CLOUDSTACK-9507: ListVM response's guest_os i...

2016-09-23 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1686 CLOUDSTACK-9507: ListVM response's guest_os id should be a UUID The listVirtualMachines API response contains a guest os_type_id field which is an integer, and therefore not useful or easily c

[GitHub] cloudstack issue #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1686 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208194 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtModifySshKeysCommandWrapper.java --- @@ -108,7 +108,7 @

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208106 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3229,7 +3230,7 @@ protected bool

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208121 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3255,7 +3256,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208151 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3300,7 +3301,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208295 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtPvlanSetupCommandWrapper.java --- @@ -87,7 +88,7 @@ pub

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208208 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtOvsCreateTunnelCommandWrapper.java --- @@ -40,14 +40,11

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208282 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtPvlanSetupCommandWrapper.java --- @@ -58,11 +59,11 @@ p

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208251 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtOvsVpcPhysicalTopologyConfigCommandWrapper.java --- @@

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208171 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3327,7 +3328,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208135 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3273,7 +3274,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208262 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtOvsVpcRoutingPolicyConfigCommandWrapper.java --- @@ -36

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208159 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3317,7 +3318,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208231 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtOvsDestroyTunnelCommandWrapper.java --- @@ -42,7 +42,7

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208089 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3197,7 +3198,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208062 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3150,7 +3151,7 @@ protected long

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208042 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -2973,7 +2974,7 @@ public Domain

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208075 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -3170,7 +3171,7 @@ public boolean

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80208027 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -2377,7 +2378,7 @@ public Type ge

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80207993 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -1319,7 +1320,7 @@ public Answer

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80207999 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -1373,7 +1374,7 @@ public synchro

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80207941 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -279,12 +280,12 @@

[GitHub] cloudstack pull request #1678: CLOUDSTACK-9503: Increased the VR script time...

2016-09-23 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1678#discussion_r80207975 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -1226,7 +1227,7 @@ private boolea

[GitHub] cloudstack issue #1663: [LTS/blocker] CLOUDSTACK-6432: Prevent DNS reflectio...

2016-09-23 Thread s-seitz
Github user s-seitz commented on the issue: https://github.com/apache/cloudstack/pull/1663 After patching our systemvm.iso and the respective routers, I've noticed the iptables rules changed as given in CsAddress.py. These rules don't get any packets since two identical (but unpatched

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack pull request #1685: IGNORE this PR

2016-09-23 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1685 IGNORE this PR This tests @blueorangutan and Trillian job as a ref for 4.9 branch. @blueorangutan package You can merge this pull request into a Git repository by running: $ git pul