vars: main: Split percona packages from the galera server ones.

The role uses the 'use_percona_upstream' variable to determine if
specific external packages are going to be used instead of the distro or
upstream ones. The {qpress,percona}_package_path variables only make sense
if we are going to use such external packages so we convert their static
declaration into a 'set_fact' task. This also allows distributions to
override the default package list with an empty list if they do not plan
to use local percona packages.

Change-Id: Icb65f6c8c1654e24f73e1e5310516cd9b3568aad
This commit is contained in:
Markos Chandras
2017-05-23 22:39:07 +01:00
committed by Jesse Pretorius (odyssey4me)
parent f46e152506
commit 22a87ddf74
4 changed files with 9 additions and 3 deletions

View File

@@ -13,6 +13,10 @@
# See the License for the specific language governing permissions and # See the License for the specific language governing permissions and
# limitations under the License. # limitations under the License.
- name: Add percona external packages to the galera packages list
set_fact:
galera_install_packages_list: "{{ galera_packages_list | union(percona_packages_list) }}"
- include: "galera_install_{{ ansible_pkg_mgr }}.yml" - include: "galera_install_{{ ansible_pkg_mgr }}.yml"
tags: tags:
- install-apt - install-apt

View File

@@ -104,14 +104,14 @@
update_cache: yes update_cache: yes
cache_valid_time: "{{ (add_repos | changed) | ternary('0', cache_timeout) }}" cache_valid_time: "{{ (add_repos | changed) | ternary('0', cache_timeout) }}"
with_items: with_items:
- "{{ galera_packages_list | selectattr('enabled') | rejectattr('local_pkg') | sum(attribute='packages', start=[]) }}" - "{{ galera_install_packages_list | selectattr('enabled') | rejectattr('local_pkg') | sum(attribute='packages', start=[]) }}"
- name: Install galera_server role local packages (apt) - name: Install galera_server role local packages (apt)
apt: apt:
deb: "{{ item }}" deb: "{{ item }}"
force: yes force: yes
with_items: with_items:
- "{{ galera_packages_list | selectattr('enabled') | selectattr('local_pkg') | sum(attribute='packages', start=[]) }}" - "{{ galera_install_packages_list | selectattr('enabled') | selectattr('local_pkg') | sum(attribute='packages', start=[]) }}"
- name: Remove policy-rc - name: Remove policy-rc
file: file:

View File

@@ -96,7 +96,7 @@
name: "{{ item }}" name: "{{ item }}"
state: "{{ galera_server_package_state }}" state: "{{ galera_server_package_state }}"
with_items: with_items:
- "{{ galera_packages_list | selectattr('enabled') | sum(attribute='packages', start=[]) }}" - "{{ galera_install_packages_list | selectattr('enabled') | sum(attribute='packages', start=[]) }}"
- name: Enable mysql to start at boot - name: Enable mysql to start at boot
service: service:

View File

@@ -23,6 +23,8 @@ galera_packages_list:
- packages: "{{ galera_server_percona_distro_packages }}" - packages: "{{ galera_server_percona_distro_packages }}"
enabled: "{{ use_percona_upstream }}" enabled: "{{ use_percona_upstream }}"
local_pkg: no local_pkg: no
percona_packages_list:
- packages: "{{ [percona_package_path] }}" - packages: "{{ [percona_package_path] }}"
enabled: "{{ not use_percona_upstream }}" enabled: "{{ not use_percona_upstream }}"
local_pkg: yes local_pkg: yes