[GitHub] cloudstack-docs-rn pull request: Add missing new features for 4.4

2014-06-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-rn/pull/15 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

Re: [DISCUSS] Increasing VM IOPS by separating golden image in high IOPS partition in Xen Server ?

2014-06-06 Thread Hieu LE
Hi Mike, Done that, I have added new FAQ section. In addition, I have tested that volume can take a snapshot, create volume from that snapshot and attach back to VM normally. Currently I am testing with volume migration. On Fri, Jun 6, 2014 at 11:11 AM, Mike Tutkowski

Re: SNAT iptable entry on VirtualRouter

2014-06-06 Thread Sachchidanand Vaidya
Hi, I'm seeing the same issue with 4.4 code. After further debug, I see that CS mgmt server is sending following command to XenHost, xensource.log: script /opt/cloud/bin/ipassoc.sh -A -s -f -l 10.84.59.131/24 -c eth0 -g 10.84.59.254 /script VirtualRouter's public interface is eth2. Also as per

Re: Realhostip service extended till Sep 30th

2014-06-06 Thread Andrija Panic
That's: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Procedure+to+Replace+realhostip.com+with+Your+Own+Domain+Name thanks On 6 June 2014 04:03, Nitin Mehta nitin.me...@citrix.com wrote: Please find wiki [1] containing the procedure to replace realhostip.com with Your own custom

Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/ --- Review request for cloudstack, daan Hoogland and Kishan Kavala. Bugs:

Re: Review Request 20316: CLOUDSTACK-1466: Adding automation test cases for Primary Storage Limits

2014-06-06 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20316/ --- (Updated June 6, 2014, 3:10 p.m.) Review request for cloudstack, Girish

Re: [ACS44] blocker issues?

2014-06-06 Thread Daan Hoogland
Sudha, - The point is that there are bugs being marked as blockers that where never discussed on this list. Someone has decided all by their own they are blocker and didn't ask for feedback. I don't find that acceptable. - My colleague Wilder has done some validation. I didn't memorize his report

Re: [ACS44] blocker issues?

2014-06-06 Thread Daan Hoogland
in so much words, yes please review them. All of you! On Thu, Jun 5, 2014 at 9:44 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: What I think we should do is ask people to review the list of blockers and provide comments (if you feel knowledgeable enough in that area) as to whether or

Re: [ACS4.3.1][DISCUSS] Minor release of Apache CloudStack 4.3.1

2014-06-06 Thread Daan Hoogland
Ilya, You are addressing non developpers, so I am crossposting. On Fri, Jun 6, 2014 at 4:25 AM, ilya musayev ilya.mailing.li...@gmail.com wrote: Creating a separate thread to see what community thinks on next minor release of CloudStack 4.3.1 There are more patches that need to be

Re: [ACS44] Cherry pick for CLOUDSTACK-6676 and CLOUDSTACK-6853

2014-06-06 Thread Daan Hoogland
three pulled, i noticed windows line endings and trailing space in server/test/resources/createNetworkOffering.xml please take care. On Fri, Jun 6, 2014 at 4:57 AM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: And one more – fixing the unit test (sorry for that) commit

ssh keypair value overridden by template meta data

2014-06-06 Thread Hiroki Ohashi
Dear guys I encountered a problem that a ssh public key of ssh_keypairs for a newly created instance is overridden by another ssh key in template meta data. I think this leads to security vulnerability because a template owner can login to other user's instance created from the template. So,

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/#review44885 --- Ship it! Ship It! - Kishan Kavala On June 6, 2014, 2:58 p.m.,

Review Request 17994: CLOUDSTACK-6856 :Fixed Few Data Corruption and retries issues.

2014-06-06 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17994/ --- Review request for cloudstack, daan Hoogland and SrikanteswaraRao Talluri.

Fwd: New Defects reported by Coverity Scan for cloudstack

2014-06-06 Thread Hugo Trippaers
FYI Begin forwarded message: From: scan-ad...@coverity.com Subject: New Defects reported by Coverity Scan for cloudstack Date: 6 juni 2014 13:40:40 CEST Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. Defect(s) Reported-by:

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/#review44886 --- patch looks alright but it seems to me you have reused a review

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Santhosh Edukulla
On June 6, 2014, 12:09 p.m., daan Hoogland wrote: patch looks alright but it seems to me you have reused a review board entry for a second patch. please don't do that. close this one and start a new one next time. Sure, but reviewboard some times in process throws 5xx error and a given

Re: [DISCUSS] Increasing VM IOPS by separating golden image in high IOPS partition in Xen Server ?

2014-06-06 Thread Todd Pigram
Sorry, thought you were based off the link you provided in this reply. In our case, we are using CloudStack integrated in VDI solution to provived pooled VM type[1]. So may be my approach can bring better UX for user with lower bootime ... A short change in design are followings - VM will be

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread daan Hoogland
On June 6, 2014, 12:09 p.m., daan Hoogland wrote: patch looks alright but it seems to me you have reused a review board entry for a second patch. please don't do that. close this one and start a new one next time. Santhosh Edukulla wrote: Sure, but reviewboard some times in

Anybody addressing this bug ? Overlaping IP subnets across different vlans

2014-06-06 Thread Andrija Panic
Hi, aftger upgrade to 4.3, I reported a bug where CS will not let me add additional IP ranges when there are 2 vlans using same IP range - I don't see point comparing IP ranges across two separate broadcast domains... https://issues.apache.org/jira/browse/CLOUDSTACK-6814 Thanks, -- Andrija

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/#review44889 --- Commit 91d054cb00f6a5818f2be1c7e4d3f44a2c58bfcc in cloudstack's

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/#review44890 --- Commit 6c0b43431f7f117c1fcb049c483407b96b56d961 in cloudstack's

coverity scan: resource leaks in upgrade path

2014-06-06 Thread Rajani Karuturi
Hi All, Of the 1697 issues reported by coverity in the latest scan, about 187 are in upgrade###to###.java and many of them are due to RESOURCE_LEAKS because of not closing the pstmt properly. For ex: Resource leak on an exceptional path (RESOURCE_LEAK) 299 } 300

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Daan Hoogland
Santhosh, Have you made sure this goes into master as well? On Fri, Jun 6, 2014 at 3:53 PM, ASF Subversion and Git Services asf...@urd.zones.apache.org wrote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/ Commit

RE: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Santhosh Edukulla
Hi Daan, Yes, for master planning additional approach with few more enhancements, useful for future release. It may require modifying schema, which i believe is not possible for 4.4 because of code freeze. Regards Santhosh From: Daan Hoogland

RE: New Defects reported by Coverity Scan for cloudstack

2014-06-06 Thread Sudha Ponnaganti
That is better. Hope new ones can be fixed each week while the old ones can be prioritized based on high density. Original message From: Rajani Karuturi Date:06/06/2014 6:37 AM (GMT-08:00) To: dev@cloudstack.apache.org Subject: Re: New Defects reported by Coverity Scan for

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Daan Hoogland
code freeze does not mean that the schema can't be changed for bug fixes. Any bug fix can go in if it genuinely serves the purpose of fixing a bug. On Fri, Jun 6, 2014 at 4:02 PM, Santhosh Edukulla santhosh.eduku...@citrix.com wrote: Hi Daan, Yes, for master planning additional approach with

Re: coverity scan: resource leaks in upgrade path

2014-06-06 Thread Daan Hoogland
great Rajani, I am having a look at Upgrade43to44 and will backport the fix to all others that it applies to: try(PreparedStatement selectstatement = conn.prepareStatement(SELECT * FROM 1+1); ResultSet results = selectstatement.executeQuery()) { }without finally On Fri, Jun

Review Request 22305: Fixed Coverity Issue : 1220837

2014-06-06 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22305/ --- Review request for cloudstack, daan Hoogland and Hugo Trippaers. Repository:

Re: KVM + LXC on the same host

