Re: Review Request 12905: Multiple VLAN range API need to accept a list rather than "add" or "remove" per command

> On July 24, 2013, 10:56 p.m., Alena Prokharchyk wrote: > > 1) Bharat, can you please check if your API allows you to set the vnet > > range with the multiple ranges. The command syntax should be like this - > > vnets should be specified and separated by comma: > > > > http://localhost:8096/?

Re: Review Request 12972: Update CitrixResourceBase to use new vmdata programming method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12972/ --- (Updated July 28, 2013, 4:39 a.m.) Review request for cloudstack, Prasanna Sant

Re: Review Request 12968: CLOUDSTACK-3686: Registering a template does not generate a usage event.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12968/#review24049 --- server/src/com/cloud/template/HypervisorTemplateAdapter.java

Re: Review Request 13003: Adding packaging support for RHEL 5 and 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13003/#review24047 --- Review 13003 PASSED the build test The url of build cloudstack-maste

RE: CloudStack 4.2 – Build and Packaging on RHEL 5.4

Submitted patch to support RHEL 5 and 6.2 packaging; someone please review this ? https://reviews.apache.org/r/13003/ Also I think we can change the packaging folder name form "cloudstack/packaging/centos63" to "cloudstack/packaging/centos"; after the merge. Regards, Rayees -Origi

Review Request 13003: Adding packaging support for RHEL 5 and 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13003/ --- Review request for cloudstack, edison su, Frank Zhang, Hugo Trippaers, and Wido

Re: Review Request 13001: Automation: Adding test cases for CPU limits from Limit Resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13001/#review24039 --- Review 13001 PASSED the build test The url of build cloudstack-maste

Review Request 13001: Automation: Adding test cases for CPU limits from Limit Resources.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13001/ --- Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam. Reposit

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12976/ --- (Updated July 27, 2013, 7:22 p.m.) Review request for cloudstack, Alena Prokhar

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12976/ --- (Updated July 27, 2013, 7:21 p.m.) Review request for cloudstack, Alena Prokhar

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

> On July 26, 2013, 11:51 p.m., Alena Prokharchyk wrote: > > We can't just remove the row w/o making sure its not being referenced by > > existing resources. So putting "DELETE IGNORE FROM `cloud`.`guest_os` where > > id=206" is not enough. You can go 2 ways: > > > > #1 - don't insert the reco

Re: Review Request 12968: CLOUDSTACK-3686: Registering a template does not generate a usage event.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12968/#review24036 --- server/src/com/cloud/template/HypervisorTemplateAdapter.java

Re: Review Request 12968: CLOUDSTACK-3686: Registering a template does not generate a usage event.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12968/ --- (Updated July 27, 2013, 6:58 p.m.) Review request for cloudstack and Min Chen.

Re: [VOTE] Apache Cloudstack 4.1.1 (Second Round)

-1 Details; https://issues.apache.org/jira/browse/CLOUDSTACK-3880 RHEL6 specific; risk for database corruption if MGR is shutdown via anything that changes runlevel resulting in use of cloudstack-management, cloudstack-usage stop() targets (e.g., /sbin/shutdown, "telinit 1", ACPID initiate

RE: Review Request 12658: Corrected typos in log messages

The submitter should close it out once the patch is approved and checked-in. The reviewer cannot close the patch unless they have admin privileges. If you want admin privileges you can open a ticket with Infra. > -Original Message- > From: Wido den Hollander [mailto:w...@widodh.nl] > Sen

Re: Dev Help: How to create scheduled tasks in ACS

Hi Alex, Will definitely look into it. These messages were just for research purposes, not looking to implement it for another month. Thanks, Ian On 27 July 2013 16:30, Alex Huang wrote: > Hi Ian, > > You might want to develop against vmsync branch for now, if you don't mind. > We intend to b

Review Request 12999: [GSoC] Add mid-term report

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12999/ --- Review request for cloudstack, Sebastien Goasguen and Hugo Trippaers. Repositor

Re: Review Request 12658: Corrected typos in log messages

On 07/26/2013 11:41 PM, Animesh Chaturvedi wrote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12658/ On July 18th, 2013, 9:48 a.m. UTC, *Wido den Hollander* wrote: Ship It! Wido did you commit the patch? If so this needs to be closed ou

Re: [VOTE] Apache Cloudstack 4.1.1 (Second Round)

