Closed dkubek closed 5 months ago
Please note that every PR needs to comply with the Leapp Guidelines and must pass all tests in order to be mergeable. If you want to request a review or rebuild a package in copr, you can use following commands as a comment:
Packit will automatically schedule regression tests for this PR's build and latest upstream leapp build. If you need a different version of leapp from PR#42, use /packit test oamg/leapp#42
It is possible to schedule specific on-demand tests as well. Currently 2 test sets are supported, beaker-minimal
and kernel-rt
, both can be used to be run on all upgrade paths or just a couple of specific ones.
To launch on-demand tests with packit:
kernel-rt
tests set for all upgrade pathskernel-rt
and beaker-minimal
test sets for 8.9->9.3 upgrade path[Deprecated] To launch on-demand regression testing public members of oamg organization can leave the following comment:
Please open ticket in case you experience technical problem with the CI. (RH internal only)
Note: In case there are problems with tests not being triggered automatically on new PR/commit or pending for a long time, please contact leapp-infra.
To make codespell happy we'd need to exclude the test files with --exclude-file
option https://github.com/oamg/leapp-repository/blob/master/.github/workflows/codespell.yml#L30
@dkubek update please also commit msg so it reflects all changes and problem description. I've updated the description of the PR already.
Also please rebase againt up-to-date master to fix some of crashing tests.
Original solution expected always
key: val
pair on each line. However, it has not been expected thatval
could be actually empty string. Hence the following snippet:has been parsed as:
instead of:
The new solution updates the parsing for output on RHEL 7, but also calls newly
lscpu -J
on RHEL 8+ to obtain data in the JSON format, which drops all possible parsing problems from our side.Fixes #1182