Re: Review Request 18677: CLOUDSTACK-3272

2014-05-05 Thread Sonal Ojha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18677/ --- (Updated May 6, 2014, 6:53 a.m.) Review request for cloudstack, Chiradeep Vitta

Re: 44 API breakage in UpdateUser

2014-05-05 Thread Antonio Fornié Casarrubios
Hi Ove, Sorry but I think the only thing I did here was to put in a constant a hardcoded value, but you should find who put the hardcoded value in the first place. Cheers Antonio 2014-05-06 0:44 GMT+02:00 Ove Ewerlid : > On 05/04/2014 10:19 PM, Ove Ewerlid wrote: > >> See https://issues.apach

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-05 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19270/#review42254 --- plugins/network-elements/juniper-contrail/src/org/apache/cloudstack

Re: Review Request 19270: Added service-instance delete command to Juniper Contrail Plugin and related fixes

2014-05-05 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19270/#review42253 --- plugins/network-elements/juniper-contrail/src/org/apache/cloudstack

[GitHub] cloudstack-docs-install pull request: add Managing Networks

2014-05-05 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-docs-install/pull/10#issuecomment-42269526 Thanks for the patch, it looks good. However I did not include the managing_networks section in the toc because I wanted to keep a clean installat

Re: Review Request 18677: CLOUDSTACK-3272

2014-05-05 Thread Murali Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18677/#review42248 --- Patch applies, but does not build on master. please resolve check-st

[Event] BigDataCloud Conference India 2014

2014-05-05 Thread Radhika Puthiyetath
Greetings! More than 400+ registrations for today's ‪#‎BigDataCloud mini conf happening at ‪#‎InMobi Bangalore. All th

Re: [API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-05 Thread Rohit Yadav
Hi Ilya, This section has an example of using cloudmonkey in a bash script to setup a whole zone, network etc: https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+cloudmonkey+CLI#CloudStackcloudmonkeyCLI-Usingasacommandlinetool You may use marvin and write python scripts to automate

[GitHub] cloudstack-docs-install pull request: add Managing Networks

2014-05-05 Thread darrentangdt
GitHub user darrentangdt opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/10 add Managing Networks add managing_networks to doctree. You can merge this pull request into a Git repository by running: $ git pull https://github.com/darrentangdt/cloudst

Re: How is this working?

2014-05-05 Thread Koushik Das
The dashboard in CS UI is impacted by this. On 05-May-2014, at 11:25 PM, Mike Tutkowski wrote: > OK, I can log a JIRA ticket for this. > > Thanks > > > On Mon, May 5, 2014 at 11:49 AM, Nitin Mehta wrote: > >> Ideally, we should deprecate the column since its not used and causes >> confusio

Re: [API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-05 Thread ilya musayev
Noji This is very helpful - https://gist.github.com/ynojima/c30e1d5d08e500e49e64 I've done lots of wrappers on cloudmonkey - this looks very similar to what i've done, i was missing the initial config portion of the zone, now i'm closer to the full stack. Thanks for sharing, Regards ilya O

Re: [API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-05 Thread Yoshikazu Nojima
This is a CloudMonkey script to deploy a VXLAN enabled zone with KVM hypervisor. https://gist.github.com/c30e1d5d08e500e49e64.git This is a marvin config to deploy a VXLAN enabled zone with KVM hypervisor. https://gist.github.com/ynojima/e104189410649f3a9c91 Since this scripts use VXLAN and KVM,

Re: [API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-05 Thread ilya musayev
Hi Alena, Thanks for the response -it is for Advanced Shared Zone (no vpc), i figured i will probably have to do this on my own. If you can share the api calls in the meantime, it would certainly help. I will get the delta's. Or just high level overview of what commands/components need to be

problem with VPC after upgrade to 4.3.0

2014-05-05 Thread Andrei Mikhailovsky
Hello guys, I've upgraded my ACS install from 4.2.1 to 4.3 and have noticed a problem with VPC services. Upon investigation i've noticed that on my VPCs eth1, which is supposed to be the public IP interface is not configured with IPs. all other interfaces have IPs assigned. If I do "ifconfig

Re: [API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-05 Thread Alena Prokharchyk
Ilya, is it a basic zone or advance you are interested in? I have the stack of API calls for Advance zone, but it might miss some information that you might need - configuring traffic labels for example, number of physical networks, number of traffic types supported by each physical network, etc.

[API][CLOUDMONKEY] - looking for example on creating entire zone with everything else

2014-05-05 Thread ilya musayev
For the CloudStack Automation testing project, i'm looking for entire flow of API commands or perhaps CloudMokey CLI equivalent commands to mimic what Create a Zone Wizard does. VMware example would be ideal, but i'm certain i can figure out difference if its not. I know i can probably get the

Re: [RESULT] [VOTE] Apache CloudStack CloudMonkey 5.1.0

2014-05-05 Thread Chip Childers
Disambiguation FTW! > On May 5, 2014, at 7:10 PM, Chiradeep Vittal > wrote: > > Hah! Thanks > > From: Chip Childers mailto:chip.child...@gmail.com>> > Reply-To: "dev@cloudstack.apache.org" > mailto:dev@cloudstack.apache.org>> > Date: Monday, May 5, 2014 at 4:

Re: [RESULT] [VOTE] Apache CloudStack CloudMonkey 5.1.0

2014-05-05 Thread Chiradeep Vittal
Hah! Thanks From: Chip Childers mailto:chip.child...@gmail.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Monday, May 5, 2014 at 4:08 PM To: "dev@cloudstack.apache.org" mailto:dev@cloudstack

Re: [RESULT] [VOTE] Apache CloudStack CloudMonkey 5.1.0

2014-05-05 Thread Chip Childers
Use git push origin tags/5.1.0 > On May 5, 2014, at 5:50 PM, Chiradeep Vittal > wrote: > > Looking for some help here. Both the branch and the tag are named 5.1.0 so > git naturally barfs on this: > [~/source/cloudstack-cloudmonkey ]chiradeep$ git push origin 5.1.0 > error: src refspec 5.1.0 ma

Re: 44 API breakage in UpdateUser

2014-05-05 Thread Ove Ewerlid
On 05/04/2014 10:19 PM, Ove Ewerlid wrote: See https://issues.apache.org/jira/browse/CLOUDSTACK-6570 Suggested patch in comment. /Ove Looping in the person that 'git blame' thinks changed the value of API_KEY in ApiConstants the last time. Antonio, was USER_API_KEY introduced for cases wher

Re: [RESULT] [VOTE] Apache CloudStack CloudMonkey 5.1.0

2014-05-05 Thread Chiradeep Vittal
Looking for some help here. Both the branch and the tag are named 5.1.0 so git naturally barfs on this: [~/source/cloudstack-cloudmonkey ]chiradeep$ git push origin 5.1.0 error: src refspec 5.1.0 matches more than one. error: failed to push some refs to 'https://git-wip-us.apache.org/repos/asf/clou

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Sam Schmit
Olivier, I've submitted the following review for 4.3: https://reviews.apache.org/r/21088/ It does, in fact, work to add the findByIdIncludingRemoved to the EntityManager. Thus, this should merge into 4.3 with no problems. Thanks, Sam On Mon, May 5, 2014 at 10:23 AM, Sam Schmit wrote: > Oli

Review Request 21088: CLOUDSTACK-6472 (4.3-specific) listUsageRecords: Pull information from removed items as well, fixing NPEs/Null UUIDs with usage API calls.

2014-05-05 Thread Sam Schmit
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21088/ --- Review request for cloudstack. Bugs: CLOUDSTACK-6472 https://issues.apache.

[RESULT] [VOTE] Apache CloudStack CloudMonkey 5.1.0

2014-05-05 Thread Chiradeep Vittal
Hi all, After (more than) 72 hours, the vote for CloudStack CloudMonkey 5.1.0 [1] *passes* with 5 PMC + 3 non-PMC votes. +1 (PMC / binding) Chiradeep Vittal Ilya Musayev Marcus Sorensen Sebastien Goasguen David Nalley +1 (non binding) Rajani Karuturi Nux! Rohit Yadav 0 none -1 none Thanks to

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alena Prokharchyk
> On May 5, 2014, 6:27 p.m., Alena Prokharchyk wrote: > > Alex, generic logic looks good to me. But still some things need to be > > fixed. > > > > 1) RegionVO > > > > @Column(name = "active") > > private boolean active; > > > > Explicitly set it to be active by default. Just setting it i

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alena Prokharchyk
> On May 5, 2014, 6:27 p.m., Alena Prokharchyk wrote: > > Alex, generic logic looks good to me. But still some things need to be > > fixed. > > > > 1) RegionVO > > > > @Column(name = "active") > > private boolean active; > > > > Explicitly set it to be active by default. Just setting it i

Apache Cloudstack Error!!!

2014-05-05 Thread Prakash Rao Banuka
Hi: When I issue url http://ip:8080/client in the browser, it says HTTP Status 404 - type Status report message description The requested resource () is not available. Apache Tomcat/6.0.24 Any Ideas?? Thank you Prakash

Jenkins build is back to normal : cloudstack-4.4-forward-build #39

2014-05-05 Thread jenkins
See

Jenkins build is back to normal : cloudstack-4.4-maven-build #255

2014-05-05 Thread jenkins
See

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alex Ough
> On May 5, 2014, 6:27 p.m., Alena Prokharchyk wrote: > > Alex, generic logic looks good to me. But still some things need to be > > fixed. > > > > 1) RegionVO > > > > @Column(name = "active") > > private boolean active; > > > > Explicitly set it to be active by default. Just setting it i

RE: [ACS4.4] Cherry pick for CLOUDSTACK-6483

2014-05-05 Thread Brian Federle
Daan just cp'd all my UI fixes now, so yours is ready to go in. -Brian From: Jessica Wang Sent: Monday, May 05, 2014 1:54 PM To: dev@cloudstack.apache.org Cc: Brian Federle Subject: [ACS4.4] Cherry pick for CLOUDSTACK-6483 Hugo/Daan, Could you please cherry-pick the following commit from 4.4

[ACS4.4] Cherry pick for CLOUDSTACK-6483

2014-05-05 Thread Jessica Wang
Hugo/Daan, Could you please cherry-pick the following commit from 4.4-forward to 4.4 branch? Please cherry-pick Brian's 3 commits in the email below, then cherry-pick mine: Commit hash:d5079e9cfa54a004a91295b1b2f24d2308e3000b CLOUDSTACK- 6483: [UI-Loading balancing]Multiple UI issues

Re: git commit: updated refs/heads/4.4-forward to 259ce3b

2014-05-05 Thread Daan Hoogland
On Mon, May 5, 2014 at 10:44 PM, Brian Federle wrote: > Please cp 38c99ffdab61ecbadf156f266ba4535958dcb67f first, then > 44a3f586d87fa7b4793cc753509024847270444f both in -- Daan

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alena Prokharchyk
> On May 5, 2014, 6:27 p.m., Alena Prokharchyk wrote: > > Alex, generic logic looks good to me. But still some things need to be > > fixed. > > > > 1) RegionVO > > > > @Column(name = "active") > > private boolean active; > > > > Explicitly set it to be active by default. Just setting it i

RE: git commit: updated refs/heads/4.4-forward to 259ce3b

2014-05-05 Thread Brian Federle
Daan, oops, I think the problem is I forgot to include a commit in the original cherry-pick request. Please cp 38c99ffdab61ecbadf156f266ba4535958dcb67f first, then 44a3f586d87fa7b4793cc753509024847270444f Thanks, Brian -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail

Re: 4.4 release-notes

2014-05-05 Thread Pierre-Luc Dion
Request Created under ASF infrastructure team; INFRA project. Thanks David, 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 www.cloudops.com @CloudOps_ On Mon, May 5, 2014 at 4:17 PM, D

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alex Ough
> On May 5, 2014, 6:27 p.m., Alena Prokharchyk wrote: > > Alex, generic logic looks good to me. But still some things need to be > > fixed. > > > > 1) RegionVO > > > > @Column(name = "active") > > private boolean active; > > > > Explicitly set it to be active by default. Just setting it i

Re: git commit: updated refs/heads/4.4-forward to 259ce3b

2014-05-05 Thread Daan Hoogland
On Mon, May 5, 2014 at 9:36 PM, Brian Federle wrote: > 4d84970ddc91c2269807708e722942e9bc758185 doesn't apply/cherry-pick -- Daan

Re: karma wanted

2014-05-05 Thread David Nalley
Ask for that level of karma in an infra ticket. --David On Mon, May 5, 2014 at 4:16 PM, Daan Hoogland wrote: > I am looking to publicize filters and a dashboard. Can I or can I get > rights for that? > > -- > Daan

Re: 4.4 release-notes

2014-05-05 Thread David Nalley
You'll need to file a ticket with infra. On Mon, May 5, 2014 at 2:52 PM, Daan Hoogland wrote: > Animesh, > > Can you (give me rights to) add these filters to the release dashboard? > > On Fri, May 2, 2014 at 4:43 AM, Pierre-Luc Dion wrote: >> 4.4.0 Fixed Issues >> project = CLOUDSTACK AND type =

karma wanted

2014-05-05 Thread Daan Hoogland
I am looking to publicize filters and a dashboard. Can I or can I get rights for that? -- Daan

Build failed in Jenkins: cloudstack-4.4-forward-build #38

2014-05-05 Thread jenkins
See Changes: [brian.federle] CLOUDSTACK-6342: Fix LB table listing for Firefox -- [...truncated 1323 lines...] [copy] Copying 3 files to

RE: git commit: updated refs/heads/4.4-forward to 259ce3b

2014-05-05 Thread Brian Federle
It's a minor but easy fix, so it wouldn't hurt :) Also, can you please cherry pick in 4d84970ddc91c2269807708e722942e9bc758185? This is a fix for a Firefox incompatibility for CLOUDSTACK-6342, where Firefox wasn't rendering the changes correctly (I was just going to make a separate request for

Build failed in Jenkins: cloudstack-4.4-maven-build #254

2014-05-05 Thread jenkins
See Changes: [Daan Hoogland] CLOUDSTACK-3932: UI: Fix Truncation issue in secondary storage URL -- [...truncated 1322 lines...] main: [copy] Copying 3 files to

Re: 4.4 release-notes

2014-05-05 Thread Pierre-Luc Dion
Thanks for you help Daan, It would be best if filters could be publically accessible so the RN script would work, if you can do this. I can't, from Jira doc I would not be allow to do that but some person ca since 4.3 filters exists. Pierre-Luc Dion Architecte de Solution Cloud | Cloud Solutions

Re: git commit: updated refs/heads/4.4-forward to 259ce3b

2014-05-05 Thread Daan Hoogland
On Mon, May 5, 2014 at 9:17 PM, wrote: > 259ce3b59c6a8633fc77a7eac88fbefd915107e2 so I guessed you'd want it cherry-picked ;) -- Daan

Re: Review Request 16002: CLOUDSTACK-3932 UI: Truncation issue occurred in secondary storage URL.

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16002/#review42191 --- Commit e28948ccc3f16a6c9c956fc773a26fcfbd58014b in cloudstack's bran

Re: 4.4 release-notes

2014-05-05 Thread Daan Hoogland
Pierre-Luc, I made a copy of https://issues.apache.org/jira/secure/Dashboard.jspa?selectPageId=12323241 and added your filters. I don't know how to share them with you, yet. Maybe I don't need to; it's at https://issues.apache.org/jira/secure/Dashboard.jspa?selectPageId=12323265 see if you can rea

