Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

systemd - use list-unit-files rather than list-units #72363

Merged
merged 1 commit into from
Oct 27, 2020

Conversation

samdoran
Copy link
Contributor

SUMMARY

Follow up fix for #71528 per this feedback from @ktdreyer.

list-units only displays units that are active, have pending jobs, or have failed. If a unit exists but is not enabled, it won't show up and the systemd module will not properly manage it.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lib/ansible/modules/systemd.py

list-unit-files will return all files on the system. list-units omits those
that are disabled.

Co-authored-by: Ken Dreyer <ktdreyer@ktdreyer.com>
@samdoran samdoran force-pushed the issue/71528-systemd-lies-more branch from c7eafed to 4cf9674 Compare October 27, 2020 21:26
@ansibot ansibot added affects_2.11 bug This issue/PR relates to a bug. core_review In order to be merged, this PR must follow the core review workflow. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team. system System category labels Oct 27, 2020
@samdoran samdoran merged commit d611588 into ansible:devel Oct 27, 2020
@samdoran samdoran deleted the issue/71528-systemd-lies-more branch October 27, 2020 21:43
samdoran added a commit to samdoran/ansible that referenced this pull request Oct 27, 2020
…nsible#72363)

list-unit-files will return all files on the system. list-units omits those
that are disabled.

Co-authored-by: Ken Dreyer <ktdreyer@ktdreyer.com>.
(cherry picked from commit d611588)

Co-authored-by: Sam Doran <sdoran@redhat.com>
samdoran added a commit to samdoran/ansible that referenced this pull request Oct 27, 2020
…sible#72363)

list-unit-files will return all files on the system. list-units omits those
that are disabled.

Co-authored-by: Ken Dreyer <ktdreyer@ktdreyer.com>
(cherry picked from commit d611588)
samdoran added a commit to samdoran/ansible that referenced this pull request Oct 27, 2020
…nsible#72363)

list-unit-files will return all files on the system. list-units omits those
that are disabled.

Co-authored-by: Ken Dreyer <ktdreyer@ktdreyer.com>
(cherry picked from commit d611588)
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Oct 29, 2020
@ansible ansible locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.11 bug This issue/PR relates to a bug. core_review In order to be merged, this PR must follow the core review workflow. module This issue/PR relates to a module. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team. system System category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants