JIRA 3061

2013-11-25 Thread Mandar Barve
Hi all, Problem: JIRA says list hosts API response didn't return CPU used parameter value. This bug is reported against version 4.0.2. I could not reproduce this problem with CS version 4.2. I used CloudMonkey CLI to fire API commands to the management server. With a basic zone created that has

Re: Review Request 15825: Added unit tests for Juniper Contrail Virtual Network (VN) and Virtual Machine (VM) Model classes.

2013-11-25 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15825/#review29367 ---

JIRA 285

2013-11-25 Thread Mandar Barve
Hi all, I could not reproduce this issue with vesion 4.0.2. I tried creating hourly snapshot schedule with a keep value of 4. I could see 4 snapshots retained. Then as mentioned in the bug I deleted the schedule and recreated with same parameters except changing the keep value to 3. After

Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

2013-11-25 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15667/#review29368 --- Can you rebase this patch against latest master? A lot of

Re: Review Request 15418: Fixes about: Code quality, checkstyle and cloudstack conventions

2013-11-25 Thread Hugo Trippaers
On Nov. 18, 2013, 6:30 p.m., daan Hoogland wrote: Ship It! commit 876b7e492f15154591ba132ddbfe6a8e7a4c4c3f Author: afornie afor...@schubergphilis.com Date: Mon Nov 18 12:12:07 2013 +0100 Checkstyle adjustments in code and configuration - Hugo

Re: Review Request 15418: Fixes about: Code quality, checkstyle and cloudstack conventions

2013-11-25 Thread Hugo Trippaers
On Nov. 18, 2013, 6:30 p.m., daan Hoogland wrote: Ship It! Hugo Trippaers wrote: commit 876b7e492f15154591ba132ddbfe6a8e7a4c4c3f Author: afornie afor...@schubergphilis.com Date: Mon Nov 18 12:12:07 2013 +0100 Checkstyle adjustments in code and configuration

Re: persistence layer

2013-11-25 Thread Sebastien Goasguen
On Nov 23, 2013, at 4:13 PM, Laszlo Hornyak laszlo.horn...@gmail.com wrote: Wouldn't it be a lot of work to move to JOOQ? All queries will have to be rewritten. An a non-java developer question: Will that help support different databases ? like moving to MariaDB ? On Sat, Nov 23,

Re: [DISCUSS] Reporting tool for feeding back zone, pod and cluster information

2013-11-25 Thread Sebastien Goasguen
On Nov 23, 2013, at 5:01 AM, Wido den Hollander w...@widodh.nl wrote: Hi, I discussed this during CCCEU13 with David, Chip and Hugo and I promised I put it on the ml. My idea is to come up with a reporting tool which users can run daily which feeds us back information about how they

Re: [PROPOSAL] User VM HA using native XS HA capabilities

2013-11-25 Thread Koushik Das
Thanks for the comments David. See inline. -Koushik On 22-Nov-2013, at 7:31 PM, David Nalley da...@gnsa.us wrote: Hi Koushik: In general I like the idea. A couple of comments: The upgrade section has a manual step for enabling HA manually per instance. Why a manual step? Why is

Re: [Doc] Validation Issue in Release Notes

2013-11-25 Thread Abhinandan Prateek
There are some issues with the 4.2/master docs. 4.2 is a priority. Anyone who fixes the build gets a special mention in the release notes ! Now can we have someone fix this. -abhi On 23/11/13 4:20 pm, Radhika Puthiyetath radhika.puthiyet...@citrix.com wrote: Hi, Sorry for cross-posting.

Re: [Doc] Validation Issue in Release Notes

2013-11-25 Thread Sebastien Goasguen
On Nov 25, 2013, at 5:40 AM, Abhinandan Prateek abhinandan.prat...@citrix.com wrote: There are some issues with the 4.2/master docs. 4.2 is a priority. Anyone who fixes the build gets a special mention in the release notes ! Now can we have someone fix this. I can't even locate:

Re: Review Request 15351: Fixing bugs from Coverity related to Dereferenced Null after check and as return value.

2013-11-25 Thread Wilder Rodrigues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15351/ --- (Updated Nov. 25, 2013, 11:02 a.m.) Review request for cloudstack and Hugo

RE: Unable add instance after update on CS4.2

2013-11-25 Thread Sanjay Tripathi
Good to hear that Diego!! If you think you followed the steps correctly then you can file a bug for this issue here: https://issues.apache.org/jira/browse/CLOUDSTACK --Sanjay From: Diego Spinola Castro [mailto:spinolacas...@gmail.com] Sent: Monday, November 25, 2013 5:28 PM To: Sanjay Tripathi

Re: persistence layer

2013-11-25 Thread Travis Graham
MariaDB is a drop in replacement for MySQL, so it can be used with or without the JOOQ changes. Travis On Nov 25, 2013, at 5:20 AM, Sebastien Goasguen run...@gmail.com wrote: On Nov 23, 2013, at 4:13 PM, Laszlo Hornyak laszlo.horn...@gmail.com wrote: Wouldn't it be a lot of work to move

Review Request 15832: enable custom offering support for scalevm

2013-11-25 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15832/ --- Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-5161

Re: Review Request 15455: Fixing all Coverity bugs on file Upgrade2214to30 related to resource leak. Now the statements closed and nullified before a new assignment.

2013-11-25 Thread Wilder Rodrigues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15455/ --- (Updated Nov. 25, 2013, 2:26 p.m.) Review request for cloudstack and Hugo

Re: Review Request 15455: Fixing all Coverity bugs on file Upgrade2214to30 related to resource leak. Now the statements closed and nullified before a new assignment.

2013-11-25 Thread Wilder Rodrigues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15455/ --- (Updated Nov. 25, 2013, 2:32 p.m.) Review request for cloudstack and Hugo

Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15833/ --- Review request for cloudstack, Girish Shilamkar and SrikanteswaraRao Talluri.

Review Request 15834: CLOUDSTACK-4737: Root volume metering

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

Re: Review Request 15647: Fixing coverity issues related to resource leak on FileInputStream being created anonymously.

2013-11-25 Thread Wilder Rodrigues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15647/ --- (Updated Nov. 25, 2013, 3:15 p.m.) Review request for cloudstack and Hugo

Re: Enabling AMQP/RabbitMQ Events on master

2013-11-25 Thread David Grizzanti
Murali, Would you be able to comment on how to enable the event message bus notifications on master? Thanks! --  David Grizzanti Software Engineer Sungard Availability Services e: david.grizza...@sungard.com w: 215.446.1431 c: 570.575.0315 On November 21, 2013 at 12:35:31 PM, Alena

Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread John Burwell
Darren, I originally presented my thoughts on this subject at CCC13 [1]. Fundamentally, I see CloudStack as having two distinct tiers — orchestration management and automation control. The orchestration tier coordinates the automation control layer to fulfill user goals (e.g. create a VM

NullPointerException when invalid zone is passed into UsageEventUtils (CLOUDSTACK-5220)

2013-11-25 Thread David Grizzanti
Hi All, I submitted a bug for null pointer exceptions I'm seeing when certain types of Usage Events are generated (when you have the event notification bus enabled).   Initially I had submitted two separate bugs for individual events causing null pointer exceptions (CLOUDSTACK-5023 and 

[Discuss] AutoScaling.next in CloudStack

2013-11-25 Thread tuna
Hi guys, At CCCEU13 I talked about the AutoScale without NetScaler feature working with XenServer XCP. For anyone don’t know about this feature, take a look into my slide here: http://www.slideshare.net/tuna20073882/autoscale-without-netscalerccceu13. Chiradeep and I had a short talk after

Re: Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread Darren Shepherd
You bring up some interesting points. I really need to digest this further. From a high level I think I agree, but there are a lot of implied details of what you've said. Darren On Mon, Nov 25, 2013 at 8:39 AM, John Burwell jburw...@basho.com wrote: Darren, I originally presented my

Re: Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread Darren Shepherd
I will ask one basic question. How do you forsee managing one mailbox per resource. If I have multiple servers running in an active-active mode, how do you determine which server has the mailbox? Do you create actors on demand? How do you synchronize that operation? Darren On Mon, Nov 25,

Need help in creating/posting Cloudstack plugin for Juniper's network devices

2013-11-25 Thread Pradeep HK
Hi, my name is Pradeep HK and I am a Tech Lead in Juniper Networks, Bangalore. I am leading the effort to develop Cloudstack Plugin(ie Network Guru) for Juniper's networking devices. I have developed Network Plugin for orchestration of L2 services on Juniper's Networking devices.  I need some

Re: Enabling AMQP/RabbitMQ Events on master

2013-11-25 Thread Darren Shepherd
Just create a file on the classpath META-INF/cloudstack/core/spring-event-bus-context.xml with the below contents (change server, port, username, etc) beans xmlns=http://www.springframework.org/schema/beans; xmlns:xsi=http://www.w3.org/2001/XMLSchema-instance;

RE: Need help in creating/posting Cloudstack plugin for Juniper's network devices

2013-11-25 Thread Rayees Namathponnan
Hi Pradeep, This may help you https://cwiki.apache.org/confluence/display/CLOUDSTACK/Plug-ins%2C+Modules%2C+and+Extensions https://cwiki.apache.org/confluence/display/CLOUDSTACK/Extensions Regards, Rayees -Original Message- From: Pradeep HK [mailto:pradeep...@yahoo.com] Sent:

what's the procedure for committing to 4.3?

2013-11-25 Thread Darren Shepherd
What's the procedure for making changes to 4.3? Obviously commit to master and then cherry-pick but at this point is there any other control around it? If I need to commit something in 4.3 do I just do it myself? Darren

Re: Unable to add NetScaler in 4.3 branch builds - Automation blocker

2013-11-25 Thread Darren Shepherd
It appears that I didn't commit any of the noredist spring configuration for network-elements. I will get that added in a bit once I just do some quick validation. Darren On Wed, Nov 20, 2013 at 7:41 PM, Rayees Namathponnan rayees.namathpon...@citrix.com wrote: Created below defect, EIP /

Re: what's the procedure for committing to 4.3?

2013-11-25 Thread Mike Tutkowski
Hey Darren, For 4.2 we just committed to master and then cherry picked it to 4.2 ourselves. We tended to require a JIRA ticket to be associated with the commit(s). As the release went on, the Release Manager took over cherry picking and would typically do so on our behalves, if we requested such

Re: what's the procedure for committing to 4.3?

2013-11-25 Thread Mike Tutkowski
As far as I know, we are NOT yet at the point where the Release Manager does the cherry picking for us (upon our request). Thanks On Mon, Nov 25, 2013 at 11:47 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Hey Darren, For 4.2 we just committed to master and then cherry picked it

Review Request 15840: CLOUDSTACK-5206: Ability to control the external id of first class objects

2013-11-25 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15840/ --- Review request for cloudstack. Bugs: CLOUDSTACK-5206

RE: Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread Edison Su
Won't the architecture used by Mesos/Omega solve the resource management/locking issue: http://mesos.apache.org/documentation/latest/mesos-architecture/ http://eurosys2013.tudos.org/wp-content/uploads/2013/paper/Schwarzkopf.pdf Basically, one server holds all the resource information in memory

Re: Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread John Burwell
Darren, In a peer-to-peer model such as I describe, active-active is and is not a concept. The supervision tree is responsible for identifying failure, and initiating process re-allocation for failed resources. For example, if a pod’s management process crashed, it would also crash all of

RE: Unable to add NetScaler in 4.3 branch builds - Automation blocker

2013-11-25 Thread Rayees Namathponnan
Thanks Darren, I can see your check-in in 4.3 and master branch; I will test this. Regards, Rayees From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] Sent: Monday, November 25, 2013 10:32 AM To: Rayees Namathponnan Cc: dev@cloudstack.apache.org Subject: Re: Unable to add NetScaler in

Re: Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread John Burwell
Edison, The CAP theorem applies to all distributed systems. One “master” controlling a bunch of a hypervisors being directed by orchestration engine + Zookeeper is a distributed system. In this case, a consistent system. In my very brief reading of it, CloudStack would need multiple Mesos

Re: Router requires upgrade. Unable to send command to router Error

2013-11-25 Thread Will Stevens
In trying to troubleshoot I think I have found another issue. I went looking for a 'more official' source for the system templates. I found this: http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.2.0/html/Release_Notes/upgrade-instructions.html#upgrade-from-3.0.x-to-4.0 Which gives a

Re: Router requires upgrade. Unable to send command to router Error

2013-11-25 Thread Travis Graham
Use the links from the Install Guide instead. http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.2.0/html/Installation_Guide/management-server-install-flow.html#prepare-system-vm-template Travis On Nov 25, 2013, at 3:01 PM, Will Stevens wstev...@cloudops.com wrote: In trying to

Re: Router requires upgrade. Unable to send command to router Error

2013-11-25 Thread Hugo Trippaers
The images should be downloadable from jenkins.buildacloud.org The 4.2 images are here: http://jenkins.buildacloud.org/view/4.2/job/build-systemvm-4.2/ There was something wrong with this build, but i’m working on it. Expect the images soon. Cheers, Hugo On 25 nov. 2013, at 21:14, Travis

Progress on GlusterFS support from the CCCEU Hackathon

2013-11-25 Thread David Nalley
Hi folks: Just bringing some things from the hackathon back to the mailing list. One of the things worked on there was GlusterFS support. Wido and Niels began work on this, and you can see the blog post[1] from Niels, which might be helpful to others as well for things like Sheepdog. There's

