Richard Megginson
Richard Megginson
@felixhowe This is excellent - thanks! I would say though that this is an Ansible problem, not a system roles problem, since this issue affects every Ansible user wanting to...
> [@richm](https://github.com/richm) , I've had some time to try this out on a few more systems/distros, and I now mostly agree (though maybe conditionally - see very bottom :) )...
not sure why - but this PR makes centos-7 imuxsock test fail - the last test before this one passed - https://dl.fedoraproject.org/pub/alt/linuxsystemroles/logs/tf_logging-440_CentOS-7-latest-2.9_20250410-130923/artifacts/tests_imuxsock_files-ANSIBLE-2.9-test_playbooks_parallel-FAIL.log is the new failure - https://dl.fedoraproject.org/pub/alt/linuxsystemroles/logs/tf_logging-358_CentOS-7-latest-2.9_20250405-081532/artifacts/tests_imuxsock_files-ANSIBLE-2.9-test_playbooks_parallel-SUCCESS.log is the...
Thinking about the purge code some more, and remembering why we did this the way we did (it's been years) - I should have marked this code with `HERE BE...
> This PR doesn't fundamentally change the approach It depends on what you mean by "fundamentally". The new code will *always* remove /etc/rsyslog.conf if logging_purge_confs is true, even if it...
overall - looking much better - since we don't have a good way to test idempotency, you'll need to run the `tests_purge_reset.yml` test with the old code, then the new...
> This new version works with the tox batch tests (see #439), but it [fails with the serial tests in tmt](https://dl.fedoraproject.org/pub/alt/linuxsystemroles/logs/tf_logging-440_CentOS-Stream-10-2.17_20250416-043953/artifacts/tests_remote-ANSIBLE-2.17-test_playbooks_parallel-FAIL.log). That is is "interesting". It assumes that this: >...
> I learned my lesson, this remove behaviour is (1) dug too deeply into all the assumptions here, and (2) existing users may rely on it now. So I give...
We have no way to confirm that OEL version N is identical to RHEL version N for every supported version N. I have seen subtle differences in other roles. However,...