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 http

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 n

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 > > > > > > can we please use # for inline comments in code and """ quoted > > doc-string comments und

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 S

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/#review27368 --- Ship it! d4072c6 - master - Prasanna Santhanam On Oct. 23, 2013,

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 ther

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

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 pl

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 , > > 0

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 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 adhere to semver, and acc

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 , > > 0

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 , > > 0

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 chang

[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 and

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 Srikante

[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 Sowm

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 Srikantes

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 refs/heads

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: Ha

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 Message--

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, Srikanteswar

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 Shilamkar

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 Vi

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 Sa

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 and

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 N

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/ > http://imageshack.us/photo/my-images/841

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 Sr

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

Regarding vhd-util binary

2013-10-23 Thread Rajesh Battala
Hi All, I had downloaded the vhd-util from http://download.cloud.com.s3.amazonaws.com/tools/vhd-util . I have an vhd file in the NFS share. I had mounted in my mgmt. server and tried vhd-util read -p -n "path of file" it works. But the same file when present in CIFS share, mounted on my mgmt. se

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" wrote: >On Wed, Oct 23, 2013 at 1: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. >> >> -abhi > >-1 >Who is t

Command sequence logic in agent code

2013-10-23 Thread Koushik Das
I was looking at the command sequencing logic in the agent code. Each agent maintains a sequence that gets initialised based on following logic private static final Random s_rand = new Random(System.currentTimeMillis()); _nextSequence = s_rand.nextInt(Short.MAX_VALUE) << 48; For every co

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: cloudstack-gi

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 S

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 Shi

RE: Regarding vhd-util binary

2013-10-23 Thread Donal Lafferty
Make sure your O/S support for CIFS includes read/write of files in mounted CIFS share. E.g. use vi to create and then edit a file on the shared storage from your management server. DL > -Original Message- > From: Rajesh Battala [mailto:rajesh.batt...@citrix.com] > Sent: 23 October

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: cloudstack-gi

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 S

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 /root/testvhd/template/tmpl/1/9/4382

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. R

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 > > > > > > I don't understand why a password is needed? Pas

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 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 contribut

Re: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Koushik Das
Congrats Sowmya! On 23-Oct-2013, at 4:16 PM, Prasanna Santhanam 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 contributors to con

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 Shephe

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 > > > > > > Their really should be a state field, not just a simple boolean flag > >

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 here

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 wrote: > It appears that the deployment of systemvms was broken by commit > e33ae74067d5

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 been

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: Subject: Re: [ANNOUNCE] New Committer: Krishnan, Sowmya Congrats Sowmya! On 23-Oct-2013, at 4:16 PM, Prasanna Santhanam wrote: > The Project Management

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: Prasa

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 createV

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: dev@cloudstack.

Re: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Ahmad Emneina
Congratulations Talluri! On Wed, Oct 23, 2013 at 9:59 AM, Sangeetha Hariharan < sangeetha.hariha...@citrix.com> wrote: > Congrats Talluri!! > > -Original Message- > From: Suresh Sadhu [mailto:suresh.sa...@citrix.com] > Sent: Wednesday, October 23, 2013 4:16 AM > To: dev@cloudstack.apache

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: > Subject: Re: [ANNOU

Re: [ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Koushik Das
Congrats Girish! On 23-Oct-2013, at 4:08 PM, Prasanna Santhanam wrote: > 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 co

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 Del

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!" To: dev@cloudstack.apache.org Cc: us...@cloudstack.apache.org Sent: Wednesday, October 23, 2013 10:48:40 AM

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

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 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 based VMs. > > The clien

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 wrote: > I think we could flip-flop these two lines if necessary: > > createVbd(conn, vmSpec, vmName

RE: [ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Rayees Namathponnan
Congrats Girsih -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: Shilamkar, Girish Nice work Girish! On Wed, Oct 23, 2013 at 9:59 AM, Sangeetha Hariharan <

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, Sangeeth

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 Har

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

2013-10-23 Thread jenkins
See 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 Federle

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 Laf

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 if

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" wrote: > I see one problem for us now, Marcus. > > * You have a running VM that you attach a volume to. > * You stop the VM. > * You detac

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 bre

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
You know, I forgot we supposed to be doing that! :) Multi-tasking too much today, I guess. Anyways, it must not be working because I still had a hypervisor connection after I shut down the VM. Let me investigate. On Wed, Oct 23, 2013 at 1:48 PM, Marcus Sorensen wrote: > Are we not disconnectin

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 S

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" > 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 to be carried fr

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

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, SuichI

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 comparato

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" > wrote: > > Er, sorry. That was poorly worded on my part. Some classes, like > SnapshotTest.java and all the storage providers, did not get up

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 stora

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

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
Yep. I’m running on a clean master. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 23, 2013, at 4:30 PM, Darren Shepherd wrote: > Okay let me look at that. Are you 100% sure your looking at a clean version

  1   2   >