Re: Review Request 13223: (CLOUDSTACK-2729) use file lock to prevent concurrent refreshPool/deleteVolume on KVM shared storage pool

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13223/#review24778 --- Commit 9ec8d48945f465b74284034b94f12efdd319bd0f in branch

Re: Review Request 13323: if a failure occurs while adding VM to another network (this should be the first vm in the subnet). The ip alias created as a part of this process is not removed.

2013-08-07 Thread Sheng Yang
On Aug. 7, 2013, 12:03 a.m., Sheng Yang wrote: server/src/com/cloud/network/NetworkManagerImpl.java, line 2433 https://reviews.apache.org/r/13323/diff/1/?file=337776#file337776line2433 It's better to show the semantic of if this is the last nic in the subnet in code. Say, if

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Daan Hoogland
Hi Dave, I see what you mean by colons wouldn't be allowed. Nothing changed but I went for this solution for consistency (over backward compatibility). I will change again. Let me get to work first. nice working around the globe;) Daan On Wed, Aug 7, 2013 at 3:57 AM, Sheng Yang

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Daan Hoogland
Sheng, Some of this code will be used in a feature allowing vpc gateways to be plugged to sdn networks and to allow multiple gateways to share that network. I am using that in a hack on 4.1.1 and porting the code to master now, bit by bit so to speak. This is why you see some functions that seem

Re: CONSOLE access via API

2013-08-07 Thread Antonio Petrocelli
Hi, my goal is obtain an url (API generated) to open directly the Console from my custom Cloud interface... Thanks. Regards. Il 06/08/2013 21:28, Musayev, Ilya ha scritto: Antonio, Would you explain what the end goal is? Console access is token based and works directly with your browser

Re: Breaking docs out

2013-08-07 Thread Daan Hoogland
ok, +0 then, I'm off for a coffee. On Tue, Aug 6, 2013 at 10:21 PM, Jessica Tomechak jessica.tomec...@citrix.com wrote: I'm +0 on it. Don't really mind either way. Jessica T. From: Radhika Puthiyetath [radhika.puthiyet...@citrix.com] Sent: Monday,

Re: Help needed with Cloudstack Setup

2013-08-07 Thread Dave Cahill
Hi Marcus, I wonder could we take the most recent Jenkins-built System VM template and get it hosted somewhere, then let people know the URL? We could then replace the old acton systemvm URL [1] with the new URL wherever it appears in docs etc. I'd imagine the template doesn't change all that

CloudStack Storage Plugin for CloudByte

2013-08-07 Thread Amit Das
Hi, I will be working on implementing the CS storage plugin for CloudByte's Elastistor. This new plugin should be based on the newly refactored storage plugin changes which has gone into 4.2 release already. Which CS release can accommodate this new storage plugin feature ? Regards, Amit

Re: Review Request 13316: CLOUDSTACK-3723: System VM Template for XenServer has wrong guest OS type Information on the Database.

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13316/#review24780 --- Commit 23aaeafd21aa667a82dca813e7314ab60a6b2ac7 in branch

RE: CloudStack Storage Plugin for CloudByte

