[TRAVIS] - failing for pull request #18

Hi guys, Last night I sent a pull request, after rebasing and fixing some conflicts in VirtualNetworkApplianceManagerImpl class. Unfortunately, the 3 builds are not okay on Travis. For the first job, I got the following: Still running (30 of 30): ./tools/travis/before_script.sh Timeout (30

RE: [TRAVIS] - failing for pull request #18

To: Wilder Rodrigues Cc: CloudStack Dev; int-toolkit; int-cloud Subject: Re: [TRAVIS] - failing for pull request #18 HI Wilder, I cloned your branch, compiled it and ran it in simulator mode. mvn -pl :cloud-client-ui jetty:run -D simulator I browsed about the UI for a bit, I seen an exception been

RE: [TRAVIS] - failing for pull request #18

. On 10 September 2014 10:44, Wilder Rodrigues wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com wrote: Thanks Ian, I will follow the same steps, because it’s really weird. I’m running with mvn –pl :cloud-ui jetty:run, but no simulator. Accounts page looks fine, I even executed

RE: [TRAVIS] - failing for pull request #18

for bothering you with that, I will fix it. Cheers, Wilder From: Wilder Rodrigues Sent: Wednesday, September 10, 2014 12:44 PM To: 'Ian Duffy' Cc: CloudStack Dev; int-toolkit; int-cloud Subject: RE: [TRAVIS] - failing for pull request #18 Okay… I’m cloning it in a different place, just to make sure I

RE: [TRAVIS] - failing for pull request #18

. Everything was tested using the Simulator and my environment: Cheers, Wilder From: Wilder Rodrigues Sent: donderdag 11 september 2014 11:25 To: 'Ian Duffy' Cc: CloudStack Dev; int-toolkit; int-cloud Subject: RE: [TRAVIS] - failing for pull request #18 Hi Ian, One job is fixed. What happened? I would say

[Agent] SS VM doesn't connect to the host agent

Hi all, I have been busy trying to identify why is the SS VM connection to the host agent broken. Could anyone please give me a clue? We have a massive Pull Request to be sent. This morning everything was working, tested with Basic/Advanced zones with both Simulator and XenServer. All smoke

Re: [Agent] SS VM doesn't connect to the host agent

Thanks for the reply, Wido. I have to pick up my daughter, but will try the telnet asap. Cheers, Wilder Sent from my iPhone On 02 Oct 2014, at 14:04, Wido den Hollander w...@widodh.nl wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/02/2014 01:55 PM, Wilder Rodrigues wrote

RE: [Agent] SS VM doesn't connect to the host agent

, I will create a bug, try to fix it and send a pull request separately. Will keep you updated. Cheers, Wilder -Original Message- From: Wilder Rodrigues Sent: Thursday, October 02, 2014 3:09 PM To: dev@cloudstack.apache.org Cc: int-toolkit Subject: Re: [Agent] SS VM doesn't connect

RE: [Agent] SS VM doesn't connect to the host agent

will create my PR and send the details about the changes to the community for approval. Cheers, Wilder -Original Message- From: Wilder Rodrigues Sent: Friday, October 03, 2014 6:27 AM To: dev@cloudstack.apache.org; 'w...@widodh.nl' Cc: int-toolkit Subject: RE: [Agent] SS VM doesn't connect

RE: [Agent] SS VM doesn't connect to the host agent

) -Original Message- From: Wilder Rodrigues Sent: Friday, October 03, 2014 6:54 AM To: dev@cloudstack.apache.org; 'w...@widodh.nl' Cc: int-toolkit Subject: RE: [Agent] SS VM doesn't connect to the host agent SSVM = Agent communication is coming UP again. I already created an Advanced Zone

[MASTER-SNMP Alerts] - Trying to test Master, but building is failing

Hi all, Just clone master and tried to build the simulator in order to start testing. Unfortunately, did not pass SNMP Alerts module: --- T E S T S --- Running

Re: [MASTER-SNMP Alerts] - Trying to test Master, but building is failing

Never mindŠ it¹s working again. The problem was related to the test trying to reach my machine based on the hostname. I just added the hostname on the /etc/hosts and it worked. Cheers, Wilder On 06/10/14 14:11, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi all, Just clone master

[VPC RVR] - Refactor for Redundant Virtual Router Implementation

Hi all, Just a quick update on the status of the VPC RVR refactor, which is the part 1 of the whole VPC RVR implementation. I have added some documentation to the Apache Wiki page. You can find it on the URL below:

