[MERGE] changing nonoss to noredist

2013-09-19 Thread Hugo Trippaers
Hey all, As discussed on the mailing list we want to change the name of the nonoss branch to noredist as this better reflects the reason for this separate branch. In the branch nonoss-to-noredist i've made the necessary changes to the code. After merging this branch i will also update the wiki

RE: Volunteers to Complete the 4.2 Release Notes

2013-09-19 Thread Radhika Puthiyetath
This has been updated. Could someone please validate the API section ? -Original Message- From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] Sent: Sunday, September 15, 2013 4:16 AM To: us...@cloudstack.apache.org Cc: dev@cloudstack.apache.org; us...@cloudstack.apache.org

Re: Review Request 14023: NetworkManager Exception Fix Due to call to getDhcpServiceProvider()

2013-09-19 Thread Murali Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14023/#review26265 --- Could you please rebase to master and submit the patch, fix need to

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

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:

Re: Hotels for CCC Amsterdam

2013-09-19 Thread Daan Hoogland
On Wed, Sep 18, 2013 at 11:26 PM, La Motta, David david.lamo...@netapp.com 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

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

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

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:

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

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

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

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:

Re: xenserver tools

2013-09-19 Thread Daan Hoogland
On Thu, Sep 19, 2013 at 7:35 AM, Abhinandan Prateek abhinandan.prat...@citrix.com 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

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.

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

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.,

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 ---

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

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 darren.s.sheph...@gmail.com wrote: So I run ACS from eclipse for development, but DatabaseUpgradeChecker does the

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: 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

[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 鄢洞

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

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: The

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: 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.,

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:

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

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

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 alex.hu...@citrix.com wrote: Around 4.1

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,

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 that was

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: [PROPOSAL] Storage Subsystem API Interface Additions

2013-09-19 Thread Kelven Yang
On 9/19/13 5: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 valid topic that I'd like to be a part of discussing anyway!). Mike - are you planning on implementing

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,

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

Re: DatabaseUpgradeChecker and running from eclipse

2013-09-19 Thread Daan Hoogland
On Thu, Sep 19, 2013 at 6:12 PM, Darren Shepherd darren.s.sheph...@gmail.com 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: 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: 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's

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

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

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- From: Marcus

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

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

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 animesh.chaturv...@citrix.com wrote: -Original Message- From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] Sent:

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

NetAppCitrix 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 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 major

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: 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

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,

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- From: Marcus

Re: NetAppCitrix 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

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