Review Request 23603: Fixed CLOUDSTACK-6983: unable to register lxc template

2014-07-17 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23603/ --- Review request for cloudstack, Kishan Kavala and Marcus Sorensen. Bugs:

Re: [VOTE] Apache Cloudstack 4.4.0

2014-07-17 Thread Erik Weber
Considering adding centos 7 and rhel 7 while at it. Erik 17. juli 2014 00:32 skrev Marcus shadow...@gmail.com følgende: Yes, thats it. I have no idea how these mappings are created, but we need to ensure that both upgrades and new installs work On Wed, Jul 16, 2014 at 3:25 PM, Amogh Vasekar

Re: Review Request 23604: CLOUDSTACK-7108: Reading provisioningtype in library for sparse and fat provisioning types

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23604/#review47986 --- Commit 6f9b3cc9d13bd1bc002ce0afad3ed416b590f882 in cloudstack's

Re: [DISCUSS]CLOUDSTACK-6191

2014-07-17 Thread Daan Hoogland
Seems like it could have been fixed with a smaller code replace but I guess you are right. @Santosh, Can you split your commit and reapply. At least chipping off the BridgeVifDriver bit, but preferably all chopped up as other issues will probably come up. thanks, On Wed, Jul 16, 2014 at 6:57

Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases

2014-07-17 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23605/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7074 and

Re: [VOTE] Apache Cloudstack 4.4.0

2014-07-17 Thread Daan Hoogland
On Wed, Jul 16, 2014 at 10:57 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: 822b38761f1655ef3d5d3648985a197f5c2c1262 is in -- Daan

Re: [VOTE] Apache Cloudstack 4.4.0

2014-07-17 Thread Daan Hoogland
Marcus, Amogh, WHat is the work to do and the ETA? As I read your mail, it seems we are not going to revert but I have no idea on what a forward fix would be. Just adding mappings to KVMGuestOsMapper? @Marcus: is this regression? As in do I agree with you making this a blocker. You seem to imply

Review Request 23607: CLOUDSTACK-7020:Fixed test script issue

2014-07-17 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23607/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7020

Review Request 23609: Signature changes to delete method in VirtualMachine class in base.py

2014-07-17 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23609/ --- Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri.

Review Request 23608: CLOUDSTACK-7014: Resolving test script related to affinity groups tests

2014-07-17 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23608/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7014

Review Request 23610: New script for verifying multiple shared networks with overlapped cidrs in different vlans

2014-07-17 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23610/ --- Review request for cloudstack and SrikanteswaraRao Talluri. Bugs: cs-4282

Re: Review Request 23606: CLOUDSTACK-7025: Resolving test script issue

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23606/#review47994 --- Commit 33fdfbc83407037b45624ca84ca436fab72e59ab in cloudstack's

Re: Review Request 23606: CLOUDSTACK-7025: Resolving test script issue

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23606/#review47995 --- Commit 28a4ae3713af85b95bc0aae52f7fff032ee595aa in cloudstack's

Re: Review Request 23607: CLOUDSTACK-7020:Fixed test script issue

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23607/#review47996 --- Commit b7d445873c36d523571896ea1c51a49f65024445 in cloudstack's

Re: Review Request 23608: CLOUDSTACK-7014: Resolving test script related to affinity groups tests

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23608/#review47999 --- Commit 1cc6317b5e744afb74b63af6600af32bdfedb888 in cloudstack's

Re: Review Request 23611: CLOUDSTACK-7013: Fixing test script issue in test_add_remove_network.py

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23611/#review48001 --- Commit 332191b1bf2935bcad4c05a9fd61d0ebc30f035b in cloudstack's

Review Request 23613: CLOUDSTACK-7022: Fixed typo

2014-07-17 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23613/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7022

Re: Review Request 23613: CLOUDSTACK-7022: Fixed typo

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23613/#review48006 --- Commit 4bab99972f5e87bb6059f34cd0a302db703342f7 in cloudstack's

Re: Review Request 23613: CLOUDSTACK-7022: Fixed typo

2014-07-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23613/#review48008 --- Commit 967b82f16b93bf1df4473b7ce8dd314f015098e8 in cloudstack's

changing cpu.overprovisioning.factor