2014-06-06 Thread Nguyen Anh Tu
That should be a good idea, Ilya. At that moment, i'm working on Docker support. When it's done, we can run only Docker hosts, no need to use KVM for hosting system vms. Cheers, --Tuna Sent from my GT-N7000 On Jun 5, 2014 7:58 AM, ilya musayev ilya.mailing.li...@gmail.com wrote: We are

Build failed in Jenkins: build-systemvm64-4.2 #302

2014-06-06 Thread jenkins
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/302/ -- Started by timer [EnvInject] - Loading node environment variables. Building remotely on devcloud-continuous-tests (devcloud) in workspace

Re: Review Request 22305: Fixed Coverity Issue : 1220837

2014-06-06 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22305/#review44897 --- I was thinking of private void updateVlanUris(Connection conn)

Re: Review Request 22305: Fixed Coverity Issue : 1220837

2014-06-06 Thread Santhosh Edukulla
On June 6, 2014, 2:54 p.m., daan Hoogland wrote: I was thinking of private void updateVlanUris(Connection conn) { s_logger.debug(updating vlan URIs); try(PreparedStatement selectstatement = conn.prepareStatement(SELECT id, vlan_id FROM `cloud`.`vlan` where

Re: Review Request 22305: Fixed Coverity Issue : 1220837

2014-06-06 Thread daan Hoogland
On June 6, 2014, 2:54 p.m., daan Hoogland wrote: I was thinking of private void updateVlanUris(Connection conn) { s_logger.debug(updating vlan URIs); try(PreparedStatement selectstatement = conn.prepareStatement(SELECT id, vlan_id FROM `cloud`.`vlan` where

Re: Support pure Xen as a hypervisor follow-up

2014-06-06 Thread Dave Scott
Hi, Here’s a quick status update: On 16 May 2014, at 15:22, Dave Scott dave.sc...@citrix.com wrote: Hi, On 14 May 2014, at 09:53, sebgoa run...@gmail.com wrote: On Apr 9, 2014, at 2:37 PM, Dave Scott dave.sc...@citrix.com wrote: Hi, Following up from Tim's Support pure Xen as a

Re: Support pure Xen as a hypervisor follow-up

2014-06-06 Thread Tim Mackey
Dave, I've submitted a merge review request (https://reviews.apache.org/r/22270/) yesterday. If you want to avoid having to potentially deal with a bunch of conflicts, you might want to see if your patches apply cleanly there and let me know. Happy to help with any conflict resolution. btw, I

Re: KVM + LXC on the same host

2014-06-06 Thread Pierre-Luc Dion
ilya, Let us know how it goes your hybrid of LXC+KVM. I'm interested to know how it's going, I might try that too on the side. Pierre-Luc Dion Architecte de Solution Cloud | Cloud Solutions Architect 855-OK-CLOUD (855-652-5683) x1101 - - - *CloudOps*420 rue Guy Montréal QC H3J 1S6

Re: Support pure Xen as a hypervisor follow-up

2014-06-06 Thread sebgoa
On Jun 6, 2014, at 5:17 PM, Tim Mackey tmac...@gmail.com wrote: Dave, I've submitted a merge review request (https://reviews.apache.org/r/22270/) yesterday. Unfortunately that patch did not seem to apply cleanly. I have not dug deep though.. If you want to avoid having to potentially

Re: Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
Rohit, Any chance you have time to check this one review. My git kungfu is limited and the patch seems to have conflicts. Tim did quite a bit of work to get there, so a merge could be call quickly, if this patch could apply, that would ease another round of rebasing.. thanks,, -sebastien On

Re: Review Request 17994: CLOUDSTACK-6856 :Fixed Few Data Corruption and retries issues.

2014-06-06 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17994/#review44906 --- Commit b43517c49629d46c2dd222bef0d8b1b2944abb66 in cloudstack's

Re: Review Request 17994: CLOUDSTACK-6856 :Fixed Few Data Corruption and retries issues.

2014-06-06 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17994/#review44907 --- Ship it! Ship It! - SrikanteswaraRao Talluri On June 6, 2014,

Re: Support pure Xen as a hypervisor follow-up

2014-06-06 Thread Tim Mackey
If you can give me the commit ID for your HEAD, I'll check and see what's up. It could be another conflict needing resolution. -tim On Fri, Jun 6, 2014 at 12:16 PM, sebgoa run...@gmail.com wrote: On Jun 6, 2014, at 5:17 PM, Tim Mackey tmac...@gmail.com wrote: Dave, I've submitted a merge

[MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
Folks, Tim has prepared a pretty significant patch in the review it lists below. There is also a wiki page describing the change. Basically it splits XenServer and XenProject (pure Xen) in two separate hypervisors. Until now we had used XenServer to also handle XenProject. It will allow to

Re: Anybody addressing this bug ? Overlaping IP subnets across different vlans

2014-06-06 Thread ilya musayev
Andrija I dont know if we can qualify this as a bug, this check was placed with some purpose in mind - yet its not clear what it is and why would someone think its bad. https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=a3b1a49c303a929c754561ca07fd8a9ed84e0382

Re: [ACS4.3.1][DISCUSS] Minor release of Apache CloudStack 4.3.1

2014-06-06 Thread ilya musayev
Thanks Dan. This was meant to be sent to dev list but somehow, i've sent it to users. Folks, please chime in. On 6/6/14, 4:39 AM, Daan Hoogland wrote: Ilya, You are addressing non developpers, so I am crossposting. On Fri, Jun 6, 2014 at 4:25 AM, ilya musayev ilya.mailing.li...@gmail.com

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
On Fri, Jun 6, 2014 at 10:14 PM, sebgoa run...@gmail.com wrote: Folks, Tim has prepared a pretty significant patch in the review it lists below. There is also a wiki page describing the change. Basically it splits XenServer and XenProject (pure Xen) in two separate hypervisors. Until now

Re: Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
Hi Sebastien, On Fri, Jun 6, 2014 at 9:51 PM, sebgoa run...@gmail.com wrote: Rohit, Any chance you have time to check this one review. My git kungfu is limited and the patch seems to have conflicts. Tim did quite a bit of work to get there, so a merge could be call quickly, if this patch

Re: Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
I just read the design document on the wiki https://cwiki.apache.org/confluence/display/CLOUDSTACK/Convert+Xen+usage+to+XenServer and got my answers regarding the work. Thanks. Regards. On Fri, Jun 6, 2014 at 11:48 PM, Rohit Yadav rohityada...@gmail.com wrote: Hi Sebastien, On Fri, Jun 6,

Re: Review Request 19874: CLOUDSTACK-6793 : Fixed an Invalid Domain Id Issue

2014-06-06 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19874/#review44945 --- Ship it! Hi Santosh - We should add the domain id column in the

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
Hi again, I was unable to deploy basic zone, I did: # marvin and python dependencies were pre-installed cd tools/devcloud; python ../marvin/marvin/deployDataCenter.py -i devcloud.cfg The error I got was related to agent communication, if this can help resolve the issue: Exception Occurred

Modifications of HTML/CSS/JS files without recompling

2014-06-06 Thread Ian Duffy
Hi UI Team, Last time I worked on the UI I found myself recompiling and restarting jetty every time I modified a HTML/CSS/JS File. Do we have some method to avoid doing this? I tried doing a symlink of the source directory with the target directory which did work but it was messy. Thanks, Ian

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
On Jun 6, 2014, at 8:41 PM, Rohit Yadav bhais...@apache.org wrote: Hi again, I was unable to deploy basic zone, I did: # marvin and python dependencies were pre-installed cd tools/devcloud; python ../marvin/marvin/deployDataCenter.py -i devcloud.cfg The error I got was related to

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
On Sat, Jun 7, 2014 at 12:51 AM, sebgoa run...@gmail.com wrote: On Jun 6, 2014, at 8:41 PM, Rohit Yadav bhais...@apache.org wrote: Hi again, I was unable to deploy basic zone, I did: # marvin and python dependencies were pre-installed cd tools/devcloud; python

Re: [ACS4.3.1][DISCUSS] Minor release of Apache CloudStack 4.3.1

2014-06-06 Thread Daan Hoogland
No Ilya, I did that on purpose as you where asking non-devs to participate in the release proces. On Fri, Jun 6, 2014 at 8:09 PM, ilya musayev ilya.mailing.li...@gmail.com wrote: Thanks Dan. This was meant to be sent to dev list but somehow, i've sent it to users. Folks, please chime in.

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
On Jun 6, 2014, at 9:35 PM, Rohit Yadav bhais...@apache.org wrote: On Sat, Jun 7, 2014 at 12:51 AM, sebgoa run...@gmail.com wrote: On Jun 6, 2014, at 8:41 PM, Rohit Yadav bhais...@apache.org wrote: Hi again, I was unable to deploy basic zone, I did: # marvin and python

Re: [ACS44] Cherry pick for CLOUDSTACK-6585

2014-06-06 Thread Daan Hoogland
On Fri, Jun 6, 2014 at 8:14 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: e4fc5d88427456b9f9b8d37b49a4406db8c886c1 the object is not found. Did you push it? -- Daan

Re: [ACS44] Cherry pick for CLOUDSTACK-6585

2014-06-06 Thread Daan Hoogland
did you mean 9e2db2f2210904752eea2f3abf87b16bb3080384? On Fri, Jun 6, 2014 at 9:55 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Fri, Jun 6, 2014 at 8:14 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: e4fc5d88427456b9f9b8d37b49a4406db8c886c1 the object is not found. Did you

Re: [ACS44] Cherry pick for CLOUDSTACK-6585

2014-06-06 Thread Alena Prokharchyk
Yes. On 6/6/14, 12:57 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: did you mean 9e2db2f2210904752eea2f3abf87b16bb3080384? On Fri, Jun 6, 2014 at 9:55 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Fri, Jun 6, 2014 at 8:14 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote:

Re: [ACS44] Cherry pick for CLOUDSTACK-6585

2014-06-06 Thread Daan Hoogland
done On Fri, Jun 6, 2014 at 10:04 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Yes. On 6/6/14, 12:57 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: did you mean 9e2db2f2210904752eea2f3abf87b16bb3080384? On Fri, Jun 6, 2014 at 9:55 PM, Daan Hoogland daan.hoogl...@gmail.com

Re: KVM + LXC on the same host

2014-06-06 Thread ilya musayev
Tuna Thanks for the feedback, conceptually, i was not trying to address the issue of sysvms not running on LXC. Here is the use case as i see it. Assume you roll out a farm of 10 KVM servers and your density with KVM is 200 virtual machines. Most of these VMs dont require independent

Re: DevCloud installation

2014-06-06 Thread ilya musayev
I'd boot it into single user mode and reset the password. On 6/5/14, 6:04 PM, Mike Tutkowski wrote: Hey Edison, Seif is using a copy of DevCloud named devcloud2.ova (908,991,488 bytes). It was downloaded back in August of last year. I don't use DevCloud much these days as I build this all

[DISCUSS] try-with-resouce use

2014-06-06 Thread Daan Hoogland
H, I have commited a patch to Upgrade430to440.java that removes all close() calls on PreparedStament en ResultSet objects, replacing it with the try(resouce) construct in java 7. I was tempted for a short moment to back-port it to older Upgrade classes. As those upgrade paths are to be supported

Re: DevCloud installation

2014-06-06 Thread Mike Tutkowski
Just heard from Seif. It was a character-encoding issue with his keyboard. On Friday, June 6, 2014, ilya musayev ilya.mailing.li...@gmail.com wrote: I'd boot it into single user mode and reset the password. On 6/5/14, 6:04 PM, Mike Tutkowski wrote: Hey Edison, Seif is using a copy of

RE: [DISCUSS] try-with-resouce use

2014-06-06 Thread Alex Huang
In general, as long as it works in the new jvm, I don't see us changing the code. However, if you have the time to do it. The upgrade classes are meant to be immutable as far as data goes. So the idea is we can do upgrades from any version to the current version. Because of that, then all

Re: [DISCUSS] try-with-resouce use

2014-06-06 Thread Daan Hoogland
I don't want to fix resource leaks in master in a way we can not backport. That's my point. And no I am not going to do a lot of it but want to set a policy. On Fri, Jun 6, 2014 at 10:46 PM, Alex Huang alex.hu...@citrix.com wrote: In general, as long as it works in the new jvm, I don't see us

Re: SNAT iptable entry on VirtualRouter

2014-06-06 Thread Sheng Yang
Hi Sachin, The nicDevId() you see is coming from prepareNetworkElementCommand(IpAssocCommand cmd) in CitrixResourceBase in case of Xen. You would see this: if (addVif) { // Add a new VIF to DomR String vifDeviceNum =

RE: Need to revert CLOUDSTACK-5853

2014-06-06 Thread Prachi Damle
Hi Marcus, I have reopened this ticket for 4.4 release, fix needed for this is reverting the code added as part of this in earlier versions. Thanks, Prachi -Original Message- From: Prachi Damle [mailto:prachi.da...@citrix.com] Sent: Wednesday, June 04, 2014 1:04 PM To: Marcus Cc:

[ANNOUNCE] Demetrius Tsitrelis as committer

2014-06-06 Thread John Kinsella
Folks - this one’s a little belated - we went through the invite process around the time of the mail issues, and somehow we didn’t send the announcement to dev@. I noticed while doing some housekeeping this week, and wanted to send out the announcement anyways just to give Demetrius the

[DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-06 Thread Sheng Yang
Hi all, Seems it's a good timing to bring back the discussion about the gerrit. We want to do CI, and improve our code quality. One obvious way of doing and reduce the workload of devs is introduce a tool to enforce the process. I've checked out quite a few projects using gerrit, which would

RE: [ANNOUNCE] Demetrius Tsitrelis as committer

2014-06-06 Thread Prachi Damle
Congratulations Demetrius! -Original Message- From: John Kinsella [mailto:j...@stratosec.co] Sent: Friday, June 06, 2014 4:18 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Demetrius Tsitrelis as committer Folks - this one's a little belated - we went through the invite process

RE: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-06 Thread Prachi Damle
+ 1 to gerrit -Original Message- From: Sheng Yang [mailto:sh...@yasker.org] Sent: Friday, June 06, 2014 4:27 PM To: dev@cloudstack.apache.org Subject: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality... Hi all, Seems

Re: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-06 Thread John Kinsella
+1 seems like a good idea. On Jun 6, 2014, at 4:26 PM, Sheng Yang sh...@yasker.orgmailto:sh...@yasker.org wrote: Hi all, Seems it's a good timing to bring back the discussion about the gerrit. We want to do CI, and improve our code quality. One obvious way of doing and reduce the workload of

Re: KVM + LXC on the same host

2014-06-06 Thread John Kinsella
Hey Ilya - So, for about a month now we’ve had a system running SmartOS[1], which gives a combination of containers and KVM, albeit on illumos instead of linux. In general I’m not impressed by SmartOS’s story, but we had one customer asking for it (and I sorta expect more, we’re not officially

Re: Support pure Xen as a hypervisor follow-up

2014-06-06 Thread Amit Das
It would be great if there's some kind of document in the wiki on this proposal. Regards, Amit *CloudByte Inc.* http://www.cloudbyte.com/ On Fri, Jun 6, 2014 at 9:52 PM, Tim Mackey tmac...@gmail.com wrote: If you can give me the commit ID for your HEAD, I'll check and see what's up. It

Re: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-06 Thread Rohit Yadav
Hi, On Sat, Jun 7, 2014 at 4:56 AM, Sheng Yang sh...@yasker.org wrote: Hi all, Seems it's a good timing to bring back the discussion about the gerrit. We want to do CI, and improve our code quality. One obvious way of doing and reduce the workload of devs is introduce a tool to enforce the

RE: [DISCUSS] try-with-resouce use

2014-06-06 Thread Rajani Karuturi
I think in the branches 4.4 and above we should use try-resource-construct for all the upgrade paths. Otherwise we will continue to see resource leaks due to copy-paste-modified code. ~Rajani Sent from my Windows Phone From: Daan