Re: Review Request 22717: refactor StoragePoolAllocator#filter logic

2014-06-26 Thread Yoshikazu Nojima
> On June 27, 2014, 5:46 a.m., Mike Tutkowski wrote: > > I first had a chance to run this patch through a sophisticated test tonight > > and noticed an issue with zone-wide primary storage that's based on the > > iSCSI protocol. > > > > This patch leads to iSCSI storage being filtered out for

Re: Review Request 22717: refactor StoragePoolAllocator#filter logic

2014-06-26 Thread Mike Tutkowski
> On June 26, 2014, 11:46 p.m., Mike Tutkowski wrote: > > I first had a chance to run this patch through a sophisticated test tonight > > and noticed an issue with zone-wide primary storage that's based on the > > iSCSI protocol. > > > > This patch leads to iSCSI storage being filtered out for

Re: Review Request 22717: refactor StoragePoolAllocator#filter logic

2014-06-26 Thread Yoshikazu Nojima
> On June 27, 2014, 5:46 a.m., Mike Tutkowski wrote: > > I first had a chance to run this patch through a sophisticated test tonight > > and noticed an issue with zone-wide primary storage that's based on the > > iSCSI protocol. > > > > This patch leads to iSCSI storage being filtered out for

Fwd: [jira] [Created] (CLOUDSTACK-7003) Arithmetic exception while creating a vdi on nfs volume in managed storage.

2014-06-26 Thread Punith S
hi mike, can you take a look at this logic, today i met an arithmetic exception(/ by 0) while creating a nfs volume. any suggestions to calculate the maxNumberOfTries if the unavailableSrSpace is equal to zero. thanks -- Forwarded message -- From: punith (JIRA) Date: Fri, Jun 2

[ACS4.4] cherry pick commit dc22566c642e400014332045a224e5545d33a11c and 48646ae186eb75052da3da385404a823bd785444

2014-06-26 Thread Sanjay Tripathi
Hi Daan, Could you please cherry-pick following commits to 4.4 branch. Commit: dc22566c642e400014332045a224e5545d33a11c CLOUDSTACK-6453: [GPU] Windows 2012 Server instance created with vGPU offering is not coming up after installing PV drivers. Commit: 48646ae186eb75052da3da385404a823bd785444 C

Re: [ACS4.4, 4.4-forward] Please revert commit

2014-06-26 Thread Mike Tutkowski
Yeah, I just looked at the Review Request: https://reviews.apache.org/r/22717/#review46838 It says it's for master (4.5), so I'm not sure how this ended up in 4.4 or 4.4-forward. On Fri, Jun 27, 2014 at 12:09 AM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Hi Daan, > > Please rever

Re: [ACS4.4] Please cherry-pick 9c2e6f5ed45522ff68131556028f3fb4ff91ee90

2014-06-26 Thread Mike Tutkowski
Hi Daan, I'm not seeing this commit in 4.4. Did I miss something? Thanks! Mike On Thu, Jun 26, 2014 at 11:16 AM, Daan Hoogland wrote: > On Thu, Jun 26, 2014 at 6:53 PM, Mike Tutkowski > wrote: > > 9c2e6f5ed45522ff68131556028f3fb4ff91ee90 > > > is in > > -- > Daan > -- *Mike Tutkowski* *

[ACS4.4, 4.4-forward] Please revert commit

