Commit b1623087 by Michael Roytman Committed by GitHub

Merge pull request #3981 from edx/mroytman/fixup-browsers-tasks

fixup download/install logic
parents 9293145d 91b7a3ea
...@@ -44,7 +44,7 @@ ...@@ -44,7 +44,7 @@
get_url: get_url:
dest: /tmp/{{ item.name }} dest: /tmp/{{ item.name }}
url: "{{ item.url }}" url: "{{ item.url }}"
register: download_deb register: download_trusty_deb
with_items: "{{ trusty_browser_s3_deb_pkgs }}" with_items: "{{ trusty_browser_s3_deb_pkgs }}"
when: ansible_distribution_release == 'trusty' when: ansible_distribution_release == 'trusty'
tags: tags:
...@@ -55,7 +55,7 @@ ...@@ -55,7 +55,7 @@
get_url: get_url:
dest: /tmp/{{ item.name }} dest: /tmp/{{ item.name }}
url: "{{ item.url }}" url: "{{ item.url }}"
register: download_deb register: download_xenial_deb
with_items: "{{ browser_s3_deb_pkgs }}" with_items: "{{ browser_s3_deb_pkgs }}"
when: ansible_distribution_release == 'xenial' when: ansible_distribution_release == 'xenial'
tags: tags:
...@@ -65,7 +65,7 @@ ...@@ -65,7 +65,7 @@
- name: install trusty browser packages - name: install trusty browser packages
shell: gdebi -nq /tmp/{{ item.name }} shell: gdebi -nq /tmp/{{ item.name }}
with_items: "{{ trusty_browser_s3_deb_pkgs }}" with_items: "{{ trusty_browser_s3_deb_pkgs }}"
when: download_deb.changed and when: download_trusty_deb.changed and
ansible_distribution_release == 'trusty' ansible_distribution_release == 'trusty'
tags: tags:
- install - install
...@@ -74,7 +74,7 @@ ...@@ -74,7 +74,7 @@
- name: install xenial browser packages - name: install xenial browser packages
shell: gdebi -nq /tmp/{{ item.name }} shell: gdebi -nq /tmp/{{ item.name }}
with_items: "{{ browser_s3_deb_pkgs }}" with_items: "{{ browser_s3_deb_pkgs }}"
when: download_deb.changed and when: download_xenial_deb.changed and
ansible_distribution_release == 'xenial' ansible_distribution_release == 'xenial'
tags: tags:
- install - install
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment