Re: [Merge] Minimal Hyper-V Plugin

2013-09-19 Thread Chiradeep Vittal
Thanks for the preliminary testing. Questions: 1. More for the community: should the C# code be in a separate repo? According to the merge request, mono and maven can be used to build the agent. 2. Packaging: how is the C# agent installed? 3. What does minimal mean? What works? What doesn't? Local

Re: NetApp&Citrix Storage plugin meeting notes -- Sep 17, 2013

2013-09-19 Thread Amit Das
Thanks Edison, the meeting notes were helpful. I am implementing a storage plugin for CloudByte & am using *_details table for storing the plugin related info. Can anyone explain what kind/type of plugin data is not suitable in *_details table (referring to point 5). Regards, Amit *CloudByte Inc

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Animesh Chaturvedi
> -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Thursday, September 19, 2013 5:15 PM > To: dev@cloudstack.apache.org > Subject: RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth > round) > > > > > -Original Message- > > Fr

Re: conflicting dependencies between CloudStack and Whirr

2013-09-19 Thread Hugo Trippaers
The Nicira NVP plugin is also depending on gson. If we make any changes we need to validate against their api to ensure that the interface still works. If we put the changes in a separate branch i'd be happy to run the changes against the api. Cheers, Hugo On Sep 20, 2013, at 12:08 PM, Darr

Re: conflicting dependencies between CloudStack and Whirr

2013-09-19 Thread Darren Shepherd
After much searching I found the two lines of code I needed to get it to serialize right. I'll keep looking and see if I find other snags. Maybe we can just move to jackson we'll see, haven't tried to deserialize yet. Darren > On Sep 19, 2013, at 4:25 PM, Alex Huang wrote: > > Darren,

Tips for git rebase

2013-09-19 Thread Nguyen Anh Tu
Hi guys, Could anyone share some tips or GUI tool for making git rebase easier? I have to rebase my working branch with master but get many many conflicts which I have no idea about these pieces of code. Thanks for any help, -- N.g.U.y.e.N.A.n.H.t.U

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Animesh Chaturvedi
> -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Thursday, September 19, 2013 4:46 PM > To: dev@cloudstack.apache.org > Subject: Re: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth > round) > > I prefer a respin, as painful as that seems. VPC is a

NetApp&Citrix Storage plugin meeting notes -- Sep 17, 2013

2013-09-19 Thread Edison Su
Meeting notes: Date: Sep 17, 2013 Attendees: NetApp: Chris, David Citrix: Alex, Animesh, Edison, Srinivas 1. TakeSnapshot method on storage plugin. Chris: TakeSnapshot method is implemented, but backup snapshot is failed on the hypervisor side, as the default backup snapshot procedure

Re: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Marcus Sorensen
I prefer a respin, as painful as that seems. VPC is a major feature, do we really want to release something that we know will break anyone who tries to upgrade? I could deal with #1 if we are able to include the hotfix script in the packaging, such that the release notes can provide dead simple in

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Indra Pramana
Can we do option (1) please? Many users, including myself, have been waiting for 4.2 to be released for months. On 20 Sep 2013 05:59, "Animesh Chaturvedi" wrote: > > > > -Original Message- > > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > > Sent: Thursday, September 1

RE: conflicting dependencies between CloudStack and Whirr

2013-09-19 Thread Alex Huang
Darren, When I looked into updating to the latest gson, the problem, IIRC, is things that weren't considered cyclical dependencies are suddenly considered cyclical with the latest. I don't recall where though. --Alex > -Original Message- > From: Darren Shepherd [mailto:darren.s.sheph

RE: cloudstack source code in build dir

2013-09-19 Thread Alex Huang
Hi黑洞, CloudStack has a few properties files with tokens within them. These tokens are usually replace during installation of cloudstack. For developers, replace.properties contains replacement values files for these tokens so that after you build, cloudstack already have the proper values for

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Animesh Chaturvedi
> -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Thursday, September 19, 2013 10:43 AM > To: dev@cloudstack.apache.org > Subject: RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth > round) > > > > > -Original Message- > > F

