[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1610 Done #1621 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1610 Good catch and indeed a small fix. @milamberspace can you open a new PR to fix this for 4.9.1? Would like to have the credits for the commit go to you :) --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1610 @rhtyd I've tested with a Ubuntu 14.04 / CS 4.7.1->4.9.0 (on virtual CS-over-CS): works fine. But with a Ubuntu 14.04 from CS 4.4.0->4.5.1->4.7.0->4.7.1->4.9.0 (real machines): found this issue. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1610 @milamberspace I tested upgrade scenario using debian packages, so at least this works for debian mgmt server hosts. I think you hit the issue with CentOS based mgmt server host? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1610 The workaround is to edit the db.properties file and insert a carriage return before "db.cloud.driver=jdbc:mysql" and restart the cloudstack-management service --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1610 @wido This PR has been include with the latest CS 4.9.0, unfortunately if the last line of the db.properties file hasn't a return line (i.e. the last line isn't an empty line), the first line of your patch (db.cloud.driver=jdbc:mysql) will be concatenate to the last line: example: db.cloud.secondsBeforeRetryMaster=3600db.cloud.driver=jdbc:mysql db.usage.driver=jdbc:mysql db.simulator.driver=jdbc:mysql And after the management service don't start correctly. Probably need to be fix in the 4.9.1 and a mention need to be had in the Release Notes. For trace (and google search), the error messages: 2016-07-31 15:05:10,684 ERROR [c.c.u.d.DriverLoader] (main:null) DB driver type null is not supported! 2016-07-31 15:05:10,709 WARN [c.c.u.d.T.Transaction] (main:null) Unable to load db configuration, using defaults with 5 connections. Falling back on assumed datasource on localhost:3306 using username:password=cloud:cloud. Please check your configuration com.cloud.utils.exception.CloudRuntimeException: DB driver type null is not supported! [...] 2016-07-31 15:05:10,979 ERROR [c.c.u.d.Merovingian2] (main:null) Unable to get a new db connection java.sql.SQLException: Access denied for user 'cloud'@'localhost' (using password: YES) [...] --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1610 Thx. I will get this merged... --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1610 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 3h 56m 40s ``` **Associated Uploads** **`/tmp/MarvinLogs/DeployDataCenter__Jul_14_2016_15_30_56_L489PO:`** * [dc_entries.obj](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/DeployDataCenter__Jul_14_2016_15_30_56_L489PO/dc_entries.obj) * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/DeployDataCenter__Jul_14_2016_15_30_56_L489PO/failed_plus_exceptions.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/DeployDataCenter__Jul_14_2016_15_30_56_L489PO/runinfo.txt) **`/tmp/MarvinLogs/test_network_885HWT:`** * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/test_network_885HWT/failed_plus_exceptions.txt) * [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/test_network_885HWT/results.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/test_network_885HWT/runinfo.txt) **`/tmp/MarvinLogs/test_vpc_routers_M5IZWP:`** * [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/test_vpc_routers_M5IZWP/failed_plus_exceptions.txt) * [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/test_vpc_routers_M5IZWP/results.txt) * [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1610/tmp/MarvinLogs/test_vpc_routers_M5IZWP/runinfo.txt) Uploads will be available until `2016-09-13 02:00:00 +0200 CEST` *Comment created by [`upr comment`](https://github.com/cloudops/upr).* --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1610 ð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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1610 Ah, good point @rhtyd! Added it to CentOS 6 as well. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1610 @wido lgtm, though we would also want the same change in the centos6's spec file. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---