Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-08 Thread Chip Childers
We are already (generally) AP for most infra changes really. I'd use that model. Eventual consistency is better in this scenario. > On Nov 8, 2013, at 6:49 PM, Chiradeep Vittal > wrote: > > I'd also like to highlight that it isn't a trivial problem. > Let's say there's 3 regions: this means t

Re: Review Request 15349: Overprivisoning changes needed

2013-11-08 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15349/#review28602 --- engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java

Regarding the ssl key store change

2013-11-08 Thread Sheng Yang
Hi Wei, I found this change in the MASTER. commit 57ba367f3c985e80ea1b34267e298b481a353298 Author: Wei Zhou Date: Thu Nov 7 11:09:06 2013 +0100 CLOUDSTACK-5042: change cloud.keystore to cloudmanagementserver.keystore and install it (cherry picked from commit de448ec4792eda5b47d79b26e9cb8c

RE: [ACS43] Schedule reminder : feature freeze today 11/08

2013-11-08 Thread Animesh Chaturvedi
Folks one final reminder I will cut the 4.3 branch tomorrow morning. > -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Monday, November 04, 2013 9:58 PM > To: dev@cloudstack.apache.org > Subject: [ACS43] Schedule reminder : feature freeze this

Re: Reviewboard needs your attention

2013-11-08 Thread Mike Tutkowski
Hi Animesh, I get the same results using Firefox. I assume there are supposed to be buttons to the left of the Review button? If I make a new Review Request, I see more buttons. Thanks On Fri, Nov 8, 2013 at 11:28 AM, Animesh Chaturvedi < animesh.chaturv...@citrix.com> wrote: > Mike is this

Review Request 15349: Overprivisoning changes needed

2013-11-08 Thread Animesh Chaturvedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15349/ --- Review request for cloudstack and Nitin Mehta. Bugs: Cloudstack-5077. https

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-08 Thread Chiradeep Vittal
I'd also like to highlight that it isn't a trivial problem. Let's say there's 3 regions: this means there are 3 copies of the user database that are geographically separated by network links that fail quite often (orders of magnitude more than intra-DC networks). Here we run into the consequences

Re: Review Request 13901: fix long cast to double

2013-11-08 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13901/ --- (Updated Nov. 8, 2013, 10:17 p.m.) Review request for cloudstack, Alena Prokhar

Re: [ACS 421] Upgrade testing

2013-11-08 Thread David Nalley
On Thu, Oct 31, 2013 at 6:31 PM, Nux! wrote: > On 31.10.2013 02:51, Sudha Ponnaganti wrote: >> >> Hi All, >> >> Want to check if anyone is testing upgrades from prior versions to >> 4.2.1. As Release date is getting close, If there are gaps, would like >> to cover some scenarios in our lab. >> >>

Review Request 15362: Fixed issue with detail view fields marked as 'isPassword' not being obfuscated

2013-11-08 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15362/ --- Review request for cloudstack, Brian Federle and Jessica Wang. Repository: clou

Re: api incompatibility between 4.1 and 4.2 in ACLs

2013-11-08 Thread Marcus Sorensen
Yes, that would certainly maintain api compatibility if one creates an ACL without specifying aclid, it creates a new list and applies it to the given network. On Fri, Nov 8, 2013 at 12:28 PM, Animesh Chaturvedi wrote: > Actually use this link to the message in that thread http://s.apache.org/QKI

RE: api incompatibility between 4.1 and 4.2 in ACLs

2013-11-08 Thread Animesh Chaturvedi
Actually use this link to the message in that thread http://s.apache.org/QKI > -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Friday, November 08, 2013 11:24 AM > To: dev@cloudstack.apache.org > Cc: Kishan Kavala > Subject: RE: api incompatib

Re: 4.2.1 BVT automation results

2013-11-08 Thread David Nalley
Thats less than ideal, but better than no testing. Did bugs get filed for the failing tests in 4.2.1? --David On Fri, Nov 8, 2013 at 10:43 AM, Rayees Namathponnan wrote: > 4.2.1 automation not executed from jenkins.buildacloud.org, > > Automation executed in side my company including vmware, > >

Re: api incompatibility between 4.1 and 4.2 in ACLs

2013-11-08 Thread Alena Prokharchyk
Kishan, weren't we supposed to create a default ACL container (list) and assign the rule to it, if no ACL list is passed to the call? Shouldn't be hard to fix From: Animesh Chaturvedi mailto:animesh.chaturv...@citrix.com>> Reply-To: "dev@cloudstack.apache.org"

RE: api incompatibility between 4.1 and 4.2 in ACLs

2013-11-08 Thread Animesh Chaturvedi
I will let Kishan comment but found this thread http://markmail.org/thread/fxzki6ftqacyrylk > -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Friday, November 08, 2013 9:13 AM > To: dev@cloudstack.apache.org > Subject: Re: api incompatibility between 4.1

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Daan Hoogland
my code doesn't work when the vlan comes in as numeric. I don't claim that. it only works when it comes in as vlan://. It is then and only then properly parsed into a numeric value. On Fri, Nov 8, 2013 at 7:23 PM, Sheng Yang wrote: > Here is the part in your commit about SRX: > > diff --git > a/p

Re: SRX inline with VR

2013-11-08 Thread Sheng Yang
Sorry, I meant the CloudStack major release after 4.3... Got mixed up a little bit... --Sheng On Fri, Nov 8, 2013 at 9:57 AM, Sheng Yang wrote: > Hi Geoff, > > Currently we don't support VR as load balancing alone, there are some VR > limitations need to be solved. I think the current plan is

RE: Reviewboard needs your attention

2013-11-08 Thread Animesh Chaturvedi
Mike is this the only review that you cannot see, or it's a general problem you are seeing with reviewboard. May be try another browser > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Friday, November 08, 2013 4:47 AM > To: dev@cloudstack.apache.o

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Sheng Yang
Here is the part in your commit about SRX: diff --git a/plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java b/plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java index 3bcbf2d..46ef332 100644 --- a/plugins/network-elem

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Sheng Yang
Hi Daan, Your commit has broken BOTH VMware and JuniperSRX, and these are only two functional codes in your commit. The other two modification are comments. So I don't know how your testing can possibly work. Alena has already fixed the Juniper part at: commit 3ab8d8d8f20c453fdc684f177a612922ea

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Daan Hoogland
Min, I realize that you fixed something and didn't just change code because of no reason. The point is that a vlan might be identified both as and as vlan://. the patch is part of a series that addresses these situations. I will look at this case and make sure either only one is possible or both

Re: SRX inline with VR

2013-11-08 Thread Sheng Yang
Hi Geoff, Currently we don't support VR as load balancing alone, there are some VR limitations need to be solved. I think the current plan is to fix it in Felton. --Sheng On Fri, Nov 8, 2013 at 6:19 AM, Geoff Higginbottom < geoff.higginbot...@shapeblue.com> wrote: > All, > > > > Is it possibl

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Min Chen
Daan, I am curious about this: you mentioned that your patch has been running fine on a mixed xen/vmware cloud environment. How come you didn't encounter the blocker bug filed by Rayees on our automation environment on master branch: https://issues.apache.org/jira/browse/CLOUDSTACK-5046. A

Re: [ACS 421] Upgrade testing

2013-11-08 Thread Chip Childers
On Fri, Nov 08, 2013 at 12:52:13PM +, Abhinandan Prateek wrote: > Sudha, > We can still cut RC but will wait for GA till Daan and others have done > their testing. Abhi, An "RC" here is really a release artifact that's being voted on, so no... please don't do that until you are satisfied t

Re: api incompatibility between 4.1 and 4.2 in ACLs

2013-11-08 Thread Marcus Sorensen
So I take the silence to simply be a collective "oops". I guess this just should serve as a reminder to not break API compatibility without a discussion. Perhaps our tests will surface this better in the future (although I need to look, I wonder if any ACL tests were also simply changed to accomod

Re: [PROPOSAL] Liaison with ETSI NFV ISG

2013-11-08 Thread Chip Childers
On Fri, Nov 08, 2013 at 08:58:17AM +0700, Nguyen Anh Tu wrote: > 2013/11/7 Chiradeep Vittal > > > While I appreciate the confidence, there are weekly calls to attend. > > The particularly interesting one (MANO WG) happens on Wednesdays at 6am > > PST. > > I've tried but there's very little chance

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Daan Hoogland
Sheng, Min, The use of the word semantics has triggered a line of though I want to share on this. The word vlanid is being used as a homonym in cloudstack, meaning both vlan-number and broadcast-uri-pertaining-to-a-vlan. This has been so since 4.0 and maybe even longer. As I see it; Assuming a vla

Re: Review Request 14605: Fill the creationMonitors based on priority

2013-11-08 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14605/#review28546 --- Ship it! Laszlo can you commit this? - Darren Shepherd On Oct. 1

Re: Review Request 15344: Enabling DB HA for cloud stack data bases using mysql active-active solution

2013-11-08 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15344/#review28545 --- I don't agree with this approach. Following this logic for every po

RE: 4.2.1 BVT automation results

2013-11-08 Thread Rayees Namathponnan
4.2.1 automation not executed from jenkins.buildacloud.org, Automation executed in side my company including vmware, Regards, Rayees -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Friday, November 08, 2013 12:20 AM To: dev@cloudstack.apache.org Subject: Re

Re: Review Request 15303: Reserved SCSI device number prevents CloudStack to attach more than 6 data volumes to CLOUDSTACK-4790

2013-11-08 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15303/#review28542 --- Ship it! commit 97348217c47ad1cb4e242a04ddfcc3696c2a280f - Kishan

Review Request 15351: Fixing bugs from Coverity related to Dereferenced Null after check and as return value.

2013-11-08 Thread Wilder Rodrigues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15351/ --- Review request for cloudstack and Hugo Trippaers. Repository: cloudstack-git

SRX inline with VR

2013-11-08 Thread Geoff Higginbottom
All, Is it possible to use a Juniper SRX Firewall for firewall features, and a VR for Load Balancing, or do you need to use an F5 to get 'in line' mode If you can use a SRX, what IP gets load balanced, public or private etc Regards Geoff Higginbottom CTO / Cloud Architect [Description: Mail L

Re: Review Request 14058: Including tests for VPC VM Lifecycle on Tagged hosts

2013-11-08 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14058/ --- (Updated Nov. 8, 2013, 1:48 p.m.) Review request for cloudstack, Girish Shilamk

Re: Review Request 15350: CLOUDSTACK-5107: Decreased memory in service offering so as not to run out of resources

2013-11-08 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15350/#review28540 --- Commit a2117d46f90ae6a96e3d046e306e524756609f5d in branch refs/heads

Re: Review Request 15350: CLOUDSTACK-5107: Decreased memory in service offering so as not to run out of resources

2013-11-08 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15350/#review28539 --- Commit 9e6a1eaffe3df0981703dc26ab4cc7043d4ca23f in branch refs/heads

Re: [ACS 421] Upgrade testing

2013-11-08 Thread Abhinandan Prateek
Sudha, We can still cut RC but will wait for GA till Daan and others have done their testing. On 08/11/13 5:36 pm, "Sudha Ponnaganti" wrote: >Abhinandan (RM for 4.2.1), > >You cannot cut RC at least till end of next week i.e Nov 15th. > >Thanks >/sudha > >-Original Message- >From: Daan

Re: Reviewboard needs your attention

2013-11-08 Thread Mike Tutkowski
This is what I see when logged in: http://i.imgur.com/RZvXuy1.png Am I missing something? Thanks! On Thu, Nov 7, 2013 at 10:01 PM, Animesh Chaturvedi < animesh.chaturv...@citrix.com> wrote: > I can see it fine, were you logged in at that time > > > -Original Message- > > From: Mike Tu

Review Request 15350: CLOUDSTACK-5107: Decreased memory in service offering so as not to run out of resources

2013-11-08 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15350/ --- Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri. B

Re: Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-08 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15348/ --- (Updated Nov. 8, 2013, 12:24 p.m.) Review request for cloudstack, Kishan Kavala

Re: Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-08 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15348/#review28538 --- Ship it! Ship It! - Koushik Das On Nov. 8, 2013, 12:02 p.m., Har

RE: [ACS43] Schedule reminder : feature freeze this Friday 11/08

2013-11-08 Thread Devdeep Singh
Merged the changes to master. Regards, Devdeep -Original Message- From: Chip Childers [mailto:chip.child...@gmail.com] Sent: Friday, November 8, 2013 12:29 AM To: dev@cloudstack.apache.org Subject: Re: [ACS43] Schedule reminder : feature freeze this Friday 11/08 I was off - this morning

RE: [ACS 421] Upgrade testing

2013-11-08 Thread Sudha Ponnaganti
Thanks Wei for covering this path -Original Message- From: Wei ZHOU [mailto:ustcweiz...@gmail.com] Sent: Friday, November 08, 2013 4:04 AM To: dev@cloudstack.apache.org Subject: Re: [ACS 421] Upgrade testing I am working on upgrade from 2.2.14 to 4.2.1 some issues have been fixed , exce

RE: [ACS 421] Upgrade testing

2013-11-08 Thread Sudha Ponnaganti
Abhinandan (RM for 4.2.1), You cannot cut RC at least till end of next week i.e Nov 15th. Thanks /sudha -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Friday, November 08, 2013 3:08 AM To: dev Cc: int-tool...@schubergphilis.com; cloud-team Subject: Re: [AC

Re: [ACS 421] Upgrade testing

2013-11-08 Thread Wei ZHOU
I am working on upgrade from 2.2.14 to 4.2.1 some issues have been fixed , except CLOUDSTACK-5076 will fix it today 2013/11/8 Sudha Ponnaganti > Hi, > > Looks like Schuberg Philis is covering 4. 1.1 -> 4.2.1 upgrade > We ( Citrix systems) would cover 4.2 -> 4.2.1 > > I think these two paths mi

Re: Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-08 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15348/ --- (Updated Nov. 8, 2013, 12:02 p.m.) Review request for cloudstack, Kishan Kavala

Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-08 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15348/ --- Review request for cloudstack and Kishan Kavala. Bugs: CLOUDSTACK-5105 http

Strange info on secondary storage after upgrade to CloudStack 4.2.0

2013-11-08 Thread Indra Pramana
Dear all, On CloudStack GUI, under Infrastructure > Secondary Storage, I remember we used to have a proper name for each secondary storage. However, after the upgrade to CloudStack 4.2.0, the name disappears and it now becomes the ID like this: NameURLProviderQuickviewc02da448-b9f4-401b-b8d5-83e8

Re: Review Request 15307: Increasing the timeout values to 600 seconds from 300 seconds

2013-11-08 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15307/#review28535 --- 9495fbc87317cbcf8b77630d06e52d52dfc19060 - SrikanteswaraRao Talluri

Re: Review Request 15263: Added fix for bug 5056.

2013-11-08 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15263/#review28534 --- commit cf7532e8480385bcc9c5dfd63fb82cd836282028 - SrikanteswaraRao

Re: Review Request 14976: [SSL Termination support] Part2 : Assign certificates to LBs

2013-11-08 Thread Murali Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14976/#review28533 --- Ship it! commit 0076307863e9155273d9e4c14282de429388c9e9 Author: Sy

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/#review28532 --- Ship it! Ship It! - SrikanteswaraRao Talluri On Nov. 8, 2013, 10

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/#review28531 --- Ship it! Ship It! - Santhosh Edukulla On Nov. 8, 2013, 10:43 a.m

Re: [ACS 421] Upgrade testing

2013-11-08 Thread Daan Hoogland
Sudha, give us till the end of next week please. I'll report back on friday, ok? On Fri, Nov 8, 2013 at 3:12 AM, Sudha Ponnaganti wrote: > Hi, > > Looks like Schuberg Philis is covering 4. 1.1 -> 4.2.1 upgrade > We ( Citrix systems) would cover 4.2 -> 4.2.1 > > I think these two paths might be e

Re: Review Request 14648: Added few changes related to 4870

2013-11-08 Thread Girish Shilamkar
> On Nov. 8, 2013, 9:15 a.m., Girish Shilamkar wrote: > > 4.2: fba874120bb40ddeceacbc7ef51217f34d3749de > > Sebastien Goasguen wrote: > if these patches apply to master, can you make sure to also apply to > master at the same time. thanks They were already applied to master but missing in

Re: Commit 25c8cee01a450ee924fe108cafe54b046485ab2b broke Vmware on Master

2013-11-08 Thread Daan Hoogland
H Sheng, All sdn implementations use the field in one way or another as uri. So if this is wrong reverting my patch won't do the trick, I am afraid. I actually think the field should be renamed to broadcastUri, as this is how it is used at the moment by a lot of elements. Going back to only allow

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/ --- (Updated Nov. 8, 2013, 10:43 a.m.) Review request for cloudstack and Srikantesw

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread Girish Shilamkar
> On Nov. 8, 2013, 10:11 a.m., Girish Shilamkar wrote: > > Santhosh, > > > > Please update the diff instead of uploading them as different file. Also the commit header should be "CLOUDSTACK-5099: " This will ensure that the jira ticket also gets updated upon merge. - Girish ---

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/#review28528 --- Santhosh, Please update the diff instead of uploading them as diffe

Re: Review Request 15208: Added fix for bug 5032. Provides custom assert facility to test features. Added the new assert to cloudstackTestCase

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15208/#review28527 --- 4.2: c76da53123f4130ac59bdf112171bc2e39af91d4 - Girish Shilamkar

Re: Review Request 14925: Added few misc changes to marvin

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14925/#review28526 --- 4.2: fba874120bb40ddeceacbc7ef51217f34d3749de - Girish Shilamkar

Re: Review Request 14875: Fixed few pep8 compliant issues

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14875/#review28525 --- 4.2: 3a31a7e6 - Girish Shilamkar On Oct. 23, 2013, 1:57 p.m., San

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/#review28524 --- 4.2: 87983e09e21c51dbef6b2a8a0252b039e32915ad - Girish Shilamkar

Re: [Cloudmonkey] assignVirtualMachine API causes index out of range error

2013-11-08 Thread sebgoa
On Nov 8, 2013, at 10:29 AM, Ryan Lei wrote: > Super. The Cloudmonkey crash is fixed now. Thanks a a lot, Sebastien!! > > You are right about this line: > - response.setResponseName(DeployVMCmd.getResultObjectName()); > + response.setResponseName(getCommandName()); > > Please also commit this

Re: [Cloudmonkey] assignVirtualMachine API causes index out of range error

2013-11-08 Thread Ryan Lei
Super. The Cloudmonkey crash is fixed now. Thanks a a lot, Sebastien!! You are right about this line: - response.setResponseName(DeployVMCmd.getResultObjectName()); + response.setResponseName(getCommandName()); Please also commit this change to master. ---

Re: Review Request 14648: Added few changes related to 4870

2013-11-08 Thread Sebastien Goasguen
> On Nov. 8, 2013, 9:15 a.m., Girish Shilamkar wrote: > > 4.2: fba874120bb40ddeceacbc7ef51217f34d3749de if these patches apply to master, can you make sure to also apply to master at the same time. thanks - Sebastien --- This is an aut

Re: Review Request 14595: Adding https support to marvin

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14595/#review28521 --- 4.2: 6e1821f585df5c643873ab0ce1e7d91d1d94f00d - Girish Shilamkar

Re: Review Request 14648: Added few changes related to 4870

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14648/#review28520 --- 4.2: fba874120bb40ddeceacbc7ef51217f34d3749de - Girish Shilamkar

Re: [DOC] 4.2.0 Templates

2013-11-08 Thread Sebastien Goasguen
On Oct 27, 2013, at 7:02 AM, Marty Sweet wrote: > Hi, > > Is there any update on this? > My commit > https://github.com/apache/cloudstack/commit/f5e7f46dadda741f10e5b674d0578ade9ba719d7 > made > it into the 4.2.0 docs but > https://github.com/apache/cloudstack/commit/922ef76224d4a8534f67f47b97c

Re: Review Request 13922: Minimal Hyper-V Plugin

2013-11-08 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13922/#review28519 --- Ship it! The feature was merged and changes committed to master in

Re: Review Request 13922: Minimal Hyper-V Plugin

2013-11-08 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13922/#review28518 --- Ship it! The feature was merged and changes committed to master in

Re: Review Request 13745: CLOUDSTACK-4329: [DOC] Added Linux template creation documentation

2013-11-08 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13745/#review28517 --- Marty, it looks like this was lost somehow when we moved to the docs

Re: Review Request 14784: [CLOUDSTACK-4805] Installation instructions drops support for Ubuntu

2013-11-08 Thread Sebastien Goasguen
> On Nov. 7, 2013, 7:38 p.m., Sebastien Goasguen wrote: > > this does not seem to apply. Can you please pull the latest and see what's > > going on > > > > thanks > > Travis Graham wrote: > Do you want this based against master or 4.2? The patch was originally > created against 4.2. I t

Re: [Cloudmonkey] assignVirtualMachine API causes index out of range error

2013-11-08 Thread Sebastien Goasguen
Ryan, I found another bug in that class, can you try again with a fresh pull of 4.2 branch ? thanks On Nov 8, 2013, at 3:40 AM, Sebastien Goasguen wrote: > Can you get on IRC ? > > On Nov 8, 2013, at 3:27 AM, Ryan Lei wrote: > >> Finally got a new environment to build from source code (4.2

Review Request 15344: Enabling DB HA for cloud stack data bases using mysql active-active solution

2013-11-08 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15344/ --- Review request for cloudstack, Abhinandan Prateek and Darren Shepherd. Bugs: ht

Re: [Cloudmonkey] assignVirtualMachine API causes index out of range error

2013-11-08 Thread Sebastien Goasguen
Can you get on IRC ? On Nov 8, 2013, at 3:27 AM, Ryan Lei wrote: > Finally got a new environment to build from source code (4.2 branch). > Unfortunately, it still crashes Cloudmonkey. The error messages are exactly > the same. Here's the new Cloudmonkey log: > > 2013-11-08 16:15:34,242 - reques

Re: [Cloudmonkey] assignVirtualMachine API causes index out of range error

2013-11-08 Thread Ryan Lei
Finally got a new environment to build from source code (4.2 branch). Unfortunately, it still crashes Cloudmonkey. The error messages are exactly the same. Here's the new Cloudmonkey log: 2013-11-08 16:15:34,242 - requester.py:45 - [DEBUG] START Request 2013-11-08 16:15:34,242 -

Re: 4.2.1 BVT automation results

2013-11-08 Thread Sebastien Goasguen
On Nov 7, 2013, at 9:54 AM, Rayees Namathponnan wrote: > Hi Sebastian, > > I don't think so we are running automation 4.2.1 branch in > jenkins.buildacloud.org, below result is from master branch; > So how did you come up with the results for the 4.2.1 branch ? > http://jenkins.buildacl

Re: Review Request 15175: Added a verifyElementInList utility function

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15175/#review28513 --- 4.2: b18e730108e6a24e00c372e3c56c655ff51106ba - Girish Shilamkar

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15021/#review28512 --- 4.2: e7b6ee10eda85a90c594c9de5d36cc062e069576 - Girish Shilamkar