-
-
Notifications
You must be signed in to change notification settings - Fork 355
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
SystemdService.exists
fails with an error due to a malformed command
#757
Comments
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
Apr 30, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 3, 2024
This functionality from pytest-testinfra is currently broken. See pytest-dev/pytest-testinfra#757 for more details. Once pytest-dev/pytest-testinfra#754 has been merged and a new release of pytest-testinfra is created the Service.exists line can be restored.
I believe this issue is resolved by #767, which was released in version 10.1.0. |
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 28, 2024
This can be done now that pytest-dev/pytest-testinfra#767 has been merged and released in version 10.1.1: https://github.com/pytest-dev/pytest-testinfra/releases/tag/10.1.1 See also: - pytest-dev/pytest-testinfra#757 - pytest-dev/pytest-testinfra#767
7 tasks
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
May 28, 2024
This can be done now that pytest-dev/pytest-testinfra#767 has been merged and released in version 10.1.1: https://github.com/pytest-dev/pytest-testinfra/releases/tag/10.1.1 See also: - pytest-dev/pytest-testinfra#757 - pytest-dev/pytest-testinfra#767
jsf9k
added a commit
to cisagov/ansible-role-systemd-resolved
that referenced
this issue
Jun 25, 2024
This can be done now that pytest-dev/pytest-testinfra#767 has been merged and released in version 10.1.1: https://github.com/pytest-dev/pytest-testinfra/releases/tag/10.1.1 See also: - pytest-dev/pytest-testinfra#757 - pytest-dev/pytest-testinfra#767
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The command being run by
SystemdService.exists
:pytest-testinfra/testinfra/modules/service.py
Line 179 in 54f3b5c
does not run successfully as it appears to be malformed. When attempting to use it in a molecule test configuration the following error is seen:
I verified that I receive the same error if I login to a running test container:
It appears that #754 fixes this issue in addition to adding testing as verified by both the tests in that PR and my own double-check in the same molecule container:
The text was updated successfully, but these errors were encountered: