linux-system-roles / nbde_client

Ansible role for configuring Network Bound Disk Encryption clients (e.g. clevis)
https://linux-system-roles.github.io/nbde_client/
MIT License
14 stars 24 forks source link

fix: facts being gathered unnecessarily #127

Closed richm closed 1 year ago

richm commented 1 year ago

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.

Signed-off-by: Rich Megginson rmeggins@redhat.com