2014-06-26 Thread Mike Tutkowski
Hi Daan, Please revert commit 99dd86e588fd28dedd5fb3b830297a8a4f885760 from 4.4. Also, please revert commit 45f0c7367680f4bfbcee470139b708d69322be78 from 4.4-forward. These commits actually break zone-wide primary storage. I was not aware that they ended up in 4.4 and 4.4-forward (I was thinkin

Re: Review Request 23084: Making the "Adding primary storage form" support adding primary storage to CS that is based on storage plug-ins

2014-06-26 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23084/#review46839 --- Ship it! Committed the changes in 9a27f201b02fe33cdba1dcca7da63497b

Re: Review Request 22717: refactor StoragePoolAllocator#filter logic

2014-06-26 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22717/#review46838 --- I first had a chance to run this patch through a sophisticated test

Re: Cloudstack MS failover

2014-06-26 Thread Abhinandan Prateek
http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.1.0/html/Instal lation_Guide/management-server-install-flow.html Section 4.5.7 has instructions on adding additional MS. -abhi On 26/06/14 10:55 pm, "Tejas Gadaria" wrote: >I want to setup HA of MS with HAproxy OR Keepalived. > >I hav

RE: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Kishan Kavala
Alex, You are correct. It should be Integer and not Long. > -Original Message- > From: Alex Ough [mailto:alex.o...@sungardas.com] > Sent: Thursday, 26 June 2014 8:09 PM > To: Kishan Kavala > Cc: cloudstack > Subject: Re: Review Request 20099: Domain-Account-User Sync Up Among > Multiple R

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread John Burwell
All, I apologize for joining this conversation late.  I understand that this patch was submitted back in February.  Around this time, my family had a significant medical event, and I was disengaged from all work activities — missing the original conversation. Reading through the specification,

RE: [VMWARE][ACS430] Traffic Shaping

2014-06-26 Thread Sateesh Chodapuneedi
Yes, it's tied to networking offering. Implemented network settings follows the network offering. Regards, Sateesh > -Original Message- > From: Nux! [mailto:n...@li.nux.ro] > Sent: 27 June 2014 05:32 > To: dev@cloudstack.apache.org > Subject: Re: [VMWARE][ACS430] Traffic Shaping > > Ilya

Re: [VMWARE][ACS430] Traffic Shaping

2014-06-26 Thread Nux!
Ilya, Isn't this tied into the network/service offering? (it is for other HVs, defaults to 200 Mbps afaik) Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "ilya musayev" > To: dev@cloudstack.apache.org > Sent: Thursday, 26 Ju

Review Request 23098: Updated Marvin code to support more properties.

2014-06-26 Thread Vania Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23098/ --- Review request for cloudstack and Mike Tutkowski. Repository: cloudstack-git

Re: Review Request 21817: [UI] New Zones tab for Templates and ISOs

2014-06-26 Thread Jessica Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21817/#review46807 --- Ship it! Ship It! - Jessica Wang On May 22, 2014, 4:52 p.m., Gab

Re: Review Request 22863: CLOUDSTACK-6823 : First code drop for Brocade Network plugin to orchestrate Brocade VDX switches for L2 connectivity.

2014-06-26 Thread Ritu Sabharwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22863/ --- (Updated June 26, 2014, 10:25 p.m.) Review request for cloudstack. Changes --

Re: [ISSUE] can not parse [10.1.1.0] error while creating Guest Network for CIDR

2014-06-26 Thread Daan Hoogland
that would be strange. It is not a IPv6 cidr. It is IPv4. On Thu, Jun 26, 2014 at 11:51 PM, Ritu Sabharwal wrote: > Sorry for the typo in earlier mail. > > I gave 10.1.10.1/23 in the IPv6 CIDR field and get an error can not parse > [10.1.10.1]. This was working with 4.3 > > Ritu. > > -Origin

RE: [ISSUE] can not parse [10.1.1.0] error while creating Guest Network for CIDR

2014-06-26 Thread Ritu Sabharwal
Sorry for the typo in earlier mail. I gave 10.1.10.1/23 in the IPv6 CIDR field and get an error can not parse [10.1.10.1]. This was working with 4.3 Ritu. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Thursday, June 26, 2014 2:47 PM To: dev Subject: Re:

Re: [ISSUE] can not parse [10.1.1.0] error while creating Guest Network for CIDR

2014-06-26 Thread Daan Hoogland
PS Don't know of a format change. On Thu, Jun 26, 2014 at 11:46 PM, Daan Hoogland wrote: > H Ritu, > > Are you sure you entered 10.1.10/23? it seems to me it would have to > be 10.1.10.0/23. > and did you enter it in the field for IPv6? this is an ipv4 address format > > Daan > > On Thu, Jun 26,

Re: [ISSUE] can not parse [10.1.1.0] error while creating Guest Network for CIDR

2014-06-26 Thread Daan Hoogland
H Ritu, Are you sure you entered 10.1.10/23? it seems to me it would have to be 10.1.10.0/23. and did you enter it in the field for IPv6? this is an ipv4 address format Daan On Thu, Jun 26, 2014 at 11:40 PM, Ritu Sabharwal wrote: > Hi, > > I am trying to create a guest network for a network off

[ISSUE] can not parse [10.1.1.0] error while creating Guest Network for CIDR

2014-06-26 Thread Ritu Sabharwal
Hi, I am trying to create a guest network for a network offering. I am giving in all the values and when I give IPv6 cidr value to 10.1.10/23 I get error on UI. can not parse [10.1.1.0]. I tried this with 4.3 and it was workin. It does not work with master branch codebase. Please let me know

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alex Ough
Sounds like it goes back to what I said I wish they have been involved more actively from the start. Thanks but really making me tired. Alex Ough On Thu, Jun 26, 2014 at 5:17 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > I did logic review according to the FS assuming that

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alena Prokharchyk
I did logic review according to the FS assuming that the FS (and the design described there) was approved on the [PROPOSAL] stage, BEFORE the code was put it to the review board. Was it approved at that stage? Alex, the feature is not small, and considering that it raised so many questions and

[VMWARE][ACS430] Traffic Shaping

2014-06-26 Thread ilya musayev
Are we enabling traffic shaping on vmware standard switches/portgroups and if so, how do we change the behavior or turn it off complete?

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alex Ough
Alena, Didn't you say that you guys already "did logic review" in the previous email? Thanks Alex Ough On Thu, Jun 26, 2014 at 2:59 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > Alex, sorry to hear that it took so long to get on the review process. > The question still remains

Re: Review Request 23084: Making the "Adding primary storage form" support adding primary storage to CS that is based on storage plug-ins

2014-06-26 Thread Seifeddine JEMLI
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23084/ --- (Updated June 26, 2014, 8:49 p.m.) Review request for cloudstack and Mike Tutko

Re: Review Request 23084: Making the "Adding primary storage form" support adding primary storage to CS that is based on storage plug-ins

2014-06-26 Thread Seifeddine JEMLI
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23084/ --- (Updated June 26, 2014, 8:47 p.m.) Review request for cloudstack and Mike Wang.

Re: Review Request 22019: CLOUDSTACK-6732: [OVS][UI] Network Service Providers page displays two ovs providers

2014-06-26 Thread Jessica Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22019/#review46798 --- Ship it! Ship It! - Jessica Wang On May 29, 2014, 2:16 p.m., Gab

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alena Prokharchyk
Alex, sorry to hear that it took so long to get on the review process. The question still remains – before you started working on implementation, and posted your plugin’s code, was the FS approved/reviewed as a part of [PROPOSAL] discussion? We should never start the development until you get th

Re: NetworkOrchestrator selects 2 NetworkGurus at one time....

2014-06-26 Thread Chiradeep Vittal
For 4.3/4.4, I’m guessing this is the same solution. For 4.5, here’s a couple of options we could implement: 1. New isolation provider (“BrocadeVLAN” or “JuniperEXVLAN”) 2. When creating the network offering, the administrator gets to select the guru 3. New VLAN provider mechanism. Fro

Re: Review Request 22019: CLOUDSTACK-6732: [OVS][UI] Network Service Providers page displays two ovs providers

2014-06-26 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22019/#review46776 --- Commit 5c36fe84b69441714c2577b40b42afaf8f65a1ce in cloudstack's bran

Re: Review Request 22019: CLOUDSTACK-6732: [OVS][UI] Network Service Providers page displays two ovs providers

2014-06-26 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22019/#review46775 --- Commit 6d5d48f460438c7df43e3d0e3da9bd91866c53d5 in cloudstack's bran

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alex Ough
Alena, It has been reduced almost twice because a lot has been separated from the CS and moved to the plug-in not because they are 'unnecessary'. Please remember that my initial implementation was inside the CS not as a plug-in as I said in the previous email. Of course, I asked and urged the rev

RE: [ACS4.4] i18n problems in Add Primary Storage dialog

2014-06-26 Thread Jessica Wang
Vetrivel, Mike has reviewed it. Mike, thanks a lot! Jessica From: Vetrivel Chinnasamy Sent: Thursday, June 26, 2014 5:07 AM To: Mike Tutkowski; dev@cloudstack.apache.org Cc: Brian Federle; Alena Prokharchyk; Jessica Wang Subject: RE: [ACS4.4] i18n problems in Add Primary Storage dialog Hi Mike,

Cloudstack MS failover

2014-06-26 Thread Tejas Gadaria
I want to setup HA of MS with HAproxy OR Keepalived. I have MS1 & DB1 installed on 10.1.1.2 & MS2 & DB2 installed on 10.1.1.3 also DB has master - master replication setup. Need help on this how can i setup failover for MS. Regards, Tejas

Re: [ACS4.4] Please cherry-pick 9c2e6f5ed45522ff68131556028f3fb4ff91ee90

2014-06-26 Thread Daan Hoogland
On Thu, Jun 26, 2014 at 6:53 PM, Mike Tutkowski wrote: > 9c2e6f5ed45522ff68131556028f3fb4ff91ee90 is in -- Daan

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alena Prokharchyk
Alex, By “huge” I’ve meant that there was a lot of repetitive hardcoded things, lot of unnecessary changes to the CS orchestration layer. If you compare a number of changes now and originally, you can see that it reduced almost twice. But lets discuss the complains about lack of initial review

[ACS4.4] Please cherry-pick 9c2e6f5ed45522ff68131556028f3fb4ff91ee90

2014-06-26 Thread Mike Tutkowski
Hi Daan, Please cherry pick 9c2e6f5ed45522ff68131556028f3fb4ff91ee90. This is the i18n issue I referred to yesterday that was in the Add Primary Storage window. Thanks! -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e: mike.tutkow...@solidfire.com o: 303.746.7302 Advancing t

Re: Review Request 23008: Reverted the hardcoding fix for "SR-Label:" and "Path:" strings

2014-06-26 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23008/#review46755 --- Ship it! Due to time constraints with regards to RC1 of 4.4 (tentat

Re: Review Request 23008: Reverted the hardcoding fix for "SR-Label:" and "Path:" strings

2014-06-26 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23008/#review46754 --- This line looks incorrect to me: -

Re: [ACS4.4] i18n problems in Add Primary Storage dialog

2014-06-26 Thread Mike Tutkowski
Hi Vetri, No problem! It happens to all of us. :) I appreciate your efforts in making these files more i18n friendly. Thanks for fixing the issue so quickly. That helps a lot! Talk to you later, Mike On Thu, Jun 26, 2014 at 6:06 AM, Vetrivel Chinnasamy < vetrivel.chinnas...@citrix.com> wrote:

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-26 Thread Alex Ough
Kishan, The type of region id is Integer, not Long, so I'm wondering why it should be Long. Alex Ough On Thu, Jun 26, 2014 at 2:08 AM, Kishan Kavala wrote: >This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20099/ > > Alex, > As discussed on the

Re: [DISCUSS] [PROPOSAL] Implementation of DNS Provider for Bind (for 4.5)

2014-06-26 Thread Silvano Nogueira Buback
Thank you David. I put design documents on wiki: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Bind+and+PowerDNS+integration+by+Globo+DNSAPI. I create an issue https://issues.apache.org/jira/browse/CLOUDSTACK-6998 too. I look forward to hearing your feedbacks. []'s, Silvano Buback On

Re: [ACS5.0] IAM feature postponed from 4.4 to 5.0?

2014-06-26 Thread Meghna Kale
Thanks Daan. With completion I meant the documentation part. On Thu, Jun 26, 2014 at 6:49 PM, Daan Hoogland wrote: > Megha, the page you mention is a collection bin for all things planned > that are going to require a major version upgrade as they change the > application programming interfa

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-26 Thread Leo Simons
+1! I’ve been slowly trying to figure out which part of the public marvin test infrastructure we can run against our test infrastructure (and then, later on, add our own tests to the public set…). Having some kind of a defined set of “these tests belong with and pass against 4.4.0 (in the citrix Q

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-26 Thread Daan Hoogland
if it is test code it will hardly interfere with the release, If it does it is extra important to know about it early. The only reason to not put them in the 4.4 branch is because you don't want them in the 4.4.0 release. On Thu, Jun 26, 2014 at 1:33 PM, Abhinandan Prateek wrote: > > > On 25/06/1

Re: [ACS5.0] IAM feature postponed from 4.4 to 5.0?

2014-06-26 Thread Daan Hoogland
Megha, the page you mention is a collection bin for all things planned that are going to require a major version upgrade as they change the application programming interface. It is not just for the IAM extensions planned. It is completed only when 5.0 is out ;) Feel free to add to it or to propos

