Re: Review Request 18970: CLOUDSTACK-6215: Updating test case to accomodate condition for VMware

2014-03-14 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18970/#review37165 --- Ship it! Ship It! - SrikanteswaraRao Talluri On March 10, 2014,

Re: Review Request 18970: CLOUDSTACK-6215: Updating test case to accomodate condition for VMware

2014-03-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18970/#review37166 --- Commit 286974488eab03efddfea35962161a224e997ab0 in cloudstack's

Re: Review Request 18970: CLOUDSTACK-6215: Updating test case to accomodate condition for VMware

2014-03-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18970/#review37168 --- Commit 768e91ef4560b15deb4804e3593b3ad22fa18eef in cloudstack's

Re: Review Request 18971: CLOUDSTACK-6216: Skip the test if run on already existing setup

2014-03-14 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18971/#review37169 --- test/integration/smoke/test_secondary_storage.py

Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

2014-03-14 Thread Mandar Barve
I tried to reproduce the issue the way you mentioned with few changes. I don't have iSCSI SAN on my setup. I connected a 2 GB disk that I presented as storage tagged NFS primary to CS. I created a 1GB disk offering and then deployed a VM on this new primary. Took a couple of snapshots like you

Re: Anybody saw this error on master?

2014-03-14 Thread Marcus
I see this on KVM as well. I don't recall hearing about the GPU feature, sounds interesting. 014-03-14 01:14:06,302 DEBUG [agent.manager.AgentManagerImpl] (StatsCollector-1:ctx-b3b2e81f) Details from executing class com.cloud.agent.api.GetGPUStatsCommand: Unsupported command

Re: Anybody saw this error on master?

2014-03-14 Thread Marcus
oh yes, I do remember this. We probably need to fix the null pointer, but I think it's more or less just noise. On Fri, Mar 14, 2014 at 1:15 AM, Marcus shadow...@gmail.com wrote: I see this on KVM as well. I don't recall hearing about the GPU feature, sounds interesting. 014-03-14

RE: Anybody saw this error on master?

2014-03-14 Thread Sanjay Tripathi
Hi Min, Which version of XS you are working on? --Sanjay -Original Message- From: Min Chen [mailto:min.c...@citrix.com] Sent: Friday, March 14, 2014 6:18 AM To: dev@cloudstack.apache.org Subject: Anybody saw this error on master? Hi there, Anybody has seen this error on master? It is

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Marcus
I have no idea if its related to this branch merge or not, but I'm unable to start the ssvm on master since I pulled about an hour ago. I can deploy a fresh zone, and the ssvm will actually start, but it can't transition state in the DB, so it kills the vm. 2014-03-14 01:57:46,356 DEBUG

Build failed in Jenkins: build-master-slowbuild #426

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-slowbuild/426/changes Changes: [koushik] CLOUDSTACK-6052: List VM enhancement to support querying with multiple VM IDs [talluri] CLOUDSTACK-6215: Updating test case to accomodate condition for VMware

RE: Anybody saw this error on master?

