Re: [DISCUSS: FINDBUG] FW: Build failed in Jenkins: build-master-slowbuild #3120

2016-01-31 Thread Daan Hoogland
What do you mean Abhi, I see only two in the report that are failing this build. For those a PR is out there. On Mon, Feb 1, 2016 at 4:20 AM, Abhinandan Prateek < abhinandan.prat...@shapeblue.com> wrote: > The slow build shows lot more findbug errors than when the findbug is run > in the dev envi

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51385879 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public void r

[GitHub] cloudstack pull request: CLOUDSTACK-8830 CLONE - VM snapshot fails...

2016-01-31 Thread maneesha-p
Github user maneesha-p commented on the pull request: https://github.com/apache/cloudstack/pull/798#issuecomment-177814501 @alexandrelimassantana Updated plz check.Thank you. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

Build failed in Jenkins: build-master-slowbuild #3125

2016-01-31 Thread jenkins
See -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[DISCUSS: FINDBUG] FW: Build failed in Jenkins: build-master-slowbuild #3120

2016-01-31 Thread Abhinandan Prateek
The slow build shows lot more findbug errors than when the findbug is run in the dev environment. My guess will be that in dev environment when running findbug on modules, then findbug is not able to catch issues created by dependencies. Can someone who is more familiar with findbug throw some l

Build failed in Jenkins: build-master-slowbuild #3124

2016-01-31 Thread jenkins
See -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-8609. [VMware] VM is not acces...

2016-01-31 Thread pedro-martins
Github user pedro-martins commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/556#discussion_r51372149 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -1765,7 +1767,18 @@ protected StartAnswer

Build failed in Jenkins: build-master-slowbuild #3123

2016-01-31 Thread jenkins
See -- [...truncated 28685 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Build failed in Jenkins: build-master-slowbuild #3122

2016-01-31 Thread jenkins
See Changes: [Remi Bergsma] CLOUDSTACK-8300: Set indexes on event table -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-qui

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread rodrigo93
Github user rodrigo93 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51367120 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public void rebo

[GitHub] cloudstack pull request: CLOUDSTACK-8910: The reserved_capacity fi...

2016-01-31 Thread alexandrelimassantana
Github user alexandrelimassantana commented on the pull request: https://github.com/apache/cloudstack/pull/892#issuecomment-177591098 @SudharmaJain could you turn that added piece of code into a method and create some testCases? Manually testing works, but making some unit tests helps

Re: Build failed in Jenkins: build-master-slowbuild #3119

2016-01-31 Thread Ramesh Babu
Kindly add my account for this Linux & Unix & Citrix, Apache, proxy, VMware, Storage, Network, Cloud information On Sun, Jan 31, 2016 at 5:39 PM, wrote: > See > > -- > [...truncated 28675 li

[GitHub] cloudstack pull request: CLOUDSTACK-8931: Fail to deploy VM instan...

2016-01-31 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/907#issuecomment-177578048 @kansal It seems your PR failed because of a Travis issue. If you're still interested in working on this, I'd suggest you rebase against the current master, imp

Build failed in Jenkins: build-master-slowbuild #3121

2016-01-31 Thread jenkins
See -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Emit event UUIDs on template deletion

2016-01-31 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/1378#issuecomment-177565744 @ProjectMoon Even if you've managed to test this in an actual environment, I think a small unit test just to assert that the correct values for the class name

[GitHub] cloudstack pull request: CLOUDSTACK-8300: Set indexes on event tab...

2016-01-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1387 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Re: Build failed in Jenkins: build-master-slowbuild #3119

2016-01-31 Thread Ramesh Babu
Hi Team, Kindly add my account for this Linux & Unix & Citrix, Apache, proxy, VMware, Storage, Network, Cloud information Thanks & Regards, Ramesh On Sun, Jan 31, 2016 at 5:39 PM, wrote: > See > > ---

[GitHub] cloudstack pull request: CLOUDSTACK-8300: Set indexes on event tab...

2016-01-31 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1387#issuecomment-177549166 Yes, simple but sane fix indeed. LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project d

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51362827 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public voi

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51362259 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public voi

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51362146 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public void r

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51362068 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public voi

Build failed in Jenkins: build-master-slowbuild #3120

2016-01-31 Thread jenkins
See -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51361803 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public void r

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51360756 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public voi

[GitHub] cloudstack pull request: [4.7] vmware: improve support for disks

2016-01-31 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1365#issuecomment-177495811 @bhaisaab Thanks for the test and StringUtils fix. **-** Could you please change the Access Levels of diskDeviceBusName and diskChain from public to

[GitHub] cloudstack pull request: CLOUDSTACK-8931: Fail to deploy VM instan...

2016-01-31 Thread pedro-martins
Github user pedro-martins commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/907#discussion_r51360120 --- Diff: server/src/com/cloud/network/IpAddressManagerImpl.java --- @@ -680,10 +681,14 @@ public IPAddressVO doInTransaction(TransactionStatus stat

Build failed in Jenkins: build-master-slowbuild #3119

2016-01-31 Thread jenkins
See -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51359410 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public voi

[GitHub] cloudstack pull request: Allow VM snapshots and volume snapshots t...

2016-01-31 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/672#discussion_r51358645 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -4251,6 +4257,34 @@ public void r

[GitHub] cloudstack pull request: Fixes regarding VOLUME_DELETE events resu...

2016-01-31 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1373#issuecomment-177461984 [1373.network.results.txt](https://github.com/apache/cloudstack/files/111219/1373.network.results.txt) [1373.vpc.results.txt](https://github.com/apache/clo

Build failed in Jenkins: build-master-slowbuild #3118

2016-01-31 Thread jenkins
See -- [...truncated 28675 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack