Skip to content
This repository has been archived by the owner on Mar 6, 2023. It is now read-only.

move installing selinux package dependencies into separate tasks #132

Merged
merged 1 commit into from
Jan 6, 2020

Conversation

paulfantom
Copy link
Member

Changing mechanism responsible for installing package dependencies to allow installing role on operating systems that are not yet supported (such as archlinux, PR #130).

The only dependencies are related to selinux, so new mechanism installs them only if selinux is enabled on whitelisted OS distributions. The installation mechanism is abstracted to a separate file for across-role portability reasons.

Mechanism is adapted from similar solution in kubespray.

Closing #130

@github-actions github-actions bot added area/tasks Logic behind ansible role area/vars Ansible variables used in role labels Dec 20, 2019
@paulfantom paulfantom changed the title tasks,vars: move installing selinux package dependencies into separate tasks move installing selinux package dependencies into separate tasks Dec 20, 2019
@paulfantom
Copy link
Member Author

paulfantom commented Dec 20, 2019

Idea is to sync selinux.yml file across all repos in @cloudalchemy with repo sync job.

Copy link
Collaborator

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/tasks Logic behind ansible role area/vars Ansible variables used in role
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants