Skip to content

Commit

Permalink
Change the platform testing package name according to the hadoop flavor
Browse files Browse the repository at this point in the history
Also, uses the cdh or hdp version number to differentiate the gobblin
build depending on the hadoop version used to compile.

PNDA-4899
  • Loading branch information
stephanesan committed Aug 30, 2018
1 parent e880296 commit 7b267b4
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 7 deletions.
6 changes: 4 additions & 2 deletions salt/gobblin/init.sls
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,11 @@
{% set gobblin_version = pillar['gobblin']['release_version'] %}

{% if grains['hadoop.distro'] == 'HDP' %}
{% set gobblin_package = 'gobblin-distribution-' + gobblin_version + '-HDP.tar.gz' %}
{% set hadoop_version = pillar['hdp']['hadoop_version'] %}
{% set gobblin_package = 'gobblin-distribution-' + gobblin_version + '-HDP-' + hadoop_version + '.tar.gz' %}
{% else %}
{% set gobblin_package = 'gobblin-distribution-' + gobblin_version + '-CDH.tar.gz' %}
{% set hadoop_version = pillar['cloudera']['hadoop_version'] %}
{% set gobblin_package = 'gobblin-distribution-' + gobblin_version + '-CDH-' + hadoop_version + '.tar.gz' %}
{% endif %}

{% set pnda_modules_version = pillar['platform_gobblin_modules']['release_version'] %}
Expand Down
12 changes: 7 additions & 5 deletions salt/platform-testing/cdh.sls
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,13 @@
{% set platform_testing_version = salt['pillar.get']('platform_testing:release_version', '0.1.1') %}
{% set platform_testing_directory = salt['pillar.get']('platform_testing:release_directory', '/opt/pnda') %}

{% set platform_testing_package = 'platform-testing-cdh' %}

{%- if grains['hadoop.distro'] == 'CDH' -%}
{% set platform_testing_service = 'cdh' %}
{% set platform_testing_package = 'platform-testing-CDH' %}
{% set cm_port = '7180' %}
{%- else -%}
{% set platform_testing_package = 'platform-testing-HDP' %}
{% set platform_testing_service = 'hdp' %}
{% set cm_port = '8080' %}
{%- endif -%}
Expand Down Expand Up @@ -84,7 +85,6 @@ platform-testing-cdh-create-venv:
- pip: python-pip-install_python_pip
- archive: platform-testing-cdh-dl-and-extract


platform-testing-cdh-create-link:
file.symlink:
- name: {{ platform_testing_directory }}/{{ platform_testing_package }}
Expand All @@ -93,15 +93,17 @@ platform-testing-cdh-create-link:
platform-testing-cdh-install-requirements-cdh:
pip.installed:
- bin_env: {{ virtual_env_dir }}
- requirements: {{ platform_testing_directory }}/{{platform_testing_package}}-{{ platform_testing_version }}/plugins/cdh/requirements.txt
- requirements: {{ platform_testing_directory }}/{{platform_testing_package}}-{{ platform_testing_version }}/plugins/{{ platform_testing_service }}/requirements.txt
- index_url: {{ pip_index_url }}
- require:
- virtualenv: platform-testing-cdh-create-venv
- onlyif:
- test -f {{ platform_testing_directory }}/{{platform_testing_package}}-{{ platform_testing_version }}/plugins/{{ platform_testing_service }}/requirements.txt

platform-testing-cdh_service:
file.managed:
- source: salt://platform-testing/templates/platform-testing-{{ platform_testing_service }}.service.tpl
- name: /usr/lib/systemd/system/platform-testing-cdh.service
- name: /usr/lib/systemd/system/platform-testing-{{ platform_testing_service }}.service
- mode: 644
- template: jinja
- context:
Expand All @@ -118,7 +120,7 @@ platform-testing-cdh-crontab-cdh:
cron.present:
- identifier: PLATFORM-TESTING-CDH
- user: root
- name: /bin/systemctl start platform-testing-cdh
- name: /bin/systemctl start platform-testing-{{ platform_testing_service }}
- require:
- pip: platform-testing-cdh-install-requirements-cdh
- file: platform-testing-cdh_service
Expand Down

0 comments on commit 7b267b4

Please sign in to comment.