[GitHub] cloudstack pull request: This branch implements the CSV and native...

2016-05-18 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/351#issuecomment-219942609 @swill @rhtyd I am facing permission denied exceptions for some APIs in UI. Seems like they are after the merge of PR #1489. --- If your project is set up for i

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1511#discussion_r63655061 --- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java --- @@ -341,179 +350,172 @@ public SslCertResponse createCertResponse(SslC

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219944062 @swill @rhtyd Some APIs are giving permission exceptions. They can be seen by just logging into UI. I have added the commands.properties to overcome these except

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1511#discussion_r63655432 --- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java --- @@ -341,179 +350,172 @@ public SslCertResponse createCertResponse(SslC

[GitHub] cloudstack pull request: Lower the time we wait for interfaces to ...

2016-05-18 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1471#issuecomment-219944851 @swill Thanks for running the tests again. The error is `Failed to delete network` so that's cleanup related. The failed one, is that the one that randomly? Can

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219945994 @DaanHoogland Not able to see diffs for NetscalerResource.java, are the changes mostly related to adding 'final' keyword? Provided some comments, rest of the co

RE: Migrating CloudStack content from download.cloud.com

2016-05-18 Thread Wido den Hollander
> Op 17 mei 2016 om 10:34 schreef Giles Sirett : > > > This is a MAJOR problem and needs to be figured out before the current > download site is removed - @Raja - have Citrix committed to it being there > another year ? > > It is my understanding that Cloudstack relies on this download at ini

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-18 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-219949545 @koushik-das @swill @mike-tutkowski This has broken Hyper-V support. And this is happening because of variable renaming in command. --- If your project is set up

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-219950557 @DaanHoogland I haven't touched this code much. @The-Loeki ran into this issue when working on our Private Cloud (which is where we test upgrades first). We do

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-219950787 @swill Is the failure related to this PR you think? Having this in 4.9 would be a good to have. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread linas-h1p
Github user linas-h1p commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-219951759 Disclaimer: i'm very new to the whole ACS, but i have built and deployed custom releases for ourselves. I understand the project structure somewhat. So..

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219953009 @anshul1886 can you share the details, which APIs. Also, are you facing this on master or on your personal/feature branch? --- If your project is set up for it, you

RE: Migrating CloudStack content from download.cloud.com

2016-05-18 Thread Paul Angus
Pulling a few threads together - if the ASF would create a download.cloudstack.org page on ASF infra with the mirrorlist json in it as suggested by Nux, we can maintain just that page and add/remove mirrors at will. Kind regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chand

[GitHub] cloudstack pull request: dynamic-roles: remove unnecessary order b...

2016-05-18 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1551 dynamic-roles: remove unnecessary order by ID On some MySQL server envs, this may cause a SQL statement error, though I was unable to reproduce it. Since it's not needed, an order by 'sort_or

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219971144 @koushik-das I will look at those two comment, thanks and no, quite significant changes were done in the NetscalerResource. You are right that a lot of them

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219971607 @koushik-das most significantly around line 690 --- 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: 4.9/master bountycastle changes

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219973474 tag:needslove @swill Looking into @koushik-das remarks I noticed global loadbalancing code different in this one compared to the previous merge commit! --

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-219974055 @wido the travis problems seem to be in xen functionality. I haven't got a working xen setup atm. The eror from @swill his run is not specific to this PR ---

[GitHub] cloudstack pull request: [WIP][4.10+/long-term] Moving towards Jav...

2016-05-18 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1546#issuecomment-219975396 We use, when ever possible, openjdk as we get support from the distribution. Support for openjdk be it 7 or 8, manually patching oraclejdk is time consuming.

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-18 Thread koushik-das
GitHub user koushik-das reopened a pull request: https://github.com/apache/cloudstack/pull/1251 CLOUDSTACK-9180: Optimize concurrent VM deployment operation on same … …network Check if VR needs to be allocated for a given network and only acquire lock if required

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-18 Thread koushik-das
Github user koushik-das closed the pull request at: https://github.com/apache/cloudstack/pull/1251 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219980293 @rhtyd There are many APIs. I have not prepared any list. But creating setup from scratch should reproduce it. Some operations in which it can be observed

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-18 Thread olivierlemasle
Github user olivierlemasle commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-219980996 Ok Still the same error. I still don't think it's related to my change. The same error occured before, for example for https://github.com/apache/

Problem enabling Intermediate SSL Certificate on Console VM

2016-05-18 Thread Özhan Rüzgar Karaman
Hi Developers; My console vm works successfully over SSL connections. Yesterday we realised that firefox could not validate our SSL certificate and it gives us certificate validation errors. We checked keystore table on database and noticed that we have not imported intermediate certificate and co

[GitHub] cloudstack pull request: [WIP][4.10+/long-term] Moving towards Jav...

2016-05-18 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1546#issuecomment-219985859 @resmo have you found if Travis specific reliance on oraclejdk7/8 causes any issue with building/testing CloudStack? --- If your project is set up for it, you can re

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219986204 @anshul1886 I'm unable to reproduce the shared issues nor they were caught by CI/Jenkins/Travis runs. The role specific rules in role_permissions table have been copi

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219987354 @rhtyd Is commands.properties exist in your setup or not? If it exists then things are working fine. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219988321 @anshul1886 with dynamic-roles feature, we no longer want to use or promote use of commands.properties file which is why this has been removed in master. We no longer

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-219990167 @rhtyd I have rebuilt my branch which was rebased yesterday. I am creating new setup. There is no new API introduced in my branch. I am talking about existing AP

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Nick LIVENS
Hey all, So I've figured it out what's causing the issue, in commit c22659d76d73f00f41c13776c490e17a50aacd20, there has been a change in the classpath in cloud-management.sysconfig The change removes the mysql-connector-java.jar from the classpath. Before : CLASSPATH=/etc/cloudstack/management:/u

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219993865 @swill the GLB code missing is false alarm. I had removed unused code months ago to get rid of warnings. I am compiling a version without pedantic formatting t

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-219994116 @DaanHoogland Good! So that means there is no direct blocker for this PR, right? This is a great addition for CloudStack which I personally would like to see

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1511#discussion_r63683861 --- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java --- @@ -341,179 +350,172 @@ public SslCertResponse createCertResponse(Ssl

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-219995275 @swill Thanks for the review. I have updated the test. Now its green. finally. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-219996391 I don't know who uses scalevm on xen @wido... @remibergsma, do you guys? anybosy else? --- If your project is set up for it, you can reply to this email and h

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1511#discussion_r63688438 --- Diff: server/src/org/apache/cloudstack/network/lb/CertServiceImpl.java --- @@ -341,179 +350,172 @@ public SslCertResponse createCertResponse(Ssl

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread The-Loeki
Github user The-Loeki commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-220004141 @linas-h1p thanks for noticing, that fix got lost in commit msg translation. I've amended the `CsAddress.py` commit to (un)fix the whitespacing & readded

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-220005496 @anshul1886 okay, let's debug; - After you mvn re-built cleanly, did you run mvn deploydb? - If yes, what is the output of: select * from configuration where n

Re: repository/nexus down

2016-05-18 Thread Anton Vinogradov
Down again. On Tue, May 17, 2016 at 7:25 PM, Will Stevens wrote: > Everything seems to be back to good. Thank you for your hard work. 👍 > > On Tue, May 17, 2016 at 11:46 AM, Will Stevens > wrote: > >> Thank you sir. I will let you know if I run into any issues. >> >> On Tue, May 17, 2016 at

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread The-Loeki
Github user The-Loeki commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1547#discussion_r63691374 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsConfig.py --- @@ -58,25 +58,35 @@ def get_level(self): return self.__LOG_LEVEL

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1547#discussion_r63692269 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsConfig.py --- @@ -82,7 +82,10 @@ def get_dns(self): conf = self.cmdline().id

RE: Migrating CloudStack content from download.cloud.com

2016-05-18 Thread Will Stevens
I like this approach as well. Seems like a nice solution and builds in redundancy if a specific mirror is down for some reason. On May 18, 2016 4:06 AM, "Paul Angus" wrote: > Pulling a few threads together - if the ASF would create a > download.cloudstack.org page on ASF infra with the mirrorlist