Re: Review Request 16002: CLOUDSTACK-3932 UI: Truncation issue occurred in secondary storage URL.

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16002/#review42190 --- Commit 259ce3b59c6a8633fc77a7eac88fbefd915107e2 in cloudstack's bran

Re: Review Request 16002: CLOUDSTACK-3932 UI: Truncation issue occurred in secondary storage URL.

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16002/#review42189 --- Commit 41684505d6626bc77d422017b5d7956a90a047e6 in cloudstack's bran

Re: Review Request 16002: CLOUDSTACK-3932 UI: Truncation issue occurred in secondary storage URL.

2014-05-05 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16002/#review42188 --- Ship it! Ship It! - Brian Federle On Dec. 6, 2013, 5:08 p.m., Da

RE: old review https://reviews.apache.org/r/16002

2014-05-05 Thread Brian Federle
Yes, it looks like it indeed is still an issue in some cases. I'll take a look at it. Thanks, Brian -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Monday, May 05, 2014 11:19 AM To: Brian Federle; Jessica Wang Cc: dev Subject: old review https://reviews.apa

Re: Review Request 21087: Remove hard-coding for platform emulators in Xen Server

2014-05-05 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21087/ --- (Updated May 5, 2014, 7:02 p.m.) Review request for cloudstack and Nitin Mehta.

Review Request 21087: Remove hard-coding for platform emulators in Xen Server

2014-05-05 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21087/ --- Review request for cloudstack and Nitin Mehta. Bugs: https://issues.apache.org/

Re: 4.4 release-notes

2014-05-05 Thread Daan Hoogland
Animesh, Can you (give me rights to) add these filters to the release dashboard? On Fri, May 2, 2014 at 4:43 AM, Pierre-Luc Dion wrote: > 4.4.0 Fixed Issues > project = CLOUDSTACK AND type = Bug AND affectedVersion in (4.2.0, 4.2.1, > 4.3.0) AND fixVersion = 4.4.0 AND resolution != "\"Unresolved

Re: Review Request 17495: CLOUDSTACK-5976: Typo in "ssh_keypairs" table's foreign key constraints on the Upgraded Setup

2014-05-05 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17495/#review42183 --- Ship it! db154a0d7937d1c01c561e510bac8c7ed74e7130 on master a521c02

Re: Review Request 17495: CLOUDSTACK-5976: Typo in "ssh_keypairs" table's foreign key constraints on the Upgraded Setup

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17495/#review42181 --- Commit 5ccde5de93db2a9b7768771912d2111cedbb1ed4 in cloudstack's bran

Re: Review Request 17495: CLOUDSTACK-5976: Typo in "ssh_keypairs" table's foreign key constraints on the Upgraded Setup

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17495/#review42179 --- Commit a521c02c54dc670b48d9e5d81fb51e688f6f36a8 in cloudstack's bran

Re: Review Request 17495: CLOUDSTACK-5976: Typo in "ssh_keypairs" table's foreign key constraints on the Upgraded Setup

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17495/#review42178 --- Commit db154a0d7937d1c01c561e510bac8c7ed74e7130 in cloudstack's bran

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-05 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review42177 --- Alex, Move all new API commands/responses you've added (ListSyncAc

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/#review42175 --- Alex, generic logic looks good to me. But still some things need to

old review https://reviews.apache.org/r/16002

2014-05-05 Thread Daan Hoogland
Brian, Jessica, Can you please judge and apply or discard this review request? it is quite old. -- Daan

Re: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

2014-05-05 Thread Nux Ro
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20921/#review42174 --- Ship it! Ship It! - Nux Ro On April 30, 2014, 10:24 p.m., David

Re: console view not coming

2014-05-05 Thread Amogh Vasekar
http://goo.gl/IsnRFw Thanks, Amogh On 5/5/14 10:28 AM, "raja sekhar" wrote: >hii all, > >After upgrading from cloudstack 4.2 to cloudstack 4.3,when i click on >console view it shows that google crome(The server refused the connection) >and firefox(can't establish a connection to the server).

Re: How is this working?

2014-05-05 Thread Mike Tutkowski
OK, I can log a JIRA ticket for this. Thanks On Mon, May 5, 2014 at 11:49 AM, Nitin Mehta wrote: > Ideally, we should deprecate the column since its not used and causes > confusion. > Setting to this value wouldn't help because this column is never updated > regularly. > > On 05/05/14 10:28 AM

Re: How is this working?

2014-05-05 Thread Nitin Mehta
Ideally, we should deprecate the column since its not used and causes confusion. Setting to this value wouldn't help because this column is never updated regularly. On 05/05/14 10:28 AM, "Mike Tutkowski" wrote: >How's about I just check this code into master? > >poolVO.setUsedBytes(mspAnswer.ge

Heart Bleed - Apache Cloud Stack - open ssl

2014-05-05 Thread Prakash Rao Banuka
Hi: It is learnt that openssl library is being called when console of system vms (secondarystorage , console proxy vm and router vm) is launched. Is it mandatory to update openssl library in Apache Cloud Stack? If yes, is there any official document. Thank you Prakash

console view not coming

2014-05-05 Thread raja sekhar
hii all, After upgrading from cloudstack 4.2 to cloudstack 4.3,when i click on console view it shows that google crome(The server refused the connection) and firefox(can't establish a connection to the server). regards, rajasekhar

Re: How is this working?

2014-05-05 Thread Mike Tutkowski
How's about I just check this code into master? poolVO.setUsedBytes(mspAnswer.getPoolInfo().getCapacityBytes() - mspAnswer.getPoolInfo().getAvailableBytes()); It is patterned off of the PrimaryDataStoreHelper.attachHost logic, which looks like this: pool.setUsedBytes(existingInfo.getCapacityByte

Re: Review Request 20007: CLOUDSTACK-2697 - Populated the clusterId with actual value instead of Null in the alert message logs

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20007/#review42165 --- Commit 0babb92d62c08afc04aa0df1bbf053539a53c523 in cloudstack's bran

Re: Review Request 20007: CLOUDSTACK-2697 - Populated the clusterId with actual value instead of Null in the alert message logs

2014-05-05 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20007/#review42166 --- Ship it! 0babb92d62c08afc04aa0df1bbf053539a53c523 - daan Hoogland

Re: Review Request 19779: Fixed[CLOUDSTACK-5150]:-Creating template from a VM in Simulator results in incorrect size

2014-05-05 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19779/#review42162 --- plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorStora

RE: How is this working?

2014-05-05 Thread Nitin Mehta
This column is not used for calculating capacity for pool. We have always used op host capacity table. Nevertheless please do file a bug Thanks, -Nitin From: Mike Tutkowski [mike.tutkow...@solidfire.com] Sent: Monday, May 05, 2014 9:12 PM To: dev@cloudstack

Re: Review Request 21070: Fixed few exception cases.

2014-05-05 Thread sanjeev n
> On May 5, 2014, 4:06 p.m., sanjeev n wrote: > > Ship It! Applied patch to 4.4-automation branch commit ef286da8ccc005fee58c4be5d508c5d7ddfc2a6c Author: sanjeevneelarapu Date: Tue May 6 02:57:08 2014 +0530 Fixed few exception cases - sanjeev -

Re: Review Request 21070: Fixed few exception cases.

2014-05-05 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21070/#review42157 --- Ship it! Ship It! - sanjeev n On May 5, 2014, 6:49 a.m., Santhos

Re: [QUESTION] Renaming files in Git

2014-05-05 Thread Mike Tutkowski
OK, thanks, Daan. I can try it out in master at some point. On Mon, May 5, 2014 at 2:26 AM, Daan Hoogland wrote: > use > $ git mv > and you should be allright. run git mv --help for options. One gotcha. > Any ref to the file you should do in the same commit but is not done > by this action. > >

Re: How is this working?

2014-05-05 Thread Mike Tutkowski
My storage plug-in actually uses a custom host listener, so I have not encountered this issue. I don't remember off hand if it was in 4.2 or 4.3, but at some point someone changed the storage_pool table's available_bytes column to be used_bytes. It looks like this code you reference was missed.

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Sam Schmit
Olivier, I'm still checking into it, but it may work - as you said, it should be a supported method. I've already made the change on my dev machine, and I'm getting set up with a development cloud to test it out. I should know later today if it will work for 4.3. Sam On Mon, May 5, 2014 at 10

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Olivier Lemasle
Hi Sam, I was going to suggest the same patch than you (with findByIdIncludingRemoved), and I hit the same issue: no method findByIdIncludingRemoved in com.cloud.utils.db.EntityManager. But I see that the implementing class com.cloud.dao.EntityManagerImpl still has findByIdIncludingRemoved(). Do y

Re: HyperV agent not able to connect with Jenkins and community builds blocked

2014-05-05 Thread Daan Hoogland
Rayees, I consulted with Hugo, but we both do not know what jenkins slave this is and where it should run. Did you find out more by now? regards, Daan On Thu, Apr 24, 2014 at 3:41 PM, Rayees Namathponnan wrote: > Hugo / Daan, > > > > Can you please help on this ? in case you have any idea ? > >

Re: Review Request 19902: CLOUDSTACK-6223. removeNicFromVirtualMachine fails if another instance in another domain has a nic with the same ip and a forwarding rule configured on it

2014-05-05 Thread Daan Hoogland
Alena, I see nothing that keeps us from aplying this patch, do you? regards, On Wed, Apr 23, 2014 at 4:41 PM, upendra moturi wrote: >This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19902/ > > On April 21st, 2014, 6:17 p.m. UTC, *daan Hoogland* wro

Re: Cleanly remove management server

2014-05-05 Thread Daan Hoogland
H Michael, Did you investigate this further? still planning to implement? On Fri, Apr 18, 2014 at 3:33 PM, Michael Phillips wrote: > Last week we talked about a GUI option to cleanly remove a management server. > I created a feature request for this as well. Hopefully this will get some > tra

Re: Review Request 18677: CLOUDSTACK-3272

2014-05-05 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18677/#review42154 --- File Attachment: Works on commit 6c44c0661dae0e9de4ed53fad8951e72db

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Sam Schmit
Sebastien, I took a look at the 4.3 code, and I was mistaken - it will not commit cleanly. The EntityManager does not have the findIdIncludingRemoved function added yet. I'll have to spend more time looking into how to get this into 4.3 cleanly. Sam On Fri, May 2, 2014 at 3:57 PM, Sebastien

Re: Review Request 16876: db sync fix : make sure that db sync task gets executed once: master branch fix

2014-05-05 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16876/#review42152 --- Suresh, can I delete this request? It doesn't contain a diff. - daa

Re: Review Request 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

2014-05-05 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20357/#review42151 --- 601827e6b34cb7debe67a8415a09440c389ba4a1 on master 2c4ae0883be04840a

Re: Review Request 21070: Fixed few exception cases.

2014-05-05 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21070/#review42142 --- Please rebase with master since patch apply failed on master branch.

Re: Review Request 20891: CLOUDSTACK-6536: Code cleanup - removing unnecessary waits after VM stop operation

2014-05-05 Thread sanjeev n
> On May 5, 2014, 10:20 a.m., Santhosh Edukulla wrote: > > Ship It! I tried to apply the patch on master, however it failed. Please rebase it with master. Following are the erros: [root@cent62-Sanjeev acs-repo]# git apply /root/0002-CLOUDSTACK-6536-Code-cleanup-removing-unnecessary-wa.patch e

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-5077

2014-05-05 Thread Daan Hoogland
thanks Bharat, It is in. I run git show on every commit that I then git cherry-pick so you don't need to send me a link. the full id will do. regards, Daan On Mon, May 5, 2014 at 12:27 PM, Bharat Kumar wrote: > Hi Daan, > > updating the link to diff. > https://git-wip-us.apache.org/repos/asf?p=

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-6463 and CLOUDSTACK-6466

2014-05-05 Thread Daan Hoogland
both in On Mon, May 5, 2014 at 11:56 AM, Harikrishna Patnala wrote: > Hi Daan, > > Could you cherry pick the following commits to 4.4 branch > > 1) Commit ce3074e9a23e71855798cb5decd01d7f9958f6c0 in cloudstack's branch > refs/heads/4.4-forward > [ https://git-wip-us.apache.org/repos/asf?p=clouds

Re: Review Request 20519: CLOUDSTACK-6466: cpu and ram is not getting updated correctly in usage_vm_instance table for usage type 2

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20519/#review42140 --- Commit 0c60ea2e01a80d5201b23415ebef818984db8730 in cloudstack's bran

Re: Review Request 20516: CLOUDSTACK-6463: password is not set for VMs created from password enabled template

2014-05-05 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20516/#review42139 --- Commit df0564f14d35c02d5e745ffdea3b3e1e6d69698b in cloudstack's bran

Re: unit test failures in files unchanged for more then 3 months??

2014-05-05 Thread Daan Hoogland
I recalled the ln command from long ago after installing 1.7 I did a $ sudo ln -sf /Library/Java/JavaVirtualMachines/jdk1.7.0_55.jdk/Contents CurrentJDK (forgot the -n option but it works) Somehow oracle overwrites the links when installing 8 but not when installing 7 over it. On Mon, May 5, 2014

RE: cherry-pick requests

2014-05-05 Thread Rajesh Battala
those are .cs files. As I was working with them on windows it could have got introduced. I will correct them and request for a new cherry-pick. Thanks Rajesh Battala -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Monday, May 5, 2014 2:52 PM To: dev Subject

Re: unit test failures in files unchanged for more then 3 months??

2014-05-05 Thread Daan Hoogland
please note, installing java 8 on your system causes these failures. Even if java 7 is still installed and used in your builds! Somehow a java 8 version of java.util.concurrrent is used/called. I am now re-installing java 7 to see if this solves the issue On Mon, May 5, 2014 at 1:00 PM, Daan Hoo

[GitHub] cloudstack-docs-install pull request: doc for enabling CSP support...

2014-05-05 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/9 doc for enabling CSP support on xenserver 6.1 https://issues.apache.org/jira/browse/CLOUDSTACK-6574 You can merge this pull request into a Git repository by running: $ git pull

Re: unit test failures in files unchanged for more then 3 months??

2014-05-05 Thread Daan Hoogland
my present 4.4 does not build either! :( On Mon, May 5, 2014 at 12:49 PM, Daan Hoogland wrote: > Did anybody else see this? > > Running com.cloud.utils.backoff.impl.ConstantTimeBackoffTest > 2014-05-05 12:38:52,966 DEBUG [backoff.impl.ConstantTimeBackoffTest] > (main:) thread started > 2014-05-05

Re: Review Request 19399: CLOUDSTACK-6257: Resolving issue in template creation from user VM volume, added method to check the state of VM before template creation

2014-05-05 Thread sanjeev n
> On May 5, 2014, 5:53 a.m., ASF Subversion and Git Services wrote: > > Commit d81c5c44f112df8c0e4388e9d88300b24eefe3d0 in cloudstack's branch > > refs/heads/4.4-forward from Girish Shilamkar > > [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d81c5c4 ] > > > > Revert "CLOUDSTACK-6

Re: Re: Upgrade from 4.1.1 + XS 6.0.2 to 4.3 + XS 6.2

2014-05-05 Thread Daan Hoogland
It would seem that if you first upgrade cloudstack to 4.3 and then upgrade your hosts this should be possible. You might want to remove your hosts, upgrade and then add again, one by one, though. to be on the save side. On Mon, May 5, 2014 at 12:24 PM, France wrote: > Is upgrade from 4.1.1 + XS 6

Re: Review Request 19399: CLOUDSTACK-6257: Resolving issue in template creation from user VM volume, added method to check the state of VM before template creation

2014-05-05 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19399/#review42136 --- Tried to apply it on master but it failed. Please rebase with master

  1   2   >