Skip to content

Commit

Permalink
feat: support openSuse Tumbleweed and Leap (#55)
Browse files Browse the repository at this point in the history
* Found a more general solution to identify rpm and deb envs
  • Loading branch information
mwester117 authored Nov 21, 2024
1 parent f5998c7 commit f032aed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions roles/install_agent/vars/main.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
# vars file for install_agent
pkg_format: "{% if ansible_facts.distribution in ['RedHat', 'CentOS', 'Fedora', 'SLES', 'Suse'] %}rpm{%
elif ansible_facts.distribution in ['Debian', 'Ubuntu'] %}deb{%
pkg_format: "{% if ansible_facts.pkg_mgr in ['yum', 'dnf', 'zypper'] %}rpm{%

Check warning on line 3 in roles/install_agent/vars/main.yml

View workflow job for this annotation

GitHub Actions / ansible-lint / Ansible Lint

jinja[spacing]

Jinja2 spacing could be improved: {% if ansible_facts.pkg_mgr in ['yum', 'dnf', 'zypper'] %}rpm{% elif ansible_facts.pkg_mgr == 'apt' %}deb{% elif ansible_facts.os_family == 'Windows' and not win_use_exe %}msi{% elif ansible_facts.os_family == 'Windows' and win_use_exe %}exe{% else %}unknown{% endif %} -> {% if ansible_facts.pkg_mgr in ['yum', 'dnf', 'zypper'] %}rpm{% elif ansible_facts.pkg_mgr == 'apt' %}deb{% elif ansible_facts.os_family == 'Windows' and not win_use_exe %}msi{% elif ansible_facts.os_family == 'Windows' and win_use_exe %}exe{% else %}unknown{% endif %}
elif ansible_facts.pkg_mgr == 'apt' %}deb{%
elif ansible_facts.os_family == 'Windows' and not win_use_exe %}msi{%
elif ansible_facts.os_family == 'Windows' and win_use_exe %}exe{%
else %}unknown{% endif %}"
Expand Down

0 comments on commit f032aed

Please sign in to comment.