Jenkins build is still unstable: simulator-singlerun #107

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 24599: CLOUDSTACK-7284: Fixed test script related to expunge VM in test_add_remove_network.py

2014-08-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24599/#review50417 --- Commit 76cecc325fca5ead8b610c49608cb8fe8a4395c5 in cloudstack's

Re: Review Request 24544: CLOUDSTACK-7294: Passing listall =True for listUsers api admin call

2014-08-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24544/#review50421 --- Commit 9d7b85153747dfe378031455fc1231bcd6088d08 in cloudstack's

Jenkins build is still unstable: simulator-singlerun #108

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: [SHOW] Authentication refactoring

2014-08-13 Thread Daan Hoogland
On Wed, Aug 13, 2014 at 5:10 AM, David Nalley da...@gnsa.us wrote: Code freeze means that we discuss every commit, and only critical or blocker bug fixes make it in. Which in practice means: RM reviews quickly and cherry-picks/merges it. -- Daan

Re: Should we remove all the -forward branches?

2014-08-13 Thread Sebastien Goasguen
On Aug 12, 2014, at 11:12 PM, David Nalley da...@gnsa.us wrote: On Tue, Aug 12, 2014 at 6:04 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: I remember we agreed to not use the -forward branches, so can we get rid of them please? I was reviewing someone’s patch and they had targeted

Jenkins build is still unstable: simulator-singlerun #109

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: [SHOW] Authentication refactoring

2014-08-13 Thread Rohit Yadav
Hi David, On 13-Aug-2014, at 5:10 am, David Nalley da...@gnsa.us wrote: On Tue, Aug 12, 2014 at 5:35 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: Hi Min, For for master, we don’t have any code freeze currently, I think merge request/reviews are not mandatory and I was confident with

Re: Should we remove all the -forward branches?

2014-08-13 Thread Rohit Yadav
Hey, On 13-Aug-2014, at 5:12 am, David Nalley da...@gnsa.us wrote: On Tue, Aug 12, 2014 at 6:04 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: I remember we agreed to not use the -forward branches, so can we get rid of them please? I was reviewing someone’s patch and they had targeted

Re: Review Request 24544: CLOUDSTACK-7294: Passing listall =True for listUsers api admin call

2014-08-13 Thread Gaurav Aradhye
On Aug. 13, 2014, 1:43 p.m., Santhosh Edukulla wrote: tools/marvin/marvin/cloudstackTestClient.py, line 348 https://reviews.apache.org/r/24544/diff/1/?file=657577#file657577line348 Make listall as keyword argument, instead of always making it to set to true. Are you suggesting

Re: Review Request 24544: CLOUDSTACK-7294: Passing listall =True for listUsers api admin call

2014-08-13 Thread Santhosh Edukulla
On Aug. 13, 2014, 8:13 a.m., Santhosh Edukulla wrote: tools/marvin/marvin/cloudstackTestClient.py, line 348 https://reviews.apache.org/r/24544/diff/1/?file=657577#file657577line348 Make listall as keyword argument, instead of always making it to set to true. Gaurav Aradhye

Re: Review Request 24238: Fix mvn marvin.sync profile

2014-08-13 Thread Rohit Yadav
On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: tools/marvin/setup.py, line 35 https://reviews.apache.org/r/24238/diff/1/?file=650483#file650483line35 Please tying like this to as a dependency on pom.xml, may unnecessarily lead to tight coupling with mvn logic inside of

Re: Should we remove all the -forward branches?

2014-08-13 Thread Daan Hoogland
4.4-forward should be removed. I have let it be as for a cooldown period. It has conflicts with 4.4 and I want to have a branch per fix/backport to merge. Leo's advice was to merge them as --no-ff. I haven't which has resulted in no merge commits in some cases. For single commits this is allright.

Re: Review Request 24557: CLOUDSTACK-7308 - Adds tagging support for security group rules

2014-08-13 Thread Stephen Hoogendijk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24557/#review50438 --- Ship it! Ship It! - Stephen Hoogendijk On Aug. 12, 2014, 2:39

Re: Should we remove all the -forward branches?

2014-08-13 Thread Sebastien Goasguen
On Aug 13, 2014, at 4:25 AM, Rohit Yadav rohit.ya...@shapeblue.com wrote: Hey, On 13-Aug-2014, at 5:12 am, David Nalley da...@gnsa.us wrote: On Tue, Aug 12, 2014 at 6:04 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: I remember we agreed to not use the -forward branches, so can we

Re: Review Request 24550: CLOUDSTACK-7306 Don't run test_01_primary_storage_iscsi on KVM or HyperV

2014-08-13 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24550/#review50440 --- Patch failed to apply on master. error: patch failed:

Re: [DISCUSS] Removing template URL format checking logic

2014-08-13 Thread Daan Hoogland
If this fixes the CLVM issue in KVM environments, I want it in 4.4. Rohit, can you expand on this? On Tue, Aug 12, 2014 at 11:40 PM, Min Chen min.c...@citrix.com wrote: Completely remove the checkFormat() will cause trouble for debugging issues later if the template format is wrong. I would

Review Request 24642: List vGPU host details and add non GPU host to GPU hosts cluster

2014-08-13 Thread sailaja mada
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24642/ --- Review request for cloudstack, Doug Clark and Sanjay Tripathi. Repository:

Re: Review Request 24550: CLOUDSTACK-7306 Don't run test_01_primary_storage_iscsi on KVM or HyperV

2014-08-13 Thread Alex Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24550/ --- (Updated Aug. 13, 2014, 10:36 a.m.) Review request for cloudstack and Santhosh

Re: Review Request 24642: List vGPU host details and add non GPU host to GPU hosts cluster

2014-08-13 Thread sailaja mada
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24642/ --- (Updated Aug. 13, 2014, 10:39 a.m.) Review request for cloudstack, Doug Clark

Jenkins build is still unstable: simulator-singlerun #111

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 24550: CLOUDSTACK-7306 Don't run test_01_primary_storage_iscsi on KVM or HyperV

2014-08-13 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24550/#review50441 --- test/integration/smoke/test_primary_storage.py

Re: Review Request 24550: CLOUDSTACK-7306 Don't run test_01_primary_storage_iscsi on KVM or HyperV

2014-08-13 Thread Alex Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24550/ --- (Updated Aug. 13, 2014, 10:44 a.m.) Review request for cloudstack and Santhosh

Re: Review Request 24550: CLOUDSTACK-7306 Don't run test_01_primary_storage_iscsi on KVM or HyperV

2014-08-13 Thread Alex Brett
On Aug. 13, 2014, 10:43 a.m., Santhosh Edukulla wrote: test/integration/smoke/test_primary_storage.py, line 158 https://reviews.apache.org/r/24550/diff/2/?file=659231#file659231line158 This check is not required right? As well, i just pushed a similar change to master, post the

Re: Review Request 24550: CLOUDSTACK-7306 Don't run test_01_primary_storage_iscsi on KVM or HyperV

2014-08-13 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24550/#review50443 --- Ship it! Ship It! - Santhosh Edukulla On Aug. 13, 2014, 10:44

Review Request 24643: CLOUDSTACK-7330: fixed ListNetworks issue in test_portable_ip.py

2014-08-13 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24643/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: CLOUDSTACK-7330

Re: [DISCUSS] Removing template URL format checking logic

2014-08-13 Thread Rohit Yadav
Hi Daan, On 13-Aug-2014, at 12:34 pm, Daan Hoogland daan.hoogl...@gmail.com wrote: If this fixes the CLVM issue in KVM environments, I want it in 4.4. Not related to CLVM directly, just template/url stuff. Rohit, can you expand on this? Not a major bugfix, can be even viewed as a feature

Versions on Jira and web

2014-08-13 Thread Alex Brett
Hello, I'm not sure who has the power to change this, but in the Apache Jira the versions for the CLOUDSTACK project are a bit out of sync with reality - we currently incorrectly list the following as unreleased: 4.1.1 4.2.1 4.3.0 4.4.0 Also, on the downloads page

Review Request 24645: Disabling failed test cases.

2014-08-13 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24645/ --- Review request for cloudstack and Santhosh Edukulla. Repository:

Re: Review Request 24644: CLOUDSTACK-7329: Fixed issues in test_region_vpc.py

2014-08-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24644/#review50446 --- Commit 525ee4a4aada2aa90bf53b301151f20111788e65 in cloudstack's

Re: Review Request 24643: CLOUDSTACK-7330: fixed ListNetworks issue in test_portable_ip.py

2014-08-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24643/#review50448 --- Commit b7c4259d9750cffe0cd0796b1434b39bff185b4d in cloudstack's

Jenkins build is still unstable: simulator-singlerun #112

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 24599: CLOUDSTACK-7284: Fixed test script related to expunge VM in test_add_remove_network.py

2014-08-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24599/#review50449 --- Commit f6bde7b46bb09aa13cb35049716bd7e1fc5b7774 in cloudstack's

Review Request 24646: CLOUDSTACK-7337: Volume state remains in allocated after volume creation faliure from snapshot

