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

2016-01-23 Thread jenkins
See -- [...truncated 28713 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-9228: Network update with mist...

2016-01-23 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1333#issuecomment-174176375 Hi @nitin-maharana , The lines from 970-1040 and lines from 1048-1118 are the same code. Would you mind extracting them to a function? Then you co

[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

2016-01-23 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1361#issuecomment-174177981 Hi @ nvazquez, Are you sure that the problem is caused because we are not telling the NFS protocol version to the mount command? If you a loo

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

2016-01-23 Thread jenkins
See -- [...truncated 28713 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 #3056

2016-01-23 Thread jenkins
See -- [...truncated 28713 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: engine/schema: Use semantically correct u...

2016-01-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1279#issuecomment-174196682 For the record: the upgrade works: ``` 2016-01-23 16:06:44,796 INFO [c.c.u.DatabaseUpgradeChecker] (main:null) (logid:) DB version = 4.6.2 Code Ver

Re: [VOTE] Apache CloudStack 4.8.0

2016-01-23 Thread Nux!
Hello, Can this run with 4.7 SysVM? If not, can http://jenkins.buildacloud.org/job/build-systemvm64-master/ be used? Additionally, I am encountering problems when building it. ./package.sh --distribution centos63 ./package.sh: unrecognized option '--distribution' This works though: ./package.sh

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

2016-01-23 Thread jenkins
See -- [...truncated 28713 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: More VR performance!

2016-01-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1356 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9245 - Cannot delete non-attac...

2016-01-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1358#issuecomment-174210050 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \ component/test_

[GitHub] cloudstack pull request: CLOUDSTACK-9245 - Cannot delete non-attac...

2016-01-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1358#issuecomment-174210065 Ping @borisroman --- 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

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-23 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-174210179 Could you replace the logic in your 'if' at lines 673 and 523 "(networks == null || networks.size() == 0)" to CollectionUtils.isEmpty(networks) from org.

Re: [VOTE] Apache CloudStack 4.8.0

2016-01-23 Thread Remi Bergsma
Yes, the same from 4.6/4.7 works also in 4.8. On 23/01/16 19:16, "Nux!" wrote: >Hello, > >Can this run with 4.7 SysVM? If not, can >http://jenkins.buildacloud.org/job/build-systemvm64-master/ be used? > >Additionally, I am encountering problems when building it. >./package.sh --distribution

[GitHub] cloudstack pull request: engine/schema: Use semantically correct u...

2016-01-23 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1279#issuecomment-174211155 I've heard everyone's comments on this, I stand by my PR. The argument that DB upgrade works is not relevant to what this PR fixes. --- If your project is set up

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-23 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1360#discussion_r50623870 --- Diff: server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java --- @@ -666,18 +666,10 @@ public ConsoleProxyVO startNew(long dataCenterI

[GitHub] cloudstack pull request: Quota: consolidated lockable account chec...

2016-01-23 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/1350#issuecomment-174211441 I see you've put a bunch of different test cases into a single `testLockableAccount` method. I'd strongly suggest you put each of those different cases into th

Re: [VOTE] Apache CloudStack 4.7.1

2016-01-23 Thread Remi Bergsma
Hi Pierre-Luc, I found the cause of the issue. PR 1134 addresses a change for affinity groups and alters this file [1]. However, it was merged om Dec 3 while the 4.6.1 RC was send on Nov 30. No way it could have ended up in 4.6.1, so it ended up afterwards in 4.6.2 and 4.7.0 etc. We should hav

[GitHub] cloudstack pull request: engine/schema: Use semantically correct u...

2016-01-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1279#issuecomment-174211604 @bhaisaab Let me rephrase: I tested your PR and that works :-) --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-23 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1360#issuecomment-174212703 @ProjectMoon Could you please do the following changes (1, 2, 3 and 4)? **1** - create a method for the code between lines 672 and 676; **2** - c

[GitHub] cloudstack pull request: CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREA...

2016-01-23 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/1157#issuecomment-174213096 @ SudharmaJain why are you passing null to the method that you created (“createTemplateAsyncCallBack”) at line 503? That will cause a null pointer at

Issue with VR after upgrade to 4.6.2

2016-01-23 Thread Daniel Mezentsev
Hi All, I've got some issue after i did cloudstack upgrade 4.5 -> 4.6.2. Environment has 8 VR, 6 were upgraded and restarted successfully, 2 of them stuck. I removed them - the same result, i can't boot them, restarted network with clean-up - no luck. VR actually is starting, i can ssh to it, but

[GitHub] cloudstack pull request: UI: improve filter dropdown width

2016-01-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1354#issuecomment-174215209 LGTM https://cloud.githubusercontent.com/assets/1630096/12532310/4bf04906-c211-11e5-9120-0988f28ebc1c.png";> --- If your project is set up for it, you can

[GitHub] cloudstack pull request: UI: improve filter dropdown width

2016-01-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1354 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9174: A deleted account result...

2016-01-23 Thread cristofolini
Github user cristofolini commented on the pull request: https://github.com/apache/cloudstack/pull/1254#issuecomment-174217394 @agneya2001 Could you please extract the code at lines 126-135 to a separate method? It would make that bit of code much easier to read. :) --- If your proje

Re: [VOTE] Apache CloudStack 4.7.1

2016-01-23 Thread Daan Hoogland
On Fri, Jan 22, 2016 at 11:11 AM, Daan Hoogland wrote: > As a matter of order, I have until Saturday afternoon, right? I am seeing > reasonably good results so far but I need to find time to test in an > advanced zone with security groups and zone wide primary storage as we use > in Leaseweb. (te

Re: [VOTE] Apache CloudStack 4.8.0

2016-01-23 Thread Daan Hoogland
did a minimal verification creating an advanced zone with security groups and shared network. instantiated a vm which started and was visible from the hypervisor, in the gui and from cloudmonkey. ​It is all the verification I can spare this round​. Did it on both 4.7.1 and 4.8.0 +1 (binding) for

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

2016-01-23 Thread jenkins
See Changes: [resmo] UI: improve filter dropdown width [Boris Schrijver] More VR performance! -- [...truncated 28713 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs

[GitHub] cloudstack pull request: CLOUDSTACK-9254: Make longer names displa...

2016-01-23 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1362 CLOUDSTACK-9254: Make longer names display pretty in UI The arrow always fell off when I log in ;-) Before change it looked like this: https://cloud.githubusercontent.com/assets

[GitHub] cloudstack pull request: CLOUDSTACK-9254: Make longer names displa...

2016-01-23 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1362#issuecomment-174225635 Finally! :-) LGTM :+1: Based on code-review and screenshots! --- If your project is set up for it, you can reply to this email and have your reply appe

Re: Issue with VR after upgrade to 4.6.2

2016-01-23 Thread Milamber
Hello, I've have a similiar issue without solution (but not with an upgrade from 4.5), please see https://issues.apache.org/jira/browse/CLOUDSTACK-9255 Milamber On 24/01/2016 03:32, Daniel Mezentsev wrote: Hi All, I've got some issue after i did cloudstack upgrade 4.5 -> 4.6.2. Environment

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

2016-01-23 Thread jenkins
See -- [...truncated 28723 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 #3060

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

Re: Issue with VR after upgrade to 4.6.2

2016-01-23 Thread Daniel Mezentsev
Hi, Seems like you have completely different environment, mine is XenServer + SAN (FC) storage + centos as VMs, but with the same simptoms. Definetely need some input from dev guys. I did research through internet, there are several issues with the same description, but no solution. I'd mark is

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

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

Re: Issue with VR after upgrade to 4.6.2

2016-01-23 Thread Remi Bergsma
Hi, We have seen this issue sometimes in the VRs from 4.6 and on. There have been several improvements in the code in 4.7. Also, setting router.aggregation.command.each.timeout global setting to 15 or 20 also may help (restart mgt server after change). @Milamber can you test this setting with