Skip to content

Commit

Permalink
refactor: remove JDBC connector install and put it in tdp-prerequisits
Browse files Browse the repository at this point in the history
  • Loading branch information
SteBaum committed Aug 5, 2024
1 parent 7db0dd8 commit fde5908
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 19 deletions.
5 changes: 0 additions & 5 deletions roles/hive/metastore/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,6 @@
name: tosit.tdp.hive.common
tasks_from: install_hive

- name: Install jdbc connector
ansible.builtin.package:
name: "{{ hive_jdbc_connector_package }}"
state: present

- name: Create a symbolic link to jdbc.jar
ansible.builtin.file:
src: /usr/share/java/{{ hive_jdbc_connector_package }}.jar
Expand Down
5 changes: 0 additions & 5 deletions roles/ranger/admin/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,6 @@
owner: root
mode: "770"

- name: Install jdbc connector
ansible.builtin.package:
name: "{{ ranger_jdbc_connector_package }}"
state: present

- name: Template Ranger Admin service file
ansible.builtin.template:
src: ranger-admin.service.j2
Expand Down
5 changes: 0 additions & 5 deletions roles/ranger/kms/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,11 +79,6 @@
owner: root
mode: "770"

- name: Install jdbc connector
ansible.builtin.package:
name: "{{ ranger_jdbc_connector_package }}"
state: present

- name: Template Ranger KMS service file
ansible.builtin.template:
src: ranger-kms.service.j2
Expand Down
2 changes: 1 addition & 1 deletion tdp_vars_defaults/hive/hive.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ hadoop_conf_dir: /etc/hadoop/conf
java_home: /usr/lib/jvm/jre-1.8.0-openjdk
hdfs_user: hdfs

# JDBC connector to be installed
# JDBC connector to be used
hive_jdbc_connector_package: mysql-connector-java

# Hive Metastore database properties
Expand Down
3 changes: 0 additions & 3 deletions tdp_vars_defaults/ranger/ranger.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,6 @@ ranger_keystore_password: Keystore123!
ranger_truststore_location: /etc/ssl/certs/truststore.jks
ranger_truststore_password: Truststore123!

# JDBC connector to be installed
ranger_jdbc_connector_package: mysql-connector-java

# Ranger Admin install.properties
install_properties:
setup_mode: SeparateDBA
Expand Down

0 comments on commit fde5908

Please sign in to comment.