Re: Review Request 14218: Add spring lifecycle framework to handle modularization

2013-09-19 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14218/ --- (Updated Sept. 19, 2013, 9:41 p.m.) Review request for cloudstack, Alex Huang a

Re: Review Request 14218: Add spring lifecycle framework to handle modularization

2013-09-19 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14218/ --- (Updated Sept. 19, 2013, 9:31 p.m.) Review request for cloudstack, Alex Huang a

Re: Master nonoss is broken?

2013-09-19 Thread Darren Shepherd
Probably a victim of not doing a "clean compile." Maven works completely on timestamps of source and dest and does not check dependencies of code. So if you build juniper code once, then changed IpAddressTO, and then did compile, maven won't detect that JuniperSrxResource should be rebuilt. Maven'

Re: conflicting dependencies between CloudStack and Whirr

2013-09-19 Thread Darren Shepherd
Alright, I looked into this and it will take a bit more work to switch to Jackson. The snag I hit was how we serialize commands for the agents. We use a customer typeadatper in gson to produce a format like { "org.MyClass" : {...} }. In jackson I don't see producing a similar format as being so

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Daan Hoogland
On Thu, Sep 19, 2013 at 6:12 PM, Darren Shepherd wrote: > Only to test builds before commit. mvn is too slow. That's probably it. I run from eclipse but usually do a build first. That is going to change one minute ago, thanks. i'll keep an eye on the dbupgradechecker

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Marcus Sorensen
We do need to ensure that we have the db upgrade fix that was mentioned on the other thread, otherwise people going from 4.1 to 4.2 will have their VPCs break. Looks like we are waiting on a script. It sounds like the plan will be to provide instructions in the release notes. Really wish we would h

RE: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Alex Huang
Before cloudstack was donated to apache, it had an open source version and a non-open source portion (which included vmware, f5, and juniper). Now, everything is open source but that division is sometimes still in the files. You can basically merge those files now. As for jdk version, that's

Re: [PROPOSAL] Storage Subsystem API Interface Additions

2013-09-19 Thread Kelven Yang
On 9/19/13 5:16 AM, "SuichII, Christopher" wrote: >John - any chance we can get your input on the original topic. Mikes >comment was a kind of unrelated (but a completely valid topic that I'd >like to be a part of discussing anyway!). > >Mike - are you planning on implementing the snapshotting

RE: [DISCUSS] CSS framework for CloudStack UI

2013-09-19 Thread Brian Federle
Yeah, that would be a great idea...it would drastically reduce initial load time for the UI. OK, I've created a new branch called ui-css-framework. Right now all it does is rename cloudstack3.css -> cloudstack3.scss. This of course will make the UI have no style if running right now, but once a

RE: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Alex Huang
Ah...got it. Makes sense. --Alex > -Original Message- > From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] > Sent: Thursday, September 19, 2013 10:38 AM > To: dev@cloudstack.apache.org > Subject: Re: DatabaseUpgradeChecker and running from eclipse > > Its getting the JDK version

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Animesh Chaturvedi
> -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Thursday, September 19, 2013 9:49 AM > To: dev@cloudstack.apache.org > Subject: RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth > round) > > We do need to ensure that we have the db upgrade fix t

Re: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Simon Weller
I think one of the issues is that practically all the community testing is based on new installs. Since we didn't appear to have a published upgrade procedure available for 4.x to 4.2, it made it very hard to test in a reliable way. I did try a couple of upgrades from 4.02 and 4.1 in my lab, an

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Darren Shepherd
Its getting the JDK version because of this.getClass().getSuperclass().getPackage().getImplementationVersion(); which it only check if this.getClass().getPackage().getImplementationVersion() is null. So parent is java.lang.Object, so it gets the JDK version. I understand your really looking for 4

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Darren Shepherd
So I just looked at DatabaseCreator and noticed it calls PremiumDatabaseUpgradeChecker, but on startup we call DatabaseUpgradeChecker. What is this madness? Why are there two db upgrade classes? Darren On Thu, Sep 19, 2013 at 10:04 AM, Alex Huang wrote: > Around 4.1 timeframe, Rohit added Da

