[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1645 @kiwiflyer Noted. Looking at the code diff, I dont think it will cause any regressions. But, just wanted to run the tests once. --- If your project is set up for it, you can reply to this ema

Re: [DISCUSS] Replacing the VR

2016-09-15 Thread ilya
Hi folks, please kindly keep in mind the open source licensing when choosing the next router. AGPL or GPL v3 are "no go" for many shops. I could not find the license for Vyatta, i do know Vyatta dev folks were open to working with ACS few years back, but somehow the initiative was dropped. On 9/

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi I think John is just asking if we can run test_ssvm.py (a smoke test) on VMware and KVM. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell My test run is on NFS. I dont have managed storage. Are you running the tests on managed storage with KVM and VMWare? --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Oh @jburwell I thought you were asking for new regression tests to be created. I think you and I are on the same page now. --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1642 @mike-tutkowski the reason I would like the regression tests run is to confirm that assumption. --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: [GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread Tutkowski, Mike
I renamed it to “delay”, but I can’t remember now what I did that. ☺ On 9/15/16, 10:20 PM, "jburwell" wrote: Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r79104400 --- Diff: test/integratio

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-15 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1615 @nvazquez I will review the test case in the morning. In the interim, please squash your commits. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r79104522 --- Diff: test/integration/plugins/solidfire/TestManagedSystemVMs.py --- @@ -0,0 +1,585 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r79104400 --- Diff: test/integration/plugins/solidfire/TestManagedSystemVMs.py --- @@ -0,0 +1,585 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell At present, this is really only supported on XenServer. As long as our other integration tests don't fail due to anything in here, we should be OK. --- If your project is set

[GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r79104336 --- Diff: test/integration/plugins/solidfire/TestManagedSystemVMs.py --- @@ -0,0 +1,585 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1642 @mike-tutkowski sorry I forgot the precise way to do it -- thanks for digging it out. It makes tests far more resilient to differences between test environments. As I mentioned previo

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-15 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1615 @jburwell @koushik-das I added new marvin test. It requires adding "nfsVersion" key and desired value on config file, if it is not provided test is skipped --- If your project is set up for i

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell I now retrieve the template in a non-hard-coded manner. I attached a diff because I squashed the commit and force pushed it up to GitHub. https://cloud.githubusercontent.com/

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 I think I know how to acquire the template name without hard-coding it now: cls.template = get_template( cls.apiClient, zone.id,

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Hey @syed I like your idea here. How's about we work on a separate PR for it so that I can incorporate your idea into all of the integration scripts I have? --- If your project is set u

Re: [DISCUSS] Replacing the VR

2016-09-15 Thread Will Stevens
Ya, we would need to add a daemon for VPN as well. Load balancing is another aspect which we will need to consider if we went this route. Something like https://traefik.io/ could potentially be a good fit due to its API driven configuration, but it may be more than what we need. We should probabl

Re: [DISCUSS] Replacing the VR

2016-09-15 Thread Simon Weller
I do agree with you that this probably isn't the right place the password service and user data. Having said that, after taking a cursory look at the dev docs, it doesn't seem that difficult to add new daemons: https://opensnaproute.github.io/docs/developer.html#creating-new-component

Re: [DISCUSS] Replacing the VR

2016-09-15 Thread Will Stevens
A big part of why I know about it is because it is written in Go. :P Yes, it is definitely interesting for the routing and traffic handling aspects of the VR. We will likely have to rethink some of the pieces a little bit like the password server and userdata if we are to adopt a different VR ap

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-15 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/872 Thanks for getting back to us @jayapalu. I will hold off focusing on this until you have made your update. If you can get @pdion891 the config, that will help us get to the point where we can sta

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-15 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/872 @jayapalu do you have a configuration example of strongswan for the remote management VPN? I would validate that the generated configuration will work with Windows and OSx clients. Tha

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @syed Yeah, I'd like to externalize that data. I just haven't taken the time to investigate how that works with Marvin. :) --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1642 @mike-tutkowski Currently, all the info in the `TestData` is hardcoded. Would it be possible to add a system to override some values so that I can test this as well (IP addresses for example)? Toda

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi I had posted this earlier in the PR. Is this OK? Also, I have run the following tests (all successfully): TestVolumes.py (both with

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @koushik-das "Currently is solidfire the only managed storage provider?" CloudByte is a managed-storage provider, but I have not seen them active in our community for years now. These Sol

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 @jburwell Do you know what that standard way of acquiring a template name is? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

Re: [DISCUSS] Replacing the VR

2016-09-15 Thread Simon Weller
I hadn't heard of Flexswitch until you mentioned it. It looks pretty cool! It even supports ONIE install. To be honest, the ipsec feature could be added, or we could offload it to separate vm if we needed to. The fact it is so feature rich from a routing perspective (and all API driven) is real

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1642 @koushik-das didn't notice that path, but completely agree. I'd like to see a generic test requirement for managed storage which make the test cases more reusable for future storage drivers.

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-15 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1645 @karuturi - So, note that this feature addition only affects KVM when using RBD as the back-end storage. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1642 @mike-tutkowski Thanks for adding marvin tests. Some observations regarding the tests. I see that you have put the tests under solidfire folder. Currently is solidfire the only managed s

RE: [DISCUSS] Replacing the VR

2016-09-15 Thread Will Stevens
Ya. I don't think it covers our whole use case, but what it does cover is all api driven... On Sep 15, 2016 1:48 AM, "Marty Godsey" wrote: > Though I don’t see VPN in Snaproute.. Makes sense since it was not > intended to do IPSec. > > It seems as though VyOS is starting to look like the best op

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

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

[GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r78934831 --- Diff: test/integration/plugins/solidfire/TestManagedSystemVMs.py --- @@ -0,0 +1,586 @@ +# Licensed to the Apache Software Foundation (AS

[GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r78934051 --- Diff: test/integration/plugins/solidfire/TestManagedSystemVMs.py --- @@ -0,0 +1,586 @@ +# Licensed to the Apache Software Foundation (AS

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1642 That is exactly what I said in my last comment. I am waiting for mike to resolve it so that I can merge. That is the only pending item on this PR. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi there remains the issue of the hard coded template name. Therefore, I am -1 until the issue of the hardcoded template name is addressed. These hardcoded values cause massive portabi

[GitHub] cloudstack pull request #1673: CLOUDSTACK-9071: Properly parse stats.output....

2016-09-15 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/1673 CLOUDSTACK-9071: Properly parse stats.output.uri in StatsCollector Both host and path could have been NULL which causes the StatsCollector no to start properly. By checking if the Stri

Re: System VM's are getting stuck in starting mode

2016-09-15 Thread ned dogg
Hi Wei, Here is the new management server log: http://pastebin.com/gQMDx2Fh On Thu, Sep 15, 2016 at 10:50 AM, Wei ZHOU wrote: > I cannot see any error/exception in agent.log. > it seems something wrong in management server ? Can you paste the latest > management server log ? > /var/log/cloudsta

Re: System VM's are getting stuck in starting mode

2016-09-15 Thread Wei ZHOU
I cannot see any error/exception in agent.log. it seems something wrong in management server ? Can you paste the latest management server log ? /var/log/cloudstack/management/management-server.log -Wei

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-09-15 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/872 @swill @kiwiflyer Sorry for the late response. I am started looking into this. I am looking into getting template with strongswan 5.2. Once it is done I will update the code changes.

[GitHub] cloudstack issue #1642: System VMs on Managed Storage

2016-09-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1642 @mike-tutkowski can you address @jburwell's comment? other than that, its ready to merge. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack pull request #1642: System VMs on Managed Storage

2016-09-15 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1642#discussion_r78916846 --- Diff: test/integration/plugins/solidfire/TestManagedSystemVMs.py --- @@ -0,0 +1,586 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1645 I will start our internal CI run on this(it runs the smoke tests and will post the results here). Once that is successful, I will merge this. --- If your project is set up for it, you can rep