Bug 2223032 - facts being gathered unnecessarily [NEEDINFO]
Summary: facts being gathered unnecessarily
Keywords:
Status: ON_QA
Alias: None
Product: Red Hat Enterprise Linux 9
Classification: Red Hat
Component: rhel-system-roles
Version: 9.3
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: 9.3
Assignee: Rich Megginson
QA Contact: Jakub Haruda
URL:
Whiteboard: role:ALL
Depends On:
Blocks: 2223036
TreeView+ depends on / blocked
 
Reported: 2023-07-14 21:45 UTC by Rich Megginson
Modified: 2023-08-10 13:48 UTC (History)
4 users (show)

Fixed In Version: rhel-system-roles-1.22.0-0.16.el9
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.
Clone Of:
: 2223036 (view as bug list)
Environment:
Last Closed:
Type: ---
Target Upstream Version:
Embargoed:
rmeggins: needinfo? (djez)
rmeggins: needinfo? (vdanek)

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHELPLAN-162425 0 None None None 2023-07-14 21:53:54 UTC

Description Rich Megginson 2023-07-14 21:45:31 UTC 
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:24:56 UTC 
Here is the script which can check logs https://github.com/linux-system-roles/auto-maintenance/pull/286
Note You need to log in before you can comment on or make changes to this bug.