Re: Jenkins job setup request

2013-10-23 Thread Yoshikazu Nojima
Hi Prasanna, Thank you for setting up jenkins job and adding config file. It seems I forgot to include it in the patch, sorry. Everything looks good, Thank you! Noji 2013/10/22 Prasanna Santhanam t...@apache.org: On Tue, Oct 22, 2013 at 06:39:45PM -0600, Yoshikazu Nojima wrote: Hi, Is

Re: Review Request 14867: api call to import ldap users to the same domains in cloudstack

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14867/#review27362 --- How is the import behaviour when OUs contain other OUs? - Prasanna

Re: [ASF4.2.1] default to 64-bit system VM template

2013-10-23 Thread Wido den Hollander
On 10/23/2013 07:32 AM, Abhinandan Prateek wrote: We are planning to make 64-bit system vm templates as default offering in 4.2.1. This is an initial email to have thoughts from the community on this. I'd say -1. Since it is going to be a minor upgrade and although this seems like a

Re: Review Request 14867: api call to import ldap users to the same domains in cloudstack

2013-10-23 Thread Rajani Karuturi
On Oct. 23, 2013, 6:22 a.m., Prasanna Santhanam wrote: How is the import behaviour when OUs contain other OUs? That is still not done. (Its the second part of TODO:1, nested hierarchy) - Rajani --- This is an automatically generated

Re: Review Request 14867: api call to import ldap users to the same domains in cloudstack

2013-10-23 Thread Prasanna Santhanam
On Oct. 23, 2013, 6:22 a.m., Prasanna Santhanam wrote: How is the import behaviour when OUs contain other OUs? Rajani Karuturi wrote: That is still not done. (Its the second part of TODO:1, nested hierarchy) Ah, didn't notice the TODO. Is there a spec file on how this is planned to

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

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/ --- Review request for cloudstack and Prasanna Santhanam. Repository:

Review Request 14868: Bugfix: listNetworks API doesn't return vlan id / VNI ( Virtual Network Identifier in VXLAN )

2013-10-23 Thread Yoshikazu Nojima
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14868/ --- Review request for cloudstack, Alena Prokharchyk, Chiradeep Vittal, Murali

Re: Review Request 14868: CLOUDSTACK-4932: Bugfix: listNetworks API doesn't return vlan id / VNI ( Virtual Network Identifier in VXLAN )

2013-10-23 Thread Yoshikazu Nojima
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14868/ --- (Updated Oct. 23, 2013, 6:38 a.m.) Review request for cloudstack, Alena

Re: CS-Management HA Networking

2013-10-23 Thread Marty Sweet
Hi Darren, Thanks for getting back to me. I will set the networking config up again and run the commands you sent me over the next couple of days. Thanks, Marty On Tue, Oct 22, 2013 at 11:39 PM, Darren Shepherd darren.s.sheph...@gmail.com wrote: Well that wasn't very useful message. If you

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

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/#review27366 --- tools/marvin/marvin/cloudstackConnection.py

Review Request 14871: CLOUDSTACK-4927: ISO and Template test cases failing duing listiso api call

2013-10-23 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14871/ --- Review request for cloudstack and Kishan Kavala. Bugs: CLOUDSTACK-4927

RE: No running ssvm is found, so command will be sent to LocalHostEndPoint

2013-10-23 Thread Sanjay Tripathi
Hi Punith, I encountered the same issue with latest master and this happens because the agent is not able to talk to the management server. To resolve it, you need to set the value of global config parameter host to your management server IP. Also, destroy the existing systemVMs to get the

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

2013-10-23 Thread Santhosh Edukulla
On Oct. 23, 2013, 7:25 a.m., Prasanna Santhanam wrote: tools/marvin/marvin/cloudstackConnection.py, line 169 https://reviews.apache.org/r/14869/diff/1/?file=369602#file369602line169 can we please use # for inline comments in code and quoted doc-string comments under methods,

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

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/ --- (Updated Oct. 23, 2013, 8:19 a.m.) Review request for cloudstack and Prasanna