Re: [MERGE] virtual network appliance manager refactorring

Thanks, Daan! :) More info here: Just a quick update on the status of the VPC RVR refactor, which is the part 1 of the whole VPC RVR implementation. I have added some documentation to the Apache Wiki page. You can find it on the URL below:

[MASTER-TESTS] Just finished testing the latest Master

Hi all, I went through the latest Master branch and did some tests on it. Below few details + URL for the tests results: * Simulator * Advanced Zone * 81 smoke tests executed and passing * Basic Zone * 26 smoke tests executed and passing * XenServer

Re: [VPC RVR] - Refactor for Redundant Virtual Router Implementation

No worriesŠ I think there won¹t be changes in the area we have touched. So, it won¹t create conflicts whilst merging. I will add another diagram to the wiki page, so it makes the bigger picture more visible. Thanks for taking this one. Cheers, Wilder On 08/10/14 09:44, Rohit Yadav

[ACS-4.4] Too many unapproved licenses: 2

Hi there guys and gals, I started doing some tests in the ACS 4.4, just to check how it's getting altogether. First thing I did was to run the license check. Unfortunately, it's failing due to 2 unapproved licenses. See below: mvn --projects='org.apache.cloudstack:cloudstack'

[ACS4.4-Tests] Tests on ACS 4.4, latest code.

Hi all, I have been busy performing some basic tests on ACS 4.4 latest code. All manual tests, but still valuable. So far I did: · As Admin o Created 2 Advanced Zones § Summer Zone with Local Storage enabled § Spring Zone with Local Storage disabled o Created 3 VMs § 2 in

[ACS-Master] Cannot load compute offerings: NPE

Hi guys, I'm facing a problem when loading service offerings on ACS Master branch. Is anyone aware of this? Is there any issue open? 2014-06-18 15:54:52,495 DEBUG [c.c.a.ApiServlet] (19418424@qtp-9894122-6:ctx-3a0f1094) ===START=== 0:0:0:0:0:0:0:1 -- GET command=listServ

Why to have API Commands for Admin actions?

Hi all, We are currently working on the redundant VPC implementation. In order to take the right steps from the beginning, we started analyzing the existing code base, from the API commands into the VPCVirtualNetworkAppliancaManagerImpl. Although it's not related to the feature itself, we

RE: Why to have API Commands for Admin actions?

response view with IAM policy. Hope that this can give you some context on this part of code. Thanks. -min On 6/24/14 4:55 AM, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi all, We are currently working on the redundant VPC implementation. In order to take the right

RE: Why to have API Commands for Admin actions?

if you can use CreateVpcCmd as an example to give some code snippet to illustrate your idea, as well as your proposed flyweight pattern. Thanks -min On 6/25/14 4:27 AM, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi there, Thanks to Min, Rohit and Prachi for the details

RE: Looking for test folks on the community!

I'm in! Cheers, Wilder -Original Message- From: Raja Pullela [mailto:raja.pull...@citrix.com] Sent: Thursday, February 27, 2014 7:01 AM To: CloudStack Dev (dev@cloudstack.apache.org) Subject: Looking for test folks on the community! Hi, Can you please respond if you are actively

Review Request 18568: Fix for returning null when expected is boolean; comparing objects with == instead of equals()

+ Console Proxy and System VMS Create 1 instance (tiny Linux) + Guest Network + 1 SourceNAT + 1 extra pub ip Set up firewall for port 22 + ICMP + port forwarding 22 == instance SSH into the instance Thanks, Wilder Rodrigues

Re: Review Request 18568: Fix for returning null when expected is boolean; comparing objects with == instead of equals()

--- On Feb. 27, 2014, 8:03 a.m., Wilder Rodrigues wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18568

RE: [DevCloud2] Problems with SR type == file

