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

2015-12-28 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

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-28 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1296#issuecomment-167672958 @remibergsma the code looks ok, would you mind to squash your commits? --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: trailing commas in javascripts removed

2015-12-28 Thread borisroman
Github user borisroman commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1288#discussion_r48466763 --- Diff: ui/plugins/plugins.js --- @@ -16,7 +16,7 @@ // under the License. (function($, cloudStack) { cloudStack.plugins = [ -

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

2015-12-28 Thread jenkins
See Changes: [nicovazquez90] CLOUDSTACK-9074: API Changes: Add l2gatewayserviceuuid to NVP devices [nicovazquez90] CLOUDSTACK-9074: API Changes: Add nsxLogicalSwitch and [nicovazquez90] CLOUDSTACK-9074: New NiciraNVP classes

[GitHub] cloudstack pull request: Quota: findbug fixes

2015-12-28 Thread agneya2001
GitHub user agneya2001 opened a pull request: https://github.com/apache/cloudstack/pull/1289 Quota: findbug fixes Findbug fixes for cloud-framework-quota and cloud-plugin-database-quota. You can merge this pull request into a Git repository by running: $ git pull

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

2015-12-28 Thread jenkins
See Changes: [nicovazquez90] CLOUDSTACK-9074: API Changes: Add l2gatewayserviceuuid to NVP devices [nicovazquez90] CLOUDSTACK-9074: API Changes: Add nsxLogicalSwitch and [nicovazquez90] CLOUDSTACK-9074: New NiciraNVP

Re: VMware vSphere 6

2015-12-28 Thread Kevin Walker
Hi Rohit Ok thats great news, I will deploy CloudStack and perform some testing. Kind regards Kevin Walker +968 99371303 +44 7714303479 On Dec 28, 2015, at 06:17, Rohit Yadav wrote: Hi Kevin, vSphere 6 is already supported in the latest 4.7 release. I’ve tested

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-28 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-167536985 @miguelaferreira I see no issues so LGTM (no integration tests done!) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: Quota: findbug fixes

2015-12-28 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1289#issuecomment-167554252 Based on the code LGTM, tests running --- 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

[GitHub] cloudstack pull request: Quota: findbug fixes

2015-12-28 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1289#discussion_r48476072 --- Diff: framework/quota/src/org/apache/cloudstack/quota/vo/ServiceOfferingVO.java --- @@ -310,14 +269,12 @@ public String getDeploymentPlanner()

[GitHub] cloudstack pull request: Quota: findbug fixes

2015-12-28 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1289#discussion_r48474787 --- Diff: framework/quota/src/org/apache/cloudstack/quota/vo/ServiceOfferingVO.java --- @@ -312,7 +271,7 @@ public String getDeploymentPlanner() {

[GitHub] cloudstack pull request: CLOUDSTACK-8895: Verify if storage on sto...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/869#issuecomment-167551878 I did a code walk-through and looks good. LGTM !! --- 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-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48475810 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8895: Verify if storage on sto...

2015-12-28 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/869#discussion_r48475788 --- Diff: test/integration/testpaths/testpath_attach_disk_zwps.py --- @@ -207,3 +208,188 @@ def test_01_attach_datadisk_to_vm_on_zwps(self):

[GitHub] cloudstack pull request: CLOUDSTACK-9186: Root admin cannot see VP...

2015-12-28 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1258#issuecomment-167539882 @nitin-maharana thanks for checking. LGTM --- 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-9132: API createVolume takes e...

2015-12-28 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1273#issuecomment-167540899 @nitin-maharana I did not notice that the other PR was to 4.6, I thought it was to master. I would just ask you to use a Javadoc block over

[GitHub] cloudstack pull request: trailing commas in javascripts removed

2015-12-28 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1288#discussion_r48466808 --- Diff: ui/plugins/plugins.js --- @@ -16,7 +16,7 @@ // under the License. (function($, cloudStack) { cloudStack.plugins = [ -

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-28 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-167517043 @DaanHoogland @remibergsma (cc @nvazquez) can we merge this PR before master drifts away again? --- If your project is set up for it, you can reply to

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

2015-12-28 Thread jenkins
See -- [...truncated 28722 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48475772 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48475782 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1094 --- 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

Let’s discuss database upgrades

2015-12-28 Thread Rafael Weingärtner
Hi all devs, First of all, sorry the long text, but I hope we can start a discussion here and improve that part of ACS. A while ago I have faced the code that Apache CloudStack (ACS) uses to upgrade from a version to newer one and that did not seem to be a good way to execute our upgrades.

[GitHub] cloudstack pull request: CLOUDSTACK-9091: Update testpath for para...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1130#issuecomment-167562545 This is one line change and looks good. I will get test results for this. LGTM!! --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-9041: Modifying template creat...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1041#issuecomment-167562620 I will post test results. 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

use android keyboard on cloudstack console

2015-12-28 Thread Alireza Eskandari
HiDo you know any workaround to force android smartphones to popup keyborad when we tap on console of a VM?Regards

master marvin install problem

2015-12-28 Thread Daan Hoogland
People, I tried to install marvin 4.8.0-SNAPSHOT this afternoon and found mysql-connector-python missing from pypi. Is this expected? I tried last week and everything was dandy. -- Daan

Re: Let’s discuss database upgrades

2015-12-28 Thread Wido den Hollander
On 28-12-15 14:16, Rafael Weingärtner wrote: > Hi all devs, > First of all, sorry the long text, but I hope we can start a discussion > here and improve that part of ACS. > > A while ago I have faced the code that Apache CloudStack (ACS) uses to > upgrade from a version to newer one and that

[GitHub] cloudstack pull request: CLOUDSTACK-9202 Bump ssh timeout for VR c...

2015-12-28 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1290 CLOUDSTACK-9202 Bump ssh timeout for VR commands It seems the VR needs more time for some of its commands. Until we figured out the root cause, this allows the VRs to start again.

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2015-12-28 Thread miguelaferreira
GitHub user miguelaferreira opened a pull request: https://github.com/apache/cloudstack/pull/1292 Add Health Check Command to NSX plugin The NSX plugin does not support the HeathCheckCommand. Instead it fakes a PingCommand as a call tot he control cluster status API. However,

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

2015-12-28 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

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2015-12-28 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/1293#issuecomment-167586479 I've built the entire project running the existing and additional unit tests. I've also deployed this jar into our beta environment to check that it

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2015-12-28 Thread wido
Github user wido closed the pull request at: https://github.com/apache/cloudstack/pull/1054 --- 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-9202 Bump ssh timeout for VR c...

2015-12-28 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1291 CLOUDSTACK-9202 Bump ssh timeout for VR commands It seems the VR needs more time for some of its commands. Until we figured out the root cause, this allows the VRs to start again.

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2015-12-28 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/1292#issuecomment-167586010 closing since this should not target master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2015-12-28 Thread miguelaferreira
Github user miguelaferreira closed the pull request at: https://github.com/apache/cloudstack/pull/1292 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9202 Bump ssh timeout for VR c...

2015-12-28 Thread remibergsma
Github user remibergsma closed the pull request at: https://github.com/apache/cloudstack/pull/1290 --- 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

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2015-12-28 Thread miguelaferreira
GitHub user miguelaferreira opened a pull request: https://github.com/apache/cloudstack/pull/1293 Add Health Check Command to NSX plugin The NSX plugin does not support the HeathCheckCommand. Instead it fakes a PingCommand as a call tot he control cluster status API. However,

Re: Let’s discuss database upgrades

2015-12-28 Thread Rafael Weingärtner
Thanks for your contribution Wido, I have not seen Rohit’s email; I will take a look at it. About database schema changes happening only in X.Y, I also agree with you (that is a convention we all could agree on, and such as conding and release procedures we could have a wiki page for that).

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48476636 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8308:Adding automation testcas...

2015-12-28 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/384#discussion_r48476586 --- Diff: test/integration/testpaths/testpath_delta_snapshots.py --- @@ -0,0 +1,539 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2015-12-28 Thread wido
GitHub user wido reopened a pull request: https://github.com/apache/cloudstack/pull/1054 CLOUDSTACK-8818: Use MySQL native connector with Python MySQLdb has been deprecated and is also not supported in Python 3. mysql.connector is a connector written in Python which talks

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2015-12-28 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1285#issuecomment-167585886 LGTM, works as expected. ![screen shot 2015-12-28 at 16 08

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-28 Thread remibergsma
Github user remibergsma closed the pull request at: https://github.com/apache/cloudstack/pull/1295 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-28 Thread remibergsma
Github user remibergsma closed the pull request at: https://github.com/apache/cloudstack/pull/1249 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-28 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1249#issuecomment-167595996 Created PR against `4.7`, see #1296 --- 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

Re: Let’s discuss database upgrades

2015-12-28 Thread Rohit Yadav
Hi Rafael and Wido, Thanks for starting a conversation in this regard, I could not pursue the Chimp tool due to other $dayjob work though it’s good to see some discussion has started again. Hope we’ll solve this in 2016. In my opinion, we will need to first separate the database init/migration

[GitHub] cloudstack pull request: Implement a NSX API request execution cou...

2015-12-28 Thread miguelaferreira
GitHub user miguelaferreira opened a pull request: https://github.com/apache/cloudstack/pull/1294 Implement a NSX API request execution counter per thread The NSX plugin has a execution counter to prevent infinite recursion (and as a result a stack overflow exception). However, the

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-28 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1295 CLOUDSTACK-9181 Prevent syntax error in checkrouter.sh Added quotes to prevent syntax errors in weird situations. Error seen in mgt server: ``` 2015-12-15 14:30:32,371

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

2015-12-28 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1296 CLOUDSTACK-9181 Prevent syntax error in checkrouter.sh Added quotes to prevent syntax errors in weird situations. Error seen in mgt server: ``` 2015-12-15 14:30:32,371

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

2015-12-28 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

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2015-12-28 Thread borisroman
Github user borisroman commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1285#discussion_r48488856 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -111,19 +114,27 @@ public

Re: Let’s discuss database upgrades

2015-12-28 Thread Wido den Hollander
On 28-12-15 16:21, Rafael Weingärtner wrote: > Thanks for your contribution Wido, > I have not seen Rohit’s email; I will take a look at it. > Ok, he has a FS here: https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+Chimp > About database schema changes happening only in X.Y, I

Re: Let’s discuss database upgrades

2015-12-28 Thread Rafael Weingärtner
Hi Wido, Rohit, I have just read the feature suggestion. Wido, I am not trying to complicate things, quite the opposite, I just illustrate a simple thing that can happen and is happening; I just pointed how it can be easily solved. About the release of .Z, releases more constant and others, I do

[GitHub] cloudstack pull request: Quota: findbug fixes

2015-12-28 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1289#issuecomment-167625390 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 does not have this feature

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2015-12-28 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1285#discussion_r48496732 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -111,19 +114,27 @@ public

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2015-12-28 Thread borisroman
Github user borisroman commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1285#discussion_r48498393 --- Diff: plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java --- @@ -111,19 +114,27 @@ public

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

2015-12-28 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

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-28 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1273#issuecomment-167721727 @rafaelweingartner : I have updated the branch. Can you please review the change. Thanks. --- If your project is set up for it, you can reply to this email

Re: master marvin install problem

2015-12-28 Thread Srikanteswararao talluri
Looks like Prasanna filed a bug for mysql : https://bugs.mysql.com/bug.php?id=68549 Daan, Problem you are facing is because pypi is being updated manually. It should go away, try after some time or manually install that package. Thanks, ~Talluri On Mon, Dec 28, 2015 at 6:21 PM, Daan Hoogland

[GitHub] cloudstack pull request: CLOUDSTACK-9202 Bump ssh timeout for VR c...

2015-12-28 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/1291#issuecomment-167720561 @remibergsma I am facing the same issue while running the tests for strongswan s2s vpn. I will apply this patch and run the tests once. --- If your

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

2015-12-28 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

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

2015-12-28 Thread jenkins
See -- [...truncated 28733 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

Re: master marvin install problem

2015-12-28 Thread Jayapal Reddy Uradi
I got the mysql-connector-python version issue on updating the marvin. I removed marvin, installed again and it working without issues. Thanks, Jayapl > On 29-Dec-2015, at 11:24 am, Srikanteswararao talluri > wrote: > > Looks like Prasanna filed a bug for mysql : >