Re: Migrating CloudStack content from download.cloud.com

2016-05-18 Thread Erik Weber
I guess we could even host it with github pages? On Wed, May 18, 2016 at 2:28 PM, Will Stevens wrote: > I like this approach as well. Seems like a nice solution and builds in > redundancy if a specific mirror is down for some reason. > On May 18, 2016 4:06 AM, "Paul Angus" wrote: > > > Pulling

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread linas-h1p
Github user linas-h1p commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-220014434 Seems to be ok now. Thanks for the quick fix. Another issue: With the remade apache config generation, i see you are either putting multiple Listen di

Re: repository/nexus down

2016-05-18 Thread Jeff Hair
Indeed. Apache Status site currently says: "We are experiencing ongoing issues with nexus performance. Expect occasional 502 errors on deployments while we work on moving the service. There will be a prolonged emergency outage of approximately 6 hours starting around UTC for data synchronizati

Re: Problem enabling Intermediate SSL Certificate on Console VM

2016-05-18 Thread Özhan Rüzgar Karaman
Hi; I solved the issue, somehow Alpha SSL has an extra space character in their intermediate CA file. So i removed the space character and reinstall the intermediate CA an now Console VM is fine. I will also remind Alpha SSL about the extra space character in their intermediate CA file. Thanks for

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-18 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220019059 Hi @anshul1886 Can you be more specific about the class that is given you problems and the variables? Thanks! --- If your project is set up for it, you can r

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-18 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220019415 @remibergsma, @DaanHoogland, @swill, I've added an update to this PR. We're stepping away from the no-redist model :) --- If your project is set up for it, you

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-220019839 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 1 Duration: 11h 12m 43s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220020636 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 9h 11m 45s ``` **Associ

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220021664 CI came back with a clean bill of health this time... The scale-vm error was also seen in a different PR, so I don't think it is related to this one. I think

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread The-Loeki
Github user The-Loeki commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-220022831 @linas-h1p tnx for the intel. Strange that I didn't encounter this, I'll go check --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: SystemVM cleanups

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1414#issuecomment-220023082 ### CI RESULTS ``` Tests Run: 83 Skipped: 0 Failed: 0 Errors: 2 Duration: 8h 32m 12s ``` **Summary of the p

[GitHub] cloudstack pull request: SystemVM cleanups

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1414#issuecomment-220023373 I think this one is ready... --- 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

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-220024998 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 0 Duration: 4h 18m 27s ``` **Summary of the pr

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-220025191 This shows that it does not break anything, but I don't think we have any verification that this feature works as expected yet... --- If your project is set up for it

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-220025466 ### CI RESULTS ``` Tests Run: 44 Skipped: 0 Failed: 0 Errors: 0 Duration: 3h 20m 40s ``` **Associ

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-220025750 The tests that were failing are not failing anymore, I think this is ready... --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack pull request: dynamic-roles: remove unnecessary order b...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220030012 Do you have reports already of this causing problems? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

Re: Problem enabling Intermediate SSL Certificate on Console VM

2016-05-18 Thread Daan Hoogland
Thanks for reporting back Ohzan, sorry we couldn't be of much help ;) On Wed, May 18, 2016 at 2:52 PM, Özhan Rüzgar Karaman < oruzgarkara...@gmail.com> wrote: > Hi; > I solved the issue, somehow Alpha SSL has an extra space character in their > intermediate CA file. So i removed the space charac

[GitHub] cloudstack pull request: This branch implements the CSV and native...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/351#issuecomment-220031084 @rhtyd can you work with @anshul1886 to understand what is going on here? This is important as we are about to freeze master and cut 4.9. Thanks... --- If your proj

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-18 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220031468 great Nick, can you explain your choosen path for prosterity? --- If your project is set up for it, you can reply to this email and have your reply appear on G

Re: Migrating CloudStack content from download.cloud.com

2016-05-18 Thread Will Stevens
Haha. I thought of that and didn't want to mention it because I thought it would be too fragile and not very ASFy. But if we have a ​ mirror ​ list, it would be a good option as one of the items in the list as a backup. On May 18, 2016 8:33 AM, "Erik Weber" wrote: I guess we could even host it

[GitHub] cloudstack pull request: Lower the time we wait for interfaces to ...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1471#issuecomment-220032965 @remibergsma that failure is not one I commonly see, but I definitely do have periodic failures in my environment which complain about "not being able to connect to t

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Simon Weller
Nick, mysql-connector-java.jar was moved into the tomcat common.loader in /etc/cloudstack/management/catalina.properties: common.loader=${catalina.base}/lib,${catalina.base}/lib/*.jar,${catalina.home}/lib,${catalina.home}/lib/*.jar,/usr/share/java/mysql-connector-java.jar,/usr/share/cloudstack-

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63707401 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) under

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Nick LIVENS
Simon, If I look at the changes of the commit I've mentioned, I can only see a change for /usr/share/cloudstack-mysql-ha/lib/*.jar (*jar -> *.jar) The mysql-connector-java was already part of it. This is the entry in our catalina.properties : common.loader=${catalina.base}/lib,${catalina.base}/li

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63707528 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-220037863 If we have to revert this, it will be a mess. Lets try to get to the bottom of this ASAP as the freeze is basically here... --- If your project is set up for it, yo

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Simon Weller
It prevented /usr/share/cloudstack-mysql-ha/lib/*.jar from loaded. They need to be loaded together it seems and pulling the ha jar into the classpath doesn't work either. So this change functions for all the folks that have tested it thus far and passed CI, so I wonder why your environment is h

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-220038585 Im not sure. I need to check the mailing list to see if there are other PRs with similar errors. It seems to have the same errors for the past 3 runs, so it seems p

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread The-Loeki
Github user The-Loeki commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-220043104 'aight, pulling that string reveals a few more problems to fix unfortunately. * `cloud-init-early` shell script creates/modifies `ports.conf`, `default`,

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-18 Thread NuxRo
Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-220033959 Thanks buddies. --- 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 fea

[GitHub] cloudstack pull request: CLOUDSTACK-9380: fix NPE in listDomains A...

2016-05-18 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1550#issuecomment-220046965 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 4 Duration: 8h 18m 52s ``` **Summary of the p

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63715139 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) un

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Nick LIVENS
Java : [root@csc-1 ~]# java -version java version "1.7.0_85" OpenJDK Runtime Environment (rhel-2.6.1.2.el7_1-x86_64 u85-b01) OpenJDK 64-Bit Server VM (build 24.85-b03, mixed mode) Tomcat : [root@csc-1 ~]# tomcat version Server version: Apache Tomcat/7.0.54 Server built: Mar 24 2015 07:49:05 Ser

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-18 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1423 [CLOUDSTACK-9296] Start ipsec for client VPN This fix starts the IPSEC daemon when enabling client side vpn You can merge this pull request into a Git repository by running: $ git pull http

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-18 Thread syed
Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1423 --- 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 e

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-18 Thread syed
Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1423 --- 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 e

Re: Migrating CloudStack content from download.cloud.com

2016-05-18 Thread Nux!
Yeah, I also left it out because it's a too frequent target of DDOS attacks and not as available as we'd like. Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Will Stevens" > To: dev@cloudstack.apache.org > Sent: Wednesday, 1

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-18 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1423 [CLOUDSTACK-9296] Start ipsec for client VPN This fix starts the IPSEC daemon when enabling client side vpn You can merge this pull request into a Git repository by running: $ git pull http

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-18 Thread syed
Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1423 --- 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 e

[GitHub] cloudstack pull request: [CLOUDSTACK-9296] Start ipsec for client ...

2016-05-18 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1423 [CLOUDSTACK-9296] Start ipsec for client VPN This fix starts the IPSEC daemon when enabling client side vpn You can merge this pull request into a Git repository by running: $ git pull http

How to fix Jenkins git workspace issues

2016-05-18 Thread Will Stevens
If Jenkins has problems with fragments being left over in it's workspace which is causing following runs against other PRs to fail, how do we fix that? Thanks...

Re: How to fix Jenkins git workspace issues

2016-05-18 Thread Rafael Weingärtner
Is Jenkins configure to clean the workspace before it starts a new build? It should have an option such as "Delete workspace before build starts". On Wed, May 18, 2016 at 12:00 PM, Will Stevens wrote: > If Jenkins has problems with fragments being left over in it's workspace > which is causing

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63720420 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63721020 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) under

Additional Subnets for networks and VR dhcp

2016-05-18 Thread Linas Žilinskas
This is in relation to https://github.com/apache/cloudstack/pull/1547 But since it seems that my observations are beyond the scope of it, I'm posting it here. I'm having hard time understanding what's going on, but something seems off. Like i mentioned in the PR comments, I'm using advanced networ

Additional Subnets for networks and VR dhcp

2016-05-18 Thread Linas Žilinskas
This is in relation to https://github.com/apache/cloudstack/pull/1547 But since it seems that my observations are beyond the scope of it, I'm posting it here. I'm having hard time understanding what's going on, but something seems off. Like i mentioned in the PR comments, I'm using advanced networ

Re: Additional Subnets for networks and VR dhcp

2016-05-18 Thread Linas Žilinskas
woops, please ignore this. On Wed, 2016-05-18 at 15:09 +, Linas Žilinskas wrote: > This is in relation to https://github.com/apache/cloudstack/pull/1547 > But since it seems that my observations are beyond the scope of it, I'm > posting it here. > > I'm having hard time understanding what's

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-18 Thread linas-h1p
Github user linas-h1p commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-220058896 Alright I posted in dev@ regarding other issues i'm noticing (Subject: Additional Subnets for networks and VR dhcp), because it seems it doesn't apply to thi

Re: Additional Subnets for networks and VR dhcp

2016-05-18 Thread Ronald van Zantvoort
On 18/05/16 17:10, Linas Žilinskas wrote: This is in relation to https://github.com/apache/cloudstack/pull/1547 But since it seems that my observations are beyond the scope of it, I'm posting it here. I'm having hard time understanding what's going on, but something seems off. Like i mentioned i

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63722337 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-05-18 Thread dmabry
Github user dmabry commented on the pull request: https://github.com/apache/cloudstack/pull/1410#issuecomment-220065889 @swill sorry for the late reply. I pulled this patch into a 4.7 bubble and manually tested it and it worked as expected. LGTM based on manual testing. --- If your

Re: How to fix Jenkins git workspace issues

2016-05-18 Thread Rafael Weingärtner
Parallel builds will use different workspace, so that should not be a problem. Maybe the “@” symbol it uses to differentiate the workspace folders? I mean, when it runs parallel builds, it will use a “@” and then it will append a number to identify the builds. For instance, if we have workspace c

Re: How to fix Jenkins git workspace issues

2016-05-18 Thread Daan Hoogland
It also does a mvn clean so that should not be an issue. I am thinking more along the line of parallel builds on the same node. On Wed, May 18, 2016 at 5:04 PM, Rafael Weingärtner < rafaelweingart...@gmail.com> wrote: > Is Jenkins configure to clean the workspace before it starts a new build? > >

Re: How to fix Jenkins git workspace issues

2016-05-18 Thread Daan Hoogland
I hav no access to the nodes so I wouldn't know about that. On Wed, May 18, 2016 at 5:36 PM, Rafael Weingärtner < rafaelweingart...@gmail.com> wrote: > Parallel builds will use different workspace, so that should not be a > problem. > > Maybe the “@” symbol it uses to differentiate the workspace

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63731204 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF) un

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Nick LIVENS
I've digged a bit deeper, and I've got it working by explicitly loading the MySQL JDBC driver with the following code change in TransactionLegacy.java : static { try { Class.forName("com.mysql.jdbc.Driver"); } catch (ClassNotFoundException e) { throw new

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-18 Thread Simon Weller
Are you able to test this on a cleanly installed OS? It sounds like you've potentially got quite a few customizations and I think it would be helpful to see whether this can be reproduced on a clean install. From: Nick LIVENS Sent: Wednesday, May 18, 201

  1   2   >