RE: [ACS4.4] i18n problems in Add Primary Storage dialog

2014-06-26 Thread Vetrivel Chinnasamy
Hi Mike, Kindly accept my apology for the issue. I have used script to identify certain pattern of hardcoded strings and fixed them. Some exceptions like this got escaped from my unit testing also. I have reverted the changes as suggested and created a patch for review. Brian/Jessica, Could yo

Review Request 23009: Fix for test_portable_ip.py script issues

2014-06-26 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23009/ --- Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri.

Review Request 23008: Reverted the hardcoding fix for "SR-Label:" and "Path:" strings

2014-06-26 Thread Vetrivel Chinnasamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23008/ --- Review request for cloudstack, Brian Federle and Jessica Wang. Repository: clou

RE: [Issue]: Cannot start virtual router

2014-06-26 Thread Rajesh Battala
Hi Is the VR started successfully? Is the systemvms(ssvm, cpvm) have got configured successfully! Were you able to view the console of VMs! Share the logs why vm deployment is failing. Am not able to see the screenshot. Thanks Rajesh Battala From: huangchunmei [mailto:huangchun...@internetware.c

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-26 Thread Abhinandan Prateek
On 25/06/14 11:30 pm, "Daan Hoogland" wrote: >On Wed, Jun 25, 2014 at 7:58 PM, Santhosh Edukulla > wrote: >... >> Team uses 4.4-forward marvin to test the changes. >You should really not test 4.4-forward, but 4.4! It is mainly about testing and fixing the Marvin framework. Didn¹t want to interf

[Issue]: Cannot start virtual router

2014-06-26 Thread huangchunmei
Hi, I am a CloudStack user, below issues blocked me, would you please help to check? For hyper-v, systemvms are running successfully. But when creating a hyper-v VM, could not start the virtual router as following, the Link Local IP Address is always 0.0.0.0 Thanks, Chunmei

Re: OpenVM.eu - repository of Cloudstack images and appliances

2014-06-26 Thread Sebastien Goasguen
[removing users@, marketing@ etc…] On Jun 26, 2014, at 3:30 AM, Nux! wrote: > Daan, > > Yup, a link will do just fine for now. > In addition to that I'll require also the kickstart file used to build it. > The images will have to be rebuilt regularly so we don't ship > vulnerable/obsolete st

Re: [ACS5.0] IAM feature postponed from 4.4 to 5.0?

2014-06-26 Thread Meghna Kale
Hi All, I have been following the IAM functionality work from quite sometime. And I am interested in this work and would like to contribute in the API changes and discussions. If there are any design documents or any Jira tickets related to these changes can you please point me to them that will b

Re: Review Request 22934: CLOUDSTACK-6984: Fixing few issues found durign simulator run

2014-06-26 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22934/#review46728 --- Commit 7bc997f4fbaa80ed6335ba3658baaee5c9cb8e48 in cloudstack's bran

Re: OpenVM.eu - repository of Cloudstack images and appliances

2014-06-26 Thread Nux!
Daan, Yup, a link will do just fine for now. In addition to that I'll require also the kickstart file used to build it. The images will have to be rebuilt regularly so we don't ship vulnerable/obsolete stuff. I'll have to write some sort of FAQ on this. Lucian - Original Message - >

Re: OpenVM.eu - repository of Cloudstack images and appliances

2014-06-26 Thread Daan Hoogland
love it Lucian, How will the procedure for submissions look? (I'm thinking a mail to you with a download link) On Thu, Jun 26, 2014 at 1:18 AM, David Nalley wrote: > This is awesome on Nux's part. I am thrilled to see this come to fruition. > > This is not an official CloudStack deployment. There

Re: [ACS45][ACS50][PROPOSAL] move forward feature freeze

2014-06-26 Thread Daan Hoogland
On Wed, Jun 25, 2014 at 11:11 PM, Animesh Chaturvedi wrote: > In response to Brocade I see your response > > "feature should be done (in it's branch) by 19th july. merging and fixing > issues may take to mid august" that essentially means feature freeze (cutting > the branch) by mid august it