Review Request 14872: CLOUDSTACK-4934: Rename Limit resources tests so that they have unique names

2013-10-23 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14872/ --- Review request for cloudstack and Rayees Namathponnan. Bugs: CLOUDSTACK-4934

Review Request 14873: CLOUDSTACK-4934: Rename Limit resources tests so that they have unique names

2013-10-23 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14873/ --- Review request for cloudstack and Rayees Namathponnan. Bugs: CLOUDSTACK-4934

Re: Review Request 14867: api call to import ldap users to the same domains in cloudstack

2013-10-23 Thread Ian Duffy
On Oct. 23, 2013, 6:22 a.m., Prasanna Santhanam wrote: How is the import behaviour when OUs contain other OUs? Rajani Karuturi wrote: That is still not done. (Its the second part of TODO:1, nested hierarchy) Prasanna Santhanam wrote: Ah, didn't notice the TODO. Is there a spec

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

2013-10-23 Thread Santhosh Edukulla
On Oct. 22, 2013, 6:37 a.m., Prasanna Santhanam wrote: This looks fine except pep8 is broken in a few places. Can you please run pep8 *.py in tools/marvin/marvin to ensure the formatting is maintained? Please upload the patch that corrects these issues. The changes for this went in

Re: Review Request 14199: Adding missing test cases: Snapshots Improvement

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14199/#review27371 --- Patch will require few changes before applying to master. Can you

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14319/ --- (Updated Oct. 23, 2013, 9:01 a.m.) Review request for cloudstack, bharat kumar

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14319/#review27372 --- test/integration/component/test_non_contiguous_vlan.py

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread Gaurav Aradhye
On Oct. 23, 2013, 2:38 p.m., Gaurav Aradhye wrote: The naming of the tests should be improved. test_01/02/03 doesn't tell much about the test. Also is the ordering strictly required? Can I reorder the tests and still expect the tests to pass? If not, we should remove the 01 , 02 , 03

Re: [ASF4.2.1] default to 64-bit system VM template

2013-10-23 Thread David Nalley
On Wed, Oct 23, 2013 at 1:32 AM, Abhinandan Prateek abhinandan.prat...@citrix.com wrote: We are planning to make 64-bit system vm templates as default offering in 4.2.1. This is an initial email to have thoughts from the community on this. -abhi -1 Who is this we? I agree with Wido We

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread Prasanna Santhanam
On Oct. 23, 2013, 9:08 a.m., Gaurav Aradhye wrote: The naming of the tests should be improved. test_01/02/03 doesn't tell much about the test. Also is the ordering strictly required? Can I reorder the tests and still expect the tests to pass? If not, we should remove the 01 , 02 , 03

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread Girish Shilamkar
On Oct. 23, 2013, 9:08 a.m., Gaurav Aradhye wrote: The naming of the tests should be improved. test_01/02/03 doesn't tell much about the test. Also is the ordering strictly required? Can I reorder the tests and still expect the tests to pass? If not, we should remove the 01 , 02 , 03

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14319/#review27377 --- test/integration/component/test_non_contiguous_vlan.py

[ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Prasanna Santhanam
The Project Management Committee (PMC) for Apache CloudStack has asked Srikanteswara Talluri to become a committer and we are pleased to announce that they have accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit

[ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Prasanna Santhanam
The Project Management Committee (PMC) for Apache CloudStack has asked Girish Shilamkar to become a committer and we are pleased to announce that they have accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit changes

New Committer: Krishnan, Sowmya

2013-10-23 Thread Prasanna Santhanam
The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya Krishnan to become a committer and we are pleased to announce that they have accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit changes and

Re: Review Request 14319: CLOUDSTACK 2238: Automation - Non Contiguous VLAN Ranges

2013-10-23 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14319/#review27374 --- test/integration/component/test_non_contiguous_vlan.py

Can't create cluster in CS 4.2

2013-10-23 Thread Denis Finko
Hello CloudStack community, Could you please help found source of following issue and how to fix it. I am tring to configure CloudStack 4.2 (installed from cloudstack repo baseurl=http://cloudstack.apt-get.eu/rhel/4.2/) with VMware 5.1. During adding new zone into UI I have got following error:

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Sanjeev Neelarapu
Congrats Talluri !! -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:08 PM To: CloudStack Dev Subject: [ANNOUNCE] New Committer: Talluri, Srikanteswara The Project Management Committee (PMC) for Apache CloudStack has asked

[ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Prasanna Santhanam
The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya Krishnan to become a committer and we are pleased to announce that they have accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit changes and

RE: New Committer: Krishnan, Sowmya

2013-10-23 Thread Radhika Puthiyetath
Party time, Sowmya :-) Welcome aboard... -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:08 PM To: CloudStack Dev Subject: New Committer: Krishnan, Sowmya The Project Management Committee (PMC) for Apache CloudStack has asked

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Radhika Puthiyetath
Congratz, Talluri. -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:08 PM To: CloudStack Dev Subject: [ANNOUNCE] New Committer: Talluri, Srikanteswara The Project Management Committee (PMC) for Apache CloudStack has asked

Re: Can't create cluster in CS 4.2

2013-10-23 Thread Denis Finko
Links to attached files: http://imageshack.us/photo/my-images/594/7oi1.png/ http://imageshack.us/photo/my-images/841/w1i7.png/ http://imageshack.us/photo/my-images/856/e31y.png/ http://imageshack.us/photo/my-images/201/x45.png/ http://imageshack.us/photo/my-images/17/avd4.png/ On 10/23/2013 01:40

Re: Review Request 14871: CLOUDSTACK-4927: ISO and Template test cases failing duing listiso api call

2013-10-23 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14871/#review27378 --- Commit 2a6905a9ce5c6499a5632afb3d8b7e8af71ef7fa in branch

Re: Review Request 14871: CLOUDSTACK-4927: ISO and Template test cases failing duing listiso api call

2013-10-23 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14871/#review27379 --- Ship it! commit 2a6905a9ce5c6499a5632afb3d8b7e8af71ef7fa Author:

RE: New Committer: Krishnan, Sowmya

2013-10-23 Thread Sanjeev Neelarapu
Congrats Sowmya !!! -Original Message- From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com] Sent: Wednesday, October 23, 2013 4:19 PM To: dev@cloudstack.apache.org Subject: RE: New Committer: Krishnan, Sowmya Party time, Sowmya :-) Welcome aboard... -Original

Re: Review Request 14738: Testscript for DeployVM User data enhancement

2013-10-23 Thread suresh sadhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14738/ --- (Updated Oct. 23, 2013, 10:57 a.m.) Review request for cloudstack,

RE: [ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Radhika Puthiyetath
Congratz, Girish -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:08 PM To: CloudStack Dev Subject: [ANNOUNCE] New Committer: Shilamkar, Girish The Project Management Committee (PMC) for Apache CloudStack has asked Girish

Re: Review Request 14827: hyperv unit tests

2013-10-23 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14827/ --- (Updated Oct. 23, 2013, 11:05 a.m.) Review request for cloudstack, Chiradeep

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Suresh Sadhu
Congrats Talluri -Original Message- From: Sanjeev Neelarapu [mailto:sanjeev.neelar...@citrix.com] Sent: 23 October 2013 16:13 To: dev@cloudstack.apache.org Subject: RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara Congrats Talluri !! -Original Message- From: Prasanna

RE: New Committer: Krishnan, Sowmya

2013-10-23 Thread Suresh Sadhu
Congrats Sowmya!! -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: 23 October 2013 16:08 To: CloudStack Dev Subject: New Committer: Krishnan, Sowmya The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya Krishnan to become a committer

RE: New Committer: Krishnan, Sowmya

2013-10-23 Thread Sailaja Mada
Congrats Sowmya. -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:08 PM To: CloudStack Dev Subject: New Committer: Krishnan, Sowmya The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya Krishnan to become a

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Sailaja Mada
Congrats Talluri. -Original Message- From: Suresh Sadhu [mailto:suresh.sa...@citrix.com] Sent: Wednesday, October 23, 2013 4:46 PM To: dev@cloudstack.apache.org Subject: RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara Congrats Talluri -Original Message- From: Sanjeev

Re: Can't create cluster in CS 4.2

2013-10-23 Thread Denis Finko
Also links to screenshots of our networking configuration: http://imageshack.us/f/855/zbtu.png/ http://imageshack.us/f/708/840z.png/ On 10/23/2013 01:51 PM, Denis Finko wrote: Links to attached files: http://imageshack.us/photo/my-images/594/7oi1.png/

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Rajesh Battala
Hearty Congratulations :) -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:08 PM To: CloudStack Dev Subject: [ANNOUNCE] New Committer: Talluri, Srikanteswara The Project Management Committee (PMC) for Apache CloudStack has asked

RE: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Rajesh Battala
Hearty Congratulations :) -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 23, 2013 4:17 PM To: CloudStack Dev Subject: [ANNOUNCE] New Committer: Krishnan, Sowmya The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya

Re: [ASF4.2.1] default to 64-bit system VM template

2013-10-23 Thread Abhinandan Prateek
On 23/10/13 3:48 pm, David Nalley da...@gnsa.us wrote: On Wed, Oct 23, 2013 at 1:32 AM, Abhinandan Prateek abhinandan.prat...@citrix.com wrote: We are planning to make 64-bit system vm templates as default offering in 4.2.1. This is an initial email to have thoughts from the community on

Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14874/ --- Review request for cloudstack and Prasanna Santhanam. Repository:

RE: [Proposal] Improve VR upgrades

2013-10-23 Thread Kishan Kavala
Spec for this feature is available at [1]. For 4.3, as mentioned in the spec, enough flexibility is provided but the communication with older version systemVms will be limited. Please provide your feedback. [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Improve+SystemVm+upgrades

Re: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14874/ --- (Updated Oct. 23, 2013, 1:22 p.m.) Review request for cloudstack and Prasanna

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-23 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14459/ --- (Updated Oct. 23, 2013, 1:33 p.m.) Review request for cloudstack and Girish

Review Request 14875: Fixed few pep8 compliant issues

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14875/ --- Review request for cloudstack and Prasanna Santhanam. Repository:

Re: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14874/ --- (Updated Oct. 23, 2013, 1:53 p.m.) Review request for cloudstack and Prasanna

RE: Regarding vhd-util binary

2013-10-23 Thread Rajesh Battala
Hi Donal, I had mounted the cifs with rw permission. I was able to touch and write data to file and read it. But when used vhd-util to query/read the the vhd files present on cifs am getting error. root@rajesh-HVM-domU:~/testvhd# /root/vhd-util read -p -n

Re: [ASF4.2.1] default to 64-bit system VM template

2013-10-23 Thread Marty Sweet
What would be the main reasoning behind this change? Surely the minimalistic specifications that SystemVMs are assigned suit 32-bit better in terms of Memory? The main disadvantage of 64-bit architectures is that, relative to 32-bit architectures, the same data occupies more space in memory (due

Review Request 14876: CLOUDSTACK 2233: Automation test cases for Add Remove Networks to VM

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

Re: Review Request 14855: [Ssl Termination Support] Certificate managenemt interface

2013-10-23 Thread Syed Ahmed
On Oct. 22, 2013, 9:56 p.m., Darren Shepherd wrote: api/src/org/apache/cloudstack/api/command/user/loadbalancer/UploadSslCertCmd.java, lines 34-35 https://reviews.apache.org/r/14855/diff/1/?file=369298#file369298line34 I don't understand why a password is needed? Password is

RE: Regarding vhd-util binary

2013-10-23 Thread Donal Lafferty
I have seen problems with an out of date vhd-util, e.g. http://dlafferty.blogspot.co.uk/2013/08/using-cloudstacks-log-files-xenserver.html You could check that you are using the version of vhd-util you downloaded using which vhd-util. However, why don't you first rule out CIFS as the problem?

Re: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Koushik Das
Congrats Talluri! On 23-Oct-2013, at 4:07 PM, Prasanna Santhanam t...@apache.org wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Srikanteswara Talluri to become a committer and we are pleased to announce that they have accepted. Being a committer allows many

Re: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Koushik Das
Congrats Sowmya! On 23-Oct-2013, at 4:16 PM, Prasanna Santhanam t...@apache.org wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya Krishnan to become a committer and we are pleased to announce that they have accepted. Being a committer allows many

maven parallel builds

2013-10-23 Thread Darren Shepherd
Just throwing this out there. We all know the maven build is slow. It never donned on me until now that maybe maven would have a parallel build option and it does [1]. I wouldn't recommend this for official builds or jenkins jobs, but for developers use the parallel build option. Basically add

Re: Review Request 14855: [Ssl Termination Support] Certificate managenemt interface

2013-10-23 Thread Syed Ahmed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14855/ --- (Updated Oct. 23, 2013, 4:17 p.m.) Review request for cloudstack, Darren

Re: Review Request 14855: [Ssl Termination Support] Certificate managenemt interface

2013-10-23 Thread Syed Ahmed
On Oct. 22, 2013, 9:56 p.m., Darren Shepherd wrote: engine/schema/src/com/cloud/network/dao/LoadBalancerCertMapVO.java, line 31 https://reviews.apache.org/r/14855/diff/1/?file=369304#file369304line31 Their really should be a state field, not just a simple boolean flag for revoke.

Deploy systemvms broken on master

2013-10-23 Thread SuichII, Christopher
It appears that the deployment of systemvms was broken by commit e33ae74067d5d7ff165098028c0bd6897f14bbd2, specifically, the change

Re: Deploy systemvms broken on master

2013-10-23 Thread SuichII, Christopher
Adding the commit author. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco Red Hat On Oct 23, 2013, at 12:39 PM, SuichII, Christopher chris.su...@netapp.com wrote: It appears that the deployment of systemvms was broken by

RE: [Proposal] Improve VR upgrades

2013-10-23 Thread Alex Huang
+1. That's the way to do it with no downtime. Tighter control needs to be formed on the VR to VR version compatibility though. It's one reason I always support that we break VR into smaller pieces. Redundant VRs work great as routing but then if you have to handle DHCP, DNS, Load Balancing,

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-10-23 Thread Mike Tutkowski
Little problem that I wanted to get your take on, Marcus. When a VM is being started, we call createVdb before calling connectPhysicalDisksViaVmSpec. The problem is that createVdb calls getPhysicalDisk and my volume has not yet been connected because connectPhysicalDisksViaVmSpec has not yet

RE: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Sangeetha Hariharan
Congrats Sowmya! -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Wednesday, October 23, 2013 8:43 AM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New Committer: Krishnan, Sowmya Congrats Sowmya! On 23-Oct-2013, at 4:16 PM, Prasanna Santhanam

RE: [ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Sangeetha Hariharan
Congrats Girish!! -Original Message- From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com] Sent: Wednesday, October 23, 2013 3:59 AM To: dev@cloudstack.apache.org Subject: RE: [ANNOUNCE] New Committer: Shilamkar, Girish Congratz, Girish -Original Message- From:

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Sangeetha Hariharan
Congrats Talluri!! -Original Message- From: Suresh Sadhu [mailto:suresh.sa...@citrix.com] Sent: Wednesday, October 23, 2013 4:16 AM To: dev@cloudstack.apache.org Subject: RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara Congrats Talluri -Original Message- From: Sanjeev

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-10-23 Thread Marcus Sorensen
I think we could flip-flop these two lines if necessary: createVbd(conn, vmSpec, vmName, vm); _storagePoolMgr.connectPhysicalDisksViaVmSpec(vmSpec); I haven't actually tried it though. But in general I don't see the Libvirt DiskDef using size at all, which is what

Re: [ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Ahmad Emneina
Nice work Girish! On Wed, Oct 23, 2013 at 9:59 AM, Sangeetha Hariharan sangeetha.hariha...@citrix.com wrote: Congrats Girish!! -Original Message- From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com] Sent: Wednesday, October 23, 2013 3:59 AM To:

Re: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Ahmad Emneina
Woohooo!!! Congrats Sowmya! On Wed, Oct 23, 2013 at 9:54 AM, Sangeetha Hariharan sangeetha.hariha...@citrix.com wrote: Congrats Sowmya! -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Wednesday, October 23, 2013 8:43 AM To: dev@cloudstack.apache.org

How does HA work?

2013-10-23 Thread Nux!
Hello, Since I'm not a programmer reading the code doesn't make too much sense to me. Can anyone explain to me how HA works (for VMs expecially)? Who monitors these HA VMs? Can I run multiple monitors, for HA? Any recommendations, best practices, gotchas? Thanks, Lucian -- Sent from the

Re: How does HA work?

2013-10-23 Thread Kelcey Jamison Damage
I can tell you it uses 'Storage-Heartbeat method for HA, unless that's changed. That means the heartbeat ring is via the primary storage. - Original Message - From: Nux! n...@li.nux.ro To: dev@cloudstack.apache.org Cc: us...@cloudstack.apache.org Sent: Wednesday, October 23, 2013

Re: Review Request 14823: Refactor StrategyPriority

2013-10-23 Thread Darren Shepherd
On Oct. 22, 2013, 1:19 p.m., Chris Suich wrote: Darren, the design of all of this has already changed here: https://reviews.apache.org/r/14522/. Take a look at the newer revisions which haven't made it in yet (still waiting on a final review from John Burwell). Could you take a look

Re: Review Request 14549: Rename net.juniper.contrail to org.apache.cloudstack.network.contrail

2013-10-23 Thread Pedro Marques
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14549/ --- (Updated Oct. 23, 2013, 6:26 p.m.) Review request for cloudstack. Changes

Re: [VOTE] Accept the donation of RDP client code into Apache CloudStack

2013-10-23 Thread Chip Childers
On Mon, Oct 21, 2013 at 2:11 PM, Donal Lafferty donal.laffe...@citrix.com wrote: As stated in a previous thread [1], Citrix is proposing the donation of source for an RDP client. After donation, the client would be integrated with the console system VM in order to provide access to Hyper-V

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-10-23 Thread Mike Tutkowski
OK, whatever way you prefer then, Marcus (createVdb first or second). If I leave createVdb first and return 0, it does seem to work. On Wed, Oct 23, 2013 at 11:13 AM, Marcus Sorensen shadow...@gmail.comwrote: I think we could flip-flop these two lines if necessary:

RE: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Rayees Namathponnan
Congrats Talluri -Original Message- From: Ahmad Emneina [mailto:aemne...@gmail.com] Sent: Wednesday, October 23, 2013 10:33 AM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New Committer: Talluri, Srikanteswara Congratulations Talluri! On Wed, Oct 23, 2013 at 9:59 AM,

RE: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Rayees Namathponnan
Congrats Sowmya -Original Message- From: Ahmad Emneina [mailto:aemne...@gmail.com] Sent: Wednesday, October 23, 2013 10:34 AM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New Committer: Krishnan, Sowmya Woohooo!!! Congrats Sowmya! On Wed, Oct 23, 2013 at 9:54 AM, Sangeetha

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

2013-10-23 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/1584/changes Changes: [jessicawang] CLOUDSTACK-4024: UI (1) detailView widget - add action handler for prepareObjectStoreMigration. [laszlo.hornyak] Integer instantation removed [laszlo.hornyak] Safe properties loader

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-23 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14522/ --- (Updated Oct. 23, 2013, 7:19 p.m.) Review request for cloudstack, Brian

RE: [VOTE] Accept the donation of RDP client code into Apache CloudStack

2013-10-23 Thread Clayton Weise
+1 -Original Message- From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] Sent: Monday, October 21, 2013 3:36 PM To: dev@cloudstack.apache.org Subject: RE: [VOTE] Accept the donation of RDP client code into Apache CloudStack +1 -Original Message- From: Donal

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-10-23 Thread Mike Tutkowski
I see one problem for us now, Marcus. * You have a running VM that you attach a volume to. * You stop the VM. * You detach the volume. * You start up the VM. The VM will not be connected to the volume (which is good), but the hypervisor will still be connected to the volume. It would be great

Re: Review Request 14381: KVM: add connect/disconnect capabilities to StorageAdaptors so that external storage services can attach/detach devices on-demand

2013-10-23 Thread Marcus Sorensen
Are we not disconnecting when we stop the vm? There's a method for it, we should be. disconnectPhysicalDiskViaVmSpec On Oct 23, 2013 1:28 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: I see one problem for us now, Marcus. * You have a running VM that you attach a volume to. * You

StrategyPriority changes w/ Spring Changes

2013-10-23 Thread Darren Shepherd
Chris, Edison, You guys just committed 'Support Revert VM Disk from Snapshot.' At the same time I was merging both my txn-refactor and spring-modularization branches. They are really tricky merges and each time I have to rebase it takes awhile to figure out. Anyhow, your change + my changes

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
It looks like the changes from us didn’t make it through your merge at all. I should have been more clear in my review - there are logic changes that needed to be carried from my changes into yours, not simply a rebase. I will work on those changes and try to get a patch up shortly. -- Chris

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread Darren Shepherd
I can fix it, let me see what I broke. Darren On Oct 23, 2013, at 1:03 PM, SuichII, Christopher chris.su...@netapp.com wrote: It looks like the changes from us didn’t make it through your merge at all. I should have been more clear in my review - there are logic changes that needed

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
And it looks like some of your changes may have not merged correctly. I’m getting compile errors like: The method close() is undefined for the type Transaction This shouldn’t have come from our merge. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud

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

2013-10-23 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/1585/changes

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread Darren Shepherd
The transaction API was changed in the merge. I could have maybe missed updating a class. Let me check. When you said It looks like the changes from us didn’t make it through your merge at all, can you point to something specific that got lost? Darren On Wed, Oct 23, 2013 at 1:05 PM,

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
Er, sorry. That was poorly worded on my part. Some classes, like SnapshotTest.java and all the storage providers, did not get updated references to your refactoring. They still reference StrategyPriority.pickStrategy(), etc. Additionally, I changed the pickStrategy() logic from using a

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread Darren Shepherd
Okay let me look at that. Are you 100% sure your looking at a clean version of master? Darren On Oct 23, 2013, at 1:17 PM, SuichII, Christopher chris.su...@netapp.com wrote: Er, sorry. That was poorly worded on my part. Some classes, like SnapshotTest.java and all the storage

Tags on storagePool

2013-10-23 Thread Alena Prokharchyk
I came across a potential bug in the way allocators do volumes placement on storage, based on storage tags. Prachi, can you please confirm if the bug is real. The tags are passed to the createStoragePool API in form of tags='tag1,tag2,..': ?command=createStoragePool...tags=alena and stored

Re: Tags on storagePool

2013-10-23 Thread Mike Tutkowski
I agree - this is a bug. We should not assume because a value is true that the name is a storage tag. On Wed, Oct 23, 2013 at 2:36 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: I came across a potential bug in the way allocators do volumes placement on storage, based on storage

Re: Tags on storagePool

2013-10-23 Thread Mike Tutkowski
If someone logs a JIRA ticket on this, please note that the updateStoragePool command would need to be corrected, as well, because it has special knowledge of a true value meaning that the name is a storage tag. On Wed, Oct 23, 2013 at 2:45 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote:

  1   2   >