Ensure those two tasks are compatible with t-h-t import

In t-h-t, we call "tasks_from" in order to exclusively call those two
files in different location.
We therefore need to ensure we actually set the right variables, even in
this case.

Change-Id: Ifbccd663373f0e29301810dcc088b83d968f6203
Related-Bug: #1891465
This commit is contained in:
Cédric Jeanneret 2020-08-13 12:53:19 +02:00
parent 3ec04b419f
commit 11f365d00b
4 changed files with 41 additions and 17 deletions

View File

@ -0,0 +1,32 @@
---
# Copyright 2019 Red Hat, Inc.
# All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
# "tripleo_podman" will search for and load any operating system variable file
# found within the "vars/" path. If no OS files are found the task will skip.
# Putting this in a dedicated file allows to just include it wherever is needed.
# Please check https://bugs.launchpad.net/tripleo/+bug/1891465 for more context.
- name: Gather variables for each operating system
include_vars: "{{ item }}"
with_first_found:
- skip: true
files:
- "{{ ansible_distribution | lower }}-{{ ansible_distribution_version | lower }}.yml"
- "{{ ansible_distribution | lower }}-{{ ansible_distribution_major_version | lower }}.yml"
- "{{ ansible_os_family | lower }}-{{ ansible_distribution_major_version | lower }}.yml"
- "{{ ansible_distribution | lower }}.yml"
- "{{ ansible_os_family | lower }}-{{ ansible_distribution_version.split('.')[0] }}.yml"
- "{{ ansible_os_family | lower }}.yml"

View File

@ -14,23 +14,8 @@
# License for the specific language governing permissions and limitations # License for the specific language governing permissions and limitations
# under the License. # under the License.
- name: Load vars
# "tripleo_podman" will search for and load any operating system variable file import_tasks: load_vars.yaml
# found within the "vars/" path. If no OS files are found the task will skip.
- name: Gather variables for each operating system
include_vars: "{{ item }}"
with_first_found:
- skip: true
files:
- "{{ ansible_distribution | lower }}-{{ ansible_distribution_version | lower }}.yml"
- "{{ ansible_distribution | lower }}-{{ ansible_distribution_major_version | lower }}.yml"
- "{{ ansible_os_family | lower }}-{{ ansible_distribution_major_version | lower }}.yml"
- "{{ ansible_distribution | lower }}.yml"
- "{{ ansible_os_family | lower }}-{{ ansible_distribution_version.split('.')[0] }}.yml"
- "{{ ansible_os_family | lower }}.yml"
tags:
- always
- name: Remove docker containers - name: Remove docker containers
import_tasks: tripleo_docker_stop.yml import_tasks: tripleo_docker_stop.yml

View File

@ -14,6 +14,10 @@
# License for the specific language governing permissions and limitations # License for the specific language governing permissions and limitations
# under the License. # under the License.
- name: Load vars if needed
when: tripleo_podman_purge_packages is undefined or tripleo_podman_purge_packages|length == 0
import_tasks: load_vars.yaml
- name: Check if docker has some data - name: Check if docker has some data
stat: stat:
path: /var/lib/docker path: /var/lib/docker

View File

@ -14,6 +14,9 @@
# License for the specific language governing permissions and limitations # License for the specific language governing permissions and limitations
# under the License. # under the License.
- name: Load vars id needed
when: tripleo_podman_packages is undefined or tripleo_podman_packages|length == 0
import_tasks: load_vars.yaml
- name: Install block - name: Install block
become: true become: true