Re: Review Request 21070: Fixed few exception cases.

Hi Santhosh, I would like to mention one concern here. If we are raising exception through "marvinRequest" function in cloudstackConnection.py, instead of returning FAILED, then this will need many test cases to be changed too, where we are explicitly asserting the output of particular marvin comm

Build failed in Jenkins: build-master #748

See Changes: [rajesh.battala] CLOUDSTACK-6528 SetupGuestNetwork command is not deleting the guest network configured on the eth device [rajesh.battala] CLOUDSTACK-6518 [Hyper-V] Efficient way of finding the empty nic in VR/VpcVR to

RE: Build failed in Jenkins: build-master #748

Am working on fixing the unit test. -Original Message- From: jenk...@cloudstack.org [mailto:jenk...@cloudstack.org] Sent: Monday, May 5, 2014 12:56 PM To: dev@cloudstack.apache.org; Rajesh Battala Subject: Build failed in Jenkins: build-master #748 See

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

See Changes: [rajesh.battala] CLOUDSTACK-6528 SetupGuestNetwork command is not deleting the guest network configured on the eth device [rajesh.battala] CLOUDSTACK-6518 [Hyper-V] Efficient way of finding the empty nic

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

See

RE: Review Request 21070: Fixed few exception cases.

Some places we are explicitly verifying for exception, EX: with assert for exception, with assert for not exception from CS, at some places we are verifying for return value say failed. If we return failed, then cases above are not clear. So, made it uniform for raising exception. Even earlier,

RE: Build failed in Jenkins: build-master #748

Fixed the test cases -Original Message- From: Rajesh Battala [mailto:rajesh.batt...@citrix.com] Sent: Monday, May 5, 2014 1:09 PM To: dev@cloudstack.apache.org Subject: RE: Build failed in Jenkins: build-master #748 Am working on fixing the unit test. -Original Message- From: j

Re: [QUESTION] Renaming files in Git

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. On Mon, May 5, 2014 at 7:41 AM, Mike Tutkowski wrote: > Hi, > > I was interested in renaming a couple files to improve the casin

Re: [ACS4.4] Cherry pick for CLOUDSTACK-6571

On Mon, May 5, 2014 at 5:32 AM, Yoshikazu Nojima wrote: > 03156ecd6f52bfa0c73f1408ac4baca5332d1cad done -- Daan

Jenkins build is back to normal : build-master #749

See

cherry-pick requests

Hi Daan, Please cherry-pick these commits to 4.4 branch. 1 6cd1c7ab32a5e4ca3228b25aa6bd6158cb57963c https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=6cd1c7ab32a5e4ca3228b25aa6bd6158cb57963c CLOUDSTACK-6528 SetupGuestNetwork command is not deleting the guest network configure

[Event]BigDataCloud Conference India 2014

Greetings ! Join us at the BigDataCloud 2014 Mini Conference at InMobi. Various top-level projects under Apache Software Foundation (ASF) are coming together for the first time to bring you the first edition of BigDataCloud Mini Conference in Bangalore. Jointly organized by Apache CloudStack, A

Can not find systemvmiso - CS42 with XenServer

I am running CS42 from my development environment. I am managing XenServer 6.2 from my CS mgmt server. When I try to create instances, I am getting this error: ** RuntimeException due to com.cloud.utils.exception.CloudRuntimeException: can n

Re: cherry-pick requests

cf41ccaa5b6475dace0bddbe6681c98cc5149189 is introducing ^M line endings. please make sure your env is clean. I hate to be nitpicking but can you fix/prevent this? I pushed the other two to wip-git. On Mon, May 5, 2014 at 10:44 AM, Rajesh Battala wrote: > Hi Daan, > > Please cherry-pick these com

Re: Cloudstack 4.3 instances can't access outside world

Serg, You marked the bug as to be solved in 4.3.1. There is no plan for a 4.3.1 and a 4.4.0 is planned for the coming weeks. I think some more conversation on this list is needed for this issue. If you have a dev env, please find the location of the bug and contact whoever has been working on thos

Re: Review Request 21009: Added Network API tests to test_escalation.py

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21009/#review42133 --- Ship it! Ship It! - Santhosh Edukulla On May 2, 2014, 8:35 a.m.,

Re: Review Request 20652: CLOUDSTACK-6531: VR alerts for command failures during start and stop the router incase of failures

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20652/ --- (Updated May 5, 2014, 9:47 a.m.) Review request for cloudstack, Abhinandan Prat

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

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=cloudstack.git;h=ce3074e ] CLOUDSTACK-6463: password is not set for VMs created from pas

Re: Review Request 20652: CLOUDSTACK-6531: VR alerts for command failures during start and stop the router incase of failures

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20652/ --- (Updated May 5, 2014, 10:02 a.m.) Review request for cloudstack, Abhinandan Pra

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

Hi Daan, please cherry pick the following commit to 4.4 branch. commit 489bb0c7edf6ab011895d5f082328eb5fe48aac1 in cloudstack 4.4-forward branch. link to diff Cloudstack-5077: reserve cpu and memory only when vmware.reserve.cpu/mem are …

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20891/#review42134 --- Ship it! Ship It! - Santhosh Edukulla On May 2, 2014, 2:23 p.m.,

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

Is upgrade from 4.1.1 + XS 6.0.2 to 4.3 + XS 6.2 officially supported / possible? Original Message Subject:Re: Upgrade from 4.1.1 + XS 6.0.2 to 4.3 + XS 6.2 Date: Tue, 29 Apr 2014 10:31:56 +0200 From: France Reply-To: us...@cloudstack.apache.org To: us..

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

Hi Daan, updating the link to diff. https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=99b4cf788eded0f6f155ff8cda51aeb2e505f764 Thanks Bharat. On 05-May-2014, at 3:36 pm, Bharat Kumar mailto:bharat.ku...@citrix.com>> wrote: Hi Daan, please cherry pick the following commit to

How is this working?

I came across this code snippet in hostConnect() method in DefaultHostListener.java. Look at the line where the used bytes is set on the poolVO. This looks like a serious bug. Looking at the history this code has been there since a year. Has anyone encountered any issues with primary storage ca

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

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 12:38:52,966 DEBUG [backoff.impl.ConstantTimeBackoffTest] (Thread-0:) before 2014-05-05 12:38:53,068 DEBUG

Re: Review Request 18677: CLOUDSTACK-3272

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18677/ --- (Updated May 5, 2014, 10:56 a.m.) Review request for cloudstack, Chiradeep Vitt

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

--- 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

Re: [REMINDER] [REVIEW] CLOUDSTACK-3272

I have posted a new patch file in the review board, kindly review it. On Mon, May 5, 2014 at 12:03 PM, Sonal Ojha wrote: > I have updated the patch almost thrice but due to delay in reviewing the > changes it doesnt work on the latest commit. I have mentioned the commit id > against which the pa

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

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

> 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: unit test failures in files unchanged for more then 3 months??

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

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

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??

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

RE: cherry-pick requests

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??

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: Review Request 20516: CLOUDSTACK-6463: password is not set for VMs created from password enabled template

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20516/#review42139 --- Commit df0564f14d35c02d5e745ffdea3b3e1e6d69698b in cloudstack's bran

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20519/#review42140 --- Commit 0c60ea2e01a80d5201b23415ebef818984db8730 in cloudstack's bran

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

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: [ACS 4.4] cherry-pick request for CLOUDSTACK-5077

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: Review Request 20891: CLOUDSTACK-6536: Code cleanup - removing unnecessary waits after VM stop operation

> 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: Review Request 21070: Fixed few exception cases.

--- 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 20357: [CS-5907] [CS-6396] Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM volumes (not able to snapshot)

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

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

--- 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 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

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 18677: CLOUDSTACK-3272

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

Re: Cleanly remove management server

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 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

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: HyperV agent not able to connect with Jenkins and community builds blocked

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 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

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: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

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: How is this working?

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: [QUESTION] Renaming files in Git

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: Review Request 21070: Fixed few exception cases.

--- 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: Review Request 21070: Fixed few exception cases.

> 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: How is this working?

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 19779: Fixed[CLOUDSTACK-5150]:-Creating template from a VM in Simulator results in incorrect size

--- 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: Review Request 20007: CLOUDSTACK-2697 - Populated the clusterId with actual value instead of Null in the alert message logs

--- 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

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

Re: How is this working?

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

console view not coming

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

Heart Bleed - Apache Cloud Stack - open ssl

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

Re: How is this working?

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

Re: How is this working?

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: console view not coming

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: Review Request 20921: [PATCH] CLOUDSTACK-6552 Cloudstack-Management install package creates log directory that is never used

--- 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

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

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

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

--- 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

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

--- 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 17495: CLOUDSTACK-5976: Typo in "ssh_keypairs" table's foreign key constraints on the Upgraded Setup

--- 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 17495: CLOUDSTACK-5976: Typo in "ssh_keypairs" table's foreign key constraints on the Upgraded Setup

--- 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

--- 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

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

Re: 4.4 release-notes

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

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

--- 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: Review Request 21087: Remove hard-coding for platform emulators in Xen Server

--- 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.

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

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 16002: CLOUDSTACK-3932 UI: Truncation issue occurred in secondary storage URL.

--- 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: Review Request 16002: CLOUDSTACK-3932 UI: Truncation issue occurred in secondary storage URL.

--- 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.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16002/#review42190 --- Commit 259ce3b59c6a8633fc77a7eac88fbefd915107e2 in cloudstack's bran

Re: 4.4 release-notes

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.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16002/#review42191 --- Commit e28948ccc3f16a6c9c956fc773a26fcfbd58014b in cloudstack's bran

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

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

Re: 4.4 release-notes

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

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

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

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

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-forward-build #38

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

karma wanted

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

Re: 4.4 release-notes

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 =

Re: karma wanted

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: git commit: updated refs/heads/4.4-forward to 259ce3b

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

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

> 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: 4.4 release-notes

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: git commit: updated refs/heads/4.4-forward to 259ce3b

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: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

> 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

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

[ACS4.4] Cherry pick for CLOUDSTACK-6483

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: [ACS4.4] Cherry pick for CLOUDSTACK-6483

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

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

> 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

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

See

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

See

  1   2   >