2013-08-07 Thread Devdeep Singh
The next release will probably be 4.3. You can target the CloudByte plugin for that release. The can find more information on CloudStack release cycle here [1]. [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Releases Regards, Devdeep -Original Message- From: Amit Das

Re: Help needed with Cloudstack Setup

2013-08-07 Thread Prasanna Santhanam
I think Citrix has done this, at least that's why comes out of the db now: | http://download.cloud.com/templates/4.2/systemvmtemplate-2013-06-12-master-xen.vhd.bz2 | | http://download.cloud.com/templates/4.2/systemvmtemplate-2013-06-12-master-kvm.qcow2.bz2 | |

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12849/ --- (Updated Aug. 7, 2013, 8:50 a.m.) Review request for cloudstack, Chiradeep

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Dave Cahill
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12849/#review24787 --- Three minor comments, but my main concerns are addressed by this

Re: weird test failure

2013-08-07 Thread Daan Hoogland
I under-kinda-stand, Does this mean I should live with the whole build failing for a while once in a while? Meaning should I change this test and submit a patch or is there some windows or java setting I should look at? On Tue, Aug 6, 2013 at 8:19 PM, Chiradeep Vittal chiradeep.vit...@citrix.com

Re: IP Address Allocation

2013-08-07 Thread Daan Hoogland
things are falling into place in slow-motion. Thanks Alex! And Soheil for bringing this up. Let's have a hack session on this at the next CCC. On Tue, Aug 6, 2013 at 10:10 PM, Alex Huang alex.hu...@citrix.com wrote: I'll say it's a bad design. He he I designed it. :( We had to make some

Re: api calls in 4.1.1

2013-08-07 Thread Daan Hoogland
will do, to busy now but is bound to get priority without further pushing from my side:| On Wed, Aug 7, 2013 at 1:37 AM, Musayev, Ilya imusa...@webmd.net wrote: Daan, Take this back - the error was on my part it seems (I've used cloudmonkey with api/secret keys for another test user). Once

Re: Unable to create image on Cloudcentral

2013-08-07 Thread Daan Hoogland
Prabakaran, Can you make 'I got null pointer exception' more explicit? regards, On Thu, Jul 25, 2013 at 12:56 PM, Prabakaran R praba...@cusdelight.com wrote: Hi, I am trying to create image of virtual machine in cloudcentral using JClouds. Following is my code snippet

Re: zone create wizard issue with create storagepool

2013-08-07 Thread Daan Hoogland
Talluri, did you check the history of the files involved in creating a zone? regards, On Fri, Jul 26, 2013 at 4:57 AM, Srikanteswararao Talluri srikanteswararao.tall...@citrix.com wrote: Createstoragepool fails during zone creation while creating zone using zone create wizard. This is

Re: Force delete domain through test case

2013-08-07 Thread Daan Hoogland
On Fri, Aug 2, 2013 at 4:57 PM, Gaurav Aradhye gaurav.arad...@clogeny.com wrote: While forcefully deleting a domain through test case, it fails. The resources in the domain and the domain itself gets cleaned up, but the code fails with following error. After cleaning up the resources, it says

Re: Review Request 12098: Add PCI Passthrough to CloudStack on KVM Hypervisor

2013-08-07 Thread Pawit Pornkitprasan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12098/#review24788 --- I will rebase it in a few days. This patch only includes support

Review Request 13373: keep the existing changes in cloud-set-guest-sshkey.in to make sure we do not break the backward compatibility.

2013-08-07 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13373/ --- Review request for cloudstack and Jayapal Reddy. Bugs: Cloudstack-4138

Re: Review Request 13325: preserve data in iso_id column while upgrading form 3.0.7 to 4.2

2013-08-07 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13325/#review24789 --- setup/db/db/schema-307to410-cleanup.sql

Review Request 13374: Automation: Affinity Groups: Removing invalid test case

2013-08-07 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13374/ --- Review request for cloudstack and Prasanna Santhanam. Repository:

RE: [Doc] IP Address Reservation is Ready for Review

2013-08-07 Thread Radhika Puthiyetath
Thanks Saksham. They have been incorporated. The guide is refreshed and posted to https://issues.apache.org/jira/browse/CLOUDSTACK-850. From: Saksham Srivastava Sent: Wednesday, August 07, 2013 3:06 PM To: Radhika Puthiyetath; us...@cloudstack.apache.org; dev@cloudstack.apache.org; Abhinav Roy

Re: Review Request 13370: CLOUDSTACK-4101: Update the Xenserver template/checksum in template.sql to the one verified.

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13370/#review24791 --- Commit d3666b96cd6aec4a5ac8b968f89d9f60a638572c in branch

Re: Review Request 13324: CLOUDSTACK-4104: VM Creation should fail if a zone is dedicated to a domain and user of the same domain deploys vm without using affinity group Explicit Dedication

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13324/#review24792 --- Commit a3ba198e5aba35297201bdf6b1ef127c0a5b01a5 in branch

Re: Review Request 13372: CLOUDSTACK-4062: SLES 11 SP2, SLES 11 SP3, CENTOS 6.4, RHEL 6.3, RHEL 6.4 OS type missing from OS Type drop down list when registering template.

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13372/#review24790 --- Commit 5dd155d02b5e03d20faf6826cc86f570b95b6d37 in branch

Re: Review Request 13324: CLOUDSTACK-4104: VM Creation should fail if a zone is dedicated to a domain and user of the same domain deploys vm without using affinity group Explicit Dedication

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13324/#review24795 --- Commit 61c7aa7afea078239146c8957f49e8ea07ebd5db in branch

Re: Review Request 13370: CLOUDSTACK-4101: Update the Xenserver template/checksum in template.sql to the one verified.

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13370/#review24794 --- Commit d9568c89d7afea84387f4e265e3b400c33c4dda9 in branch

Re: Review Request 13372: CLOUDSTACK-4062: SLES 11 SP2, SLES 11 SP3, CENTOS 6.4, RHEL 6.3, RHEL 6.4 OS type missing from OS Type drop down list when registering template.

2013-08-07 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13372/#review24793 --- Commit 3de6cdb4f9422264babf14728e3ffa939d72c0b1 in branch

Re: Review Request 13324: CLOUDSTACK-4104: VM Creation should fail if a zone is dedicated to a domain and user of the same domain deploys vm without using affinity group Explicit Dedication

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13324/#review24796 --- Ship it! Ship It! - Devdeep Singh On Aug. 6, 2013, 12:53 p.m.,

Re: Review Request 13370: CLOUDSTACK-4101: Update the Xenserver template/checksum in template.sql to the one verified.

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13370/#review24797 --- Ship it! Ship It! - Devdeep Singh On Aug. 7, 2013, 8:12 a.m.,

Re: Review Request 13372: CLOUDSTACK-4062: SLES 11 SP2, SLES 11 SP3, CENTOS 6.4, RHEL 6.3, RHEL 6.4 OS type missing from OS Type drop down list when registering template.

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13372/#review24798 --- Ship it! Ship It! - Devdeep Singh On Aug. 7, 2013, 9:07 a.m.,

Review Request 13375: CLOUDSTACK-4144 Make specifyVlan to false for shared network

2013-08-07 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13375/ --- Review request for cloudstack and Prasanna Santhanam. Bugs: CLOUDSTACK-4144

RE: Unable to create image on Cloudcentral

2013-08-07 Thread Prabakaran
Thanks, but it resolved. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Wednesday, August 07, 2013 3:43 PM To: dev; Prabakaran R Subject: Re: Unable to create image on Cloudcentral Prabakaran, Can you make 'I got null pointer exception' more explicit?

Re: Review Request 13325: preserve data in iso_id column while upgrading form 3.0.7 to 4.2

2013-08-07 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13325/ --- (Updated Aug. 7, 2013, 1:02 p.m.) Review request for cloudstack and Kishan

Re: Review Request 13325: preserve data in iso_id column while upgrading form 3.0.7 to 4.2

2013-08-07 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13325/ --- (Updated Aug. 7, 2013, 1:02 p.m.) Review request for cloudstack and Kishan

Re: Review Request 13325: preserve data in iso_id column while upgrading form 3.0.7 to 4.2

2013-08-07 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13325/#review24799 --- Ship it! commit 95f36798559c71d1de9f34fedad597d12f09766b - Kishan

Re: Review Request 13373: keep the existing changes in cloud-set-guest-sshkey.in to make sure we do not break the backward compatibility.

2013-08-07 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13373/ --- (Updated Aug. 7, 2013, 1:25 p.m.) Review request for cloudstack, Jayapal Reddy

Review Request 13376: Fix QuickCloud secondary storage service ListTemplateCommand and ListVolumeCommand

2013-08-07 Thread Donal Lafferty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13376/ --- Review request for cloudstack, Chiradeep Vittal and Devdeep Singh. Repository:

Re: Ruby dev wanted

2013-08-07 Thread Isaac Chiang
Hi Chip, I just sent the pull request for the cloudstack_ruby_client on Github. Please have a look! I did some restructure works, let me know if you have any concerns, thanks. Regards Isaac On Thu, Aug 1, 2013 at 9:38 PM, Chip Childers chip.child...@sungard.comwrote: Happy to accept

Re: Review Request 13374: Automation: Affinity Groups: Removing invalid test case

2013-08-07 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13374/#review24800 --- Ship it! - Prasanna Santhanam On Aug. 7, 2013, 11:18 a.m.,

Re: Ruby dev wanted

2013-08-07 Thread Chip Childers
On Wed, Aug 07, 2013 at 10:05:13PM +0800, Isaac Chiang wrote: Hi Chip, I just sent the pull request for the cloudstack_ruby_client on Github. Please have a look! I did some restructure works, let me know if you have any concerns, thanks. Regards Isaac Looks great Isaac. I

RE: [Responsiveness report] 2013w31 dev

2013-08-07 Thread Musayev, Ilya
Daan, thanks a million for doing this! -Original Message- From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] Sent: Wednesday, August 07, 2013 6:24 AM To: dev@cloudstack.apache.org Subject: [Responsiveness report] 2013w31 dev the three from week 30 where still open. I asked

Re: IP Address Allocation

2013-08-07 Thread Murali Reddy
On 07/08/13 1:40 AM, Alex Huang alex.hu...@citrix.com wrote: I'll say it's a bad design. He he I designed it. :( We had to make some compromises due to time constraints. I always intended to move the ip allocation into NetworkGurus and actually attempted to several times but, at the end

Review Request 13378: Test assumes storagetype to be local therefore test_egress_fw_rules fail.

2013-08-07 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13378/ --- Review request for cloudstack, Ashutosh Kelkar and Prasanna Santhanam.

Re: Comment about Data Disk Offering when creating a VM from an ISO

2013-08-07 Thread Mike Tutkowski
That sounds reasonable to me. Do you know if a bug exists for this or perhaps I should create one? On Wed, Aug 7, 2013 at 2:03 AM, Ryan Lei ryan...@cht.com.tw wrote: I noticed this issue long ago, and I agree with you. In my understanding, while the tab in the VM wizard always says Data

Re: [Responsiveness report] 2013w31 dev

2013-08-07 Thread Daan Hoogland
less and less a problem :P On Wed, Aug 7, 2013 at 4:34 PM, Musayev, Ilya imusa...@webmd.net wrote: Daan, thanks a million for doing this! -Original Message- From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] Sent: Wednesday, August 07, 2013 6:24 AM To:

Re: Review Request 13377: CLOUDSTACK-4021: [Automation] TestExplicitDedication.test_01_deploy_vm_with_explicit_dedication failed to deploy VM

2013-08-07 Thread Saksham Srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13377/ --- (Updated Aug. 7, 2013, 4:18 p.m.) Review request for cloudstack, Devdeep Singh

Re: CloudStack Storage Plugin for CloudByte

2013-08-07 Thread Mike Tutkowski
Feel free to take a look at the SolidFire plug-in I developed for 4.2. I built it to support QoS (Min and Max IOPS guarantees), which may be of interest to you. In 4.2, it works for Disk Offerings only (hopefully Compute Offerings, as well, in 4.3). Also, since the hypervisor plug-ins needed to

Re: CloudStack Storage Plugin for CloudByte

2013-08-07 Thread Chiradeep Vittal
We have time based releases, so as long as your feature is code complete (not necessarily bug-free) by 9/30, it will hit 4.3. But first you have to formally propose the feature. Please see https://cwiki.apache.org/confluence/x/0h-VAQ https://cwiki.apache.org/confluence/x/ni-VAQ

[ACS42]Attach volumes to Instance which has VM snapshots

2013-08-07 Thread Sailaja Mada
Hi, I have an instance which has VM snapshots. (VMWARE hypervisor) Now I am trying to attach DATA volume to this instance. It failed saying Unable to attach volume, please specify a VM that does not have VM snapshots Is there any limitation not to add additional volumes if it has VM

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Donal Lafferty
On Aug. 6, 2013, 11:40 p.m., Sheng Yang wrote: api/src/com/cloud/network/Networks.java, line 79 https://reviews.apache.org/r/12849/diff/5/?file=337785#file337785line79 I don't quite understand this part. Why // is missing in this case? Who would require // and who wouldn't?

Re: CloudStack Storage Plugin for CloudByte

2013-08-07 Thread Amit Das
Thanks Mike. I have started looking into the storage related changes due to SolidFire storage plugin. I shall start the formal proposal process communicate my git feature branch soon. Regards, Amit *CloudByte Inc.* http://www.cloudbyte.com/ On Wed, Aug 7, 2013 at 9:50 PM, Mike Tutkowski

Re: Review Request 13008: Fix usage of vm.instancename.flag when generating VM names on ESX hypervisor

2013-08-07 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13008/#review24812 --- Ship it! commit 059e4b2ae0a44205a8567a79bf9a154a54a83492 Author:

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-08-07 Thread Daan Hoogland
Donal, The mix of vlan://number and number is one of the issues i had to deal with doing this. The vlan not being handled by my first patch correctly is why this second one was made. for vlans it reverts what I did before. What is it you are seeing? And how does it relate to the

Re: Review Request 13298: Put in upgrade path from 4.1.1 to 4.1.2

2013-08-07 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13298/#review24813 --- Vijay, as the schema is empty, you don't have to have this file.

[KVM] Helper for agent during HA operations

2013-08-07 Thread Wido den Hollander
Hi, In our production setups we have seen some crashes of the KVM agent. This could happen for all kinds of reasons, but that's not what I wanted to discuss. Also see this issue: https://issues.apache.org/jira/browse/CLOUDSTACK-3954 What I've been writing for a PoC in our company is a small

RE: IP Address Allocation

2013-08-07 Thread Soheil Eizadi
I think this is a consensus of where we are: We would follow the path of NetworkElement meaning the infrastructure to create a service offering, the provider concept and the use of agent/host infrastructure to support external providers. (The external NetworkGuru also uses many of the same

Re: weird test failure

2013-08-07 Thread Chiradeep Vittal
I'd say the test needs to be changed to allow for a negative skew. On 8/7/13 2:05 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: I under-kinda-stand, Does this mean I should live with the whole build failing for a while once in a while? Meaning should I change this test and submit a patch or

[ACS42] Closing Resolved Defects

2013-08-07 Thread Sudha Ponnaganti
Hi All, Below are total and Blocker/Critical defects in resolved state. Pl take some time to close them. Even if you get to close Cortical/Blocker that would be great. Reporter Total Blocker/Critical Abhinandan Prateek 1 1 Abhinav Roy 8 7

Re: Values within commands.properties.in

2013-08-07 Thread Chiradeep Vittal
I think 'private zones' are assigned to RESOURCE_DOMAIN_ADMIN On 8/7/13 1:29 AM, Ryan Lei ryan...@cht.com.tw wrote: Curious for long. When does the rule RESOURCE_DOMAIN_ADMIN come in place in CloudStack? A CloudStack account can only be root admin, domain admin, or user. Does

RE: [KVM] Helper for agent during HA operations

2013-08-07 Thread Edison Su
-Original Message- From: Wido den Hollander [mailto:w...@widodh.nl] Sent: Wednesday, August 07, 2013 10:53 AM To: dev@cloudstack.apache.org Cc: shadow...@gmail.com Subject: [KVM] Helper for agent during HA operations Hi, In our production setups we have seen some crashes of

Re: Review Request 13376: Fix QuickCloud secondary storage service ListTemplateCommand and ListVolumeCommand

2013-08-07 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13376/#review24814 --- Ship it! In the future, please format patches so that they can be

Re: Review Request 13252: SHA256 timing attack and brute force attack fix

2013-08-07 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13252/ --- (Updated Aug. 7, 2013, 7:01 p.m.) Review request for cloudstack and John

Re: Question about creating a template

2013-08-07 Thread Chiradeep Vittal
Looks like a bug to me. On 8/6/13 11:28 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Hi, I have a small Linux ISO (http://www.damnsmalllinux.org/download.html) I've uploaded to my secondary storage NFS location. I can create VM instances just fine with it. I stopped one of my VMs

Re: Review Request 13252: SHA256 timing attack and brute force attack fix

2013-08-07 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13252/#review24818 --- Ship it! Ship It! - Chiradeep Vittal On Aug. 7, 2013, 7:01

Re: weird test failure

2013-08-07 Thread Daan Hoogland
ok, in the mean while please have another look at https://reviews.apache.org/r/12849/ please? On Wed, Aug 7, 2013 at 8:01 PM, Chiradeep Vittal chiradeep.vit...@citrix.com wrote: I'd say the test needs to be changed to allow for a negative skew. On 8/7/13 2:05 AM, Daan Hoogland

Re: [KVM] Helper for agent during HA operations

2013-08-07 Thread Jörgen Maas
why not use systemd and launchd facilities ? On Wed, Aug 7, 2013 at 8:28 PM, Edison Su edison...@citrix.com wrote: -Original Message- From: Wido den Hollander [mailto:w...@widodh.nl] Sent: Wednesday, August 07, 2013 10:53 AM To: dev@cloudstack.apache.org Cc:

Re: Help needed with Cloudstack Setup

2013-08-07 Thread Sarath Chandra
Hi All, Thanks for your inputs. I have tried with the suggested changes but could not get it working. I started from scratch again to see if it helps. But I seem to be going wrong in some place. Can some one kinldy go though the below setup and tell me if its right. I am stuck since a week

4.2 build failing with latest head

2013-08-07 Thread Rayees Namathponnan
4.2 build failing with latest head, someone please look this ASAP ? [INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ cloud-api --- [debug] execute contextualize [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] skip non existing resourceDirectory

Re: Help needed with Cloudstack Setup

2013-08-07 Thread Marcus Sorensen
Local storage wasn't working at all yesterday in 4.2. Fixed last night. Not sure if that is related. On Aug 7, 2013 1:32 PM, Sarath Chandra sarath.cloudst...@gmail.com wrote: Hi All, Thanks for your inputs. I have tried with the suggested changes but could not get it working. I started from

Re: Help needed with Cloudstack Setup

2013-08-07 Thread Chip Childers
On Wed, Aug 07, 2013 at 01:50:28PM -0600, Marcus Sorensen wrote: Local storage wasn't working at all yesterday in 4.2. Fixed last night. Not sure if that is related. The master / 4.2 quality seems to be churning a ton... this isn't a good sign IMO. Did we cram in too much at the end again?

Re: Help needed with Cloudstack Setup

2013-08-07 Thread Marcus Sorensen
No idea, it's been mostly off of my radar. I just happened to try to test a few fixes and 4.2 was having problems with local storage. At the least I'd say it's not a good idea for someone to use it (or any non-production version) right now if they're setting up cloudstack for the first time. On

RE: 4.2 build failing with latest head

2013-08-07 Thread Rayees Namathponnan
Latest build works; please ignore -Original Message- From: Rayees Namathponnan [mailto:rayees.namathpon...@citrix.com] Sent: Wednesday, August 07, 2013 12:45 PM To: dev@cloudstack.apache.org Subject: 4.2 build failing with latest head 4.2 build failing with latest head, someone

[DISCUSS] KVM HA

2013-08-07 Thread Edison Su
There is long time issue related to KVM HA, see bug: CLOUDSTACK-3535. Basically, HA won't be triggered, if KVM agent is stopped either normally nor abnormally, HA only be triggered if the network between mgt server and kvm host is disconnected and the network between KVM hosts in the same

Re: [KVM] Helper for agent during HA operations

2013-08-07 Thread Marcus Sorensen
For me the most useful part is that I can take down the agent without anything freaking out. It's fairly common to upgrade the agent (or stop for a minute or two) without putting the host into maintenance and dealing with migrations. That's by far the normal case for when the agent isn't running,

Re: [DISCUSS] KVM HA

2013-08-07 Thread Marcus Sorensen
Does KVMInvestigator work on all shared primary storage, or just NFS? I'm only familiar with the NFS KVMHA directories. From this it seems like a clean stop of the KVM agent still shouldn't trigger any issues/HA, correct? On Wed, Aug 7, 2013 at 2:28 PM, Edison Su edison...@citrix.com wrote:

Re: Review Request 13323: if a failure occurs while adding VM to another network (this should be the first vm in the subnet). The ip alias created as a part of this process is not removed.

2013-08-07 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13323/#review24826 --- Ship it! Ship It! - Sheng Yang On Aug. 7, 2013, 8:53 a.m.,

Re: [DISCUSS] KVM HA

2013-08-07 Thread Marcus Sorensen
I'm not sure we can rely on IPMI to tell us much about the host status itself. It's easy to use it for checking on basic poweron/poweroff, temperature, etc, but not so easy to tell if something is wrong with the OS, config, or at the software level. However, I did mention support in that thread

Re: IP Address Allocation

2013-08-07 Thread Chiradeep Vittal
Yes, I think so On 8/7/13 10:59 AM, Soheil Eizadi seiz...@infoblox.com wrote: I think this is a consensus of where we are: We would follow the path of NetworkElement meaning the infrastructure to create a service offering, the provider concept and the use of agent/host infrastructure to support

Re: Review Request 13323: if a failure occurs while adding VM to another network (this should be the first vm in the subnet). The ip alias created as a part of this process is not removed.

2013-08-07 Thread Sheng Yang
On Aug. 7, 2013, 8:46 p.m., Sheng Yang wrote: Ship It! Please uppercase the bug id(CLOUDSTACK- rather than Cloudstack-xxx) next time, otherwise jira won't recognize it. I also removed the unnecessary import and blank line. - Sheng

Re: Review Request 13323: if a failure occurs while adding VM to another network (this should be the first vm in the subnet). The ip alias created as a part of this process is not removed.

2013-08-07 Thread Sheng Yang
On Aug. 7, 2013, 8:46 p.m., Sheng Yang wrote: Ship It! Sheng Yang wrote: Please uppercase the bug id(CLOUDSTACK- rather than Cloudstack-xxx) next time, otherwise jira won't recognize it. I also removed the unnecessary import and blank line. Pushed to 4.2 Bharat, please send

Re: Comment about Data Disk Offering when creating a VM from an ISO

2013-08-07 Thread Mike Tutkowski
I went ahead and logged a JIRA ticket for this issue. On Wed, Aug 7, 2013 at 10:14 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: That sounds reasonable to me. Do you know if a bug exists for this or perhaps I should create one? On Wed, Aug 7, 2013 at 2:03 AM, Ryan Lei

RE: [ACS42] Closing Resolved Defects

2013-08-07 Thread Animesh Chaturvedi
I bulk updated the comment for these tickets asking for verifying the issues Animesh -Original Message- From: Sudha Ponnaganti [mailto:sudha.ponnaga...@citrix.com] Sent: Wednesday, August 07, 2013 11:02 AM To: dev@cloudstack.apache.org Subject: [ACS42] Closing Resolved Defects Hi

RE: [DISCUSS] KVM HA

2013-08-07 Thread Edison Su
-Original Message- From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Wednesday, August 07, 2013 1:42 PM To: dev@cloudstack.apache.org Subject: Re: [DISCUSS] KVM HA Does KVMInvestigator work on all shared primary storage, or just NFS? Right now, the agent only writes

[DISCUSS] Binary diff and hot patch

2013-08-07 Thread Musayev, Ilya
I was wondering if we can use binary diffs to hotpatch ACS. Currently, the way for me to identify what patch could have caused issue X, would be - try to find the suspects - it could be several. Revert 1 commit, rebuild and retry, if failed, restore previously reverted commit, revert 2nd

RE: [DISCUSS] KVM HA

2013-08-07 Thread Edison Su
-Original Message- From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Wednesday, August 07, 2013 1:46 PM To: dev@cloudstack.apache.org Subject: Re: [DISCUSS] KVM HA I'm not sure we can rely on IPMI to tell us much about the host status itself. It's easy to use it for

Re: Review Request 13298: Put in upgrade path from 4.1.1 to 4.1.2

2013-08-07 Thread Marcus Sorensen
Let's get this applied! 4.1 branch has been broken for over a week! On Wed, Aug 7, 2013 at 11:48 AM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: --- This is an automatically generated e-mail. To reply, visit:

RE: Review Request 13298: Put in upgrade path from 4.1.1 to 4.1.2

2013-08-07 Thread Vijayendra Bhamidipati
Will send out a patch shortly. Regards, Vijay -Original Message- From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Wednesday, August 07, 2013 3:09 PM To: dev@cloudstack.apache.org; Alena Prokharchyk Cc: Min Chen; Chip Childers; cloudstack; Vijayendra Bhamidipati Subject: Re:

RE: Question about creating a template

2013-08-07 Thread Soheil Eizadi
Hi Mike, The problem is with handling the command org.apache.cloudstack.storage.command.CopyCommand, look at the log before this error and see if there is any problem with the registered agent that is suppose to handle this command. -Soheil From: Mike

Re: Review Request 13373: keep the existing changes in cloud-set-guest-sshkey.in to make sure we do not break the backward compatibility.

2013-08-07 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13373/#review24838 --- Look fine to me. Make sure when you test it, you tested both

Re: Review Request 13298: Put in upgrade path from 4.1.1 to 4.1.2

2013-08-07 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13298/#review24840 --- Pushed the patch to 4.1 branch with commit id

Re: Review Request 13298: Put in upgrade path from 4.1.1 to 4.1.2

2013-08-07 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13298/#review24841 --- Ship it! Ship It! - Alena Prokharchyk On Aug. 7, 2013, 10:44

Re: Review Request 13298: Put in upgrade path from 4.1.1 to 4.1.2

2013-08-07 Thread Alena Prokharchyk
On Aug. 7, 2013, 10:58 p.m., Alena Prokharchyk wrote: Ship It! Vijay will open a separate reviewboard request for 411-412 upgrade path for master branch. The patch is going to be different given the presence of 4.2 release. - Alena

Re: [jira] [Commented] (CLOUDSTACK-2729) [Automation] Libvirt failed find primary storage and VM deployment failed

2013-08-07 Thread Marcus Sorensen
Here's the issue... local storage isn't mounted like remote storage is. This is invalid for local storage: String lockFile = _mountPoint + File.separator + pool.getUUIDString() + File.separator + _lockfile; Need to look at storage pool type and change lockfile to path for local storage On Wed,

Re: [jira] [Commented] (CLOUDSTACK-2729) [Automation] Libvirt failed find primary storage and VM deployment failed

2013-08-07 Thread Marcus Sorensen
Here's the debugging I put in: 2013-08-07 17:39:12,286 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-3:null) attempting to lock pool 2a9c418e-dd56-3f2c-9081-032b877d5644 with file '/mnt/2a9c418e-dd56-3f2c-9081-032b877d5644/KVMFILELOCK/.lock' 2013-08-07 17:39:12,425 DEBUG

RE: [jira] [Commented] (CLOUDSTACK-2729) [Automation] Libvirt failed find primary storage and VM deployment failed

2013-08-07 Thread Edison Su
The local path of storage pool should be able to get from LibvirtStoragePoolDef. -Original Message- From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Wednesday, August 07, 2013 4:43 PM To: dev@cloudstack.apache.org Cc: cloudstack-iss...@incubator.apache.org Subject: Re:

Re: [ACS42]Attach volumes to Instance which has VM snapshots

2013-08-07 Thread Kelven Yang
It is currently a limitation. After VM is running in VM-snapshot mode, it is not allowed to change VM's disk membership status Kelven On 8/7/13 10:09 AM, Sailaja Mada sailaja.m...@citrix.com wrote: Hi, I have an instance which has VM snapshots. (VMWARE hypervisor) Now I am trying to attach

Review Board messages

2013-08-07 Thread Dave Cahill
Hi, I noticed that Review Board messages are still going to cloudstack-...@incubator.apache.org. Should we switch that to dev@cloudstack.apache.org ? If so, who has the access to do so? Thanks, Dave.

  1   2   >