ManageIQ / manageiq-rpm_build

Code to build RPMs for ManageIQ appliances and container images
Apache License 2.0
10 stars 25 forks source link

Fix performance issue of looking up rpms #510

Closed kbrock closed 2 months ago

kbrock commented 2 months ago

Thanks @Fryguy Something in your comment triggered this thought.

Resolves performance issue introduced in #509 when it looks up all python packages associated with the system rpms. (dropping an N+1 for the win)

time VERBOSE=true ./requirements.rb requirements.txt \
     /usr/lib/python3.9/site-packages/ansible_collections/ > new_requirements.txt

Before

real    0m5.443s

After

real    0m0.778s

But it feels 20x faster