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

2014-07-21 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23605/#review48203 --- Commit 43dffaad5fd197c87a8dab087066f677a095dee7 in cloudstack's

Re: Review Request 22799: Golden (Base) Primary Storage feature

2014-07-21 Thread Hieu LE
Thank you, Sudha and Mike, So I will implement some regression test on Marvin to ensure this feature. BRs. On Mon, Jul 21, 2014 at 11:30 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Thanks for the note, Sudha. I was not aware we were treating Marvin tests are a requirement. That

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

2014-07-21 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23605/#review48204 --- Why would we want to disable test cases that fail? Doesn't this

Replace primary secondary storage

2014-07-21 Thread Tejas Gadaria
Hi, I have production vms running on ACS 4.3 with xen 6.2 SP1. I have requirement to change primary Secondary storage. I am planning live storage migration for all running vms, after adding new storage as primary storage storage in same cluster. ( correct me if i am missing anything)..but how

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-21 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23282/#review48208 --- Ship it! commit 03de9cc33507400e0e06ccd84a36334a4660ef4e Author:

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-21 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23282/#review48207 --- Commit 03de9cc33507400e0e06ccd84a36334a4660ef4e in cloudstack's

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-21 Thread Hugo Trippaers
Heya all, I’ve pushed support for the NuageVSP feature just now. Technically two days after the intended feature freeze for 4.6, but i think i can get away with it for the following reasons: * The feature was submitted on the review board before the feature freeze * The feature includes

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

2014-07-21 Thread Gaurav Aradhye
On July 21, 2014, 1:03 p.m., Hugo Trippaers wrote: Why would we want to disable test cases that fail? Doesn't this mean we need to fix something else so they don't fail anymore? Hi Hugo, Whenever we found a test case failing, we create bug for that, may it be a test script issue or

Build failed in Jenkins: build-master #1171

2014-07-21 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/1171/changes Changes: [htrippaers] CLOUDSTACK-6845 : NuageVsp Network plugin -- [...truncated 4511 lines...] [INFO] Compiling 146 source files to

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

2014-07-21 Thread Hugo Trippaers
Hey Gaurav, That doesn’t make a lot of sense to me. If the test sequence fails because of a bug it should keep failing until the bug is resolved, otherwise the tests will report that the situation is OK while in all reality it is not. These test should be in indication that Apache CloudStack

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Stephen Turner
In the case that it's a product bug, wouldn't it be better to keep running the test even if you know it's going to fail? That way, you get a consistent view of the overall pass rate from build to build. If you disable all the tests that are failing, you're going to get a 100% pass rate, but you

Re: Build failed in Jenkins: build-master #1171

2014-07-21 Thread Hugo Trippaers
Should be fixed by commit 40d133dd7358be1940edd8e2a776e035c230c226 I didn’t see the error on my dev system as the proper jar file was present in my local maven cache so it could be found by mvm during the regular compile. Good find by Jenkins. Cheers, Hugo On 21 jul. 2014, at 11:01,

Build failed in Jenkins: build-master #1172

2014-07-21 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/1172/changes Changes: [htrippaers] NuageVSP is a noredist component -- Started by user Hugo [EnvInject] - Loading node environment variables. Building remotely on cloudstack-buildslave-centos6-a84

Re: [DISCUSS] Acquire New Ip from a different range on shared networks

2014-07-21 Thread Murali Reddy
On 15/07/14 3:29 AM, Silvano Nogueira Buback silv...@corp.globo.com wrote: Hi guys, At Globo.com we are working in a load balancer plugin for Cloudstack with a network api developed internally. This api manages shared networks and is working with cloudstack 4.3 (as a network guru

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

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

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Gaurav Aradhye
Hugo, Stephen, We have been following this practice as part of Continuous Integration changes as defined in doc [1]. I personally think that tagging test case with BugId is good idea to map the test cases with bugs, but the test case should not be skipped when tagged. We can have discussion on

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-21 Thread Hugo Trippaers
On 19 jul. 2014, at 02:49, Suresh Ramamurthy suresh.ramamur...@nuagenetworks.net wrote: snip I had following questions regarding compiling only nuagevsp plugin a) To build only nuagevsp, is below command correct. mvn clean install -P developer,nuagevsp Correct b) To run client

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

