theforeman / foreman_ansible_inventory

Foreman dynamic inventory script for ansible - Now merged into Ansible itself
GNU General Public License v3.0
70 stars 31 forks source link

Fix some things to make inventory script perform better. #41

Closed wenottingham closed 7 years ago

wenottingham commented 7 years ago

Given the large number of API requests this script makes, this improves things quite a bit.

This is less of a raw performance issue, but there was not any obvious to me reason why it was rewriting the cache for every single host.

wenottingham commented 7 years ago

... if accepted, this should also go into https://github.com/ansible/ansible/pull/19510

agx commented 7 years ago

Merged. Thanks!