2014-03-14 Thread Sanjay Tripathi
I have resolved this issue under this ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-6238 --Sanjay -Original Message- From: Marcus [mailto:shadow...@gmail.com] Sent: Friday, March 14, 2014 1:09 PM To: dev@cloudstack.apache.org Cc: Sanjay Tripathi Subject: Re: Anybody saw this

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Marcus
creating a new router does the same... 2014-03-14 02:06:41,446 DEBUG [vm.dao.VMInstanceDaoImpl] (Work-Job-Executor-5:Job-41/Job-42 ctx-91928df8) Unable to update VM[DomainRouter|r-5-VM]: DB Data={Host=1; State=Running; updated=3; time=Fri Mar 14 02:06:11 MDT 2014} New Data: {Host=1;

RE: Release cadence

2014-03-14 Thread Paul Angus
I think it's a very difficult subject, because we're relying on the developers to write tests which will show whether the code in their feature is good, and think of every possible failure mode in order to ratify their code. Obviously if they can think of the failure, they'll probably have

RE: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Paul Angus
Will these automatically go into 4.3.1, do we know if they've gone into master as well? Otherwise does this mean we have a load of bug fixes which we're not putting into 4.3.0 which could potentially become orphaned in the 4.3-forward branch? (this query may be just my ignorance regarding ACS

Re: Master is broken?

2014-03-14 Thread Antonio Fornié Casarrubios
Sorry, I see what the problem is. I've been looking for other possible side effects. But this should only affect anything in ApiDispatcher after standardDispatchChain.dispatch(new DispatchTask(cmd, params)); because the params used are not the same object processed in the dispatchChain. I'm going

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
On March 11, 2014, 12:28 p.m., Donal Lafferty wrote: 1. How do you guarantee that 'sudo' is never added to scripts created by the Script class? See (utils/src/com/cloud/utils/script/Script.java) 2. How do you version control the .exe? 3. WiX can be run on Linux in a WINE

Re: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Daan Hoogland
Paul, they might be disapproved patches as well. On Fri, Mar 14, 2014 at 9:38 AM, Paul Angus paul.an...@shapeblue.com wrote: Will these automatically go into 4.3.1, do we know if they've gone into master as well? Otherwise does this mean we have a load of bug fixes which we're not putting

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/ --- (Updated March 14, 2014, 9:16 a.m.) Review request for cloudstack, Abhinandan

Re: Release cadence

2014-03-14 Thread Daan Hoogland
Our problem is in the test coverage of the existing code base not in new features! As \Paul said dev will code tests they can think of. It's the ones they didn't think of that will break stuff. But we don't mind if new features aren't fully functional. We mind regression the most. You can not

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
On March 11, 2014, 12:28 p.m., Donal Lafferty wrote: 1. How do you guarantee that 'sudo' is never added to scripts created by the Script class? See (utils/src/com/cloud/utils/script/Script.java) 2. How do you version control the .exe? 3. WiX can be run on Linux in a WINE

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review37185 --- The exe which was a major issue for this patch has been removed.

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/ --- (Updated March 14, 2014, 9:42 a.m.) Review request for cloudstack, Abhinandan

Re: 4.4 Feature Freeze

2014-03-14 Thread Nux!
On 11.03.2014 15:28, Hugo Trippaers wrote: Hey guys, I didn’t go and tally all the +1s and -1s for the shift of the feature freeze, but with so many reactions i feel sticking to the schedule is the only way to go. We should only change dates if there is a consensus and there clearly is none at

Re: Master is broken?

2014-03-14 Thread Antonio Fornié Casarrubios
The params are converted to lowercase, the problem is the processed params are available only inside the dispatch chain, so later when the rest of the method ApiDispatcher#dispatch tries to get the param, in the map where it was looking up it was still with Camel Case name. I think the proper way

Re: Review Request 19038: CLOUDSTACK-6090: Virtual Router Service Failure Alerting

2014-03-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19038/#review37187 --- Commit f7337527cf9dbb35ec4c60621d4b37173635660c in cloudstack's

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review37186 --- Can you add some doc or ref to doc that explains how to test this

RE: [Merge] Storage motion for hyper-v

2014-03-14 Thread Devdeep Singh
I'll merge these changes later today. Regards, Devdeep -Original Message- From: Devdeep Singh [mailto:devdeep.si...@citrix.com] Sent: Wednesday, March 12, 2014 7:32 AM To: dev@cloudstack.apache.org Subject: [Merge] Storage motion for hyper-v Hi, I would like to merge storage

Re: Master is broken?

2014-03-14 Thread Daan Hoogland
If you have an account on the wiki, I can add some rights to it (or any pmc member could) On Fri, Mar 14, 2014 at 10:47 AM, Antonio Fornié Casarrubios antonio.for...@gmail.com wrote: The params are converted to lowercase, the problem is the processed params are available only inside the

Re: Master is broken?

2014-03-14 Thread Antonio Fornié Casarrubios
I do have for Antonio Fornie antonio.for...@gmail.com 2014-03-14 11:02 GMT+01:00 Daan Hoogland daan.hoogl...@gmail.com: If you have an account on the wiki, I can add some rights to it (or any pmc member could) On Fri, Mar 14, 2014 at 10:47 AM, Antonio Fornié Casarrubios

Re: Master is broken?

2014-03-14 Thread Daan Hoogland
On Fri, Mar 14, 2014 at 11:08 AM, Antonio Fornié Casarrubios antonio.for...@gmail.com wrote: antonio.fornie is in -- Daan

Re: Review Request 19038: CLOUDSTACK-6090: Virtual Router Service Failure Alerting

2014-03-14 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19038/#review37188 --- Ship it! master - f7337527cf9dbb35ec4c60621d4b37173635660c,

slow build

2014-03-14 Thread Daan Hoogland
Guys `n' dolls, One if the new tools for maintaining our quality is the new job slowbuild. It does a build and then runs findbugs (and other tools) against our code base. Since a while this has been going on and it succeeds if no new warnings where introduced since last successful build. For a

RE: [Merge] CLOUDSTACK-6106 vpc support on Hyper-V

2014-03-14 Thread Rajesh Battala
I will be merging my branch changes later today. Thanks Rajesh Battala -Original Message- From: Rajesh Battala [mailto:rajesh.batt...@citrix.com] Sent: Wednesday, March 12, 2014 11:42 AM To: dev@cloudstack.apache.org Subject: RE: [Merge] CLOUDSTACK-6106 vpc support on Hyper-V Fixed all

Build failed in Jenkins: build-master-slowbuild #427

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-slowbuild/427/changes Changes: [sanjay.tripathi] CLOUDSTACK-6238: NPE in HostStatsCollector. [koushik] CLOUDSTACK-6090: Virtual Router Service Failure Alerting -- [...truncated 18054 lines...] Tests

Re: Review Request 19036: CLOUDSTACK-6092: Storage OverProvisioning as a Per Primary Basis

2014-03-14 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19036/ --- (Updated March 14, 2014, 11:21 a.m.) Review request for cloudstack, Devdeep

Re: Review Request 19036: CLOUDSTACK-6092: Storage OverProvisioning as a Per Primary Basis

2014-03-14 Thread Sateesh Chodapuneedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19036/#review37190 --- Ship it! Ship It! - Sateesh Chodapuneedi On March 14, 2014,

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

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/2469/changes Changes: [Devdeep Singh] CLOUDSTACK-6143: Storage motion support for hyper-v. With these changes a volume on a shared [muralimmreddy] -introduces 'DistributedRouter' as capability to 'Connectivity' service.

Build failed in Jenkins: build-master #475

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/475/changes Changes: [muralimmreddy] -introduces 'DistributedRouter' as capability to 'Connectivity' service. [muralimmreddy] -add check to ensure 'Connectivity' service provider specified in [muralimmreddy] mark VPC to be using distributed

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

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/476/changes

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 13.03.2014 21:24, Animesh Chaturvedi wrote: [Animesh] Did you see this with prior RC too? [Animesh] Nux, security group support for advanced zone is limited and that too was developed in 4.2. I don’t think any changes have been made to that support since then. Can you call out what specific

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/ --- (Updated March 14, 2014, 12:10 p.m.) Review request for cloudstack, Abhinandan

Re: Review Request 19036: CLOUDSTACK-6092: Storage OverProvisioning as a Per Primary Basis

2014-03-14 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19036/ --- (Updated March 14, 2014, 12:10 p.m.) Review request for cloudstack, Devdeep

Re: Review Request 19036: CLOUDSTACK-6092: Storage OverProvisioning as a Per Primary Basis

2014-03-14 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19036/ --- (Updated March 14, 2014, 12:10 p.m.) Review request for cloudstack, Devdeep

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
On March 14, 2014, 9:58 a.m., daan Hoogland wrote: server/src/com/cloud/server/ConfigurationServerImpl.java, line 881 https://reviews.apache.org/r/18964/diff/3/?file=519572#file519572line881 why is this condition needed if both the if and the else branch take the same action.

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

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

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review37191 --- Hi Daan I have updated the FS in wiki regarding how to build and

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review37192 --- Hi Daan I have updated the FS in wiki regarding how to build and

Re: Review Request 19036: CLOUDSTACK-6092: Storage OverProvisioning as a Per Primary Basis

2014-03-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19036/#review37193 --- Commit 48f8a95b06b0348ba1349cb5434183c2c18710db in cloudstack's

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 14.03.2014 12:06, Nux! wrote: It looks like the traffic doesn't go in the right chains, all traffic is accepted as FORWARD is set to ACCEPT. There are zero packets going through BF-breth0-109. Here's outputs from: iptables-save: http://paste.fedoraproject.org/85337/47982321/raw/

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-14 Thread Punith S
hi guys, i have resolved all issues with fixes and refactoring of plugin code, its good to go now , i'll submit once i get the ship it message. thanks for the review. On Fri, Mar 14, 2014 at 11:19 AM, punith s punit...@cloudbyte.com wrote: This is an automatically generated e-mail. To

Re: Master is broken?

2014-03-14 Thread Srikanteswararao Talluri
I am seeing another issue on master. Filed a bug for this https://issues.apache.org/jira/browse/CLOUDSTACK-6239 Thanks, ~Talluri On 14/03/14 3:50 pm, Daan Hoogland daan.hoogl...@gmail.com wrote: On Fri, Mar 14, 2014 at 11:08 AM, Antonio Fornié Casarrubios antonio.for...@gmail.com wrote:

Re: 4.4 Feature Freeze

2014-03-14 Thread Marcus
Will be doing it today, however it seems that there have been some new issues that have cropped up in master that break basic functionality. I'm not sure if anyone should merge or not while that is the case. I was attempting to validate that the latest merges played well in my branch, but I

RE: 4.4 Feature Freeze

2014-03-14 Thread Sudha Ponnaganti
Can the offending check-ins be reverted or master be blocked for further check-ins till blocking issue is fixed. Talluri is running BVTs and can that be taken as basic passing criteria to re-enable check-ins. -Original Message- From: Marcus [mailto:shadow...@gmail.com] Sent: Friday,

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Ove Ewerlid
-1 given the regression in Security Groups for advanced zones. I was able to duplicate NUX reported issues, e.g., works in ACS421 but does not work at all in ACS43. /Ove On 03/13/2014 01:26 AM, Animesh Chaturvedi wrote: Hi All, I've created a 4.3.0 release, with the following artifacts up

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Ove Ewerlid
It should be noted that my tests use a single IP per VM. I believe NUX mentioned using multiple IP's. When SG in advanced zone is enabled, only one NIC can be assigned per VM. /Ove On 03/14/2014 02:41 PM, Ove Ewerlid wrote: On 03/14/2014 01:57 PM, Nux! wrote: On 14.03.2014 12:06, Nux! wrote:

Build failed in Jenkins: cloudstack-4.4-maven-build #1

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/1/ -- [...truncated 60 lines...] [INFO] Apache CloudStack Plugin - Host Allocator Random [INFO] Apache CloudStack Plugin - Dedicated Resources [INFO] Apache CloudStack Plugin - Hypervisor

Re: Review Request 19036: CLOUDSTACK-6092: Storage OverProvisioning as a Per Primary Basis

2014-03-14 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19036/#review37196 --- Commit 48f8a95b06b0348ba1349cb5434183c2c18710db in cloudstack's

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Wei ZHOU
Did you test on fresh 4.2.1, or upgraded platform? 2014-03-14 14:51 GMT+01:00 Ove Ewerlid ove.ewer...@oracle.com: It should be noted that my tests use a single IP per VM. I believe NUX mentioned using multiple IP's. When SG in advanced zone is enabled, only one NIC can be assigned per VM.

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Wei ZHOU
Is this similar to CLOUDSTACK-5144? 2014-03-14 14:57 GMT+01:00 Wei ZHOU ustcweiz...@gmail.com: Did you test on fresh 4.2.1, or upgraded platform?

Build failed in Jenkins: build-master-slowbuild #428

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-slowbuild/428/changes Changes: [koushik] CLOUDSTACK-6090: Virtual Router Service Failure Alerting [Devdeep Singh] CLOUDSTACK-6143: Storage motion support for hyper-v. With these changes a volume on a shared [muralimmreddy] -introduces

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 13.03.2014 18:58, Nux! wrote: On 13.03.2014 00:26, Animesh Chaturvedi wrote: Hi All, I've created a 4.3.0 release, with the following artifacts up for a vote: -1 In Adv SG zone assigning secondary IPs to a NIC doesn't update the ipset accordingly on the agent/hv; it requires

Jenkins build is back to normal : cloudstack-4.4-maven-build #2

2014-03-14 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/2/changes

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 14.03.2014 13:59, Wei ZHOU wrote: Is this similar to CLOUDSTACK-5144? Yes, but it's only exhibited on KVM. My colleague who is testing 4.3 with Xenserver is reporting the SG rules work as expected (though ipset update still requires VM stop/start). Lucian -- Sent from the Delta

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

2014-03-14 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review37198 --- server/src/com/cloud/server/ConfigurationServerImpl.java

RE: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Sudha Ponnaganti
Yes that is my understanding that all of these would go in to next maintenance that is why they are staged in forward branch. -Original Message- From: Paul Angus [mailto:paul.an...@shapeblue.com] Sent: Friday, March 14, 2014 1:38 AM To: dev@cloudstack.apache.org Subject: RE:

Re: [CS4.1] Error during ip range creation

2014-03-14 Thread nicolas.lamirault
any idea ? Le 13/03/2014 16:55, nicolas.lamira...@orange.com a écrit : Yes : mysql select * from vlan where vlan_gateway = '10.200.244.129' and vlan_netmask = '255.255.255.192' ;

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Wei ZHOU
Nux and Ove, Did you test on fresh 4.2.1 platform? -Wei 2014-03-14 15:07 GMT+01:00 Nux! n...@li.nux.ro: On 14.03.2014 13:59, Wei ZHOU wrote: Is this similar to CLOUDSTACK-5144? Yes, but it's only exhibited on KVM. My colleague who is testing 4.3 with Xenserver is reporting the SG rules

Re: Review Request 19021: Cloudbyte Elastistor storage plug-in

2014-03-14 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19021/#review37199 --- Ship it! Ship It! - Mike Tutkowski On March 13, 2014, 9:16

Re: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Mike Tutkowski
Right, I think it's dangerous to sync 4.3.1 to 4.3 because developers might have changes in 4.3.1 that are not yet in master. This is not my situation personally, but I imagine some devs might be in this state. On Fri, Mar 14, 2014 at 8:18 AM, Sudha Ponnaganti sudha.ponnaga...@citrix.com

RE: 4.4 Feature Freeze

2014-03-14 Thread Marcus
I'm not sure what the offending ones are at the moment, and there was recently a feature merge that was not squashed, so it might be a little difficult to weed through. On Mar 14, 2014 7:45 AM, Sudha Ponnaganti sudha.ponnaga...@citrix.com wrote: Can the offending check-ins be reverted or master

Re: 4.4 Feature Freeze

2014-03-14 Thread Marcus
It may even just be a behavioral change or something simple I missed that is now required with a recent check-in. Obviously I don't feel comfortable merging in my feature branch though when I pull from master and can no longer run my own feature tests. On Fri, Mar 14, 2014 at 8:35 AM, Marcus

RE: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Sudha Ponnaganti
Yes - 4.3-forward is unstable branch and should be merged while 4.3 is going through RC. It is meant for maintenance release. -Original Message- From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Friday, March 14, 2014 7:36 AM To: dev@cloudstack.apache.org Subject: Re:

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 14.03.2014 14:29, Wei ZHOU wrote: Nux and Ove, Did you test on fresh 4.2.1 platform? Yes. Fresh 4.2.1 doesn't have the problem, fresh 4.3 RC8 has the problem. HTH Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro

Re: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Mike Tutkowski
Yes, and we definitely cannot just take all of 4.3.1 and put it in 4.3 as many of those changes are intended for 4.3.1 and not 4.3. On Fri, Mar 14, 2014 at 8:40 AM, Sudha Ponnaganti sudha.ponnaga...@citrix.com wrote: Yes - 4.3-forward is unstable branch and should be merged while 4.3 is

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Wei ZHOU
Can you try to revert commit 175549f3ab952bbd39318c16c269c16526255475 on ./scripts/vm/network/security_group.py ? git checkout 0898a264a5463b85c4cab3033f9c3161c5ef83f8 ./scripts/vm/network/security_group.py scp to all hosts, and try again. -Wei 2014-03-14 15:07 GMT+01:00 Nux! n...@li.nux.ro:

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Wei ZHOU
OK, then ignore my previous email. it looks no issue with security_group.py 2014-03-14 15:43 GMT+01:00 Nux! n...@li.nux.ro: On 14.03.2014 14:29, Wei ZHOU wrote: Nux and Ove, Did you test on fresh 4.2.1 platform? Yes. Fresh 4.2.1 doesn't have the problem, fresh 4.3 RC8 has the problem.

Re: 4.4 Feature Freeze

2014-03-14 Thread Mike Tutkowski
Yes, having a temp freeze of the codebase while a build is broken is standard procedure at several places I've worked at. We should probably employ such a technique here, too. It's nothing enforced by the VCS - just something devs control when they see such an e-mail. On Fri, Mar 14, 2014 at

Re: Git Push Summary

2014-03-14 Thread Murali Reddy
On 14/03/14 7:09 PM, h...@apache.org h...@apache.org wrote: Repository: cloudstack Updated Branches: refs/heads/4.4 [created] 3ee1fc28d Hugo, I was holding off merge to master, as it seems to be broken now. I see that you already created 4.4 branch. Can we wait till master is ready to be

RE: 4.4 Feature Freeze

2014-03-14 Thread Sudha Ponnaganti
Other alternative is to block further check-ins till blocking issue is fixed. As long as BVTs are good then it can be opened up for further check ins. Just a suggestion. -Original Message- From: Marcus [mailto:shadow...@gmail.com] Sent: Friday, March 14, 2014 7:40 AM To:

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 14.03.2014 14:43, Wei ZHOU wrote: Can you try to revert commit 175549f3ab952bbd39318c16c269c16526255475 on ./scripts/vm/network/security_group.py ? git checkout 0898a264a5463b85c4cab3033f9c3161c5ef83f8 ./scripts/vm/network/security_group.py scp to all hosts, and try again. Do you happen

Re: 4.4 Feature Freeze

2014-03-14 Thread Marcus
The build is not broken. I, personally, just cannot start VMs in my environment, even after starting afresh. It may still be something I'm doing (like a missed behavioral change), but I've seen at least one bug submitted overnight that looks like a stack trace I got. It would be nice to see if

Assigning a JIRA ticket to someone

2014-03-14 Thread Mike Tutkowski
Hi, I'm trying to assign a JIRA ticket to someone who has a JIRA account, but I don't see his name in the list (I see plenty of other names in the list I recognize, though). Any thoughts on what I might be missing here? Thanks! -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.*

Re: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Abhinandan Prateek
Yes, I think we could have checked with people struggling to fix the master and commit there features. -1 for the timing to cut the branch ! On 14/03/14 8:19 pm, Murali Reddy murali.re...@citrix.com wrote: On 14/03/14 7:09 PM, h...@apache.org h...@apache.org wrote: Repository: cloudstack

Re: Git Push Summary

2014-03-14 Thread Hugo Trippaers
Hey Muralli, The branch is already cut and the jenkins builds are setup. So i’m not going to recreate the 4.4 branch from master as all the version number are already updated. Which merge request are you merging in? If you can squash it it will be easy to pull into master and 4.4. Cheers,

Re: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Hugo Trippaers
I announced the time when i would cut the branch in advance, so it shouldn’t be a surprise. But like i replied to Muralli, if you have a pending merge request on the ML i see no problem with merging it in in the next few hours because of the trouble with master. Just make sure it hits both

[ACS4.4] RE: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Sudha Ponnaganti
Can time be given till PST EOD as many developers are also in this time zone? If specific features are being pulled in to 4.4, we might as well cut branch a little later where approved merge queue is cleared. -Original Message- From: Abhinandan Prateek

Re: Git Push Summary

2014-03-14 Thread Murali Reddy
On 14/03/14 8:24 PM, Hugo Trippaers h...@trippaers.nl wrote: Hey Muralli, The branch is already cut and the jenkins builds are setup. So i¹m not going to recreate the 4.4 branch from master as all the version number are already updated. Which merge request are you merging in? If you can squash

Re: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Abhinandan Prateek
Thanks Hugo ! Saw your reply bit late. On 14/03/14 8:34 pm, Hugo Trippaers h...@trippaers.nl wrote: I announced the time when i would cut the branch in advance, so it shouldn¹t be a surprise. But like i replied to Muralli, if you have a pending merge request on the ML i see no problem with

Re: [ACS4.4] RE: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Hugo Trippaers
The branch is already cut and master has already been switched to 4.5.0. I’ve announced that the switch would be at 14:00 CET, you can’t expect me to get up in the middle of the night to cut a release branch. Cheers, Hugo On 14 mrt. 2014, at 16:09, Sudha Ponnaganti

Re: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Mike Tutkowski
Since a lot of devs are basing their features that have not yet been placed in Git off of master (which is now 4.5), we need to make sure it is clear not to rebase yourself beyond 48f8a95b06b0348ba1349cb5434183c2c18710db or 4.5 code that should not be in 4.4 could be brought into 4.4. On Fri,

Re: [ACS4.4] RE: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Marcus
Yes, you did. I for one was up until 2AM trying to test my branch against the current master to try to get it in, but gave up. I think most people saw Get your merge request in today because I'll be cutting the branch on Friday. At the time that most of us in the west read that email, get your

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Ove Ewerlid
On 03/14/2014 03:29 PM, Wei ZHOU wrote: Nux and Ove, Did you test on fresh 4.2.1 platform? Yes, fresh 4.2.1. /Ove -Wei 2014-03-14 15:07 GMT+01:00 Nux! n...@li.nux.ro: On 14.03.2014 13:59, Wei ZHOU wrote: Is this similar to CLOUDSTACK-5144? Yes, but it's only exhibited on KVM. My

Re: [VOTE] Apache CloudStack 4.3.0 (eighth round)

2014-03-14 Thread Nux!
On 14.03.2014 14:43, Wei ZHOU wrote: Can you try to revert commit 175549f3ab952bbd39318c16c269c16526255475 on ./scripts/vm/network/security_group.py ? git checkout 0898a264a5463b85c4cab3033f9c3161c5ef83f8 ./scripts/vm/network/security_group.py scp to all hosts, and try again. I got it from

Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-14 Thread Saksham Srivastava
On March 13, 2014, 4:46 p.m., John Burwell wrote: server/src/com/cloud/vm/UserVmManagerImpl.java, line 756 https://reviews.apache.org/r/17736/diff/3/?file=518208#file518208line756 Two queries are executed in this for loop which puts stress on the database and slows down the

Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack

2014-03-14 Thread Saksham Srivastava
On March 13, 2014, 4:55 p.m., Alena Prokharchyk wrote: Also why don't start routers in parallel? First, get all the router nics in one query like John suggested, then span start process for routers in separate threads, check the status of each job using Future. Sure I will do that. -

Re: [ACS4.4] RE: Git Push Summary : 4.4 feature branch

2014-03-14 Thread Hugo Trippaers
Fair enough, i’ll try to send a reminder for the next phases a bit earlier. Just wanted to make sure that people wouldn’t forget to put their merge requests in on time. I’m trying to be a bit strict about the timing, we are already seeing the strain on the 4.3 release which isn’t even out

RE: Differences between 4.3 and 4.3-forward

2014-03-14 Thread Animesh Chaturvedi
-Original Message- From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Friday, March 14, 2014 7:43 AM To: dev@cloudstack.apache.org Subject: Re: Differences between 4.3 and 4.3-forward Yes, and we definitely cannot just take all of 4.3.1 and put it in 4.3 as many of

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Min Chen
Thanks Marcus. I am not aware of this convention, will remember that next time when I do the merge. -min On 3/13/14 10:30 PM, Marcus shadow...@gmail.com wrote: Min, in looking at this branch merge, I need to be reminded whether we are supposed to squash feature branches when they come in, or

Re: Master is broken?

2014-03-14 Thread Antonio Fornié Casarrubios
Hi Talluri. I am trying to know what causes the issue you describe. Just to make sure I understand you, why do you think this issue is related with this change? Anybody knows anything else about it? Thanks. Cheers Antonio 2014-03-14 14:17 GMT+01:00 Srikanteswararao Talluri

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Min Chen
Before merge, I did a sanity zone and VM deployment test, and worked fine on my setup after fixing the issues introduced by Antonio's commit. I can verify this again today. From the symptom, it seems not related to IAM change. Thanks -min On 3/14/14 1:07 AM, Marcus shadow...@gmail.com wrote:

  1   2   >