leapp-repository/0105-9-9-targetcontentresolver-Make-TODO-notes.patch
Matej Matuska b31dbfa161 IPU 8.10 -> 9.9: CTC2 candidate 1
- Requires leapp-framework 6.6+
- Initial implementation of upgrades on systems with configured Software RAID
- Handle multipath device identification in upgrade environment
- Fix upgrade incorrectly resuming during SELinux relabeling
- Fix upgrades on systems with multiple LUKS devices
- Fix upgrades for systems with /boot/efi on a Software RAID
- Fix mount failures for FSTAB entries with the `nofail` option specified
- Fix AVC errors triggered by registration to insight-client during the upgrade
- Install the correct kernel for the source system kernel page size on ARM systems
- Inhibit the upgrade when network devices configured with ifcfg files could depend on NM
- Introduce rhui.obsolete_gpg_keys configuration option for the removal of obsoleted RPM GPG keys of RHUI Cloud providers
- LiveMode: Fix upgrade getting stuck when the `.leapp_upgrade_failed` file exists
- LiveMode: Fix system version determination in the upgrade environment on CentOS Stream
- Fix target system initramfs containing outdated configuration files in some circumstances
- Ensure SELinux is set to permissive mode during the upgrade even when enforcing=1 is set on the kernel cmdline
- Fix source and target distribution names in /etc/migration-results
- Fix upgrades with the RealTime kernel on CentOS Stream
- Detect misconfigured kernel & systemd API mountpoints in FSTAB
- Detect misconfigured /var/run on the source system
- Unify behaviour of upgrades on systems with enabled CRB repositories and explicitly required CRB repositories by user during the upgrade
- Drop the inconsistent report about the use of CRB (unsupported by Red Hat) repositories
- Fix upgrade crashing when dnf repofiles contain URL encoded characters

- Resolves: RHEL-3289, RHEL-17842, RHEL-36249, RHEL-56040, RHEL-56176, RHEL-60060, RHEL-76846, RHEL-104384, RHEL-145136, RHEL-148631, RHEL-162192, RHEL-185508
2026-06-29 13:26:50 +02:00

53 lines
2.6 KiB
Diff

From a3a0c1d59f72c5c367b2541a510e01d08c470266 Mon Sep 17 00:00:00 2001
From: Petr Stodulka <pstodulk@redhat.com>
Date: Mon, 22 Jun 2026 15:02:05 +0200
Subject: [PATCH 105/108] [9/9] targetcontentresolver: Make TODO notes
The current refactoring progressed significantly, but there are still
anti-patterns and bugs in the code, which cannot be easily resolved
without the second part of the refactoring. Due to lack of time, we
decided to stop here, considering the situation much better in
comparison to the previous state. Making notes in the code about
additional expected changes so we do not forget them :)
Co-authored-by: Matej Matuska <mmatuska@redhat.com>
Jira: RHEL-115867
Signed-off-by: Petr Stodulka <pstodulk@redhat.com>
---
.../libraries/targetcontentresolver.py | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/repos/system_upgrade/common/actors/targetcontentresolver/libraries/targetcontentresolver.py b/repos/system_upgrade/common/actors/targetcontentresolver/libraries/targetcontentresolver.py
index 83460d99..f969d0f9 100644
--- a/repos/system_upgrade/common/actors/targetcontentresolver/libraries/targetcontentresolver.py
+++ b/repos/system_upgrade/common/actors/targetcontentresolver/libraries/targetcontentresolver.py
@@ -142,6 +142,25 @@ def process():
source repos, installed package repos, and custom/blacklisted repo
configuration.
"""
+ # TODO: update pes_even_scanner to work just with PESIDs instead of repoids.
+ # The blocklisted events needs to be then evaluated / calculated after
+ # the setuptargetrepos calculations. Similarly we can update the blocklist
+ # library. The setuptargetrepos tells exactly what's going to be allowed
+ # (repoids) and based on that we can finish the calculations.
+ # This includes:
+ # * all reports will be generated (or directly called) from the orchestrator
+ # * all/most of messages should be produced in the orchestrator
+ # * we will need at least 2 public functions in pes_events_scanner
+ # # and repositoriesblocklist libs:
+ # # * these that we have right now
+ # # * additional ones for the final calculation / filtering based on
+ # # the real list of blocklisted repoids
+ # TL;DR; the workflow will be like this:
+ # * Make interim pes-events calculation (work just with PESIDS)
+ # * Make interim blocklist calculation
+ # * Calc expected & really blocked target repositories
+ # * take the result and finish the calculation
+ # * generate reports & messages
indata = InputData()
repomap_handler = _init_repomap_handler()
blocklisted_repoids = repositoriesblocklist.compute_blocklist(
--
2.54.0