+1 (binding) I've upgraded a couple of internal systems and even production systems to 4.1.1 We were facing a couple of bugs in production which lead me to deploy a homebrew version of 4.1.1 in our company. The only change is a very small change in the security groups script for something in

Re: Review Request 12996: Unable to Resize Volume (kvm, vmware)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12996/#review24034 --- Review 12996 PASSED the build test The url of build cloudstack-maste

Re: [GSOC][Urgent] Build docbook failed

Yeah it worked. Thanks Shiva! 2013/7/28 Shiva Teja > Change the itemizedlist in the id="Spring" to this > > > > components are declared as beans in componentcontext.xml > and applicationcontext.xml > > > >

Re: Review Request 12997: usage event is not generated for volume resize

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12997/#review24031 --- Review 12997 failed the build test : FAILURE The url of build clouds

Re: [GSOC][Urgent] Build docbook failed

Hi Shiva, I send to you my report. Please try build it. Thanks 2013/7/27 Shiva Teja > Oops, > > *paste it somewhere > > > On Sat, Jul 27, 2013 at 10:06 PM, Shiva Teja wrote: > > > Hi, > > > > Can you send the paste it somewhere and give a link? I can try. > > > > > > On Sat, Jul 27, 2013 at 9

Re: [GSOC][Urgent] Build docbook failed

Oops, *paste it somewhere On Sat, Jul 27, 2013 at 10:06 PM, Shiva Teja wrote: > Hi, > > Can you send the paste it somewhere and give a link? I can try. > > > On Sat, Jul 27, 2013 at 9:46 PM, Nguyen Anh Tu wrote: > >> Hi guys, >> >> I build my GSoC mid-term review and get an error here: >> >>

Re: [GSOC][Urgent] Build docbook failed

Hi, Can you send the paste it somewhere and give a link? I can try. On Sat, Jul 27, 2013 at 9:46 PM, Nguyen Anh Tu wrote: > Hi guys, > > I build my GSoC mid-term review and get an error here: > > Validation failed: > CloudStack_GSoC_Guide.xml:67: validity error : Element itemizedlist content >

[GSOC][Urgent] Build docbook failed

Hi guys, I build my GSoC mid-term review and get an error here: Validation failed: CloudStack_GSoC_Guide.xml:67: validity error : Element itemizedlist content does not follow the DTD, expecting (blockinfo? , (title , titleabbrev?)? , (caution | important | note | tip | warning | literallayout | p

RE: Dev Help: How to create scheduled tasks in ACS

Hi Ian, You might want to develop against vmsync branch for now, if you don't mind. We intend to bring the job queue work there over to master. I've been bringing vmsync over piece meal to master but haven't gotten to the job part yet. In the vmsync branch, what you'll find is that we've isol

Re: Review Request 12968: CLOUDSTACK-3686: Registering a template does not generate a usage event.

> On July 26, 2013, 5:38 p.m., Min Chen wrote: > > server/src/com/cloud/template/HypervisorTemplateAdapter.java, line 230 > > > > > > I don't understand why you need to do a bunch of search here. This > > method is in

RE: Do we delete an account after all users are deleted?

That's the behavior right now - we don't delete the account if users are all deleted. So we just need the test script to be removed/altered which is testing the other way. > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Saturday, July 27, 2013 8:38 PM > To

Review Request 12997: usage event is not generated for volume resize

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12997/ --- Review request for cloudstack, Devdeep Singh, Kishan Kavala, and Ram Ganesh. Re

RE: Do we delete an account after all users are deleted?

The bug is scarce on details. Assuming I understand what is asked correctly, this should not be done at all. A deletion of an account triggers an expunge of the all resources. Suppose someone made a mistake and removed all users from an account and cs automatically deleted the account and a

Review Request 12996: Unable to Resize Volume (kvm, vmware)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12996/ --- Review request for cloudstack, Devdeep Singh, edison su, Min Chen, and Ram Ganes

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/#review24028 --- Commit b21de4006abd61512b027af2dd9e8b07df1dd06f in branch refs/heads

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/#review24029 --- Ship it! commit b21de4006abd61512b027af2dd9e8b07df1dd06f - Kishan

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/#review24027 --- Commit a8ad908666b963bfd152b508fcec7259cf3a545f in branch refs/heads

RE: Do we delete an account after all users are deleted?

> -Original Message- > From: Sowmya Krishnan [mailto:sowmya.krish...@citrix.com] > Sent: Saturday, July 27, 2013 7:05 PM > To: dev@cloudstack.apache.org > Subject: RE: Do we delete an account after all users are deleted? > > > > > -Original Message- > > From: Prasanna Santhanam

Re: Do we delete an account after all users are deleted?

On Sat, Jul 27, 2013 at 01:34:25PM +, Sowmya Krishnan wrote: > > > > -Original Message- > > From: Prasanna Santhanam [mailto:t...@apache.org] > > Sent: Saturday, July 27, 2013 6:55 PM > > To: dev@cloudstack.apache.org > > Subject: Re: Do we delete an account after all users are delete

RE: Do we delete an account after all users are deleted?

> -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Saturday, July 27, 2013 6:55 PM > To: dev@cloudstack.apache.org > Subject: Re: Do we delete an account after all users are deleted? > > On Sat, Jul 27, 2013 at 12:19:31PM +, Sowmya Krishnan wrote: > > Do

Re: Do we delete an account after all users are deleted?

On Sat, Jul 27, 2013 at 12:19:31PM +, Sowmya Krishnan wrote: > Do we perform any cleanup of an account if all users in that account > are deleted? I don't find that is the current behavior. > I was going through test_accounts.py and figured that's one of the > tests. Here's the description of

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated July 27, 2013, 1:21 p.m.) Review request for cloudstack, sailaja mada,

Re: Review Request 12972: Update CitrixResourceBase to use new vmdata programming method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12972/#review24026 --- Ship it! Here's how I tested this: Reverted Sheng's revert which r

Re: Review Request 12995: [GSoC] Add midterm report to docs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12995/ --- (Updated July 27, 2013, 1:11 p.m.) Review request for cloudstack and Sebastien

Review Request 12995: [GSoC] Add midterm report to docs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12995/ --- Review request for cloudstack and Sebastien Goasguen. Repository: cloudstack-gi

Review Request 12994: CLOUDSTACK-3216 Logs in the Software router are not being rotated

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12994/ --- Review request for cloudstack, Alex Huang, Devdeep Singh, Ram Ganesh, and Hugo T

Do we delete an account after all users are deleted?

Do we perform any cleanup of an account if all users in that account are deleted? I don't find that is the current behavior. I was going through test_accounts.py and figured that's one of the tests. Here's the description of that test: # Validate the following # 1. Remove both the users

Re: Review Request 12928: update midterm review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12928/ --- (Updated July 27, 2013, 11:56 a.m.) Review request for cloudstack, Abhinandan P

Re: Review Request 12928: update midterm review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12928/#review24025 --- Put the xml as patch. Mail me the images. I am still getting some is

Re: [DISCUSS] vlan uri format change

The original patch was committed under Alex' account. Work on the networks enums is far from done with https://reviews.apache.org/r/12849/. I commented on the one you are mentioning, Prasanna. regards, Daan On Sat, Jul 27, 2013 at 12:41 PM, Koushik Das wrote: > I raised this in a separate thread

Re: rejected mail by dev.apache.org

thanks, i c&p'd a link from another mail, big sin of course. On Sat, Jul 27, 2013 at 1:23 PM, Ian Duffy wrote: > Just check your not sending emails in HTML. They get rejected unless > they are plain text :-) > > On 27 July 2013 12:21, Daan Hoogland wrote: > > Hello, > > > > I replied to a mess

Re: Unable to get the management server node id

Great thanks Alax. Much neater than my work around of commenting out code. On 27 July 2013 03:53, Alex Huang wrote: > Ian, > > Assuming you're on master, I just checked in a bunch of changes yesterday > that included this one item. Every code now retrieves the management server > id from Mana

Re: Review Request 12968: CLOUDSTACK-3686: Registering a template does not generate a usage event.

> On July 26, 2013, 5:38 p.m., Min Chen wrote: > > server/src/com/cloud/template/HypervisorTemplateAdapter.java, line 230 > > > > > > I don't understand why you need to do a bunch of search here. This > > method is in

Re: rejected mail by dev.apache.org

Just check your not sending emails in HTML. They get rejected unless they are plain text :-) On 27 July 2013 12:21, Daan Hoogland wrote: > Hello, > > I replied to a message a minute ago and got > Delivery to the following recipient failed permanently: > > dev@cloudstack.apache.org > > Techn