2014-07-17 Thread Pierre-Luc Dion
I've tried to change the cpu.overprovisioning.factor in Cloudstack 4.3 using Global Settings, even after restarting cloudstack-managment server GHz available remain the same in the zone. Turns out that changing the value in the cluster did update value automatically without restarts. Is this a

Re: changing cpu.overprovisioning.factor

2014-07-17 Thread Pierre-Luc Dion
Nitin, This is the behaviour I've observe and make sense too. Updating cluster config does work. I think our documentation need to reflect that behaviour maybe in the admin guide... Thanks ! *Pierre-Luc DION* Architecte de Solution Cloud | Cloud Solutions Architect t 855.652.5683 *CloudOps*

Re: Review Request 23617: Add Nic UUID to the context so that we can read the same in event bus after create a nic

2014-07-17 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23617/#review48016 --- server/src/com/cloud/vm/UserVmManagerImpl.java

Re: changing cpu.overprovisioning.factor

2014-07-17 Thread Nitin Mehta
Thanks for confirming. Could you please create a bug at JIRA for adding this into documentation ? -Nitin On 17/07/14 9:53 AM, Pierre-Luc Dion pd...@cloudops.com wrote: Nitin, This is the behaviour I've observe and make sense too. Updating cluster config does work. I think our documentation

Re: [VOTE] Apache Cloudstack 4.4.0

2014-07-17 Thread Marcus
Yes, it's a regression. The impact is that upgraders to 4.4 may have existing VMs that don't boot when rebooted, or templates that won't work, because the virtual hardware is changed and not what was expected. On Thu, Jul 17, 2014 at 1:56 AM, Daan Hoogland daan.hoogl...@gmail.com wrote:

Re: changing cpu.overprovisioning.factor

2014-07-17 Thread Harikrishna Patnala
Please create a doc bug only for two parameters cpu and memory overprovisioning factors. Because for the remaining parameters till the value is defined at granular level, CS uses the value at global level. So in your case if it would have some parameter other than cpu and memory over

Re: changing cpu.overprovisioning.factor

2014-07-17 Thread Pierre-Luc Dion
https://issues.apache.org/jira/browse/CLOUDSTACK-7120 updated. *Pierre-Luc DION* Architecte de Solution Cloud | Cloud Solutions Architect t 855.652.5683 *CloudOps* Votre partenaire infonuagique* | *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw

Daily Continues Integration test summary

2014-07-17 Thread Rayees Namathponnan
Hi All, We are running Continues Integration inside Citrix environment, tests are running in both mater and release branches in every 3 hours. Currently only BVT test cases executing on Simulator, Xen and KVM. We are working towards to add regression suite, vmware and external devices test

[ACS 4.4] Cherry pick request

2014-07-17 Thread Amogh Vasekar
Hi Daan, Can you please cherry-pick c8ca15b95a57a3d79b71c76c913e295f6490f05d This is for CLOUDSTACK-6358. Thanks, Amogh

4.4 and master system template jobs failing

2014-07-17 Thread Rayees Namathponnan
Hi All, 4.4 and master system template jobs failing from http://jenkins.buildacloud.org; these jobs try to download ISO from below location and fail with 404 error; http://cdimage.debian.org/mirror/cdimage/archive/7.4.0/amd64/iso-cd/debian-7.4.0-amd64-netinst.iso

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
Amogh, I couldn't help myself. please have a look at the resulting setup/db/db/schema-430to440.sql On Thu, Jul 17, 2014 at 11:35 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Thu, Jul 17, 2014 at 11:19 PM, Amogh Vasekar amogh.vase...@citrix.com wrote:

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Amogh Vasekar
Seems good, looks like was an issue with a newline somewhere. But deploydb went fine on 4.4 Thanks, Amogh On 7/17/14 2:42 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: Amogh, I couldn't help myself. please have a look at the resulting setup/db/db/schema-430to440.sql On Thu, Jul 17, 2014 at

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
sure? I saw that the last few lines where not in the last version. I'm not confortable with this bit, it has been coming up a few time before already looks like some commit on 4.4-forward is trying to sneak it's way into the release: @@ -2439,4 +2474,16 @@ CONSTRAINT

Re: Review Request 22863: CLOUDSTACK-6823 : First code drop for Brocade Network plugin to orchestrate Brocade VDX switches for L2 connectivity.

2014-07-17 Thread Ritu Sabharwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22863/ --- (Updated July 17, 2014, 10:04 p.m.) Review request for cloudstack and Hugo

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Nitin Mehta
Hi Daan - I am not sure I get your point here. These changes were put in as I want them in 4.4.1, but were not critical enough to be put in 4.4. Thanks, -Nitin On 17/07/14 2:58 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: sure? I saw that the last few lines where not in the last version.

Re: Review Request 22863: CLOUDSTACK-6823 : First code drop for Brocade Network plugin to orchestrate Brocade VDX switches for L2 connectivity.

2014-07-17 Thread Ritu Sabharwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22863/ --- (Updated July 17, 2014, 11:52 p.m.) Review request for cloudstack and Hugo

VirtualRoutingResource bugs CLOUDSTACK-7111 after refactor

2014-07-17 Thread Sheng Yang
Hi Hugo, I saw you refactor the VirtualRoutingResource, but it introduced https://issues.apache.org/jira/browse/CLOUDSTACK-7111 . Here are the lines you deleted. -ExecutionResult result = new ExecutionResult(true, No configure to be applied); -for (ConfigItem c : cfg) {

Build failed in Jenkins: build-master #1147

2014-07-17 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/1147/changes Changes: [sheng.yang] CLOUDSTACK-7124: Fix semicolon caused VPN programming issue -- [...truncated 935 lines...] Audit done. [INFO] [INFO] --- maven-remote-resources-plugin:1.3:process

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

2014-07-17 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/1148/changes

Re: [ACS4.5] When is feature freeze?

2014-07-17 Thread Mike Tutkowski
So, how do we plan on enforcing feature freeze (July 19th) without a RM for 4.5? I would take the role, but I don't have the bandwidth it demands. On Wed, Jul 16, 2014 at 5:36 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: On Wed, Jul 16, 2014 at 8:55 AM, Sebastien Goasguen run...@gmail.com

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Mike Tutkowski
Perhaps there is some confusion again as to the nature of the 4.4-forward branch. A while back, we agreed that changes put in here would not be cherry picked to 4.4 unless requested so by the developer and agreed to by the RM. Changes in 4.4-forward that do not go into 4.4 will at least go into

Re: Review Request 23410: Improvised CloudByte Storage Plugin for 4.5 and above. jira id : CLOUDSTACK-7098

2014-07-17 Thread Mike Tutkowski
On July 17, 2014, 10:06 p.m., Mike Tutkowski wrote: Ship It! Changes were put into ACS repo with this commit: f3cbb699955243007ffb53b335d9508b2e422a06 - Mike --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 23410: Improvised CloudByte Storage Plugin for 4.5 and above. jira id : CLOUDSTACK-7098

2014-07-17 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23410/#review48107 --- Ship it! Ship It! - Mike Tutkowski On July 15, 2014, 8:18 a.m.,

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
That is not the confusion Mike. The problem is that some changes that don't go into 4.4 keep causing conflicts. I made the mistake of adding the conflicting lines this time. this 4.4-forward branch is not suitable for providing cherry-picks for an RM because of this. I thin people should just

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Mike Tutkowski
I see, Daan - thanks for the clarification. This is probably another good reason why we should seriously consider implementing the branching approach Sebastien recommended here: http://nvie.com/posts/a-successful-git-branching-model/ On Thu, Jul 17, 2014 at 10:06 PM, Daan Hoogland

Re: Question regarding Quiesce VM feature in ACS 4.3 wrt XEN hypervisor

2014-07-17 Thread Amit Das
thanks Mike. I shall have a look at the implementation details. Will be looking forward to any documentation if any. Regards, Amit *CloudByte Inc.* http://www.cloudbyte.com/ On Fri, Jul 18, 2014 at 8:58 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Hi Amit, I don't really know

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
Well, maybe, we are not even far of from it. call 'master' 'develop' and create a new 'master'. next make sure every body develops in a feature branch. also call x.y-forward hotfix-x.y. Don't bet your life this approach is going to save our world. What would really help is if everybody would