Re: How to fix libvirt storage pool refresh issue?

2013-08-02 Thread Wei ZHOU
Alex, Exactly. We can also use Enable Maintenance - umount nfs point - restart cloudstack-agent - Cancel Maintenance to solve this issue. -Wei 2013/8/2 Alex Huang alex.hu...@citrix.com So I have very limited knowledge on KVM. But, from my understanding from Edison, we should consider what

RE: [Doc] Dedicated Resources: Public IPs and VLANs per Account is Ready

2013-08-02 Thread Likitha Shetty
I just have 2 comments, 1. Since this feature allows the admin to assign (and release) the IP addresses and VLANs only to an account, not both domain and account we need to modify the introduction accordingly. 2. And could you also add the following to section 15.8, If an account has

Re: Review Request 13072: Print agent host name in logging of agent commands

2013-08-02 Thread Chiradeep Vittal
This of course introduces an extra DB call during EVERY agent communication. If there was a cache, not a big deal, but as it so happens, there isn't one. On 8/1/13 10:10 PM, Koushik Das koushik@citrix.com wrote: I see your point Marcus. I felt that accessing the database would be simpler

Can not launch instances on 4.2 today

2013-08-02 Thread Thomas O'Dowd
This was working yesterday but the launching of instances for me on 4.2 is currently not working. I have twice now tried a clean build along with deploydb and deploysvr in the usual manner but no luck. On the UI I get: Unable to create a deployment for VM[User|tom1] On the management server

How to clean deployed system vm on xen host

2013-08-02 Thread Fang Wang
Hi, I started running MS , host is xen hypervisor. After 2 system vms are up, there were issues. So I made changes in source code, rebuild, did deploydb to clean DB. Now I am running ms again, but failed to bring up ssvm and cpvm, because of the existing old system vm from previous run. I

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

2013-08-02 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13223/ --- Review request for cloudstack, edison su and Wido den Hollander. Bugs:

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Prasanna Santhanam
On devcloud $ cd / $ find . -name vmdata.py Do you see a file by that name anywhere in the XCP plugins? If not - it failed to copy over into the devcloud plugins for some reason when adding the host. On Fri, Aug 02, 2013 at 03:22:20PM +0900, Thomas O'Dowd wrote: This was working yesterday but

Re: How to fix libvirt storage pool refresh issue?

2013-08-02 Thread Wei ZHOU
Edison, Please review the patch: https://reviews.apache.org/r/13223/ -Wei 2013/8/2 Wei ZHOU ustcweiz...@gmail.com Alex, Exactly. We can also use Enable Maintenance - umount nfs point - restart cloudstack-agent - Cancel Maintenance to solve this issue. -Wei 2013/8/2 Alex Huang

Re: How to clean deployed system vm on xen host

2013-08-02 Thread Prasanna Santhanam
On your xen host $ xe vm-uninstall --force --multiple And reconnect the MS again, it should kick in HA for systemVMs and restart them. On Fri, Aug 02, 2013 at 06:36:09AM +, Fang Wang wrote: Hi, I started running MS , host is xen hypervisor. After 2 system vms are up, there were issues.

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Thomas O'Dowd
Hi Prasanna, I've just finished re-initializing a brand new devcloud and am about to start things up again. root@devcloud:~# find / -name vmdata.py root@devcloud:~# Maybe it was there before in my previous install. How would it be copied to devcloud usually? For example for the s3xen plugin I

[Doc] Multiple IP Address per NIC for Review

2013-08-02 Thread Radhika Puthiyetath
Hi, Multiple IP Address per NIC documentation is ready for review. The doc is attached at https://issues.apache.org/jira/browse/CLOUDSTACK-809. Please see 15.9. Configuring Multiple IP Addresses on a Single NIC (134), and provide your feedback. Regards -Radhika

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Prasanna Santhanam
Yeah this was a recent change that Marcus made to optimize the vmdata passed to guests. I'll verify why the change is not working shortly. Running some other tests at the moment on my setup. Meanwhile, Could you manually copy that to where you see the s3xen plugin and retry? On Fri, Aug 02,

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Thomas O'Dowd
Ok. I've just copied it over. Its the only one that ends with .py and I've chmod'd it to 755 also. Will check it shortly. I'm building. Tom. root@devcloud:~# ls -l /usr/lib/xcp/plugins/ total 164 -rwxrwxrwx 1 root root 8304 Nov 5 2012 coalesce-leaf -rwxrwxrwx 1 root root 301 Nov 27 2012

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

2013-08-02 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13223/#review24511 --- Review 13223 failed the build test : FAILURE The url of build

Re: Unable to start domR(VR) in devcloud using laster 4.1/master.

2013-08-02 Thread Nguyen Anh Tu
Hi guys, So how to fix this? I got the same error Thanks, 2013/4/1 Rajesh Battala rajesh.batt...@citrix.com I had debugged and found that from devcloud unable to ssh to the router vm. When it tries to execute the GetDomRVersionCmd its getting failing and router vm is getting destroyed. I

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Thomas O'Dowd
Prasanna, when I first ran deploysvr after a new database install, I had quite a lot of starting and crashing of the system vms (which seems new to me) but I waited and it settled down. I was then able to launch an instance again after the usual other steps. The instance now tells me the

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Prasanna Santhanam
I made a mistake - the vmdata goes into the router deployed by cloudstack but is called by the router_proxy.sh script within the /usr/lib/xcp directory of the host over ssh. root@r-5-VM:/# find . -name vmdata.py ./opt/cloud/bin/vmdata.py The tests are passing now and VM deploy works fine:

Re: Can not launch instances on 4.2 today

2013-08-02 Thread Thomas O'Dowd
Thanks Prasanna, I've confirmed that the vmdata.py is on the router vm. root@r-7-VM:~# find / -name vmdata.py /opt/cloud/bin/vmdata.py Maybe some other commit fixed the problem at the same time for me. Tom. On Fri, 2013-08-02 at 14:42 +0530, Prasanna Santhanam wrote: I made a mistake - the

Re: S3 Object Store testing

2013-08-02 Thread Thomas O'Dowd
Sanjeev, Any further ideas on https://issues.apache.org/jira/browse/CLOUDSTACK-3229 John worked and I worked on it quite a lot yesterday (he had a long day or rather a short night). We're passed the S3 plugin issue but snapshots are still failing on what looks like the last step. Tom. On Tue,

Re: Review Request 13109: CLOUDSTACK-3594: Fixes related to affinity groups

2013-08-02 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13109/#review24513 --- Ship it! Ship It! - Prasanna Santhanam On July 31, 2013, 12:13

reviewer wanted

2013-08-02 Thread Daan Hoogland
Can https://reviews.apache.org/r/13192/ be reviewed please. regards, Daan

Review Request 13236: CLOUDSTACK: 3798: Fixed test_accounts.TestUserLogin.test_LoginApiUuidResponse

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

RE: How to enforce data disk selection

2013-08-02 Thread Musayev, Ilya
Mike, That is correct, we need to enforce data disk selection when user goes through deploying instance wizard in UI. Is there a config to do this or should I hack some .js files? Thanks ilya -Original Message- From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Friday,

Re: Review Request 12360: custom cpu model for kvm guest.the default model 'QEMU Virtual CPU version xxx' affect the activation of Windows OS and low performance.

2013-08-02 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12360/#review24514 --- Ship it! Ship It! - Wei Zhou On July 14, 2013, 3:38 a.m., Jijun

Re: Review Request 13236: CLOUDSTACK: 3798: Fixed test_accounts.TestUserLogin.test_LoginApiUuidResponse

2013-08-02 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13236/#review24515 --- The reason for failure of the login tests is that Marvin is not

Review Request 13237: Unable to resize disk with Zone wide storage

2013-08-02 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13237/ --- Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh. Bugs:

Re: [ACS42] Release Status Update: After Code Freeze Next Steps

2013-08-02 Thread Dave Cahill
Now that we've bumped the version number, do we need to create Upgrade420to430 and add it to engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java ? I ran mvn -P developer -pl developer,tools/devcloud -Ddeploydb (from [1]) on a fresh checkout of master and got: [ERROR] Failed to

RE: How to enforce data disk selection

2013-08-02 Thread Sanjay Tripathi
There is no any gc to enforce the datadisk selection; perhaps you can hack instanceWizard.js file in ui-custom folder to disable the No thanks option. --Sanjay -Original Message- From: Musayev, Ilya [mailto:imusa...@webmd.net] Sent: Friday, August 02, 2013 3:17 PM To:

Values within commands.properties.in

2013-08-02 Thread Ian Duffy
Hi, This query is in relation to the permissions set for commands via client/tomcatconf/commands.properties.in Within the header of the file it states: bitmap of permissions at the end of each classname, 1 = ADMIN, 2 = RESOURCE_DOMAIN_ADMIN, 4 = DOMAIN_ADMIN, 8 = USER However, throughout the

Re: How to fix libvirt storage pool refresh issue?

2013-08-02 Thread Wido den Hollander
On 08/02/2013 01:55 AM, Edison Su wrote: Hi Wei, regarding to the bug CLOUDSTACK-2729, I removed storage.refresh during getStoragePool in LibvirtStorageAdaptor, but the issue still happened in BVT. I am thinking add file lock on primary storage, seems you already have the patch, could you

Re: Review Request 12934: Tests for egress firewall rules for advance zone

2013-08-02 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12934/#review24517 --- test/integration/component/test_egress_fw_rules.py

Re: [ACS42] Release Status Update: After Code Freeze Next Steps

2013-08-02 Thread Wido den Hollander
On 08/02/2013 11:22 AM, Dave Cahill wrote: Now that we've bumped the version number, do we need to create Upgrade420to430 and add it to engine/schema/src/com/cloud/upgrade/DatabaseUpgradeChecker.java ? Yes, I forgot that. I created the SQL file, but I didn't create the Java files. Wido

RE: Values within commands.properties.in

2013-08-02 Thread Musayev, Ilya
Ian, You can combine numbers to get the specific permission, for example 15 is the sum of 1, 2, 4, and 8 - which also means everyone has access 3 is the sum of 1 and 2 and only admin and resource admin have access Regards ilya -Original Message- From: Ian Duffy

Re: Values within commands.properties.in

2013-08-02 Thread Ian Duffy
Cool, thanks Ilya. I should of spotted that! On 2 August 2013 11:32, Musayev, Ilya imusa...@webmd.net wrote: Ian, You can combine numbers to get the specific permission, for example 15 is the sum of 1, 2, 4, and 8 - which also means everyone has access 3 is the sum of 1 and 2 and only admin

Re: Review Request 13112: upgrade to ASF 4.2 cloud Database Schema Inconsistencies on the Upgraded Setup - Table volumes

2013-08-02 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13112/#review24518 --- Ship it! Ship It! - Abhinandan Prateek On Aug. 1, 2013, 6:01

Re: [4.1] getVMPassword how to decrypt

2013-08-02 Thread Antonio Petrocelli
Thanks a lot. It work. I'm unable to retrive PRIVATEKEY with API call: http://cloudstack.apache.org/docs/api/apidocs-4.1/domain_admin/listSSHKeyPairs.html the only way to retrieve is from:

Re: [4.1] getVMPassword how to decrypt

2013-08-02 Thread Wei ZHOU
Antonio, For almost all clients, they do not want the provider storing the private key (on management server). It is a big security risk for clients. You have to store the key by yourself. -Wei 2013/8/2 Antonio Petrocelli a.petroce...@netsons.com Thanks a lot. It work. I'm unable to

Re: [4.1] getVMPassword how to decrypt

2013-08-02 Thread Antonio Petrocelli
This key must alreay be on the system... Only i need to retrieve it API wan't i need a different solution Thanks Il 02/08/2013 12:51, Wei ZHOU ha scritto: Antonio, For almost all clients, they do not want the provider storing the private key (on management server). It is a big

Re: Review Request 13239: fixed the test test_non_contigiousvlan.py

2013-08-02 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13239/#review24519 --- Ship it! - Prasanna Santhanam On Aug. 2, 2013, 10:54 a.m.,

[DISCUSS] [jira] make affectedVersion field mandatory.....

2013-08-02 Thread Ram Ganesh
Hi, While triaging bugs I noticed that many bugs had affectedVersion field as empty. This makes it difficult to guess the version/release the reporter was on while filing the bug. Can we make the affectedVersion field a mandatory field instead? Thanks, RamG

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

2013-08-02 Thread Wido den Hollander
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13223/#review24520 --- I haven't tested the patch, but I'm missing some comments. Can we

Re: Review Request 13239: fixed the test test_non_contigiousvlan.py

2013-08-02 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13239/#review24521 --- Review 13239 failed the build test : FAILURE The url of build

Re: Review Request 13112: upgrade to ASF 4.2 cloud Database Schema Inconsistencies on the Upgraded Setup - Table volumes

2013-08-02 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13112/ --- (Updated Aug. 2, 2013, 11:16 a.m.) Review request for cloudstack, Abhinandan

Re: How to fix libvirt storage pool refresh issue?

2013-08-02 Thread Wei ZHOU
Wido, You applied the libvirt patch on your production system, and this issue disappeared, right? If so, that is good. I expect the redhat community can accep the patch (or the v2 https://www.redhat.com/archives/libvir-list/2013-July/msg00639.html) ASAP. -Wei 2013/8/2 Wido den Hollander

Re: Review Request 13236: CLOUDSTACK: 3798: Fixed test_accounts.TestUserLogin.test_LoginApiUuidResponse

2013-08-02 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13236/#review24522 --- Ship it! The problem will be seen only when using marvin.sync to

RE: [DISCUSS] [jira] make affectedVersion field mandatory.....

2013-08-02 Thread Koushik Das
+1 -Original Message- From: Ram Ganesh [mailto:ram.gan...@citrix.com] Sent: Friday, August 02, 2013 4:38 PM To: dev@cloudstack.apache.org Subject: [DISCUSS] [jira] make affectedVersion field mandatory. Hi, While triaging bugs I noticed that many bugs had affectedVersion

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

2013-08-02 Thread Wei Zhou
On Aug. 2, 2013, 11:09 a.m., Wido den Hollander wrote: I haven't tested the patch, but I'm missing some comments. Can we add clear comments as to WHY this is in the code? Otherwise I'm afraid this will become legacy and in 6 months new committer won't have the slightest idea of why

Re: Review Request 12360: custom cpu model for kvm guest.the default model 'QEMU Virtual CPU version xxx' affect the activation of Windows OS and low performance.

2013-08-02 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12360/#review24525 --- Review 12360 failed the build test : FAILURE The url of build

Re: How to fix libvirt storage pool refresh issue?

2013-08-02 Thread Wido den Hollander
On 08/02/2013 01:22 PM, Wei ZHOU wrote: Wido, You applied the libvirt patch on your production system, and this issue disappeared, right? Yes, I applied that patch and rebuild libvirt. The issue never came back. If so, that is good. I expect the redhat community can accep the patch (or

Review Request 13238: JIRA-3701 : VM instantiation : Sending network information to a new VM instance

2013-08-02 Thread Nicolas FOATA
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13238/ --- Review request for cloudstack. Repository: cloudstack-git Description

[Doc] Egress Firewall Rules for Review

2013-08-02 Thread Radhika Puthiyetath
Hi, Egress Firewall Rules documentation is ready for review. The doc is attached at https://issues.apache.org/jira/browse/CLOUDSTACK-2685 Please see 15.22.2. Egress Firewall Rules in Advanced Zone(165), and provide your feedback. Regards -Radhika

[Doc] IP Address Reservation is Ready for Review

2013-08-02 Thread Radhika Puthiyetath
Hi, IP Reservation documentation is ready for review. The doc is attached at https://issues.apache.org/jira/browse/CLOUDSTACK-850. Please see 15.7. IP Reservation in Isolated Guest Networks, and provide your feedback. Regards -Radhika

Re: Review Request 13238: JIRA-3701 : VM instantiation : Sending network information to a new VM instance

2013-08-02 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13238/#review24526 --- Review 13238 failed the build test : FAILURE The url of build

[Doc] PVLAN Doc is Ready for Review

2013-08-02 Thread Radhika Puthiyetath
Hi, PVLAN documentation is ready for review. The doc is attached at https://issues.apache.org/jira/browse/CLOUDSTACK-850. Please see 15.14. Isolation in Advanced Zone Using Private VLAN (141), and provide your feedback. Regards -Radhika

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

2013-08-02 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13223/ --- (Updated Aug. 2, 2013, 1:03 p.m.) Review request for cloudstack, edison su and

Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-02 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13240/ --- Review request for cloudstack. Repository: cloudstack-git Description

Re: Review Request 13127: UI Plugin Internationalization

2013-08-02 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13127/ --- (Updated Aug. 2, 2013, 1:36 p.m.) Review request for cloudstack, Brian

Re: Review Request 13128: Zone Chart Extension

2013-08-02 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13128/ --- (Updated Aug. 2, 2013, 1:37 p.m.) Review request for cloudstack, Brian

Review Request 13241: was not populating the overcommit details in uservm details when overcommit is decreased to one.

2013-08-02 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13241/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: Cloudstack-1695

Re: Review Request 13132: Field default value functions

2013-08-02 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13132/ --- (Updated Aug. 2, 2013, 1:37 p.m.) Review request for cloudstack, Brian

Re: Review Request 13131: Default field identifier

2013-08-02 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13131/ --- (Updated Aug. 2, 2013, 1:38 p.m.) Review request for cloudstack, Brian

Re: Review Request 13241: was not populating the overcommit details in uservm details when overcommit is decreased to one.

2013-08-02 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13241/#review24528 --- Ship it! Ship It! - Abhinandan Prateek On Aug. 2, 2013, 1:36

Re: Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-02 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13240/#review24529 --- Review 13240 failed the build test : FAILURE The url of build

RE: Review Request 13072: Print agent host name in logging of agent commands

2013-08-02 Thread Alex Huang
Yes. I like the idea but we should just load the hostname during the attache's creation rather than on every communication. --Alex -Original Message- From: Chiradeep Vittal Sent: Thursday, August 1, 2013 11:20 PM To: dev@cloudstack.apache.org; Marcus Sorensen Cc: cloudstack; Alex

Re: Review Request 13072: Print agent host name in logging of agent commands

2013-08-02 Thread Marcus Sorensen
Yes, I thought of that, which is one of the reasons I asked for feedback. I'm not quite sure how to do it the was Alex suggests, but I'll look into it. I'm guessing some sort of id to hostname map that populates as attaches are created. On Aug 2, 2013 12:19 AM, Chiradeep Vittal

Re: Review Request 13241: was not populating the overcommit details in uservm details when overcommit is decreased to one.

2013-08-02 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13241/#review24530 --- Review 13241 failed the build test : FAILURE The url of build

Re: [4.1] getVMPassword how to decrypt

2013-08-02 Thread David Nalley
No it doesn't. Keeping both destroys the benefit of public key cryptography. The idea is that you can hand out the public key and never need share the private. On Aug 2, 2013 7:02 AM, Antonio Petrocelli a.petroce...@netsons.com wrote: This key must alreay be on the system... Only i need to

Force delete domain through test case

2013-08-02 Thread Gaurav Aradhye
Hi all, 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 can't find the specific domain. Has anybody seen this issue before? Log:

RE: [DISCUSS] [jira] make affectedVersion field mandatory.....

2013-08-02 Thread Toshiaki Hatano
+1 Thanks, -- Toshiaki -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Friday, August 02, 2013 05:25 To: dev@cloudstack.apache.org Subject: RE: [DISCUSS] [jira] make affectedVersion field mandatory. +1 -Original Message- From: Ram Ganesh

Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

2013-08-02 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13190/#review24538 --- Ship it! Ship It! - Brian Federle On Aug. 1, 2013, 11:05 a.m.,

Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

2013-08-02 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13190/#review24541 --- Commit 977b0eca973b721fe158768f3f3c5515b9d2aab7 in branch

Re: Review Request 13190: CLOUDSTACK-4010: Change required in the UI to pass startdate/enddate parameter rather than olderthan parameter in the archive/delete alerts/events API.

2013-08-02 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13190/#review24540 --- Commit 80ae4830ae63ab73a06ab033bc3feb3dd701cbb2 in branch

RE: Reverting 17267794adb2bab923fb20515a7b943780d61921

2013-08-02 Thread Alex Huang
Ok...i spoke too soon. Just talked with Prasanna. He pointed out that it's a large change that's been in since May. So I won't revert it. But the rule is no one can change create-schema.sql until the community decided we want to based off of a new copy of the create-schema. --Alex

Re: How to enforce data disk selection

2013-08-02 Thread Mike Tutkowski
Exactly...through the current CS GUI, this is hard coded to be optional - although it probably wouldn't be hard to make required if you change instanceWizard.js. On Fri, Aug 2, 2013 at 4:21 AM, Sanjay Tripathi sanjay.tripa...@citrix.comwrote: There is no any gc to enforce the datadisk

Re: Review Request 13192: create private network test

2013-08-02 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13192/#review24547 --- server/test/com/cloud/network/CreatePrivateNetworkTest.java

Re: Unable to start domR(VR) in devcloud using laster 4.1/master.

2013-08-02 Thread Chiradeep Vittal
On 4.1? On 8/2/13 1:05 AM, Nguyen Anh Tu ng.t...@gmail.com wrote: Hi guys, So how to fix this? I got the same error Thanks, 2013/4/1 Rajesh Battala rajesh.batt...@citrix.com I had debugged and found that from devcloud unable to ssh to the router vm. When it tries to execute the

Re: Review Request 13237: Unable to resize disk with Zone wide storage

2013-08-02 Thread Rajesh Battala
On Aug. 2, 2013, 11:29 a.m., Jenkins Cloudstack.org wrote: Review 13237 failed the build test : FAILURE The url of build cloudstack-master-with-patch #106 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/106/ patch is not generated on master branch hence this failure

Re: [DISCUSS] [jira] make affectedVersion field mandatory.....

2013-08-02 Thread Chiradeep Vittal
+1 On 8/2/13 4:06 AM, Ram Ganesh ram.gan...@citrix.com wrote: Hi, While triaging bugs I noticed that many bugs had affectedVersion field as empty. This makes it difficult to guess the version/release the reporter was on while filing the bug. Can we make the affectedVersion field a mandatory

RE: [DISCUSS] [jira] make affectedVersion field mandatory.....

2013-08-02 Thread Alex Huang
+10 How can we reproduce and verify without an affected version. --Alex -Original Message- From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] Sent: Friday, August 2, 2013 10:56 AM To: dev@cloudstack.apache.org Subject: Re: [DISCUSS] [jira] make affectedVersion field

CloudStack Bay Area Meetup on August 7th

2013-08-02 Thread Animesh Chaturvedi
I am happy to announce the CloudStack Bay area meetup is on August 7th. Here is the link for meetup details http://www.meetup.com/CloudStack-SF-Bay-Area-Users-Group/events/117379992/. If you wish to be the speaker for this meetup please reach out to Karen Vuong, John Kinsella or Animesh.

CloudStack WorkShop in NYC / Free Training

2013-08-02 Thread Musayev, Ilya
I'm brain-storming hosting a CloudStack workshop in NYC. I'm under impression that meetups are too informational - and not enough hands on. End user must bring a laptop with Virtual Box pre-installed and template downloaded prior to attending the workshop. I will also have few USB sticks with

Re: Review Request 13192: create private network test

2013-08-02 Thread daan Hoogland
On Aug. 2, 2013, 5:26 p.m., Chiradeep Vittal wrote: server/test/com/cloud/network/CreatePrivateNetworkTest.java, line 163 https://reviews.apache.org/r/13192/diff/1/?file=332329#file332329line163 Use logger.error instead of printStackTrace? This way the logfile will contain the

Re: Review Request 13192: create private network test

2013-08-02 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13192/ --- (Updated Aug. 2, 2013, 8:23 p.m.) Review request for cloudstack, Koushik Das

Re: Review Request 13192: create private network test

2013-08-02 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13192/ --- (Updated Aug. 2, 2013, 8:24 p.m.) Review request for cloudstack, Chiradeep

Re: Review Request 12976: Remove duplicate row from guest_os table

2013-08-02 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12976/#review24559 --- The fix is good, but the patch failed to apply: alena@macbookpro:

Re: Review Request 13192: create private network test

2013-08-02 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13192/#review24571 --- Ship it! Needs license headers. Fixed locally and committed.

Re: Review Request 13240: dealt with some warnings in NetworkServiceImpl

2013-08-02 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13240/#review24576 --- server/src/com/cloud/network/NetworkServiceImpl.java

Re: [Discuss] Making CloudMonkey simpler to use for admin tasks

2013-08-02 Thread Rohit Yadav
On Thu, Aug 1, 2013 at 10:43 PM, Donal Lafferty donal.laffe...@citrix.comwrote: I needed a different configuration than DevCloud provided, so I turned to CloudMonkey to automate setup of my test environment. This led to a blog on automating with CloudMonkey at

Re: Review Request 12976: Remove duplicate row from guest_os table

2013-08-02 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12976/ --- (Updated Aug. 2, 2013, 10:43 p.m.) Review request for cloudstack, Alena

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

2013-08-02 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13223/#review24589 ---

Re: Review Request 12976: Remove duplicate row from guest_os table

2013-08-02 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12976/#review24590 --- Ship it! Ship It! - Alena Prokharchyk On Aug. 2, 2013, 10:43

Re: Cannot deploy db on latest 4.1 branch

2013-08-02 Thread Alena Prokharchyk
Happens because the management server version was changed to 4.1.2 on the 4.1 branch, but the db upgrade path to 4.1.2 is not present in the databaseupgrade checker. Adding 411-412 db upgrade path – empty if there were no db changes between the releases- should fix this problem. -alena. From:

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

2013-08-02 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13008/#review24602 --- Ship it! Ship It! - Kelven Yang On Aug. 2, 2013, 12:49 a.m.,

Re: How to enforce data disk selection

2013-08-02 Thread Mike Tutkowski
As an example, you might want to look at the Wizard path related to kicking off a VM that is based on an ISO (as opposed to a template). In that path, the Disk Offering is required. On Fri, Aug 2, 2013 at 11:17 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Exactly...through the