Open danzatt opened 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, e.g. from PR#42, use /packit test oamg/leapp#42
Note that first time contributors cannot run tests automatically - they will be started by a reviewer.
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.10->9.4 upgrade pathSee other labels for particular jobs defined in the .packit.yaml
file.
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.
@danzatt Hi Dan \o most likely I will get to the review during early May or later June. We are dealing now with additional stuff.
/packit test
@danzatt I haven't went through the whole code yet, but covered most of it. I found some things that could be changed, and some that needs to be changed. I do not expect I will find anything else in the rest of the code (and not sure when I will get to it), but i am letting you know about that in advance, in case you would like to wait for the full review.
Hello @pirat89 thanks for the review! I've hopefully addressed all your remarks now.
/packit copr-build
/packit copr-build
/packit copr-build
tldr; Add a
LuksScanner
actor which scans all crypt devices usingcryptsetup luksDump
. Don't inhibit, when all devices are LUKS2 with clevs TPM2 token.jira: RHEL-3294
This PR introduces new shortened URLs:
/cc @pirat89