Re: CloudStack Snapshot Question

2014-09-03 Thread Mike Tutkowski
This topic (below) is actually much more relevant to people than I initially thought it would be (hence why I just sent it to Edison in the first place). Please feel free to provide comments and suggestions. Thanks! On Tue, Sep 2, 2014 at 11:54 PM, Mike Tutkowski mike.tutkow...@solidfire.com

Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25289/ --- Review request for cloudstack, Frank Zhang and Hugo Trippaers. Bugs:

Review Request 25290: CLOUDSTACK-7441: Fixed template register issue in test_resource_limits.py

2014-09-03 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25290/ --- Review request for cloudstack, sanjeev n, Santhosh Edukulla, and

Re: Review Request 25290: CLOUDSTACK-7441: Fixed template register issue in test_resource_limits.py

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25290/#review52133 --- Ship it! LGTM - SrikanteswaraRao Talluri On Sept. 3, 2014, 6:28

Re: Review Request 25290: CLOUDSTACK-7441: Fixed template register issue in test_resource_limits.py

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25290/#review52135 --- pushed to master 396f29c135e3f384218400a1f64c63a6dbb2a76d -

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review52136 --- test/integration/smoke/test_volumes.py

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review52138 --- Ship it! Ship It! - SrikanteswaraRao Talluri On Sept. 2, 2014,

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review52141 --- Pushed to master : 9b783d19ad7e784d3751f4eaa2a672401d122a01 -

Re: Cherry-picking fix that may change 4.3.1 schema

2014-09-03 Thread Sebastien Goasguen
On Sep 2, 2014, at 11:59 PM, Rajani Karuturi raj...@apache.org wrote: this was to your suggestion of a separate tool that does (rolling) upgrades which i think is a new proposal. I should have made it clear in my mail about it and should have started it with right way according to me.

Jenkins build is back to normal : build-master-noredist #3443

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/3443/changes

Commit cd8af6a

2014-09-03 Thread Hugo Trippaers
Min, I’m checking some findbugs reports and found the following piece of code in VolumeDataStoreVO (line 352): public void setRefCnt(Long refCnt) { if (refCnt 0) { refCnt--; } else { s_logger.warn(We should not try to decrement a zero

Re: Cherry-picking fix that may change 4.3.1 schema

2014-09-03 Thread Sebastien Goasguen
On Sep 2, 2014, at 11:28 AM, Rohit Yadav rohit.ya...@shapeblue.com wrote: On 02-Sep-2014, at 5:07 pm, Francois Gaudreault fgaudrea...@cloudops.com wrote: I see. Well, I think we were impacted by that too, and we made the decision to move on 4.4.1-snap (even if it's technically less

Jenkins build is still unstable: simulator-singlerun #270

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Cherry-picking fix that may change 4.3.1 schema

2014-09-03 Thread Daan Hoogland
4.3.1 to 4.4.1 then should contain all effort that is also in 4.3.0 to 4.4.0 I don't look forward to the work but it is doable. What bothers me most is that we are putting so much enegy into 4.3.1 at the moment while 4.4.1 is about to be rc'd. -- an anonymous commit to commit upgrade advocate /

Jenkins build is still unstable: simulator-singlerun #271

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Review Request 25292: CLOUDSTACK-6898: [Hyper-V] fixed rdp console freezing during reboot.

2014-09-03 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25292/ --- Review request for cloudstack and Devdeep Singh. Bugs: CLOUDSTACK-6898

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25289/#review52151 --- client/tomcatconf/classpath.conf.in

Jenkins build is still unstable: simulator-singlerun #272

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Review Request 25293: Test script to verify vm deployment with two networks in SG enabled advanced zone

2014-09-03 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25293/ --- Review request for cloudstack, suresh sadhu, Santhosh Edukulla, and

Jenkins build is still unstable: simulator-singlerun #273

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Clarification on github or and TravisCI

2014-09-03 Thread Sebastien Goasguen
On Sep 2, 2014, at 12:31 PM, Stephen Turner stephen.tur...@citrix.com wrote: So just to be clear, Sebastien, does that mean it's acceptable for non-committers to submit their code for review via a Github pull request, instead of using ReviewBoard? (I hope so: I think that will make the

Re: Clarification on github or and TravisCI

2014-09-03 Thread Ian Duffy
It's doable but have not yet discussed and voted if we wanted to make it an official mechanism. Would love to see it happen. I recall review board being a massive pain point for a few reasons: 1) Large patches fail to upload, especially if they contain images. I recall sending many patches by

Re: Clarification on github or and TravisCI

2014-09-03 Thread Erik Weber
On Wed, Sep 3, 2014 at 1:15 PM, Ian Duffy i...@ianduffy.ie wrote: It's doable but have not yet discussed and voted if we wanted to make it an official mechanism. Would love to see it happen. I recall review board being a massive pain point for a few reasons: +1 as a non-committer it is

API calls and keys

2014-09-03 Thread Giri Prasad
Hello, I am using a html/php script to read in userid, password and domain id from the user. Using CloudStackClient.php, I am able to call most of the api's, like list vm's, and the output seems correct. And I want to use some API's via a href like:

RE: jsonObj is not defined. UI development problem.

2014-09-03 Thread Ilia Shakitko
Hi Gabor, Thank you very much for your reply. Yeah, I spent quite some time on it and it's not so straight forward. So it would be nice if contributor could assist us here. I've tried to contact the main contributor for that file, but have no answer for quite long time :\ Very good find you

Re: Cherry-picking fix that may change 4.3.1 schema

2014-09-03 Thread David Nalley
On Wed, Sep 3, 2014 at 3:02 AM, Sebastien Goasguen run...@gmail.com wrote: On Sep 2, 2014, at 11:59 PM, Rajani Karuturi raj...@apache.org wrote: this was to your suggestion of a separate tool that does (rolling) upgrades which i think is a new proposal. I should have made it clear in my mail

Re: Clarification on github or and TravisCI

2014-09-03 Thread David Nalley
On Wed, Sep 3, 2014 at 7:15 AM, Ian Duffy i...@ianduffy.ie wrote: It's doable but have not yet discussed and voted if we wanted to make it an official mechanism. Would love to see it happen. I recall review board being a massive pain point for a few reasons: 1) Large patches fail to upload,

Review Request 25301: CLOUDSTACK-7442: Fixed template permission issue in test_project_resources.py

2014-09-03 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25301/ --- Review request for cloudstack, sanjeev n, Santhosh Edukulla, and

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
Thanks Hugo for pointing that out. That is a bug caused by copy-and-paste, the code should be in decrRefCnt routine like other SnapshotDataStoreVO and TemplateDataStoreVO. I will fix this in a minute. -min On 9/3/14 12:30 AM, Hugo Trippaers h...@trippaers.nl wrote: Min, I¹m checking some

wiki on systemvm.iso

2014-09-03 Thread Nitin Mehta
I created a short write up on systemvm.iso to the best of my knowledge. Devs/Users can use this to understand how systemvm.iso is propagated and use it to debug their setup. Since its propagation depends on each hypervisor I might have missed few things so please review and correct it.

Cloudstack Build Failing

2014-09-03 Thread Deepal Jayasekara
Hi All, We are working on a cloudstack based project named Resource Aware and Policy Based VM Scheduling in Medium-scale Clouds. Our group is really new to Cloudstack and currently trying to build and install cloudstack from source. We followed the documentation at docs.cloudstack.apache.org and

Re: Review Request 25235: vGPU service offering upgrade test automation

2014-09-03 Thread sailaja mada
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25235/ --- (Updated Sept. 3, 2014, 6:12 p.m.) Review request for cloudstack, Doug Clark,

Re: Commit cd8af6a

2014-09-03 Thread Daan Hoogland
H Min, what is this fix for? On Wed, Sep 3, 2014 at 8:20 PM, Min Chen min.c...@citrix.com wrote: Fix is checked into master branch. Can I back port it to 4.4? Thanks -min On 9/3/14 9:42 AM, Min Chen min.c...@citrix.com wrote: Thanks Hugo for pointing that out. That is a bug caused by

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
Fix is checked into master branch. Can I back port it to 4.4? Thanks -min On 9/3/14 9:42 AM, Min Chen min.c...@citrix.com wrote: Thanks Hugo for pointing that out. That is a bug caused by copy-and-paste, the code should be in decrRefCnt routine like other SnapshotDataStoreVO and

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
This is a bug that will impact reference counting only for old volume on NFS to S3 object store migration. Thanks -min On 9/3/14 11:25 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: H Min, what is this fix for? On Wed, Sep 3, 2014 at 8:20 PM, Min Chen min.c...@citrix.com wrote: Fix is

Re: Commit cd8af6a

2014-09-03 Thread Daan Hoogland
guess you should go ahead and backport. On Wed, Sep 3, 2014 at 8:28 PM, Min Chen min.c...@citrix.com wrote: This is a bug that will impact reference counting only for old volume on NFS to S3 object store migration. Thanks -min On 9/3/14 11:25 AM, Daan Hoogland daan.hoogl...@gmail.com

Jenkins build is still unstable: simulator-singlerun #274

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

[REVIEW] CLOUDSTACK-7468 in master

2014-09-03 Thread Will Stevens
Here is a link to the issue: https://issues.apache.org/jira/browse/CLOUDSTACK-7468 This fix has been committed in the 4.4 branch already. I have pushed a feature branch called 'CLOUDSTACK-7468' to the ASF repo which is a branch off master with the fix applied. The fix for master is here:

Jenkins build is still unstable: simulator-singlerun #275

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread Frank Zhang
On Sept. 3, 2014, 9:32 a.m., Rajani Karuturi wrote: client/tomcatconf/classpath.conf.in, line 37 https://reviews.apache.org/r/25289/diff/1/?file=674877#file674877line37 Can we get the JAVA_HOME from installed java instead of hardcoding it? The path may be different for different

Re: wiki on systemvm.iso

2014-09-03 Thread Marcus
Thank you for this. It has been a common question than many people have just had to figure out. On Sep 3, 2014 11:40 AM, Nitin Mehta nitin.me...@citrix.com wrote: I created a short write up on systemvm.iso to the best of my knowledge. Devs/Users can use this to understand how systemvm.iso is

Jenkins build is still unstable: simulator-singlerun #277

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread Alex Brett
On Sept. 3, 2014, 9:32 a.m., Rajani Karuturi wrote: client/tomcatconf/classpath.conf.in, line 37 https://reviews.apache.org/r/25289/diff/1/?file=674877#file674877line37 Can we get the JAVA_HOME from installed java instead of hardcoding it? The path may be different for different

Jenkins build is still unstable: simulator-singlerun #278

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread David Nalley
On Wed, Sep 3, 2014 at 7:40 PM, Alex Brett alex.br...@citrix.com wrote: On Sept. 3, 2014, 9:32 a.m., Rajani Karuturi wrote: client/tomcatconf/classpath.conf.in, line 37 https://reviews.apache.org/r/25289/diff/1/?file=674877#file674877line37 Can we get the JAVA_HOME from installed

Re: Review Request 24882: CLOUDSTACK-6282 - Added skip condition when hypervisor is hyper-v for tests which are not applicable for hyper-v

2014-09-03 Thread Alex Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24882/#review52262 --- test/integration/component/test_escalations_ipaddresses.py

RE: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread Alex Brett
On 04 September 2014 00:45, David Nalley [da...@gnsa.us] wrote: On Wed, Sep 3, 2014 at 7:40 PM, Alex Brett alex.br...@citrix.com wrote: To expand on this a little bit - in RHEL 6.3 if you have both Java 1.6 and Java 7 installed, the default behaviour with the alternatives mechanism makes 1.6

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread David Nalley
On Wed, Sep 3, 2014 at 8:12 PM, Alex Brett alex.br...@citrix.com wrote: On 04 September 2014 00:45, David Nalley [da...@gnsa.us] wrote: On Wed, Sep 3, 2014 at 7:40 PM, Alex Brett alex.br...@citrix.com wrote: To expand on this a little bit - in RHEL 6.3 if you have both Java 1.6 and Java 7

Jenkins build is still unstable: simulator-singlerun #279

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Jenkins build is still unstable: simulator-singlerun #280

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 22799: Golden (Base) Primary Storage feature

2014-09-03 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22799/#review52266 --- Just checking in again to see where we're at with this feature.

Jenkins build is still unstable: simulator-singlerun #281

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 25293: Test script to verify vm deployment with two networks in SG enabled advanced zone

2014-09-03 Thread suresh sadhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25293/#review52275 --- Ship it! Ship It! - suresh sadhu On Sept. 3, 2014, 9:58 a.m.,

Re: Review Request 25293: Test script to verify vm deployment with two networks in SG enabled advanced zone

2014-09-03 Thread suresh sadhu
On Sept. 4, 2014, 4:05 a.m., suresh sadhu wrote: Ship It! addressed the preview review comments and looks good to me. - suresh --- This is an automatically generated e-mail. To reply, visit:

Jenkins build is still unstable: simulator-singlerun #282

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
Backported to 4.4 branch. -min From: Daan Hoogland daan.hoogl...@gmail.commailto:daan.hoogl...@gmail.com Date: Wednesday, September 3, 2014 11:52 AM To: dev dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org, Min Chen min.c...@citrix.commailto:min.c...@citrix.com Subject: Re: Commit

Build failed in Jenkins: cloudstack-4.4-maven-build-noredist #368

2014-09-03 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build-noredist/368/changes Changes: [min.chen] CLOUDSTACK-6236:fix a copy-and-paste error for decrRefCnt in -- [...truncated 2618 lines...] [INFO] Using 'UTF-8' encoding to copy filtered