2014 17:01, Wilder Rodrigues wrodrig...@schubergphilis.comwrote: Hi guys and gals, I'm trying to test the current master withDevCloud2, but unfortunately I'm facing few problems concerning my DevCloud2 storage. On DevCloud2 when I type xe sr-list I get the following: uuid ( RO

Re: Review Request 18568: Fix for returning null when expected is boolean; comparing objects with == instead of equals()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18568/#review35862 --- On Feb. 27, 2014, 8:03 a.m., Wilder Rodrigues wrote

[WIKI] Release test procedure contains wrong version of Java Open JDK

Hi guys and gals, Could I get rights to the Apache Wiki page? In case that's not possible, could someone please edit the page and change the version of the Java JDK? It now says 1.6 instead of 1.7. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+test+procedure sudo aptitude

RE: [WIKI] Release test procedure contains wrong version of Java Open JDK

Already got rights and page has been edited. Thanks, Daan! Cheers, Wilder -Original Message- From: Wilder Rodrigues [mailto:wrodrig...@schubergphilis.com] Sent: Wednesday, March 12, 2014 10:19 AM To: dev@cloudstack.apache.org Subject: [WIKI] Release test procedure contains wrong

RE: [VOTE] Apache CloudStack 4.3.0 (seventh round)

Hi guys and gals, Based on the findings after my first round of tests, I give a +1 to the 4.3 RC Please, find below what I have tested so far: * Environment - Management Server: Debian 7 VM under VirtualBox - DevCloud: XenServer 6.2 - MySQL: running on the DevCloud - System VM: Latest

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

Hi guys, I'm testing the CS 4.3 RC (8th round), and will keep doing so, but for now I already did the following and it looks good (sorry for the long email): * Environment - Management Server: Debian 7 VM under VirtualBox - DevCloud: XenServer 6.2 - MySQL: running on the DevCloud -

RE: [VOTE] Apache CloudStack 4.3.0 (eighth round)

Velayudham [mailto:madangan...@me.com] Sent: donderdag 13 maart 2014 11:49 To: dev@cloudstack.apache.org Subject: Re: [VOTE] Apache CloudStack 4.3.0 (eighth round) +1 On 13-Mar-2014, at 4:18 pm, Guo Star ghxand...@gmail.com wrote: +1 2014-03-13 18:27 GMT+08:00 Wilder Rodrigues wrodrig

RE: [VOTE] Apache CloudStack 4.3.0 (ninth round)

Hi there Ilya, Could you please provide which tests cases you have covered? I would like to add them to my tests round as well. Currently I'm testing multiple zones, networks, instances, ACLs, Port Forwarding and will add security groups. If you have more points, I would be glad to test them

[NFS] Adding NFS primary storage fails

Hi all, I'm trying to tests some changes I did for CLOUDSTACK-6139. Currently I have 3 XenServer 6.2 instances and my management server is running in a different machine (debian 7). I have successfully created 2 zones, which use Local Storage. Now I'm trying to create a 3rd zone, which will

Re: [ANNOUNCE] Change of Apache CloudStack PMC Chair

Congratulations, Hugo! Cheers, Wilder Sent from my iPhone On 19 Mar 2014, at 21:51, Chip Childers chipchild...@apache.org wrote: Per our project bylaws, we are changing our project's chair today! Over discussions during the last month the PMC had reached a consensus to recommend to the

RE: [NFS] Adding NFS primary storage fails

and see. thanks. On Wed, Mar 19, 2014 at 3:50 PM, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi all, I'm trying to tests some changes I did for CLOUDSTACK-6139. Currently I have 3 XenServer 6.2 instances and my management server is running in a different machine (debian 7). I

RE: [VOTE] Apache CloudStack 4.3.0 (ninth round)

Hi all, Just finished my [manual] tests here: * Environment - Management Server: Debian 7 VM under VirtualBox - 2 DevCloud: XenServer 6.2 - 1 DevCloud with xe-switch-network-backend bridge - MySQL: running on 1 of the DevCloud - System VM: Latest from

[MASTER] SQL error due to unknown column

Hi all, Got an error on latest Master and cannot even login via the UI... is anyone working on this? ERROR [c.c.n.r.VirtualNetworkApplianceManagerImpl] (RouterStatusMonitor-1:ctx-b82e43b9) Fail to complete the CheckRouterTask! com.cloud.utils.exception.CloudRuntimeException: DB Exception on:

RE: [MASTER] SQL error due to unknown column

that has necessary schema change. Can you please try now. On 21/03/14 3:41 PM, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Hi all, Got an error on latest Master and cannot even login via the UI... is anyone working on this? ERROR [c.c.n.r.VirtualNetworkApplianceManagerImpl

Re: [VOTE] [RESULTS] Apache CloudStack 4.3.0 (ninth round)

Great news! Have a nice weekend, guys and gals! Cheers, Wilder Sent from my iPhone On 21 Mar 2014, at 20:08, Animesh Chaturvedi animesh.chaturv...@citrix.com wrote: The vote has *passed* with the following results (binding PMC votes indicated with a * next to their name: +1 :

[CLOUDSTACK-6139] system.vm.use.local.storage enable with Zone scope

Hi all, I have done some work to get the feature CLOUDSTACK-6139 implemented. I did not change many things, basically I made the Config.SystemVMUseLocalStorage enum to use ConfigKey.scope.Zone in its constructor and changed the DeploymentPlanningManagerImpl. The changes are depicted below

Review Request 19686: Implementation of the issue CLOUDSTACK 6139 - System.vm.use.local.storage global setting to zone setting

to master (4.5.0-SNAPSHOT) won't have problems when performing an update. Thanks, Wilder Rodrigues

Re: Congrats on 4.3 release!

Hi, The upgrade Remi mentioned was from 4.2 to 4.3. Cheers, Wilder Sent from my iPhone On 28 Mar 2014, at 04:05, ilya musayev ilya.mailing.li...@gmail.com wrote: Remi Thanks for the friendly update. Do you mind mentioning which version do you upgrade from? Thanks ilya On

Re: Review Request 19686: Implementation of the issue CLOUDSTACK 6139 - System.vm.use.local.storage global setting to zone setting

. With the steps provided above, I can make sure that resources created with version prior to master (4.5.0-SNAPSHOT) won't have problems when performing an update. Thanks, Wilder Rodrigues

Re: Review Request 19686: Implementation of the issue CLOUDSTACK 6139 - System.vm.use.local.storage global setting to zone setting

. Done. :) - Wilder --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19686/#review40286 --- On April 16, 2014, 9:18 a.m., Wilder Rodrigues

Re: Review Request 19686: Implementation of the issue CLOUDSTACK 6139 - System.vm.use.local.storage global setting to zone setting

/#review40546 --- On April 16, 2014, 9:18 a.m., Wilder Rodrigues wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19686

Re: Review Request 19686: Implementation of the issue CLOUDSTACK 6139 - System.vm.use.local.storage global setting to zone setting

. With the steps provided above, I can make sure that resources created with version prior to master (4.5.0-SNAPSHOT) won't have problems when performing an update. Thanks, Wilder Rodrigues

Re: Review Request 19686: Implementation of the issue CLOUDSTACK 6139 - System.vm.use.local.storage global setting to zone setting

Again, everything worked as expected. With the steps provided above, I can make sure that resources created with version prior to master (4.5.0-SNAPSHOT) won't have problems when performing an update. Thanks, Wilder Rodrigues

Re: Review Request 15264: Fix for 1125389 RV: Bad use of return value

/r/15264/diff/ Testing --- Thanks, Wilder Rodrigues

Review Request 15268: Fix for 1125376 NP: Null pointer dereference - make sure the equals implementation is compliant with the contract.

for null being passed as the argument. (From FindBugs™ description) (CWE-476) Diffs - plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ModelObject.java 71d28ac Diff: https://reviews.apache.org/r/15268/diff/ Testing --- Thanks, Wilder

Review Request 15298: Fix for issues on Coverity related to IDs 1125383 [82, 80, 79, 78, 77, 76, 75, 74]

/src/org/apache/cloudstack/network/contrail/model/ModelObjectBase.java f22c7c5 Diff: https://reviews.apache.org/r/15298/diff/ Testing --- Thanks, Wilder Rodrigues

Re: Review Request 15298: Fix for issues on Coverity related to IDs 1125383 [82, 80, 79, 78, 77, 76, 75, 74]

-contrail/src/org/apache/cloudstack/network/contrail/model/ModelObject.java 71d28ac plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ModelObjectBase.java f22c7c5 Diff: https://reviews.apache.org/r/15298/diff/ Testing --- Thanks, Wilder Rodrigues

Re: Review Request 15298: Fix for issues on Coverity related to IDs cv_1125383 [82, 80, 79, 78, 77, 76, 75, 74]

plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/ModelObjectBase.java f22c7c5 Diff: https://reviews.apache.org/r/15298/diff/ Testing --- Thanks, Wilder Rodrigues

Re: Review Request 15264: Fix for cv_1125389 RV: Bad use of return value

/ContrailManagerImpl.java ae9bba9 Diff: https://reviews.apache.org/r/15264/diff/ Testing --- Thanks, Wilder Rodrigues

Re: Review Request 15298: Fix for issues on Coverity related to IDs cv_1125383 [82, 80, 79, 78, 77, 76, 75, 74]

: https://reviews.apache.org/r/15298/diff/ Testing --- Thanks, Wilder Rodrigues

Review Request 15351: Fixing bugs from Coverity related to Dereferenced Null after check and as return value.

] [INFO] Total time: 10:19.357s [INFO] Finished at: Fri Nov 08 15:06:34 CET 2013 [INFO] Final Memory: 68M/163M Thanks, Wilder Rodrigues

Review Request 15455: Fixing all Coverity bugs on file Upgrade2214to30 related to resource leak. Now the statements closed and nullified before a new assignment.

://reviews.apache.org/r/15455/diff/ Testing --- All tests passed during build and I also applied the patch to a different branch, based on Master, and built the project: all passed. Thanks, Wilder Rodrigues

Review Request 15508: Make sure that if the file does not exist an Exception is thrown and that once it exists it is also closed after the properties are loaded.

. Thanks, Wilder Rodrigues

Review Request 15647: Fixing coverity issues related to resource leak on FileInputStream being created anonymously.

. No new feature was added. Thanks, Wilder Rodrigues

Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

committed and patched, the a brand new branch was created from Master and patched with this patch. Everything worked fine. No new feature was added. Thanks, Wilder Rodrigues

Re: Review Request 15351: Fixing bugs from Coverity related to Dereferenced Null after check and as return value.

/ Testing --- All tests passed during build. [INFO] BUILD SUCCESS [INFO] [INFO] Total time: 10:19.357s [INFO] Finished at: Fri Nov 08 15:06:34 CET 2013 [INFO] Final Memory: 68M/163M Thanks, Wilder Rodrigues

Re: Review Request 15455: Fixing all Coverity bugs on file Upgrade2214to30 related to resource leak. Now the statements closed and nullified before a new assignment.

(updated) new_diff https://reviews.apache.org/media/uploaded/files/2013/11/25/aabab4c6-902d-4b37-a60a-7b52b2db13c2__fix_1116754.patch Thanks, Wilder Rodrigues

Re: Review Request 15455: Fixing all Coverity bugs on file Upgrade2214to30 related to resource leak. Now the statements closed and nullified before a new assignment.

- engine/schema/src/com/cloud/upgrade/dao/Upgrade2214to30.java 48b83b4 Diff: https://reviews.apache.org/r/15455/diff/ Testing --- All tests passed during build and I also applied the patch to a different branch, based on Master, and built the project: all passed. Thanks, Wilder Rodrigues

Re: Review Request 15647: Fixing coverity issues related to resource leak on FileInputStream being created anonymously.

for these changes. After committed and patched, the a brand new branch was created from Master and patched with this patch. Everything worked fine. No new feature was added. Thanks, Wilder Rodrigues

Re: Review Request 15647: Fixing coverity issues related to resource leak on FileInputStream being created anonymously.

created for these changes. After committed and patched, the a brand new branch was created from Master and patched with this patch. Everything worked fine. No new feature was added. Thanks, Wilder Rodrigues

Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

://reviews.apache.org/r/15667/diff/ Testing --- A full build was executed on top of the branch created for these changes. After committed and patched, the a brand new branch was created from Master and patched with this patch. Everything worked fine. No new feature was added. Thanks, Wilder

Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

--- On Nov. 27, 2013, 9:47 a.m., Wilder Rodrigues wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15667/ --- (Updated Nov

Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15667/#review29481 --- On Nov. 27, 2013, 9:47 a.m., Wilder Rodrigues wrote: --- This is an automatically

Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

/diff/ Testing --- A full build was executed on top of the branch created for these changes. After committed and patched, the a brand new branch was created from Master and patched with this patch. Everything worked fine. No new feature was added. Thanks, Wilder Rodrigues

Re: Review Request 15667: Fix for Coverity issues CID_1116744, CID_1116718 and CID_1116682, all related to resource leak.

of the branch created for these changes. After committed and patched, the a brand new branch was created from Master and patched with this patch. Everything worked fine. No new feature was added. Thanks, Wilder Rodrigues

Review Request 16384: Adding the OpenDayLight Neutron REST API

: 24, Failures: 0, Errors: 0, Skipped: 0 Thanks, Wilder Rodrigues

RE: [ANNOUNCE] New Committer: Sanjay Tripathi

Congratulations, Sanjay! With kinds regards, Wilder -Original Message- From: David Nalley [mailto:da...@gnsa.us] Sent: Thursday, January 09, 2014 11:53 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New Committer: Sanjay Tripathi The Project Management Committee (PMC) for Apache

RE: [VOTE] Apache CloudStack 4.3.0 (second round) - unable to manage Firewall, Port Forwarding or Load Balancing rules via the UI, the interfaces no longer exist.

to start from scratch. Kind regards, Wilder Rodrigues Mission Critical Engineer Schuberg Philis Boeingavenue 271 1119 PD Schiphol-Rijk _ -Original Message- From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] Sent: Thursday, January 30, 2014 12:57 AM To: Jessica

Re: [4.3.0] CloudStack diff of Commit 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC Attachments are stripped from this mailing list. From: Wilder Rodrigues wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org dev@cloudstack.apache.orgmailto:dev

[4.3.0] Smoke tests of the fourth round RC

Hi there guys, I have been trying to do a smoke test on the 4.3.0 fourth round RC for few hours already unsuccessfully. I followed the release test procedures wiki and all the checks and builds worked fine. However, when it comes to start the management server, I get errors about creating bean

Contrail plugin - VirtualMchineModel

Hi guys, We are working with FindBugs and trying to get Scariest and Scary bugs fixed on the master branch. FindBugs has reported a bug on the contrail plugin, in the VirtualMachineModel class. If you check the Checking the commit id cc2b1c4961244d9c3d8b452f1dcaa6614e56d11a, you will see the

Review Request 18066: Fixing troubling issues on contrail plugin found by FindBugs

successful + 3 unit tests added in order to cover compare methods cases. Thanks, Wilder Rodrigues

Review Request 18068: Adding fixes and unit tests for cloud-engine-api scariest bugs found by FindBugs

, Wilder Rodrigues

Review Request 18069: Fixing FindBugs scariest for replaceFirst() method call that does not assign the return value

plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/provider/CloudStackPrimaryDataStoreProviderImpl.java 410416c Diff: https://reviews.apache.org/r/18069/diff/ Testing --- Build ran successfully Thanks, Wilder Rodrigues

Review Request 18070: Fixing 12 scary from the engine-storage-volume module found by FindBugs

/VolumeServiceImpl.java 3e315da Diff: https://reviews.apache.org/r/18070/diff/ Testing --- Build passed succesfully Thanks, Wilder Rodrigues

Review Request 18072: Fixing 11 scary and 1 of concern issues found by findBugs. Adding 13 unit tests and also 1 validate method in the UuidUtils class.l

and added 13 unit tests Thanks, Wilder Rodrigues

Re: Review Request 18070: Fixing 12 scary from the engine-storage-volume module found by FindBugs

- engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java 3e315da Diff: https://reviews.apache.org/r/18070/diff/ Testing --- Build passed succesfully Thanks, Wilder Rodrigues

Re: Review Request 18069: Fixing FindBugs scariest for replaceFirst() method call that does not assign the return value

Thanks, Wilder Rodrigues

Re: Review Request 18068: Adding fixes and unit tests for cloud-engine-api scariest bugs found by FindBugs

--- Build successful + 8 unit tests added to cover the changes Thanks, Wilder Rodrigues

Re: Review Request 18066: Fixing troubling issues on contrail plugin found by FindBugs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18066/#review34463 --- On Feb. 13, 2014, 11:32 a.m., Wilder Rodrigues wrote

Re: Review Request 18066: Fixing troubling issues on contrail plugin found by FindBugs

/cloudstack/network/contrail/model/VirtualMachineModel.java https://reviews.apache.org/r/18066/#comment64588 The code is actually right: I removed the dead bit already, but they look so the same that we did not notice. - Wilder Rodrigues On Feb. 13, 2014, 11:32 a.m., Wilder Rodrigues wrote

Re: Review Request 18072: Fixing 11 scary and 1 of concern issues found by findBugs. Adding 13 unit tests and also 1 validate method in the UuidUtils class.l

--- On Feb. 13, 2014, 12:06 p.m., Wilder Rodrigues wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18072

Re: Review Request 18066: Fixing troubling issues on contrail plugin found by FindBugs

On Feb. 14, 2014, 10:18 a.m., Wilder Rodrigues wrote: This review will be discarded. The changes for the contrail plugin were squashed with another commit. A new patch will be uploaded to the review ID 18072 - Wilder

Re: Review Request 18072: Fixing 11 scary and 1 of concern issues found by findBugs. Adding 13 unit tests and also 1 validate method in the UuidUtils class.l

, Wilder Rodrigues

Re: Review Request 18072: Fixing 11 scary and 1 of concern issues found by findBugs. Adding 13 unit tests and also 1 validate method in the UuidUtils class.l

7831bea utils/test/com/cloud/utils/UuidUtilsTest.java PRE-CREATION Diff: https://reviews.apache.org/r/18072/diff/ Testing --- Build successfully and added 13 unit tests Thanks, Wilder Rodrigues

Review Request 18165: FindBugs findings: fixing equals() methods in 2 classes; commenting out dead variable in 1 class; adding 5 tests to cover the changes in the equals() methods.

/test/com/cloud/agent/manager/ConnectedAgentAttacheTest.java PRE-CREATION Diff: https://reviews.apache.org/r/18165/diff/ Testing --- Check style successfully executed; Build successfully executed 5 unit tests added in order to cover the changes. Thanks, Wilder Rodrigues

Re: Review Request 18165: FindBugs findings: fixing equals() methods in 2 classes; commenting out dead variable in 1 class; adding 5 tests to cover the changes in the equals() methods.

--- On Feb. 15, 2014, 12:42 p.m., Wilder Rodrigues wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18165

Re: Review Request 18165: FindBugs findings: fixing equals() methods in 2 classes; commenting out dead variable in 1 class; adding 5 tests to cover the changes in the equals() methods.

On Feb. 15, 2014, 12:53 p.m., Hugo Trippaers wrote: Heya, did you test this with devcloud as well? Wilder Rodrigues wrote: Unfortunately not. I got the link error when trying it in my environment over my network. Will try to refresh the mac addresses on the net adapters

[DevCloud2] Problems with SR type == file

Hi guys and gals, I'm trying to test the current master withDevCloud2, but unfortunately I'm facing few problems concerning my DevCloud2 storage. On DevCloud2 when I type xe sr-list I get the following: uuid ( RO): 9f3f9262-3f77-09cc-2df7-0d8475676260 name-label (

Re: Review Request 18165: FindBugs findings: fixing equals() methods in 2 classes; commenting out dead variable in 1 class; adding 5 tests to cover the changes in the equals() methods.

On Feb. 15, 2014, 12:53 p.m., Hugo Trippaers wrote: Heya, did you test this with devcloud as well? Wilder Rodrigues wrote: Unfortunately not. I got the link error when trying it in my environment over my network. Will try to refresh the mac addresses on the net adapters

Re: [MERGE] virtual network appliance manager refactorring

, Wilder Rodrigues wrodrig...@schubergphilis.com wrote: Thanks, Daan! :) More info here: Just a quick update on the status of the VPC RVR refactor, which is the part 1 of the whole VPC RVR implementation. I have added some documentation to the Apache Wiki page. You can find it on the URL below

[MARVIN] Test Nuage VSP import failed

Hi all, Just running all smoking tests with the simulator and noticed that a new test is not okay: File /Users/wrodrigues/sbp_dev/cloudstack/test/integration/smoke/test_nuage_vsp.py, line 23, in module from marvin.cloudstackAPI import (listPhysicalNetworks, ImportError: cannot import

Re: [ACS44]release 4.4.2 release candidate RC20141121T0341 (#2)

Hi guys, Tests results for ACS 4.4.2. I will also do the same tests, but using a dump from our betacloud database. Cheers, Wilder Environment: * XenServer host running on Betacloud (ACS 4.3.0 VMWare zone) * Management server running on MacBook Pro * MySQL Database running on MacBook Pro *

[4.3.0] System VM 64bit build disabled?

Hi all, Trying to download a systemvm-64 from jenkins.buildacloud.orghttp://jenkins.buildacloud.org but the build seems to be disabled. There is nothing there: http://jenkins.buildacloud.org/view/4.3/job/cloudstack-4.3-systemvm64/ Any ideas why? It was available earlier this week. Cheers,

Re: [4.3.0] System VM 64bit build disabled?

Actually, the whole Jenkins seems to be dead since all slaves are dead. There are several jobs in the queue to be executed. Is there anyone looking into this? Cheers, Wilder On 05 Dec 2014, at 07:57, Wilder Rodrigues wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com wrote: Hi

  1   2   3   >