RE: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Alex Huang
Around 4.1 timeframe, Rohit added DatabaseCreator to call DatabaseUpgradeChecker to create and upgrade the database but forgot to remove it from the server configuration. We should simply add a DatabaseVesionChecker for the server configuration to check if the database is at the version expect

Review Request 14231: Don't check implementation verion of Object for DatabaseUpgradeChecker

2013-09-19 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14231/ --- Review request for cloudstack, Alex Huang and daan Hoogland. Repository: clouds

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Darren Shepherd
I just created https://reviews.apache.org/r/14231/ to remove the version check that finds the JDK version. I can't see why we would ever want to find the JDK version, so might as well just not check and fall through to the existing logic that does "if (currentVersion==null) return" Darren On Th

Re: Review Request 14226: do not instantiate Long just for toString

2013-09-19 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14226/#review26269 --- Ship it! Ship It! - daan Hoogland On Sept. 19, 2013, 9:45 a.m.,

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Darren Shepherd
Its has to do with my maven setup in Eclipse really. So import all the projects then create a new Maven Build Debug Configuration with Base Directory: ${workspace_loc:/cloudstack} Goals: -pl :cloud-client-ui jetty:run Check "Resolve Workspace Artifacts" So the issue comes from "Resolve Workspace

RE: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth round)

2013-09-19 Thread Animesh Chaturvedi
> -Original Message- > From: Wido den Hollander [mailto:w...@widodh.nl] > Sent: Wednesday, September 18, 2013 4:55 PM > To: dev@cloudstack.apache.org > Subject: Re: [VOTE][RESULTS] Release Apache CloudStack 4.2.0 (fifth > round) > > > > On 09/19/2013 01:28 AM, Animesh Chaturvedi wrote:

Re: Review Request 14228: Do not instantiate long just to generate String

2013-09-19 Thread daan Hoogland
> On Sept. 19, 2013, 3:03 p.m., daan Hoogland wrote: > > Ship It! 3650f12e9b284d16f28131ba6cafecacb2d03fdc - daan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14228/#review26270 --

[Responsiveness report] dev 2013w37

2013-09-19 Thread Daan Hoogland
http://markmail.org/message/6ag3r6mtjhvqsytc ConsoleProxy RPM by Rayees Namathponnan http://markmail.org/message/xkf2heruwa6umc4d No running ssvm is found, so command will be sent to LocalHostEndPoint by Punith s http://markmail.org/message/2ql3xrzbndgbapz4 How to modify cloudstack ui by 鄢洞 http:

Re: Review Request 14226: do not instantiate Long just for toString

2013-09-19 Thread daan Hoogland
> On Sept. 19, 2013, 2:59 p.m., daan Hoogland wrote: > > Ship It! 88954053c344232189c130684c8dd3a495570045 - daan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14226/#review26269 --

Re: Review Request 14228: Do not instantiate long just to generate String

2013-09-19 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14228/#review26270 --- Ship it! Ship It! - daan Hoogland On Sept. 19, 2013, 9:49 a.m.,

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Daan Hoogland
Darren, what is your scenario to run into this? I had some upgrade paths to run through but didn't encounter this. regards, Daan On Wed, Sep 11, 2013 at 6:58 PM, Darren Shepherd wrote: > So I run ACS from eclipse for development, but DatabaseUpgradeChecker does > the following > > String current

Re: [PROPOSAL] Storage Subsystem API Interface Additions

2013-09-19 Thread Mike Tutkowski
I'm not sure on a release for when I'll implement snapshot functionality. Maybe 4.4. On Thu, Sep 19, 2013 at 6:16 AM, SuichII, Christopher < chris.su...@netapp.com> wrote: > John - any chance we can get your input on the original topic. Mikes > comment was a kind of unrelated (but a completely v

Re: ROOT disk resize

2013-09-19 Thread Marcus Sorensen
Something like this for 4.2... for 4.1 it would be pretty much the same, just that in 4.1 the code was in StorageManagerImpl.java iff --git a/server/src/com/cloud/storage/VolumeManagerImpl.java b/server/src/com/cloud/storage/VolumeManagerImpl.java index 1d6b44f..4ef57ed 100644 --- a/server/src/com

Re: Review Request 14230: Storage type was set to local due to which tests failed.

2013-09-19 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14230/#review26268 --- Ship it! - Prasanna Santhanam On Sept. 19, 2013, 1:20 p.m., Giris

Re: Review Request 9539: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-09-19 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/#review26267 --- Commit 5aee73b1a59f02c7853892d13afc58ad01f40c88 in branch refs/heads/

Review Request 14230: Storage type was set to local due to which tests failed.

2013-09-19 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14230/ --- Review request for cloudstack, Harikrishna Patnala and Prasanna Santhanam. Repo

Re: xenserver tools

2013-09-19 Thread Daan Hoogland
On Thu, Sep 19, 2013 at 7:35 AM, Abhinandan Prateek wrote: > Do file a big to track. does not compute&%$&^$@ Oh wait, you have a spellchecker active ;) yes, if someone can tell me if we can include gpl'd binaries in our git/distro I think we should. In that case I will file a bug. regards,

Review Request 14229: CLOUDSTACK: 4256 - Fixed shared network test cases

2013-09-19 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14229/ --- Review request for cloudstack and Prasanna Santhanam. Repository: cloudstack-gi

Re: [PROPOSAL] Storage Subsystem API Interface Additions

2013-09-19 Thread SuichII, Christopher
John - any chance we can get your input on the original topic. Mikes comment was a kind of unrelated (but a completely valid topic that I'd like to be a part of discussing anyway!). Mike - are you planning on implementing the snapshotting methods of the storage subsystem API anytime in the near

Re: Review Request 14227: CLOUDSTACK-4707: "sourcetemplateid" field is not getting set for derived templates

2013-09-19 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14227/ --- (Updated Sept. 19, 2013, 10:25 a.m.) Review request for cloudstack and Kishan K

Review Request 14228: Do not instantiate long just to generate String

2013-09-19 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14228/ --- Review request for cloudstack and mice xia. Repository: cloudstack-git Descri

Review Request 14227: CLOUDSTACK-4707: "sourcetemplateid" field is not getting set for derived templates

2013-09-19 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14227/ --- Review request for cloudstack, Abhinandan Prateek and Kishan Kavala. Bugs: CLOU

Review Request 14226: do not instantiate Long just for toString

2013-09-19 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14226/ --- Review request for cloudstack and Alex Huang. Repository: cloudstack-git Desc

Re: Master nonoss is broken?

2013-09-19 Thread Daan Hoogland
Sorry, it was mee, Don't know how I missed it as I did do the nonoss build. Probably had some build leftovers on my system. Does someone know if there are configurations possible where juniper works with something else then vlan? if so if (ip.getBroadcastUri() != null && !ip.getBroadcastUr

Re: Hotels for CCC Amsterdam

2013-09-19 Thread Daan Hoogland
On Wed, Sep 18, 2013 at 11:26 PM, La Motta, David wrote: > Hey everybody, I think I saw something about hotels not being decided for > CCC in Amsterdam this November. Has that been squared out? yes, I hear at Schuberg Philis there are three hotels. Nagotiations on prices are going on, hence they

Review Request 14225: CLOUDSTACK: 4706 - Adding missing method "get_region"

2013-09-19 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14225/ --- Review request for cloudstack and venkata swamy babu budumuru. Repository: clo

Re: Review Request 13559: Automation: Portable IP test cases

2013-09-19 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13559/ --- (Updated Sept. 19, 2013, 12:57 p.m.) Review request for cloudstack, Murali Redd