{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible/issues/47256","repository_url":"https://api.github.com/repos/ansible/ansible","labels_url":"https://api.github.com/repos/ansible/ansible/issues/47256/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible/issues/47256/comments","events_url":"https://api.github.com/repos/ansible/ansible/issues/47256/events","html_url":"https://github.com/ansible/ansible/issues/47256","id":371350869,"node_id":"MDU6SXNzdWUzNzEzNTA4Njk=","number":47256,"title":"[RFE] Improve Foreman Dynamic Inventory by Leveraging include=all_parameters","user":{"login":"scottd018","id":13810795,"node_id":"MDQ6VXNlcjEzODEwNzk1","avatar_url":"https://avatars3.githubusercontent.com/u/13810795?v=4","gravatar_id":"","url":"https://api.github.com/users/scottd018","html_url":"https://github.com/scottd018","followers_url":"https://api.github.com/users/scottd018/followers","following_url":"https://api.github.com/users/scottd018/following{/other_user}","gists_url":"https://api.github.com/users/scottd018/gists{/gist_id}","starred_url":"https://api.github.com/users/scottd018/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scottd018/subscriptions","organizations_url":"https://api.github.com/users/scottd018/orgs","repos_url":"https://api.github.com/users/scottd018/repos","events_url":"https://api.github.com/users/scottd018/events{/privacy}","received_events_url":"https://api.github.com/users/scottd018/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-18T03:49:14Z","updated_at":"2018-10-18T03:54:15Z","closed_at":null,"author_association":"NONE","body":"##### SUMMARY\r\nNewer versions of Foreman allow for include=all_parameters, which allows us to pull parameters back in the initial API calls and parse in memory.  This prevents from having to make several subsequent API calls back to the Foreman API to pull all_parameters.  This isn't awful for small environments, but it doesn't scale well for large environments (currently testing against 1500 hosts in Foreman and it takes 3 mins).\r\n\r\n##### ISSUE TYPE\r\n- Feature Idea\r\n\r\n##### COMPONENT NAME\r\nforeman.py - dynamic inventory script improvement\r\n\r\n##### ADDITIONAL INFORMATION\r\nSee summary above.\r\n\r\nThis same concept should be applied to facts as well.  Currently, include=facts is not implemented in the Foreman API.\r\n"},"comment":{"url":"https://api.github.com/repos/ansible/ansible/issues/comments/430866923","html_url":"https://github.com/ansible/ansible/issues/47256#issuecomment-430866923","issue_url":"https://api.github.com/repos/ansible/ansible/issues/47256","id":430866923,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMDg2NjkyMw==","user":{"login":"ansibot","id":6585283,"node_id":"MDQ6VXNlcjY1ODUyODM=","avatar_url":"https://avatars0.githubusercontent.com/u/6585283?v=4","gravatar_id":"","url":"https://api.github.com/users/ansibot","html_url":"https://github.com/ansibot","followers_url":"https://api.github.com/users/ansibot/followers","following_url":"https://api.github.com/users/ansibot/following{/other_user}","gists_url":"https://api.github.com/users/ansibot/gists{/gist_id}","starred_url":"https://api.github.com/users/ansibot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ansibot/subscriptions","organizations_url":"https://api.github.com/users/ansibot/orgs","repos_url":"https://api.github.com/users/ansibot/repos","events_url":"https://api.github.com/users/ansibot/events{/privacy}","received_events_url":"https://api.github.com/users/ansibot/received_events","type":"User","site_admin":false},"created_at":"2018-10-18T03:54:15Z","updated_at":"2018-10-18T03:54:15Z","author_association":"CONTRIBUTOR","body":"Hi @scottd018, thank you for submitting this issue!\n\n[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)\n<!--- boilerplate: thank_submitter --->"}}
{"actor":{"display_login":"ansibot"}}