Jose/ansible_proxmox_WOL
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

refactor ♻️: Refactor ExecStart to use a list for commands and conditionally execute them.
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 9s
Details

Browse Source 
This refactoring involves restructuring the `ExecStart` method to utilize a list of commands instead of a single string. This change allows for more flexible execution based on certain conditions, enhancing the modularity and maintainability of the code.
This commit is contained in:
Jose
2025-12-26 10:45:26 +01:00
parent 942625f894
commit f11ad0badc
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
templates/wol-interfaces.service.j2
View File

@@ -5,8 +5,13 @@ Wants=network.target
[Service] [Service]
Type=oneshot Type=oneshot
ExecStart=/bin/bash -c '{% for intf, enabled, supported in en_interfaces | zip(wol_enabled.results, wol_supported.results) %} ExecStart=/bin/bash -c '{% set cmds = [] %}
{% if wol_mode in supported.stdout and wol_mode not in enabled.stdout %}/sbin/ethtool -s {{ intf }} wol {{ wol_mode }}; {% endif %}{% endfor %}' {% for intf, enabled, supported in en_interfaces | zip(wol_enabled.results, wol_supported.results) %}
{% if wol_mode in supported.stdout and wol_mode not in enabled.stdout %}
{% set _ = cmds.append("/sbin/ethtool -s " ~ intf ~ " wol " ~ wol_mode) %}
{% endif %}
{% endfor %}
{{ cmds | join("; ") if cmds | length > 0 else ":" }}'
RemainAfterExit=yes RemainAfterExit=yes
[Install] [Install]