2014-07-21 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23603/ --- (Updated July 21, 2014, 9:53 a.m.) Review request for cloudstack, Kishan

Re: DNS service on VR not responding

2014-07-21 Thread Indra Pramana
Hi Vihar, Are you referring to /etc/resolv.conf on the VR itself or on the guest VM? On the VR itself, there are only two entries on /etc/resolv.conf pointing to both Google public DNS servers. On the guest VM, there are 3 entries, one to the VR and two to the Google public DNS servers. If I

Wiki page https://cwiki.apache.org/confluence/display/CLOUDSTACK/Cloudstack+-+Continuous+Integration

2014-07-21 Thread Hugo Trippaers
Hey Santosh, On the wiki page there are two URLs in the paragraph of the Log Upload Job that i (and presumably a majority of the community can’t access) can you fix this? This is bit: Log Uploader Job: This job post the CI run uploads all logs to nfs share. All logs post the run are available

RE: Wiki page https://cwiki.apache.org/confluence/display/CLOUDSTACK/Cloudstack+-+Continuous+Integration

2014-07-21 Thread Santhosh Edukulla
sure. Those are example ref links, anyways, i just updated some thing like below, please let me know if its ok. Log Uploader Job: This job post the CI run uploads all logs to nfs share. All logs post the run are available under designated nfs share at configured location in CI, EX:

Re: Guidelines on Functional Test for plugins

2014-07-21 Thread Hugo Trippaers
Santosh, Talluri, Can you guys make sure the functional tests created by Suresh are integrated in the CI runs? test/integration//component/test_nuage_vsp.py Cheers, Hugo On 15 jul. 2014, at 14:07, Suresh Ramamurthy suresh.ramamur...@nuagenetworks.net wrote: Hi All, Can any one please

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Santhosh Edukulla
All, Alex, wanted to disable test cases in between CI( continuous integration) runs for the below reason for failures. I only, provided a way to achieve the same using tags, so that it will work for dual purpose, one not to effect community and can be used in CI as well, it will not effect if

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Hugo Trippaers
Hey Santosh, For keeping track of which component fails its generally better to let jenkins figure it out. As we use nosetest (i think) we can store all the test reports and jenkins can determine which component failed in which test run without having to disable tests. Especially if

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Sudha Ponnaganti
In the beginning to get CI up and running, it would be ok to disable these handful of tests while getting fixes in, to achieve 100% pass rates. When CI runs in production, code changes need to be reverted if there are any new failures to keep CI pass rates at 100% (a known state to make CI

Re: Wiki page https://cwiki.apache.org/confluence/display/CLOUDSTACK/Cloudstack+-+Continuous+Integration

2014-07-21 Thread Hugo Trippaers
Ok, I didn’t realize they were samples, isn’t the CI system running somewhere already? So the logs should be available somewhere right? Cheers, Hugo On 21 jul. 2014, at 12:09, Santhosh Edukulla santhosh.eduku...@citrix.com wrote: sure. Those are example ref links, anyways, i just updated

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Hugo Trippaers
Hey Sudha, Sorry, but i disagree. The purpose of tests should not be to get a 100% pass rate. The tests should show an accurate state of the how the tests are doing versus the current state of the branch being tested. If tests fail we should fix why the tests fails and the system should not

RE: Wiki page https://cwiki.apache.org/confluence/display/CLOUDSTACK/Cloudstack+-+Continuous+Integration

2014-07-21 Thread Santhosh Edukulla
Yes, its running inside now and so logs are available only to internal users, but there was a discussion on dev list to make it public, which was not concluded( i believe ). Once it will be made public, we can adopt the similar approach to facilitate others for viewing\other action. Regards,

Re: Replace primary secondary storage

2014-07-21 Thread Tejas Gadaria
Hi, I found this article http://support.citrix.com/article/CTX135229 I was just going through this but could not get some of the points. Currently I have no snapshots and all templates are public another secondary storage is not added yet. 1) In above article 2nd point says Copy the snapshots

Re: [PROPOSAL] Adding a plugin to check the password strength of all users

2014-07-21 Thread Damoder Reddy
For the first part I am planning to add parameters to global settings. But for the 2nd part I think it depends on where we are integrating this plugin. To start with I will integrate it with abstract layer of Authenticator implementations and we can override in the corresponding authenticator

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Stephen Turner
Tagging is good: that will allow you to see whether a failure is new or known about. Maybe the tag could even contain a date when it was added, to spot failures which are not receiving attention. But I don’t like to see tests skipped whenever we have a bug that is causing them to fail. --

Re: [VOTE][ACS44]Apache CloudStack 4.4.0 RC 2 in branch 4.4-RC20140716T1409

2014-07-21 Thread Daan Hoogland
abandoning vote due to bad quorum :( On Sat, Jul 19, 2014 at 5:28 PM, Pierre-Luc Dion pd...@cloudops.com wrote: Ran into issue when trying to upgrade from 4.2.1. Got the following database upgrade failure: http://pastebin.com/wkkAVYu0 Jira:

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Sudha Ponnaganti
Hugo, I absolutely agree with you that tests should not be disabled and fixes should be made before check in. As per what Alex has mentioned in his CI enablement mail [1], premise of CI is that it runs at 100% pass rates and if any check in causes failure in CI Run, the bad check-in is

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

2014-07-21 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23603/#review48211 --- Commit 58bad41910970fb3b08aa4d21875af6d04c8bf2e in cloudstack's

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

2014-07-21 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23603/#review48212 --- Ship it! Ship It! - Kishan Kavala On July 21, 2014, 3:23 p.m.,

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Stephen Turner
Is there not a way we could have the best of both worlds? Tag them, but leave them running, and reject a check-in if it triggers a failure in an untagged test only? -- Stephen Turner -Original Message- From: Sudha Ponnaganti [mailto:sudha.ponnaga...@citrix.com] Sent: 21 July 2014

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Hugo Trippaers
Hey Sudha, OK, clear. I agree the CI system should be running as soon as possible. However the automated revert bit, i don’t agree with and will give a -1 on that. Cheers, Hugo On 21 jul. 2014, at 13:42, Sudha Ponnaganti sudha.ponnaga...@citrix.com wrote: Hugo, I absolutely agree with

Review Request 23735: Fix deployment of data center with marvin

2014-07-21 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/ --- Review request for cloudstack, daan Hoogland, John Dilley, Santhosh Edukulla,

Review Request 23737: Improve usability of deployDataCenter.py and advanced zone config

2014-07-21 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23737/ --- Review request for cloudstack, daan Hoogland, Hugo Trippaers, and Wei Zhou.

Re: Review Request 23735: Fix deployment of data center with marvin

2014-07-21 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/#review48216 --- tools/devcloud/devcloud-advanced.cfg

Re: Review Request 23735: Fix deployment of data center with marvin

2014-07-21 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/#review48217 --- tools/devcloud/devcloud-advanced.cfg

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

2014-07-21 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22863/#review48220 --- Can you rebase on latest master as the patch currently fails to

Re: Review Request 23737: Improve usability of deployDataCenter.py and advanced zone config

2014-07-21 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23737/#review48219 --- tools/devcloud/devcloud-advanced.cfg

Re: rfc: newsystemvm

2014-07-21 Thread Leo Simons
Hey Chiradeep, Thanks for taking a look. I’ve now re-done this work, but carefully and cleanly and on top of current master, in 37 small commits instead of 2 scary ones. Please take a look at https://github.com/schubergphilis/cloudstack/compare/systemvm-refactor Summarizing this kind of

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

2014-07-21 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23609/#review48224 --- Ship it! Ship It! - Santhosh Edukulla On July 17, 2014, 9:22

Re: Replace primary secondary storage

2014-07-21 Thread Min Chen
That article only applied to ACS 4.1 and older versions. In ACS 4.2 storage refactor, db tables are changed. template_host_ref has been deprecated and replaced with template_store_ref. Thanks -min On 7/21/14 3:47 AM, Tejas Gadaria refond.g...@gmail.com wrote: Hi, I found this article

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-21 Thread Nitin Mehta
On July 17, 2014, 4:55 p.m., Nitin Mehta wrote: server/src/com/cloud/vm/UserVmManagerImpl.java, line 960 https://reviews.apache.org/r/23617/diff/1/?file=634289#file634289line960 Why is it made create=true when it is not a BaseAyncCreate cmd ? create=true should be added only when

Re: [DISCUSS] [PROPOSAL] SAML2 plugin for SSO/SLO in CloudStack

2014-07-21 Thread Min Chen
+1. Very well-written FS and email, Rohit. Those open questions are very valid, I added a little comment in your FS regarding the flow. Thanks -min On 7/20/14 8:35 AM, Rohit Yadav rohit.ya...@shapeblue.com wrote: Hi, I'm assuming no one objects the proposal and the spec, I'll move forward with

Re: [DISCUSS] [PROPOSAL] SAML2 plugin for SSO/SLO in CloudStack

2014-07-21 Thread ilya musayev
Rohit, definite +1 Thanks, ilya On 7/20/14, 8:35 AM, Rohit Yadav wrote: Hi, I'm assuming no one objects the proposal and the spec, I'll move forward with the first implementation starting next week but will be mostly offline till 28th July. Regards. Rohit Yadav wrote: Hi guys, There has

[QUESTION] Baremetal DHCP Server - Abstracting Router VM

2014-07-21 Thread ilya musayev
We are trying to abstract router vm completely from our environment as it has dual nics which is big no no in hardened security environments. This is for shared (non-vpc) advanced security zone. CloudStack already has a support for Baremetal DHCP Server under Network Service Providers.

RE: Tomcat version issue while building cloudstack on RHEL7 RPMS

2014-07-21 Thread Rayees Namathponnan
Hi Hugo, Do you have any ETA for this ? Regards, Rayees -Original Message- From: Rayees Namathponnan Sent: Tuesday, July 15, 2014 9:27 AM To: dev@cloudstack.apache.org Subject: RE: Tomcat version issue while building cloudstack on RHEL7 RPMS Thanks Hugo, defect

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-21 Thread Suresh Ramamurthy
Hi Hugo, Thanks for reviewing NuageVsp plugin on time and making it part for CloudStack's virtual networking solution. Thanks, Suresh On Mon, Jul 21, 2014 at 1:56 AM, Hugo Trippaers h...@trippaers.nl wrote: Heya all, I’ve pushed support for the NuageVSP feature just now. Technically two

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

2014-07-21 Thread Ritu Sabharwal
On July 21, 2014, 2:02 p.m., Hugo Trippaers wrote: Can you rebase on latest master as the patch currently fails to apply. Cheers, Hugo patching file api/src/com/cloud/network/Network.java Hunk #1 FAILED at 132. Hunk #2 succeeded at 224 (offset 3 lines). 1 out of 2

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

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

Re: [DISCUSS] Acquire New Ip from a different range on shared networks

2014-07-21 Thread ilya musayev
Silvano, This is not exactly a help you've asked about. But something along the similar concept, we've requested a feature known as IP POOL. As an example, you have several networks either guest or public that can be put into 1 large pool. In case network x runs out of IP space, you would

Re: [DISCUSS] Acquire New Ip from a different range on shared networks

2014-07-21 Thread Chiradeep Vittal
Do you want to acquire IPs for the VIP (front-end)? From: Silvano Nogueira Buback silv...@corp.globo.commailto:silv...@corp.globo.com Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org Date: Monday, July 14, 2014 at 2:59

Re: [QUESTION] Baremetal DHCP Server - Abstracting Router VM

2014-07-21 Thread Chiradeep Vittal
Are you trying to get rid of the virtual router? Have you looked at the DNSAPI implementation discussed earlier? From: ilya musayev ilya.mailing.li...@gmail.commailto:ilya.mailing.li...@gmail.com Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org

Re: [VOTE][ACS44]Apache CloudStack 4.4.0 RC 2 in branch 4.4-RC20140716T1409

2014-07-21 Thread Alena Prokharchyk
Pierre-Luc, I¹ve closed the bug as invalid. Here is the explanation why: The first attempt to upgrade failed because the new system template wasn't pre-downloaded prior to upgrade (the requirement reflected in Release notes, for the cases when template is changed between CS versions) What should

Re: Tomcat version issue while building cloudstack on RHEL7 RPMS

2014-07-21 Thread Hugo Trippaers
Hey Rayees, Not really, but at least before the 4.6 feature freeze. By that time centos 7 will be stable and we will some time to properly test will all the new versions of the components. Cheers, Hugo Sent from my iPhone On 21 jul. 2014, at 20:47, Rayees Namathponnan

Need help clearing out the Async Queue

2014-07-21 Thread Mathias Mullins
Folks, Looks like I have a job stuck in the async queue that I need to clear. How do you clear out the async queue please? Thanks Matt

Daily CI test summary 07/21/14

2014-07-21 Thread Rayees Namathponnan
Hi All, You can see the current master builds automation status at https://cwiki.apache.org/confluence/x/RAGTAg Simulator 100% pass KVM 4 failures Xen 2 failures There is no new issue reported with latest build Regards, Rayees

RE: [DISCUSS]CLOUDSTACK-6191

2014-07-21 Thread Santhosh Edukulla
Its not a false positive though, the complain is about possible null de-reference, the calling method declaration and definition provides that scope, i now made a change to accommodate null checks for vnetid and protocol values, only inside few cases where it is locally applicable, submitted a

Review Request 23750: Fixed Coverity Reported Issues

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

Resource [DataCenter:6] is unreachable: Unable to apply userdata and password entry on router

2014-07-21 Thread Indra Pramana
Dear all, Suddenly I am not able to create new VMs on my zone. The error message I found on management-server.log: 2014-07-22 11:25:51,632 INFO [cloud.vm.VirtualMachineManagerImpl] (Job-Executor-3:job-17146 = [ b780cd9c-2d0b-499f-b396-269064995e30 ]) Unable to contact resource.

Ceph Question (Wido?)

2014-07-21 Thread Mike Tutkowski
Hi, I'm thinking Wido knows the answer to this, but - of course - anyone feel free to chime in. :) I was looking at this video of Wido's from CCCEU13 in Amsterdam: https://www.youtube.com/watch?v=57voBP_zdf8feature=youtu.be In it, Wido talks about cloning a template on the Ceph cluster, then

Re: Replace primary secondary storage

2014-07-21 Thread Tejas Gadaria
Hi Min, Thanks for clarification, template_store_ref provided me install_path secondary storage id information. it was quite helpful. I found this blog http://stankirdey.com/content/cloudstack-merging-secondary-storages but he is using `template_host_ref`for ACS 4.2. Is there any documented

Re: Review Request 23679: CLOUDSTACK-7130: Tagging failed test cases with product bug Id

2014-07-21 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23679/#review48342 --- Commit 88f35179ef90dc4f7696b6fe2e8ee7ca4d00586c in cloudstack's