Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Raja Pullela
Hi Rohit, same question as Rene has posted, impact on older releases – will have issues on older releases. I know that the older releases have marvin code which can be used. Also, this will require changes on the CI side to pull the correct repo for Marvin. +1, if Bharat can modify CI imp

[GitHub] cloudstack issue #1583: Update L10N resource files with 4.9 strings from Tra...

2016-07-19 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1583 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 4h 03m 22s ``` **Associated Uploads** **`/tm

[GitHub] cloudstack pull request #1594: CLOUDSTACK-9407: vm_network_map table doesnt ...

2016-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1594 --- 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 #1583: Update L10N resource files with 4.9 strings f...

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

State of the S3 secondary storage

2016-07-19 Thread Nux!
Hello, Could anyone summarise the state of S3 (and clones) for secondary storage? I read at some point that it's the only secondary storage type supported inter-zone, so it would appeal from a DR pov, but not sure how mature it is or if there are any serious gotchas involved. Thanks Lucian --

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Bharat Kumar
Hi Rohit, what are we trying to achieve by moving marvin into a separate repo.? --Bharat. From: Raja Pullela Sent: Tuesday, July 19, 2016 5:30:20 PM To: dev@cloudstack.apache.org Subject: Re: [VOTE] Split Marvin to its own repository Hi Rohit, same question

Re: Starting jetty on another port

2016-07-19 Thread Syed Ahmed
Thanks Rajani! That worked. -Syed On Tue, Jul 19, 2016 at 1:17 AM, Rajani Karuturi wrote: > Try changing in > client/pom.xmlhttps:// > github.com/apache/cloudstack/blob/master/client/pom.xml#L459 > ~ Rajanihttp://cloudplatform.accelerite.com/ > On July 19, 2016 at 1:11 AM, Syed Mushtaq > (syed1

Re: State of the S3 secondary storage

2016-07-19 Thread Wido den Hollander
> Op 19 juli 2016 om 14:38 schreef Nux! : > > > Hello, > > Could anyone summarise the state of S3 (and clones) for secondary storage? > I read at some point that it's the only secondary storage type supported > inter-zone, so it would appeal from a DR pov, but not sure how mature it is > or i

Re: State of the S3 secondary storage

2016-07-19 Thread Will Stevens
We are using Swift as secondary storage. It also offers inter-zone secondary storage. We have had to do fixes to make it work correctly, but we have pushed those fixes upstream so they are available in 4.9 (and some fixes earlier than that). Like Wido said, the staging NFS is still needed which

Re: State of the S3 secondary storage

2016-07-19 Thread Syed Ahmed
As Will put it, the implementation for using Object store as secondary storage relies on using Staging NFS. You would also loose the functionality to do differential snapshots. All your snapshots will basically be full volume snapshots. So, if you have a lot of snapshots being taken at scheduled in

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Syed Ahmed
Also I would +1 this, although my vote is non-binding :) On Tue, Jul 19, 2016 at 12:11 PM, Syed Ahmed wrote: > I believe it will make CI much smoother. Right now marvin is tied to the > Cloudstack repo which was fine if all the integration tests were running > from Cloudstack build but we are no

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Syed Ahmed
I believe it will make CI much smoother. Right now marvin is tied to the Cloudstack repo which was fine if all the integration tests were running from Cloudstack build but we are now seeing much better CI approaches with bubble and Trillian and having marvin in its own repo will facilitate that eve

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Rohit Yadav
Hi Paul, Each integration test in general probes three things - the API, the DB and remote server/host (using a ssh client). APIs are assumed to be strictly backward compatible so any test that tests an API should be backward compatible with previous CloudStack versions. Though, each CloudStac

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Rohit Yadav
Hi Rene and Raja, In general, the core Marvin library has not changed since at least 4.5. The only difference across all Marvin/cloudstack version is the contents in marvin/marvin/cloudstackAPI directory i.e. the API cmd and response classes. This has been solved by using runtime auto-generati

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Will Stevens
So how would the different versions of Marvin be tracked and how would the versions be associated with the supported ACS versions? Because the ACS API changes, a Marvin version will only support a specific set of ACS versions. We need to understand how that will work because this is bound to caus

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Rohit Yadav
Thanks Syed, you've answered it well. Bharat, what Syed has described captures the goal behind this effort. I see the pros and cons with keeping Marvin within the repo, and if there is a hesitation with splitting Marvin -- I propose another alternative: 1. Along with cloudstack deb/rpm packag

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Will Stevens
@Rohit: I think that the assumption that the API is always backwards compatible is dangerous as we know that in practice it is not always the case. For example: In ACS 4.6 the `restartVPC` introduced a new parameter called `makeredundant`. Unfortunately, the default value for this parameter (if y

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Rohit Yadav
Hi Will, I understand your concerns, the goal with this initiative is to make sure that Marvin would remain forward compatible with future versions. As for the past releases/versions, we cannot guarantee backward compatibility. My main goal was to solve and make it easier for CI systems to co

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Will Stevens
I think your alternatives makes sense. Since we are always merging into and testing 3 different branches (4.7, 4.8, and master in the case of the 4.9 release), we are opening ourselves to headaches IMHO. I don't think we can expect that the same Marvin install will ALWAYS work on all three branch

Re: State of the S3 secondary storage

2016-07-19 Thread Nux!
Thanks for your input guys. I think I'll stay with NFS for now after your replies, but it'd be pretty sweet the day when we'll be having region wide secondary storage. I'll need to think of another way to provide DR, perhaps some automatic snapshot+download of sorts. Lucian -- Sent from the D

Re: State of the S3 secondary storage

2016-07-19 Thread Syed Ahmed
Is there a reason why NFS cannot be region-wide? -Syed On Tue, Jul 19, 2016 at 2:08 PM, Nux! wrote: > Thanks for your input guys. > > I think I'll stay with NFS for now after your replies, but it'd be pretty > sweet the day when we'll be having region wide secondary storage. > > I'll need to th

Re: State of the S3 secondary storage

2016-07-19 Thread Will Stevens
Probably because ACS does not treat it as region wide, unlike the object storage implementations. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Tue, Jul 19, 2016 at 2:52 PM, Syed Ahmed wro

Re: State of the S3 secondary storage

2016-07-19 Thread Syed Ahmed
Right, but is that an artificial limitation imposed by Cloudstack of is there something that will break if we make NFS region wide? If not, we can look at moving the NFS to a region wide storage. -Syed On Tue, Jul 19, 2016 at 3:54 PM, Will Stevens wrote: > Probably because ACS does not treat it

Re: State of the S3 secondary storage

2016-07-19 Thread Will Stevens
Oh ya. Gotta love NFS syncing to servers halfway around the world. :P *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Tue, Jul 19, 2016 at 4:06 PM, Syed Ahmed wrote: > Right, but is that

Re: State of the S3 secondary storage

2016-07-19 Thread Nux!
Glusterfs ;) -- Sent from my mobile device. Please excuse my brevity. On 19 July 2016 21:24:45 BST, Will Stevens wrote: >Oh ya. Gotta love NFS syncing to servers halfway around the world. :P > >*Will STEVENS* >Lead Developer > >*CloudOps* *| *Cloud Solutions Experts >420 rue Guy *|* Montreal *

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Rohit Yadav
Thank you for voting and for your feedback. Based on the discussion on this thread, I'm calling off the voting. I'll send a PR to add two new deb/rpm packages from 4.9/master, that will allow anyone to install Marvin and integration-tests, which will also make it easier for CI systems to insta

[GitHub] cloudstack issue #1599: Marvin: Fix codegenerator to work with API discovery

2016-07-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1599 The initiative to split marvin into its own repository has been dropped for now. Meanwhile, this PR can still be accepted as it fixed the code-generator to work against the api discovery. I'll sen

[GitHub] cloudstack-docs-install pull request #27: make sure we use the same nfs path...

2016-07-19 Thread bugoff
GitHub user bugoff opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/27 make sure we use the same nfs paths. /primary vs /mnt/primary make sure we use the same nfs paths. /primary vs /mnt/primary on the next page. You can merge this pull request into a

[GitHub] cloudstack pull request #1612: packaging: Marvin and integration-tests packa...

2016-07-19 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1612 packaging: Marvin and integration-tests packages This introduces two new cloudstack packages: marvin and integration-tests. The two packages will make it easier for CI systems to install Marvi

[GitHub] cloudstack issue #1612: packaging: Marvin and integration-tests packages

2016-07-19 Thread PaulAngus
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1612 @rohit is it worth putting into straight into the apachecloudstack repo (rather than apache/cloudstack). Kind regards, Paul Angus VP Technology