rejected mail by dev.apache.org

Hello, I replied to a message a minute ago and got Delivery to the following recipient failed permanently: dev@cloudstack.apache.org Technical details of permanent failure: Google tried to deliver your message, but it was rejected by the server for the recipient domain cloudstack.apache.or

Re: Review Request 12985: CLOUDSTACK-3682: NPE in BridgeVifDriver causing systemvm startup failure in KVM

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12985/#review24023 --- this is a duplicate of https://reviews.apache.org/r/12849/ please r

Re: [DISCUSS] Compatibility issue between network plugins and hypervisors

H, isn't it the responsibility of the administrator to setup the cloud in a proper way? hypervisor and network implementations may enhance their capabilities at minor upgrades so it will not be easy to keep checks on this up to date in cloudstack. Am I missing the point here? regards, Daan On S

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12809/#review24022 --- Patch failed to apply on 4.2. Please update the patch - Jayapal Red

Re: Review Request 12928: update midterm review

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12928/ --- (Updated July 27, 2013, 10:57 a.m.) Review request for cloudstack, Abhinandan P

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

> On July 23, 2013, 12:30 p.m., Koushik Das wrote: > > api/src/com/cloud/network/Networks.java, line 225 > > > > > > Why only Vlan in case of IsolationType and Vlan, Storage and Mido in > > the case of BroadcastDomain

RE: [DISCUSS] vlan uri format change

I raised this in a separate thread. Daan created a new patch https://reviews.apache.org/r/12849/ to address backward compat. > -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Saturday, July 27, 2013 11:12 AM > To: dev@cloudstack.apache.org > Subject: Re: [DI

Re: [DISCUSS] maven dependencies...

ship it;) On Sat, Jul 27, 2013 at 12:09 AM, Alex Huang wrote: > Everyone, > > After looking around in the maven documentation, I realized the way we are > specifying dependencies is not quite right for a large project such as ours. > > Currently, almost every project declare their own dependenc

Re: Review Request 12992: CLOUDSTACK-3544: Patch to support upgrade to Xenserver 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12992/#review24021 --- Review 12992 PASSED the build test The url of build cloudstack-maste

Re: Review Request 12993: CLOUDSTACK-3874: Adding method to cloud-early-config to allow updation of systemvms log rotation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12993/#review24020 --- Commit 6403bb339d45b0a5fd0b2aa7aa59ec312c6ba2dd in branch refs/heads

Re: Review Request 12993: CLOUDSTACK-3874: Adding method to cloud-early-config to allow updation of systemvms log rotation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12993/#review24019 --- Commit a10861a57b1a0df11c80dbd04bd02cc2772de2f4 in branch refs/heads

Re: Review Request 12993: CLOUDSTACK-3874: Adding method to cloud-early-config to allow updation of systemvms log rotation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12993/#review24018 --- Ship it! Ship It! - Abhinandan Prateek On July 27, 2013, 8:55 a.

Re: Review Request 12992: CLOUDSTACK-3544: Patch to support upgrade to Xenserver 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12992/#review24017 --- Commit f33c887b0327f820429230654c040e45af2c651a in branch refs/heads

Re: Review Request 12992: CLOUDSTACK-3544: Patch to support upgrade to Xenserver 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12992/#review24016 --- Commit 65e12e1dadde0512ea1ebf51549bf6ab73c78818 in branch refs/heads

Re: Review Request 12992: CLOUDSTACK-3544: Patch to support upgrade to Xenserver 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12992/#review24015 --- Ship it! Ship It! - Abhinandan Prateek On July 27, 2013, 8:48 a.

Re: Review Request 12993: CLOUDSTACK-3874: Adding method to cloud-early-config to allow updation of systemvms log rotation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12993/#review24014 --- Review 12993 PASSED the build test The url of build cloudstack-maste

Review Request 12993: CLOUDSTACK-3874: Adding method to cloud-early-config to allow updation of systemvms log rotation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12993/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: 3874 Repository:

Review Request 12992: CLOUDSTACK-3544: Patch to support upgrade to Xenserver 6.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12992/ --- Review request for cloudstack, Abhinandan Prateek and Devdeep Singh. Bugs: CLOU