Build failed in Jenkins: simulator-4.5-singlerun #279

2015-11-22 Thread jenkins
See -- Started by upstream project "build-4.5-simulator" build number 333 originally caused by: Started by upstream project "build-4.5" build number 540 originally caused by: Started by an

Jenkins build is back to normal : build-4.5 #540

2015-11-22 Thread jenkins
See

[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-9042: VR: DHCP: fix miss...

2015-11-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1045 --- 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: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-22 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-158860647 Anyone tested this patch yet? --- 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 n

[GitHub] cloudstack pull request: Update 4.5 branch with L10N resources fil...

2015-11-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1099 --- 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-9020: UI enhancements from met...

2015-11-22 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1106 CLOUDSTACK-9020: UI enhancements from metrics view - Sortable columns - Logos with cloudmonkey - Wider UI - CSS fixes and new status icons - Configurable UI list api requests pa

[GitHub] cloudstack pull request: bugfix ui/scripts/accounts.js L1339

2015-11-22 Thread wolf29
Github user wolf29 commented on the pull request: https://github.com/apache/cloudstack/pull/1104#issuecomment-158857050 I have added in the second fix on that file, which should also fix the generated script file, I believe. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-11-22 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-158854951 @remibergsma GIve me some time I will take care of it for 4.7 --- If your project is set up for it, you can reply to this email and have your reply appear on G

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

2015-11-22 Thread jenkins
See -- [...truncated 797 lines...] 2015-11-23 05:11:58,173 DEBUG [utils.script.Script] (main:) System resource: null 2015-11-23 05:11:58,173 DEBUG [utils.script.Script] (main:) Classpath reso

[GitHub] cloudstack pull request: Shuffling the password to avoid having a ...

2015-11-22 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1058#issuecomment-158854458 @remibergsma thanks, will merge this on 4.5 soon. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack pull request: bugfix ui/scripts/accounts.js L1339

2015-11-22 Thread wolf29
Github user wolf29 commented on the pull request: https://github.com/apache/cloudstack/pull/1104#issuecomment-158854379 @ustcweizhou - I can. I have never done one of these before, and wanted to make sure I understood the process before doing a whole lot of changes. --- If your pr

[GitHub] cloudstack pull request: Debian/Ubuntu packaging issues

2015-11-22 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1105 Debian/Ubuntu packaging issues See individual issues. Before: ``` root@acs46:~# ls -la /var/lib/cloudstack total 16 drwxr-xr-x 4 root root 4096 Nov 23 00:20 . d

[GitHub] cloudstack pull request: bugfix ui/scripts/accounts.js L1339

2015-11-22 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1104#issuecomment-158817796 @wolf29 Can you create a PR with ALL related changes ? it is convenient for review and merge --- If your project is set up for it, you can reply to this em

[GitHub] cloudstack pull request: bugfix ui/scripts/accounts.js L1339

2015-11-22 Thread wolf29
GitHub user wolf29 opened a pull request: https://github.com/apache/cloudstack/pull/1104 bugfix ui/scripts/accounts.js L1339 This is a fix for one of the 2 problems in the first file in the stopper list for the https://analysis.apache.org/component_issues?id=org.apache.cloudstack%3

Jenkins build is still unstable: build-systemvm64-master #680

2015-11-22 Thread jenkins
See

[GitHub] cloudstack pull request: CID 1338016: multi catch instead of pokem...

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

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2015-11-22 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-158792594 Hi @remibergsma. I am trying to fix it to put all in one commit. I have force pushed already, but it seems that it does no work. I have tried to squash them, but t

[GitHub] cloudstack pull request: Fix NuageVsp errors for build-master-slow...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1092#issuecomment-158791293 LGTM, all integration tests pass. ![screen shot 2015-11-22 at 20 24 54 pm](https://cloud.githubusercontent.com/assets/1630096/11325793/3c600dfe-9157-11e

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-11-22 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-158790894 Hi @remibergsma. Thanks for the reminding! I was merging that commit with that actual master and I forgot to put the title in the commit. I have fixed that alre

[GitHub] cloudstack pull request: update release to 4.7.0 so it use proper ...

2015-11-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1103#issuecomment-158779047 @remibergsma you are right, we don't need to up the sysvm template version yet. For other version numbers mentioned, @pdion891 can you add thos location in the

[GitHub] cloudstack pull request: update release to 4.7.0 so it use proper ...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1103#issuecomment-158777294 @pdion891 See comments on https://issues.apache.org/jira/browse/CLOUDSTACK-9077 for possible solution for `/dev/loop0` issue. --- If your project is set up fo

[GitHub] cloudstack pull request: update release to 4.7.0 so it use proper ...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1103#issuecomment-158776432 Thanks @pdion891! I agree with all of them, except the systemvm template. There is no reason to bump that version because there is no change yet. As soon as we

[GitHub] cloudstack pull request: update release to 4.7.0 so it use proper ...

2015-11-22 Thread pdion891
GitHub user pdion891 opened a pull request: https://github.com/apache/cloudstack/pull/1103 update release to 4.7.0 so it use proper packages and systemvm This update container creation of master branch to 4.7.0 version. It doesn't fix the /dev/loop0 issue. You can merge th

[GitHub] cloudstack pull request: CLOUDSTACK-9076: Changed ownership of dir...

2015-11-22 Thread borisroman
Github user borisroman closed the pull request at: https://github.com/apache/cloudstack/pull/1095 --- 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 featur

[GitHub] cloudstack pull request: Removal of DefaultUserAuthenticator empty...

2015-11-22 Thread alexandrelimassantana
Github user alexandrelimassantana commented on the pull request: https://github.com/apache/cloudstack/pull/1100#issuecomment-158764180 Thanks for that look-up @rafaelweingartner. Yeah that class was very old, it's use in the hierarchy, as of now, was just for forwading an interface

[GitHub] cloudstack pull request: findbugs: ! equals instead of !=

2015-11-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1019 --- 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: findbugs: ! equals instead of !=

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1019#issuecomment-158753796 Since the commits changed I run the tests again. Still LGTM. ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_h

[GitHub] cloudstack pull request: Quota

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-158753409 @abhinandanprateek @bhaisaab @jburwell Hi guys, I want to run soms tests to see if we can get this in 4.7. As master is on 4.7.0-SNAPSHOT now, please also update

Build failed in Jenkins: build-master-jdk18 #400

2015-11-22 Thread jenkins
See Changes: [Carles.Figuerola] CLOUDSTACK-9048: Fix typo for public network description [David Amorim Faria] Refactor package.sh: * lint * adjust exit codes (1 for usage, 2 for [nera] CLOUDSTACK-9052 Shuffling the password

[GitHub] cloudstack pull request: CLOUDSTACK-9068: Listing Port Forwarding ...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1081#issuecomment-158752938 @DaanHoogland resizing the screen? I didn't test that. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-9048: Fix typo for public netw...

2015-11-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1051 --- 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-9048: Fix typo for public netw...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1051#issuecomment-158752800 LGTM, just to be sure run all integration tests against this branch. They all passed. https://cloud.githubusercontent.com/assets/1630096/11323652/72ab39

[GitHub] cloudstack pull request: Shuffling the password to avoid having a ...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1058#issuecomment-158752741 This was merged to 4.6, then to master as it is a security improvement that applied cleanly. @bhaisaab You may want to include this in the final 4.5 rel

[GitHub] cloudstack pull request: Shuffling the password to avoid having a ...

2015-11-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1058 --- 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: Shuffling the password to avoid having a ...

2015-11-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1058#issuecomment-158752169 LGTM, based on a set of tests that I run on this branch (which I rebased myself first). Screenshot is from an experimental Jenkins job (that runs the same tests

[GitHub] cloudstack pull request: Removal of class AgentBasedStandaloneCons...

2015-11-22 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/855#issuecomment-158746107 @alexandrelimassantana, Could you squash your commits ? --- If your project is set up for it, you can reply to this email and have your reply appear on Gi