2014-08-13 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24646/ --- Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-7337

Jenkins build is still unstable: simulator-singlerun #113

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 24599: CLOUDSTACK-7284: Fixed test script related to expunge VM in test_add_remove_network.py

2014-08-13 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24599/#review50453 --- Commit 5f893daa2cab381aef6ac59eeebe923373c4ea80 in cloudstack's

Build failed in Jenkins: build-master-noredist #3313

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/3313/changes Changes: [Rohit Yadav] CLOUDSTACK-4770: In MacAddress skip macAddress when parsed value is 0x00 [girish] CLOUDSTACK-7284: Fixed regression issue in test_escalations_instances.py

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
On Aug. 13, 2014, 9:26 a.m., Rohit Yadav wrote: Hi Daniel, the patch is failing on current master. Please submit a git-formatted patch that would apply cleanly on master. Sorry about that, we were working on an outdated fork. Will produce a new patch file that applies on current master.

Review Request 24648: CLOUDSTACK-7310 - don't try and shink a disk in the volume resize test

2014-08-13 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24648/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: CLOUSTACK-7310

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

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/3314/

Jenkins build is still unstable: simulator-singlerun #114

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Interested in contribution

2014-08-13 Thread Parag Goyal
Hello, I have experience of development in java of about three years. I want to contribute to your organisation can you help in getting started ?

Re: Interested in contribution

2014-08-13 Thread Daan Hoogland
have a look at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Development+101 (and don't take it all for granted) On Wed, Aug 13, 2014 at 4:17 PM, Parag Goyal parag0...@gmail.com wrote: Hello, I have experience of development in java of about three years. I want to contribute to your

Re: Review Request 24648: CLOUDSTACK-7310 - don't try and shink a disk in the volume resize test

2014-08-13 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24648/ --- (Updated Aug. 13, 2014, 2:22 p.m.) Review request for cloudstack and Santhosh

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24611/ --- (Updated Aug. 13, 2014, 2:43 p.m.) Review request for cloudstack, Chiradeep

Re: Interested in contribution

2014-08-13 Thread Parag Goyal
thanks for replying sir i will surely take it seriously and will get back to you after reading this :) On Wed, Aug 13, 2014 at 7:49 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: have a look at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Development+101 (and don't take it all

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
On Aug. 12, 2014, 9:52 p.m., Rohit Yadav wrote: Hi Daniel, Thank you for your work, it looks comprehensive. Appreciate the tests and design doc on the wiki. Can you make sure we don't do wildcard imports, I personally don't have any problem with using them but a lot of

Re: Zones, Domains and Templates

2014-08-13 Thread Xerex Bueno
That sounds right. The only thing I can think of is if you mark the templates as public they will be able to be seen across multiple domains. On 8/12/14, 2:10 AM, Giri Prasad g_p...@yahoo.com.INVALID wrote: Thanks for your comments. I have the seen the documentation, talking of adding

Jenkins build is still unstable: simulator-singlerun #115

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Jenkins build is still unstable: simulator-singlerun #116

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24611/ --- (Updated Aug. 13, 2014, 5:08 p.m.) Review request for cloudstack, Chiradeep

Re: Interested in contribution

2014-08-13 Thread Mike Tutkowski
Thanks for your interest in CloudStack! Please refer to this page, as well: http://cloudstack.apache.org/developers.html It should help give you an idea regarding the process of submitting code to the project. We definitely like to work on CloudStack as a team (versus being super isolated and

Dedicating a network to an account in Adv+Security Groups

2014-08-13 Thread Nux!
Hi guys, I notice that in Adv zone one can dedicate a network to a single account, trying the same in Adv+SG fails miserably. Is there any work around for this? We have a customer willing to move into our deployment, but want to bring his own IPs as well. Lucian -- Sent from the Delta

Jenkins build is still unstable: simulator-singlerun #117

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

4.4.1 tests

2014-08-13 Thread Rohit Yadav
Hi, I’m tested CloudStack 4.4 branch for the upcoming 4.4.1 target release, using KVM on Ubuntu (amd64). I’m sharing my observations before creating bugs for every issue that should be fixed. Observations: # Installation - cloudstack-cli does nothing (why do we have it here or should we add a

RE: Regarding Shrinking of Volume

2014-08-13 Thread Chandan Purushothama
Marcus, May I know your response, Thank you, Chandan. -Original Message- From: Chandan Purushothama [mailto:chandan.purushoth...@citrix.com] Sent: Tuesday, August 12, 2014 12:37 PM To: Marcus Cc: dev@cloudstack.apache.org Subject: Regarding Shrinking of Volume Hello Marcus, I see

Re: 4.4.1 tests

2014-08-13 Thread Daan Hoogland
On Wed, Aug 13, 2014 at 8:29 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: Daan , please cherry-pick the fix from master to 4.4 if needed) ​seems reasonable, done​ -- Daan

[ACS44] 4.4.1 test and RC

2014-08-13 Thread Daan Hoogland
H all, I am working on testing 4.4.1 (4.4 branch). I am leaving for a two week holiday with low connectivity on Saturday. Therefore I do not think I will make/bake an RC soon and given the lack of volunteers lately, I suggest we plan for the first week of September. -- Daan

Jenkins build is still unstable: simulator-singlerun #118

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: [ACS44] 4.4.1 test and RC

2014-08-13 Thread Erik Weber
On Wed, Aug 13, 2014 at 9:15 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: H all, I am working on testing 4.4.1 (4.4 branch). I am leaving for a two week holiday with low connectivity on Saturday. Therefore I do not think I will make/bake an RC soon and given the lack of volunteers

Re: Regarding Shrinking of Volume

2014-08-13 Thread Mike Tutkowski
This is how Marcus responded: Yes, this is true. Ideally other storage maintainers/drivers would implement it as well, but we only committed a test for shrinking clvm (over a year ago now). I'd like to keep the test itself to run regression against clvm, but since this test is being used now for

Re: [ACS44] 4.4.1 test and RC

2014-08-13 Thread Amogh Vasekar
Hi, Wondering who do we reach out to for hotfix* branch merge, if any. Have a nice vacation! Thanks, Amogh On 8/13/14 12:47 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: On 13-Aug-2014, at 9:15 pm, Daan Hoogland daan.hoogl...@gmail.com wrote: H all, I am working on testing 4.4.1 (4.4

Re: Regarding Shrinking of Volume

2014-08-13 Thread John Dilley
Hi, I'll put a condition around the block that does the shrinking only in the case of CLVM. Would the disk image type be CLVM instead of qcow2? Thanks, John On 13 Aug 2014, at 20:39, Mike Tutkowski mike.tutkow...@solidfire.com wrote: This is how Marcus responded: Yes, this is true.

Re: Regarding Shrinking of Volume

2014-08-13 Thread Marcus
If you're asking what I think, it would be RAW. The pool type would be CLVM, though. This is probably a better solution, as we likely want to grow the cases that the automated tests run. If we're testing KVM at all it is trivial to set up an LVM volume group as a storage pool to test against.

Jenkins build is still unstable: simulator-singlerun #119

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Rohit Yadav
On Aug. 12, 2014, 9:52 p.m., Rohit Yadav wrote: Hi Daniel, Thank you for your work, it looks comprehensive. Appreciate the tests and design doc on the wiki. Can you make sure we don't do wildcard imports, I personally don't have any problem with using them but a lot of

RE: Regarding Shrinking of Volume

2014-08-13 Thread Chandan Purushothama
Thanks Marcus, John Mike for all the information, FS at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Resize+Data+Volumes claims that XenServer can shrink data volumes. Can I edit it and remove that claim from FS? Kindly let me know, Thank you, Chandan. -Original Message-

Jenkins build is still unstable: simulator-singlerun #120

2014-08-13 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/changes

Re: [ACS44] 4.4.1 test and RC

2014-08-13 Thread Pierre-Luc Dion
have a good vacation Daan, Will continue on testing 4.4.1 :) *Pierre-Luc DION* Architecte de Solution Cloud | Cloud Solutions Architect t 855.652.5683 *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw

Re: Regarding Shrinking of Volume

2014-08-13 Thread Nitin Mehta
I think there are 2 bugs here Api level - If shrinking is not allowed say in XS it should fail fast and say that strinking is not allowed in XS rather than going through the resource layer and trying the operation and getting the error from XS. Test case - I would modify the test case to expect

Re: Regarding Shrinking of Volume

2014-08-13 Thread Mike Tutkowski
That's an interesting question you raise about failing fast. On the one hand, it's nice from a usability standpoint. On the other, it adds in additional business logic (the API layer would then have business logic that is duplicated on the XenServer side). On Wed, Aug 13, 2014 at 6:40 PM,

Re: Regarding Shrinking of Volume

2014-08-13 Thread Mike Tutkowski
As an example, I will note that the GUI submits requests to the management server that it could - in theory - validate locally. However, the GUI is (rightly) intended in CloudStack to be a thin layer on top of the API. It purposefully avoids duplicating business logic that is run on the backend