oamg / leapp-repository

Leapp repositories containing actors for the Leapp framework (https://github.com/oamg/leapp). Currently provides leapp repositories for in-place upgrades of RHEL systems.
Apache License 2.0
52 stars 146 forks source link

Skip checking files under .../directory-hash/ dir #1297

Open pirat89 opened 1 month ago

pirat89 commented 1 month ago

The directory has been relatively recently added to ca-certificates rpm on EL 9+ systems mostly to improve performance of OpenSSL and the content does not seem to be important for the IPU process. The high number of files takes too much time to evaluate and causes flood of logs that are not important.

This is updated solution that we drop originally: 60f500e59bb92

github-actions[bot] commented 1 month ago

Thank you for contributing to the Leapp project!

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. However, here are additional useful commands for packit:

Note that first time contributors cannot run tests automatically - they need to 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:

See 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.

packit-as-a-service[bot] commented 1 month ago

Based on your Packit configuration the settings of the @oamg/leapp Copr project would need to be updated as follows:

field old value new value
bootstrap on default

Packit was unable to update the settings above as it is missing admin permissions on the @oamg/leapp Copr project.

To fix this you can do one of the following:

Please retrigger the build, once the issue above is fixed.

pirat89 commented 1 month ago

The original solution checked the relative path of each file (one by one) instead of the directory. Seems to me that skipping the whole directory in this case it's a better option.

pirat89 commented 1 month ago

/packit copr-build

pirat89 commented 9 hours ago

@oamg/developers review please. the IPU 7 - 8 are failing due to test issue, the 8.10 -> 9.5 failed due to infra issue most likely.