----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50396/#review143399 -----------------------------------------------------------
Fix it, then Ship it! Ship It! ambari-server/src/main/resources/custom_actions/scripts/check_host.py (line 287) <https://reviews.apache.org/r/50396/#comment209174> The error message may not be correct. Maybe use: "The MySQL JDBC driver has not been set. Please ensure that you have executed 'ambari-server setup --jdbc-db=mysql --jdbc-driver=/path/to/jdbc_driver'." ambari-server/src/main/resources/custom_actions/scripts/check_host.py (line 295) <https://reviews.apache.org/r/50396/#comment209175> The error message may not be correct. Maybe use: "The Oracle JDBC driver has not been set. Please ensure that you have executed 'ambari-server setup --jdbc-db=oracle --jdbc-driver=/path/to/jdbc_driver'. ambari-server/src/main/resources/custom_actions/scripts/check_host.py (line 303) <https://reviews.apache.org/r/50396/#comment209177> The error message may not be correct. Maybe use: "The Postgres JDBC driver has not been set. Please ensure that you have executed 'ambari-server setup --jdbc-db=postgres --jdbc-driver=/path/to/jdbc_driver'. ambari-server/src/main/resources/custom_actions/scripts/check_host.py (line 311) <https://reviews.apache.org/r/50396/#comment209178> The error message may not be correct. Maybe use: "The MSSQL JDBC driver has not been set. Please ensure that you have executed 'ambari-server setup --jdbc-db=mssql --jdbc-driver=/path/to/jdbc_driver'. ambari-server/src/main/resources/custom_actions/scripts/check_host.py (line 319) <https://reviews.apache.org/r/50396/#comment209179> The error message may not be correct. Maybe use: "The SQLAnywhere JDBC driver has not been set. Please ensure that you have executed 'ambari-server setup --jdbc-db=sqlanywhere --jdbc-driver=/path/to/jdbc_driver'. - Robert Levas On July 25, 2016, 1:18 p.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50396/ > ----------------------------------------------------------- > > (Updated July 25, 2016, 1:18 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Robert Levas, and > Sumit Mohanty. > > > Bugs: AMBARI-17889 > https://issues.apache.org/jira/browse/AMBARI-17889 > > > Repository: ambari > > > Description > ------- > > While installing a cluster, I forgot to call: > {{ambari-server setup --jdbc-db=mysql > --jdbc-driver=/usr/share/java/mysql-connector-java.jar}} > > Check DB connection failed with: > {code} > 2016-07-13 00:56:32,751 - There was an unknown error while checking database > connectivity: cannot concatenate 'str' and 'NoneType' objects > Traceback (most recent call last): > File "/var/lib/ambari-agent/cache/custom_actions/scripts/check_host.py", > line 144, in actionexecute > db_connection_check_structured_output = > self.execute_db_connection_check(config, tmp_dir) > File "/var/lib/ambari-agent/cache/custom_actions/scripts/check_host.py", > line 285, in execute_db_connection_check > jdbc_url = jdk_location + jdbc_driver_mysql_name > TypeError: cannot concatenate 'str' and 'NoneType' objects > 2016-07-13 00:56:32,752 - Check db_connection_check was unsuccessful. Exit > code: 1. Message: cannot concatenate 'str' and 'NoneType' objects > > Traceback (most recent call last): > File "/var/lib/ambari-agent/cache/custom_actions/scripts/check_host.py", > line 506, in <module> > CheckHost().execute() > File > "/usr/lib/python2.6/site-packages/resource_management/libraries/script/script.py", > line 280, in execute > method(env) > File "/var/lib/ambari-agent/cache/custom_actions/scripts/check_host.py", > line 206, in actionexecute > raise Fail(error_message) > resource_management.core.exceptions.Fail: Check db_connection_check was > unsuccessful. Exit code: 1. Message: cannot concatenate 'str' and 'NoneType' > objects > {code} > > > Diffs > ----- > > ambari-server/src/main/resources/custom_actions/scripts/check_host.py > 3d9dfa8 > > Diff: https://reviews.apache.org/r/50396/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Vitalyi Brodetskyi > >
