Bug 2223036

Summary: facts being gathered unnecessarily
Product: Red Hat Enterprise Linux 8 Reporter: Rich Megginson <rmeggins>
Component: rhel-system-rolesAssignee: Rich Megginson <rmeggins>
Status: CLOSED ERRATA QA Contact: David Jež <djez>
Severity: unspecified Docs Contact: Lucie Vařáková <lmanasko>
Priority: unspecified    
Version: 8.9CC: djez, jharuda, lmanasko, rhel-cs-system-management-subsystem-qe, spetrosi, vdanek
Target Milestone: rcKeywords: Triaged
Target Release: 8.9Flags: pm-rhel: mirror+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: role:ALL
Fixed In Version: rhel-system-roles-1.22.0-0.16.el8 Doc Type: Bug Fix
Doc Text:
Cause: The comparison of the present facts with the required facts is being done on unsorted lists. Consequence: The comparison may fail if the only difference is the order. Facts are gathered unnecessarily. Fix: Use difference which works no matter what the order is. Ensure that the fact gathering subsets used are the absolute minimum required. Result: The role gathers only the facts it requires, and does not unnecessarily gather facts.
Story Points: ---
Clone Of: 2223032 Environment:
Last Closed: 2023-11-14 15:31:21 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2223032    
Bug Blocks:    
Deadline: 2023-08-21   

Description Rich Megginson 2023-07-14 21:57:58 UTC
+++ This bug was initially created as a clone of Bug #2223032 +++

Cause: The comparison of the present facts with the required facts is
being done on unsorted lists.

Consequence: The comparison may fail if the only difference is the
order.  Facts are gathered unnecessarily.

Fix: Use `difference` which works no matter what the order is.  Ensure
that the fact gathering subsets used are the absolute minimum required.

Result: The role gathers only the facts it requires, and does
not unnecessarily gather facts.

original issue https://github.com/linux-system-roles/storage/issues/373
storage fix https://github.com/linux-system-roles/storage/pull/374

Comment 5 Rich Megginson 2023-07-19 18:25:28 UTC
Here is the script which can check logs https://github.com/linux-system-roles/auto-maintenance/pull/286

Comment 11 errata-xmlrpc 2023-11-14 15:31:21 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (rhel-system-roles bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2023:6946

Comment 12 Red Hat Bugzilla 2024-03-14 04:26:03 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days