There are a few problems with the NPM commitlint
* NPM is problematic
* Having to install NPM and commitlint and dependencies is slow and heavyweight
* commitlint is overkill for our simple use case
We have our own pr_title_lint.py which is small and works exactly for our use case.
Enhance coderabbit checks for PRs and test cleanup
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
The method for determining the kernel url path for the module did not
work on all arches.
Skip the test if the kernel module is not found. Some architectures do
not provide the module.
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
We do not use molecule.
The custom requirements are no longer needed.
Remove py26 requirements
Use ansible-core - much smaller than ansible
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
The wireless test setup/cleanup had a lot of duplicated code scattered
among a few task files. This unifies the setup and cleanup code and
gets rid of redundant files.
This also enables all of the wireless tests on CentOS 7 and later,
and RHEL 8 and later.
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
Cause: The retry loop for reading from a device would bail out of the loop
before retrying because the loop should have used a greater-than-or-equal
to test the retry condition rather than a less-than.
Consequence: If the device is not stable, the loop would bail and issue
an exception rather than trying to retry the read.
Fix: Use the correct loop condition.
Result: The network role can correctly retry the read when the device
is unstable.
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
- Add info about .coderabbit.yam for AI assistants to contributing.md
- Note that CodeRabbit uses config from the main branch so this change will apply
after this PR is merged.
Signed-off-by: Sergei Petrosian <spetrosi@redhat.com>
The recently released ansible.posix 2.2.0 does not work on EL7.
Pin the version of ansible.posix to 2.1.X.
NOTE: Even though this role might not support EL7, this update
is applied to all system roles for consistency. Plus, when this
role is part of the system roles collection, all roles must use
the same version of ansible.posix - there is no way for a role
which is part of a collection to use a different version of a
dependency than the version used by the other roles.
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
