Review Request 24300: listIsos call does not return isdynamicallyscalable in the response attributes as mentioned in API docs

2014-08-05 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24300/ --- Review request for cloudstack and Kishan Kavala. Bugs: https://issues.apache.or

Re: 4.4.1 related checkins

2014-08-05 Thread Daan Hoogland
If it is trivial, make it on 4.4. If it needs review and testing make it on it's own branch. Name it something like fix-4.4- and have the RM merge it after validation. Please note that this is a way of working extracted from the new workflow but not exactly the same. thanks, your latest RM (we ar

[JENKINS] simulator check on hot fix branches

2014-08-05 Thread Hugo Trippaers
Hey all, I’ve setup a simulator run that will automagically trigger on new or changed branches that start with the name hotfix. So if you want to have your fix checked before you commit it to master, create a new branch named hotfix/whatever, push it and track the results here http://jenkins.b

Regarding CLOUDSTACK-4819

2014-08-05 Thread Namita Chaudhari
Hi, I was working on a bug https://issues.apache.org/jira/browse/CLOUDSTACK-4819 - Created template within a project does not show up. ​So the issue basically is while listing templates or ISO a parameter "projectid" is passed and based on that a list of permittedaccounts is obtanined whose ISO's

Re: implementing git workflow

2014-08-05 Thread Daan Hoogland
Leo, great work. I should volunteer you more often. (volunteer me back at will) Rajani, Yes you are absolutaly right on the one hand. No, the community had decided that it couldn't support beyond the next release as it is to small, on the other hand. In the middle: By just saying this you have pro

Re: [JENKINS] simulator check on hot fix branches

2014-08-05 Thread Daan Hoogland
Wow, that is useful stuff. I remember having type just fix-... to sugest to people. have to ammend that :) On Tue, Aug 5, 2014 at 9:31 AM, Hugo Trippaers wrote: > Hey all, > > I’ve setup a simulator run that will automagically trigger on new or changed > branches that start with the name hotfix.

Re: 4.4.1 related checkins

2014-08-05 Thread Daan Hoogland
Amogh, make that branchname 'hotfix/4.4-' + jira ticket number. That will trigger an automatic build due to some great work of our jenkins adminstrator. On Tue, Aug 5, 2014 at 9:23 AM, Daan Hoogland wrote: > If it is trivial, make it on 4.4. If it needs review and testing make > it on it's own br

Re: [JENKINS] simulator check on hot fix branches

2014-08-05 Thread Hugo Trippaers
I can add more prefixes :-) Just let me know what the final prefixes will be for the git flow stuff and i’ll add them to the list. Cheers, Hugo On 5 aug. 2014, at 09:33, Daan Hoogland wrote: > Wow, that is useful stuff. I remember having type just fix-... to > sugest to people. have to amme

Re: [JENKINS] simulator check on hot fix branches

2014-08-05 Thread Daan Hoogland
It's ok, lets start with hotfix/ and go from there. I am sure we are going to want more of them. On Tue, Aug 5, 2014 at 9:40 AM, Hugo Trippaers wrote: > I can add more prefixes :-) > > Just let me know what the final prefixes will be for the git flow stuff and > i’ll add them to the list. > > Ch

Build failed in Jenkins: simulator-hotfix-trigger #1

2014-08-05 Thread jenkins
See -- Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on simulator in workspace

Build failed in Jenkins: simulator-hotfix-trigger #2

2014-08-05 Thread jenkins
See -- [...truncated 6981 lines...] + mvn -P systemvm,simulator -pl :cloud-client-ui jetty:run + '[' 0 -lt 44 ']' + grep -q 'Management server node 127.0.0.1 is up' jetty-output.out + sleep 5 +

Review Request 24301: Fixed Marvin Issue: Hypervisor value was hardcoded to XenServer while deploying VM if not specified

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24301/ --- Review request for cloudstack and Santhosh Edukulla. Repository: cloudstack-git

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

2014-08-05 Thread Leo Simons
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24238/ --- (Updated Aug. 5, 2014, 8:43 a.m.) Review request for cloudstack and Rohit Yadav

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

2014-08-05 Thread Leo Simons
> On Aug. 4, 2014, 4:35 p.m., Santhosh Edukulla wrote: > > tools/marvin/setup.py, line 35 > > > > > > Please tying like this to as a dependency on pom.xml, may unnecessarily > > lead to tight coupling with mvn logic in

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

2014-08-05 Thread Leo Simons
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24238/#review49575 --- tools/marvin/pom.xml

Build failed in Jenkins: simulator-hotfix-trigger #4

2014-08-05 Thread jenkins
See -- [...truncated 7402 lines...] + '[' 0 -lt 44 ']' + grep -q 'Management server node 127.0.0.1 is up' jetty-output.out + mvn -P systemvm,simulator -pl :cloud-client-ui jetty:run + sleep 5 +

Re: implementing git workflow

2014-08-05 Thread Rajani Karuturi
Daan, I think we should do the branch related changes[1] and make the switch. Are we waiting for a RM for 4.5? [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Git#Git-Movetogit-flow ~Rajani On 05-Aug-2014, at 1:01 pm, Daan Hoogland mailto:daan.hoogl...@gmail.com>> wrote: Leo, g

Build failed in Jenkins: build-master #1314

2014-08-05 Thread jenkins
See Changes: [jayapal] CLOUDSTACK-7246: corrected script name of createipalias -- [...truncated 936 lines...] [INFO] [INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-fr

Build failed in Jenkins: build-master-simulator #91

2014-08-05 Thread jenkins
See Changes: [jayapal] CLOUDSTACK-7246: corrected script name of createipalias -- [...truncated 806 lines...] [INFO] Deleting

Re: Review Request 24298: CLOUDSTACK-6873: Moved test cases that run only on simulator and those should be run serially to misc folder and also tagged them with required_hardware='simulator only'

2014-08-05 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24298/#review49576 --- According to the wiki https://cwiki.apache.org/confluence/display/C

Re: implementing git workflow

2014-08-05 Thread Daan Hoogland
On Tue, Aug 5, 2014 at 10:52 AM, Rajani Karuturi wrote: > Daan, > I think we should do the branch related changes[1] and make the switch. Are > we waiting for a RM for 4.5? > > [1] > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Git#Git-Movetogit-flow > > > ~Rajani No Rajani, We don't

Build failed in Jenkins: simulator-hotfix-trigger #5

2014-08-05 Thread jenkins
See -- [...truncated 6019 lines...] [INFO] Starting audit... Audit done. [INFO] [INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-console-proxy --- [INFO] [INFO] --- ma

Re: Review Request 24298: CLOUDSTACK-6873: Moved test cases that run only on simulator and those should be run serially to misc folder and also tagged them with required_hardware='simulator only'

2014-08-05 Thread Gaurav Aradhye
> On Aug. 5, 2014, 2:27 p.m., Rajani Karuturi wrote: > > According to the wiki > > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Validating+check-ins+for+your+local+changes%2C+using+Simulator > > I think the flag should be required_hardware=false Required_hardware = false tells us tha

Review Request 24302: CLOUDSTACK-7241: Fixed error in test_escalations_ipaddresses.py

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

Jenkins build is back to normal : build-master-simulator #92

2014-08-05 Thread jenkins
See

Jenkins build is back to normal : build-master #1315

2014-08-05 Thread jenkins
See

Review Request 24304: CLOUDSTACK-7247: Default hypervisor to None in deployVirtualMachine call in marvin base.py

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

Re: Review Request 24111: Coverity findings for brocade-plugin

2014-08-05 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24111/#review49578 --- Ship it! Ship It! - Santhosh Edukulla On July 31, 2014, 5:57 p.m

Re: Review Request 24304: CLOUDSTACK-7247: Default hypervisor to None in deployVirtualMachine call in marvin base.py

2014-08-05 Thread Gaurav Aradhye
Hi, I already submitted patch for this. https://reviews.apache.org/r/24301/ Regards, Gaurav On Tue, Aug 5, 2014 at 2:53 PM, John Dilley wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 24111: Coverity findings for brocade-plugin

2014-08-05 Thread Santhosh Edukulla
> On Aug. 5, 2014, 9:31 a.m., Santhosh Edukulla wrote: > > Ship It! Pushed to master Updated Branches: refs/heads/master 1bfb1f650 -> cc725e53e - Santhosh --- This is an automatically generated e-mail. To reply, visit: https://reviews

NO commits to master

2014-08-05 Thread Rajani Karuturi
I am creating a develop branch on the latest master commit cc725e53e304781148a6bf077e05d844fe88207c and will update in a minute. please wait for my next mail and push all the commit to develop branch there after. ~Rajani

Build failed in Jenkins: simulator-hotfix-trigger #6

2014-08-05 Thread jenkins
See -- [...truncated 7384 lines...] [INFO] --- maven-install-plugin:2.3.1:install (default-install) @ cloud-developer --- [INFO] Installing

Review Request 24305: CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py

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

Re: NO commits to master

2014-08-05 Thread Rajani Karuturi
develop branch is created as of commit cc725e53e304781148a6bf077e05d844fe88207c Please do not commit anything to master and use the respective branches. Please go through the detailed wiki by Leo @ https://cwiki.apache.org/confluence/display/CLOUDSTACK/Git ~Rajani On 05-Aug-2014, at 3:08 p

Re: Review Request 24305: CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24305/ --- (Updated Aug. 5, 2014, 3:15 p.m.) Review request for cloudstack and Santhosh Ed

Re: Review Request 24302: CLOUDSTACK-7241: Fixed error in test_escalations_ipaddresses.py

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24302/ --- (Updated Aug. 5, 2014, 3:15 p.m.) Review request for cloudstack and Santhosh Ed

Re: implementing git workflow

2014-08-05 Thread Rajani Karuturi
Hi Daan, I created develop branch on commit cc725e53e304781148a6bf077e05d844fe88207c and pushed it. I am trying to create the 4.5 branch. For this, I need to change the version number in develop branch. Release Procedure wiki mention a script to change to version number. [1] Is it safe to jus

Re: NO commits to master

2014-08-05 Thread Rohit Yadav
Hi Rajani, Thanks for taking the initiative, people may still keep pushing onto master, so please give them a window of 24-48 hours so that everyone is informed, or alternatively you may sync commits from master to develop but I hope we don’t end up in branch based split-brain where some folks

RE: implementing git workflow

2014-08-05 Thread Santhosh Edukulla
Post the proposal vote, we should have given some time buffer, or prior intimation of a day or two before this switch i believe. There were still people using master till today for their development purpose, holding on it at sudden may not be appropriate. Regards, Santhosh _

Review Request 24306: Review request for CLOUDSTACK-2694 - [Firewall Rule] Able to configure duplicate firewall rule with protocol and no ports

2014-08-05 Thread Namita Chaudhari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24306/ --- Review request for cloudstack and Jayapal Reddy. Bugs: CLOUDSTACK-2694 and http

Re: Review Request 24301: CLOUDSTACK-7247: Fixed Marvin Issue: Hypervisor value was hardcoded to XenServer while deploying VM if not specified

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24301/ --- (Updated Aug. 5, 2014, 3:32 p.m.) Review request for cloudstack and Santhosh Ed

Re: NO commits to master

2014-08-05 Thread Rajani Karuturi
we had enough time since the vote passed. But, I agree we should have decided on dates. master/develop are in sync right now. So, it should be a simple branch switch to develop and push. some automated things like jenkins may need changes. other than that, I don’t see any issues. A git hook i

RE: NO commits to master

2014-08-05 Thread Koushik Das
Why not cut out a new develop branch after the 24/48 hrs window and discard the current develop branch. That is much more simpler. -Original Message- From: Rohit Yadav [mailto:rohit.ya...@shapeblue.com] Sent: Tuesday, 5 August 2014 3:20 PM To: dev@cloudstack.apache.org Subject: Re: NO co

Re: NO commits to master

2014-08-05 Thread Rajani Karuturi
Its not any simpler. Its going to be the same work. I am not convinced that delay is going to help. But, I am going to abandon the develop branch and create one 48hrs later. ~Rajani On 05-Aug-2014, at 3:46 pm, Koushik Das wrote: > Why not cut out a new develop branch after the 24/48 hrs

Build failed in Jenkins: simulator-hotfix-trigger #7

2014-08-05 Thread jenkins
See Changes: [htrippaers] Improve the error message to see what is going on -- [...truncated 7381 lines...] > Processing upgrade: com.cloud.upgrade.DatabaseUpgradeChecker [INFO

master branch will be frozen for direct commits on 7 Aug 2014

2014-08-05 Thread Rajani Karuturi
I deleted the develop branch and will be recreating it in 48hrs. As per the new git flow which we agreed, all the next release work should move to develop branch and there shouldn’t be any direct commits on master. Please make sure you go through wiki @ https://cwiki.apache.org/confluence/displ

Re: NO commits to master

2014-08-05 Thread Daan Hoogland
Rajani, I think you did great work. The fact that more communication could have helped is in no way on you. There is a wiki page with a proposal that has been voted on. No -1! Having said that let's put down a time line. One thing in there is, in my opinion that as of now no one should commit to m

Re: implementing git workflow

2014-08-05 Thread Rajani Karuturi
The switch to the new git flow is postponed to 7th aug. we have two open questions to answer before that: 1. Can we enable a git hook to prevent the commits to master post the switch? 2. should we create the release branch as well or wait for RM to do it? ~Rajani On 05-Aug-2014, at 3:16 pm, R

Re: Review Request 23837: CLOUDSTACK-7087: Downgrade openswan to previous version for VPN services to fix OSX client

2014-08-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23837/#review49581 --- Commit 2dfd278625a5d569a5cce2fec5ccb0d7c515c9e8 in cloudstack's bran

Re: implementing git workflow

2014-08-05 Thread Daan Hoogland
Santhosh, Though in principle you are right I hardly think this can be called suddenly. On Tue, Aug 5, 2014 at 11:52 AM, Santhosh Edukulla wrote: > Post the proposal vote, we should have given some time buffer, or prior > intimation of a day or two before this switch i believe. There were still

Re: Review Request 24301: CLOUDSTACK-7247: Fixed Marvin Issue: Hypervisor value was hardcoded to XenServer while deploying VM if not specified

2014-08-05 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24301/#review49582 --- Ship it! Ship It! - Santhosh Edukulla On Aug. 5, 2014, 10:02 a.m

Re: implementing git workflow

2014-08-05 Thread Daan Hoogland
ad 1. Git hooks in the central repo require assistence of apache infra. I think all committers should be able to do this. If people don't comply their work should be reverted. ad 2. We shouldn't need an RM. The gitflow model implies that committers share the load of RM. On Tue, Aug 5, 2014 at 12:3

Re: Review Request 24301: CLOUDSTACK-7247: Fixed Marvin Issue: Hypervisor value was hardcoded to XenServer while deploying VM if not specified

2014-08-05 Thread Santhosh Edukulla
> On Aug. 5, 2014, 10:41 a.m., Santhosh Edukulla wrote: > > Ship It! dfa607f..2109418 master -> master - Santhosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24301/#review49582 -

Re: Review Request 24229: upgrade system templates in 4.4.x

2014-08-05 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24229/#review49584 --- Daan, This looks good to me. Can you also please remove template up

Re: Review Request 24229: upgrade system templates in 4.4.x

2014-08-05 Thread Kishan Kavala
> On Aug. 5, 2014, 4:12 p.m., Kishan Kavala wrote: > > Daan, > > This looks good to me. > > Can you also please remove template upgrade code in Upgrade421to430.java. > > Otherwise upgrades from 4.2 will fail complaining that > > systemvm-xenserver-4.3 is not available. I mean systemvm--4.3 te

Re: implementing git workflow

2014-08-05 Thread Rohit Yadav
Hi Rajani, On 05-Aug-2014, at 12:35 pm, Rajani Karuturi wrote: > The switch to the new git flow is postponed to 7th aug. Thanks for that, this means you’ll do it again on 7th August 00:00UTC? Meanwhile this means we’re to follow our normal workflow and commit on master. > we have two open que

Review Request 24308: Enable Password Strength check for all users

2014-08-05 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24308/ --- Review request for cloudstack and Darren Shepherd. Bugs: https://issues.apache.

Build failed in Jenkins: simulator-hotfix-trigger #8

2014-08-05 Thread jenkins
See Changes: [htrippaers] It's the template that has the simulator type, not the deploy target. -- [...truncated 7393 lines...] [INFO]

Re: implementing git workflow

2014-08-05 Thread Daan Hoogland
On Tue, Aug 5, 2014 at 12:45 PM, Rohit Yadav wrote: >> 1. Can we enable a git hook to prevent the commits to master post the switch? > > It’s possible theoretically but to do that in less than 48 hours could be a > challenge, you can open a ticket on JIRA/ASF-infra or contact the fine folks > on

Re: Review Request 24305: CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py

2014-08-05 Thread Santhosh Edukulla
> On Aug. 5, 2014, 12:01 p.m., Santhosh Edukulla wrote: > > Ship It! Please change the branch to master, its still open for commits, pushed to master. 64b9ca0..c76302c master -> master - Santhosh --- This is an automatically generated

Re: Review Request 24305: CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py

2014-08-05 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24305/#review49589 --- Ship it! Ship It! - Santhosh Edukulla On Aug. 5, 2014, 9:45 a.m.

Re: Review Request 24305: CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py

2014-08-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24305/#review49591 --- Commit c76302cdd8a70498a792db5d7fbe82ad43b7f8b5 in cloudstack's bran

Re: Review Request 24305: CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24305/ --- (Updated Aug. 5, 2014, 5:33 p.m.) Review request for cloudstack and Santhosh Ed

Re: Review Request 24302: CLOUDSTACK-7241: Fixed error in test_escalations_ipaddresses.py

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24302/ --- (Updated Aug. 5, 2014, 5:34 p.m.) Review request for cloudstack and Santhosh Ed

Build failed in Jenkins: simulator-singlerun #52

2014-08-05 Thread jenkins
See Changes: [santhosh.edukulla] Fixed Marvin Issue: Hypervisor value was hardcoded to XenServer while deploying VM if not specified [kishan.kavala] CLOUDSTACK-7087: Latest OS X VPN client not working Downgrading openswan vers

Re: Review Request 24229: upgrade system templates in 4.4.x

2014-08-05 Thread Pierre-Luc Dion
Will this be part of 4.4.1 ? if so, we will have to update Release-notes for upgrade steps from 4.2.x to 4.4 which for the moment use the template named systemvm--4.3. Thanks, *Pierre-Luc DION* Architecte de Solution Cloud | Cloud Solutions Architect t 855.652.5683 *CloudOps* Votre partenaire

Re: implementing git workflow

2014-08-05 Thread Rajani Karuturi
~Rajani On 05-Aug-2014, at 4:15 pm, Rohit Yadav wrote: > Hi Rajani, > > On 05-Aug-2014, at 12:35 pm, Rajani Karuturi > wrote: > >> The switch to the new git flow is postponed to 7th aug. > > Thanks for that, this means you’ll do it again on 7th August 00:00UTC? I am planning to do this

Re: implementing git workflow

2014-08-05 Thread Rajani Karuturi
On 05-Aug-2014, at 4:47 pm, Daan Hoogland wrote: > On Tue, Aug 5, 2014 at 12:45 PM, Rohit Yadav > wrote: >>> 1. Can we enable a git hook to prevent the commits to master post the >>> switch? >> >> It’s possible theoretically but to do that in less than 48 hours could be a >> challenge, you

Re: Review Request 24306: Review request for CLOUDSTACK-2694 - [Firewall Rule] Able to configure duplicate firewall rule with protocol and no ports

2014-08-05 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24306/#review49593 --- Ship it! Ship It! - Jayapal Reddy On Aug. 5, 2014, 10 a.m., Nami

Re: Review Request 24306: Review request for CLOUDSTACK-2694 - [Firewall Rule] Able to configure duplicate firewall rule with protocol and no ports

2014-08-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24306/#review49594 --- Commit fce2880b9f20a372dd0a005c4575ffab526bac0b in cloudstack's bran

Re: Review Request 24306: Review request for CLOUDSTACK-2694 - [Firewall Rule] Able to configure duplicate firewall rule with protocol and no ports

2014-08-05 Thread Jayapal Reddy Uradi
Hi Namita, The patch is applied on master. Please close the review ticket. Thanks, Jayapal On 05-Aug-2014, at 5:57 PM, Jayapal Reddy mailto:jayapalreddy.ur...@citrix.com>> wrote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24306/ Ship it! Ship I

Re: Review Request 23805: Added "state" parameter to the "listPublicIpAddresses" API call

2014-08-05 Thread Ilia Shakitko
> On July 29, 2014, 10:20 a.m., Rohit Yadav wrote: > > Looks fine, we still need to test it before we land it on master. I > > unfortunately don't have my hardware (and some parts) setup will try with > > DevCloud or hope someone else would review, test and ship it :) > > Ilia Shakitko wrote:

Re: Review Request 24298: CLOUDSTACK-6873: Moved test cases that run only on simulator and those should be run serially to misc folder and also tagged them with required_hardware='simulator only'

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24298/ --- (Updated Aug. 5, 2014, 6:24 p.m.) Review request for cloudstack and Santhosh Ed

Re: Review Request 24232: Removing invalid and unused import from test_escalations_templates.py

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24232/ --- (Updated Aug. 5, 2014, 6:28 p.m.) Review request for cloudstack and Santhosh Ed

Build failed in Jenkins: simulator-singlerun #53

2014-08-05 Thread jenkins
See Changes: [santhosh.edukulla] Fixed a coverity reported concurrency issue [santhosh.edukulla] CLOUDSTACK-7240: Fixed expunge VM issue in test_persistent_networks.py -- [...truncated 7

Re: Review Request 24229: upgrade system templates in 4.4.x

2014-08-05 Thread Daan Hoogland
Pierre-Luc, I plan to apply Kishan's comment and add it to 4.4.1 On Tue, Aug 5, 2014 at 2:13 PM, Pierre-Luc Dion wrote: > Will this be part of 4.4.1 ? if so, we will have to update Release-notes for > upgrade steps from 4.2.x to 4.4 which for the moment use the template named > systemvm--4.3. > >

Re: implementing git workflow

2014-08-05 Thread Daan Hoogland
That sound good to me On Tue, Aug 5, 2014 at 2:15 PM, Rajani Karuturi wrote: > > On 05-Aug-2014, at 4:47 pm, Daan Hoogland wrote: > >> On Tue, Aug 5, 2014 at 12:45 PM, Rohit Yadav >> wrote: 1. Can we enable a git hook to prevent the commits to master post the switch? >>> >>> It’s po

[acs441] planning?

2014-08-05 Thread Pierre-Luc Dion
Hi, I'm willing to work on the release-notes and tests of 4.4.1. I'd like to know what is the planning of this release. - there is no version 4.4.1 in jira to follow bug fixes, - git branch = 4.4 - how can we know what would be targeted bug fixes for this release? - I thing Daan is the Release Ma

Build failed in Jenkins: simulator-singlerun #54

2014-08-05 Thread jenkins
See Changes: [jayapal] CLOUDSTACK-2694: Able to configure duplicate firewall rule with protocol and no ports -- [...truncated 7299 lines...] main: [INFO] Executed tasks [INFO] [INFO] >>

Re: [acs441] planning?

2014-08-05 Thread Daan Hoogland
Pierre-Luc, I will make a tag in Jira. The branch will remain 4.4. The release is not yet planned. I will have to do some work still on the template update issue. The silence around the other issues so far leaves me to believe that 4.4.1 will not be a significant improvement if we plan it now. I w

[ACS44] jira tag for release 4.4.1

2014-08-05 Thread Daan Hoogland
people, I have created a tag in jira for marking issues as affecting or solved in 4.4.1 Please make abundant use of it. thanks -- Daan

Review Request 24313: CLOUDSTACK-7255: Fixed marvin issue related to creating random account usernames

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

Adding README file

2014-08-05 Thread Rohit Yadav
Added a README file for users and developers landing on Github or going thorough freshly downloaded repo: https://github.com/apache/cloudstack/blob/master/README.md Feel free to edit/fix the file as needed. Daan, if you want this in 4.4 branch? Regards, Rohit Yadav Software Architect, ShapeBlue

Re: Adding README file

2014-08-05 Thread Daan Hoogland
On Tue, Aug 5, 2014 at 4:19 PM, Rohit Yadav wrote: > Daan, if you want this in 4.4 branch? > Sure, I'll have a read and see if it is worth picking -- Daan

Re: Review Request 24298: CLOUDSTACK-6873: Moved test cases that run only on simulator and those should be run serially to misc folder and also tagged them with required_hardware='simulator only'

2014-08-05 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24298/ --- (Updated Aug. 5, 2014, 8:43 p.m.) Review request for cloudstack and Santhosh Ed

Build failed in Jenkins: simulator-singlerun #55

2014-08-05 Thread jenkins
See Changes: [Rohit Yadav] README: Give CloudStack its much deserved readme file [Rohit Yadav] README: Fix links for viewing on Github -- [...truncated 7284 lines...] main: [INFO] Execut

Re: Review Request 23805: Added "state" parameter to the "listPublicIpAddresses" API call

2014-08-05 Thread Ilia Shakitko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23805/ --- (Updated Aug. 5, 2014, 4:02 p.m.) Review request for cloudstack, Alena Prokharc

Re: Review Request 23805: Added "state" parameter to the "listPublicIpAddresses" API call

2014-08-05 Thread Ilia Shakitko
> On July 29, 2014, 10:20 a.m., Rohit Yadav wrote: > > Looks fine, we still need to test it before we land it on master. I > > unfortunately don't have my hardware (and some parts) setup will try with > > DevCloud or hope someone else would review, test and ship it :) > > Ilia Shakitko wrote:

Review Request 24314: CLOUDSTACK-6695: UI: Added support for uploading a chain of certificates

2014-08-05 Thread Mihaela Stoica
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24314/ --- Review request for cloudstack, Brian Federle, Jessica Wang, and Nitin Mehta. Re

Re: Review Request 23805: Added "state" parameter to the "listPublicIpAddresses" API call

2014-08-05 Thread Rohit Yadav
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23805/#review49608 --- Ship it! Ship It! - Rohit Yadav On Aug. 5, 2014, 4:02 p.m., Ilia

Build failed in Jenkins: simulator-singlerun #56

2014-08-05 Thread jenkins
See -- [...truncated 7274 lines...] [INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ cloud-developer --- [INFO] Starting audit... Audit done. [INFO] [INFO] --- properties-

Re: Review Request 23805: Added "state" parameter to the "listPublicIpAddresses" API call

2014-08-05 Thread Rohit Yadav
> On Aug. 5, 2014, 4:12 p.m., Rohit Yadav wrote: > > Ship It! Thanks for the git formatted patch, tested to build. Merged on master: commit 07f916dfc6fb30dc928b258beeaa761253e43ab9 Author: Ilia Shakitko Date: Tue Aug 5 17:58:46 2014 +0200 CLOUDSTACK-7159 Added "state" parameter to the "

DashBoard Stats

2014-08-05 Thread mo
Hello, Could someone please tell me, how I can obtain latest stats to be included in % for Secondary Storage | Primary Storage. It shows what I have used so far, but not the precent, like the others.

Build failed in Jenkins: simulator-hotfix-trigger #9

2014-08-05 Thread jenkins
See -- [...truncated 7107 lines...] Audit done. [INFO] [INFO] <<< exec-maven-plugin:1.2.1:java (create-schema-simulator) @ cloud-developer <<< [INFO] [INFO] --- exec-maven-plugin:1.2.1:java

[ACS441] Hotfix merge request

2014-08-05 Thread Amogh Vasekar
Hi Daan, Can you please merge hotfix/4.4-7257 into 4.4, that fixes CLOUDSTACK-7257. Thanks, Amogh

Re: implementing git workflow

2014-08-05 Thread David Nalley
> > we have two open questions to answer before that: > > 1. Can we enable a git hook to prevent the commits to master post the > > switch? > > It’s possible theoretically but to do that in less than 48 hours could be a > challenge, you can open a ticket on JIRA/ASF-infra or contact the fine folk

Build failed in Jenkins: simulator-singlerun #57

2014-08-05 Thread jenkins
See Changes: [Rohit Yadav] README: Import README sections from 4.4 branch into master [Rohit Yadav] CLOUDSTACK-7159 Added "state" parameter to the "listPublicIpAddresses" API call [Rohit Yadav] server: use scoped variable in p

Re: [VOTE] git workflow

2014-08-05 Thread Min Chen
Agree with Animesh. Didn't see any gains from this, we just shift pain from one branch to another, so vote -1. -min On 8/2/14 9:50 PM, "Animesh Chaturvedi" wrote: >+0 > >While this protects master with only commits which are merges from >release branch and keeps it clean but the issues that we

RE: [VOTE] git workflow

2014-08-05 Thread Jessica Wang
Exactly. This just shifts pain from one branch to another. I don't see any gains from this, either. I vote "-1". Jessica -Original Message- From: Min Chen [mailto:min.c...@citrix.com] Sent: Tuesday, August 05, 2014 11:27 AM To: dev@cloudstack.apache.org Subject: Re: [VOTE] git workflow

Re: implementing git workflow

2014-08-05 Thread Alena Prokharchyk
I think we should hold on with the git workflow implementation till all the decisions on the items written by Leo, are discussed and finalized. The very beginning of ³git workflow vote² shows that the vote was just to get people opinion on the proposal. Before adopting it and cutting the develop b

  1   2   >