[PROPOSAL] Alert publishing via API

2013-11-25 Thread Alena Prokharchyk
Third party systems integrating with CloudStack should be able to publish custom alerts to the CS. Existing alerts might not be enough for a particular application purposes, and adding new one while utilizing the existing CS Alert notification system, can be quite useful. Currently there is no

Re: Router requires upgrade. Unable to send command to router Error

2013-11-25 Thread Will Stevens
I just used: http://jenkins.buildacloud.org/view/master/job/build-systemvm-master/lastSuccessfulBuild/artifact/tools/appliance/dist/systemvmtemplate-2013-11-24-master-xen.vhd.bz2 I ran into the same issue with this template as all the others I have been trying. Basically, the vm comes up with a

Re: Resource Management/Locking [was: Re: What would be your ideal solution?]

2013-11-25 Thread Darren Shepherd
Okay, I'll have to stew over this for a bit. My one general comment is that it seems complicated. Such a system seems like it would take a good amount of effort to construct properly and as such it's a risky endeavour. Darren On Mon, Nov 25, 2013 at 12:10 PM, John Burwell jburw...@basho.com

Re: Router requires upgrade. Unable to send command to router Error

2013-11-25 Thread Hugo Trippaers
Will, What is the error reported by the OS before the fsck is needed? I ran into a similar issue a while back and it was caused by a wrong date/time setting on the hypervisor. If the systemvm spins up with a time that is earlier than when the image was created the system will force an fsck

Re: Router requires upgrade. Unable to send command to router Error

2013-11-25 Thread Will Stevens
Thanks Hugo, that was the cause of my 'fsck' issue. NTP was broken on my dev XenServer due to a misconfigured resolv.conf. I can confirm that the recent System VM Templates are working. Thank you... On Mon, Nov 25, 2013 at 5:15 PM, Hugo Trippaers h...@trippaers.nl wrote: Will, What is the

RE: persistence layer

2013-11-25 Thread Alex Huang
Has anyone actually tried dropping in a different jdbc driver and see if CS can use another DB? I don't think the current CS DB layer prevents anyone from doing that. This is different from MariaDB which, as othes have pointed out, is drop-in replacement for MySQL. I'm talking about stuff

Re: [Doc] Validation Issue in Release Notes

2013-11-25 Thread kel...@backbonetechnology.com
Hi we had a similar issue at the conf with a nested list not contained in a listitem. It's worth a shot. I am not yet near reliable internet to look myself. -Kelcey Sent from my HTC - Reply message - From: Sebastien Goasguen run...@gmail.com To: dev@cloudstack.apache.org Subject:

Re: [Discuss] AutoScaling.next in CloudStack

2013-11-25 Thread Nguyen Anh Tu
2013/11/26 Chiradeep Vittal chiradeep.vit...@citrix.com Hi Tuna, I boldly diagrammed out what we talked about here: https://cwiki.apache.org/confluence/x/M6YTAg The idea is to keep the monitoring part separate from the autoscale decision. So, the monitoring can be SNMP/RRD/whatever.

Re: Need help in creating/posting Cloudstack plugin for Juniper's network devices

2013-11-25 Thread Pradeep HK
Thanks Rayees -Pradeep On Monday, November 25, 2013 11:15 PM, Pradeep HK pradeep...@yahoo.com wrote: Hi, my name is Pradeep HK and I am a Tech Lead in Juniper Networks, Bangalore. I am leading the effort to develop Cloudstack Plugin(ie Network Guru) for Juniper's networking devices. I

RE: [Doc] Validation Issue in Release Notes

2013-11-25 Thread Radhika Puthiyetath
That issue has been resolved. This is about a CDATA, which is still hidden to my eyes.. -Original Message- From: kel...@backbonetechnology.com [mailto:kel...@backbonetechnology.com] Sent: Tuesday, November 26, 2013 7:37 AM To: dev@cloudstack.apache.org Subject: Re: [Doc] Validation

RE: [Discuss] AutoScaling.next in CloudStack

2013-11-25 Thread Vijay Venkatachalam
Hi Chiradeep, The monitoring service seems to be collecting statistics using polling and triggers actions during threshold breach. This seems to be very tasking. Can it be designed to listen for events on threshold breach as well? For ex. a configuration

RE: [Doc] Validation Issue in Release Notes

2013-11-25 Thread Radhika Puthiyetath
Hi, I managed to use the --novalid option to create the file. Hope, it should work to generate the online version. -Original Message- From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com] Sent: Tuesday, November 26, 2013 10:00 AM To: dev@cloudstack.apache.org Subject: RE:

Re: Review Request 15834: CLOUDSTACK-4737: Root volume metering

2013-11-25 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15834/ --- (Updated Nov. 26, 2013, 5:39 a.m.) Review request for cloudstack and Koushik

Re: Review Request 15505: Usage details are not getting populated when using dynamic offerings.

2013-11-25 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15505/ --- (Updated Nov. 26, 2013, 5:57 a.m.) Review request for cloudstack, Kishan

Re: Review Request 15834: CLOUDSTACK-4737: handling usage events for dynamic compute offering

2013-11-25 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15834/ --- (Updated Nov. 26, 2013, 5:57 a.m.) Review request for cloudstack and Koushik

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15833/#review29420 --- Commit d6298302a1872eea1be52ccf5922174e469ed807 in branch

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15833/#review29421 --- Commit 5b37d38ed0b156268132ade1856032102723c36e in branch

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15833/#review29422 --- Ship it! Committed to 4.3 and master. - Girish Shilamkar On

[Responsiveness report] users 2013w46

2013-11-25 Thread Daan Hoogland
http://markmail.org/message/gha6ezktv7mpfgsh Create primary storage entry via API by Lisa B. http://markmail.org/message/l47ztbzzohqxs36h Multiple simultaneous tasks in vCenter by Sean Hamilton http://markmail.org/message/65nouxpzfbhxtjrb Major stability problems lately by Timothy Ehlers

[Responsiveness report] dev 2013w46

2013-11-25 Thread Daan Hoogland
http://markmail.org/message/ro7ahmfcj4pqppqo Snapshot file extension on nfs by Gaurav Aradhye http://markmail.org/message/zbcewsuysyvwj4ob [DISCUSS] (CLOUDSTACK-1889) by Saurav Lahiri http://markmail.org/message/rgv6vmvyjvwx67g7 help need error launching jetty by Juan Barrio

RE: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Santhosh Edukulla
Does this applies to 4,.2 and if possible prior versions as well? Santhosh From: Girish Shilamkar [nore...@reviews.apache.org] on behalf of Girish Shilamkar [gir...@clogeny.com] Sent: Tuesday, November 26, 2013 1:41 AM To: Girish Shilamkar;

Re: edit access to cwiki

2013-11-25 Thread Daan Hoogland
On Mon, Nov 18, 2013 at 7:36 AM, Shweta Agarwal shweta.agar...@citrix.com wrote: 'shweta.agar...@citrix.com added, sorry it took so long

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Ashutosh Kelkar
Yes, this a generic fix and would apply to all branches. On Tue, Nov 26, 2013 at 12:19 PM, Santhosh Edukulla santhosh.eduku...@citrix.com wrote: Does this applies to 4,.2 and if possible prior versions as well? Santhosh From: Girish Shilamkar

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Girish Shilamkar
Santhosh, I am not sure if we will be running tests against 4.2.1 so I did not merge it to 4.2 branch. Regards, Girish On 26-Nov-2013, at 12:19 PM, Santhosh Edukulla santhosh.eduku...@citrix.com wrote: Does this applies to 4,.2 and if possible prior versions as well? Santhosh

RE: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Santhosh Edukulla
But, its better to merge when tests are run in future for that branch as well. Santhosh From: Girish Shilamkar [gir...@clogeny.com] Sent: Tuesday, November 26, 2013 2:01 AM To: Santhosh Edukulla Cc: dev@cloudstack.apache.org; Srikanteswararao Talluri;

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread Girish Shilamkar
Ok, I will merge it to 4.2 as well. Regards, Girish On 26-Nov-2013, at 12:33 PM, Santhosh Edukulla santhosh.eduku...@citrix.com wrote: But, its better to merge when tests are run in future for that branch as well. Santhosh From: Girish

Re: Review Request 15508: Make sure that if the file does not exist an Exception is thrown and that once it exists it is also closed after the properties are loaded.

2013-11-25 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15508/#review29423 --- Ship it! commit 7a6751aa770eaf8065864f497bff401012f553ae Author:

Re: Review Request 15647: Fixing coverity issues related to resource leak on FileInputStream being created anonymously.

2013-11-25 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15647/#review29424 --- awsapi/src/com/cloud/bridge/service/EC2RestServlet.java

Re: Review Request 15833: CLOUDSTACK-5257: Fixed Network ACL issue related to Egress traffic

2013-11-25 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15833/#review29425 --- Commit b2dc2db269c405b6c65b8d00b74e4c84c7abeff2 in branch