Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions roles/os_images/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,3 +101,7 @@ os_images_hide: false

# Visibility of images ('public' requires admin rights.)
os_images_visibility: "{{ 'public' if os_images_public | bool else 'private' }}"

# Whether or not to install debootstrap on Linux distributions in the RedHat family.
# This allows you to build Debian based images on RedHat hosts. Requires EPEL.
os_images_rhel_debootstrap_enabled: "{{ os_images_install_epel_repo }}"
7 changes: 4 additions & 3 deletions roles/os_images/tasks/prechecks.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
---
# Here a quick check for the presence of Docker (or an alias for Podman)
# Display a warning if not found
# Check for the presence of Docker or Podman as this is
# required for building container based images.
# Error out if not found.
- name: Check if container engine is installed

Check warning on line 5 in roles/os_images/tasks/prechecks.yml

View workflow job for this annotation

GitHub Actions / lint / Ansible 2.20 lint

var-naming[no-role-prefix]

Variables names from within roles should use os_images_ as a prefix. (register: result)
ansible.builtin.command: which docker
ansible.builtin.shell: which docker || which podman
changed_when: false
ignore_errors: true
register: result
Expand Down
2 changes: 1 addition & 1 deletion roles/os_images/vars/RedHat.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
# List of package dependencies.
os_images_package_dependencies:
- debootstrap
- "{% if os_images_rhel_debootstrap_enabled | bool %}debootstrap{% endif %}"
- dosfstools
- gdisk
- "{% if os_images_git_elements | length > 0 %}git{% endif %}"
Expand Down
Loading