)]}'
{"lookup/py_pkgs.py":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"e5600393c5302dd318b8ddcc0b96555e095c61e7","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        :returns: ``list``"},{"line_number":334,"context_line":"        \"\"\""},{"line_number":335,"context_line":"        _file_names \u003d list()"},{"line_number":336,"context_line":"        file_name_words \u003d [\u0027/defaults/\u0027, \u0027/group_vars/\u0027, \u0027/vars/\u0027, \u0027/user_\u0027]"},{"line_number":337,"context_line":"        file_name_words.extend(REQUIREMENTS_FILE_TYPES)"},{"line_number":338,"context_line":"        for file_name in file_names:"},{"line_number":339,"context_line":"            if file_name.endswith(ext):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_242510ac","line":336,"updated":"2017-05-19 14:14:58.000000000","message":"could you add /host_vars/ too ?","commit_id":"f8b814355d18be61188b52bdc63e940f453a8276"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"61330e60220d3f2c2831c1d64b973559aebd3f9d","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        :returns: ``list``"},{"line_number":334,"context_line":"        \"\"\""},{"line_number":335,"context_line":"        _file_names \u003d list()"},{"line_number":336,"context_line":"        file_name_words \u003d [\u0027/defaults/\u0027, \u0027/group_vars/\u0027, \u0027/vars/\u0027, \u0027/user_\u0027]"},{"line_number":337,"context_line":"        file_name_words.extend(REQUIREMENTS_FILE_TYPES)"},{"line_number":338,"context_line":"        for file_name in file_names:"},{"line_number":339,"context_line":"            if file_name.endswith(ext):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_245f5073","line":336,"in_reply_to":"ff0f0b1f_04dd6c6f","updated":"2017-05-19 14:32:14.000000000","message":"Yeah, I\u0027m trying to stick to the same precedence as Ansible.","commit_id":"f8b814355d18be61188b52bdc63e940f453a8276"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"f40cb84770c7139ba14fbec240e82677b1241c0b","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        :returns: ``list``"},{"line_number":334,"context_line":"        \"\"\""},{"line_number":335,"context_line":"        _file_names \u003d list()"},{"line_number":336,"context_line":"        file_name_words \u003d [\u0027/defaults/\u0027, \u0027/group_vars/\u0027, \u0027/vars/\u0027, \u0027/user_\u0027]"},{"line_number":337,"context_line":"        file_name_words.extend(REQUIREMENTS_FILE_TYPES)"},{"line_number":338,"context_line":"        for file_name in file_names:"},{"line_number":339,"context_line":"            if file_name.endswith(ext):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_e464b8e7","line":336,"in_reply_to":"ff0f0b1f_242510ac","updated":"2017-05-19 14:16:12.000000000","message":"The py_pkgs lookup is for global purposes. I\u0027d like to understand the use-case for their being an override in a host_var that should be applied globally.","commit_id":"f8b814355d18be61188b52bdc63e940f453a8276"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"216cc7d9ee3139447f630ff56ea552f91b1e84f9","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        :returns: ``list``"},{"line_number":334,"context_line":"        \"\"\""},{"line_number":335,"context_line":"        _file_names \u003d list()"},{"line_number":336,"context_line":"        file_name_words \u003d [\u0027/defaults/\u0027, \u0027/group_vars/\u0027, \u0027/vars/\u0027, \u0027/user_\u0027]"},{"line_number":337,"context_line":"        file_name_words.extend(REQUIREMENTS_FILE_TYPES)"},{"line_number":338,"context_line":"        for file_name in file_names:"},{"line_number":339,"context_line":"            if file_name.endswith(ext):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_04dd6c6f","line":336,"in_reply_to":"ff0f0b1f_242510ac","updated":"2017-05-19 14:20:34.000000000","message":"looks fine. Do we want group_vars to be overridden w/ vars? that list is for precedence and while I\u0027d suspect this was an intentional placement, I just want to make sure.","commit_id":"f8b814355d18be61188b52bdc63e940f453a8276"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"396fff418d6e5c9cb834f90e29bfd9d1ff225218","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        :returns: ``list``"},{"line_number":334,"context_line":"        \"\"\""},{"line_number":335,"context_line":"        _file_names \u003d list()"},{"line_number":336,"context_line":"        file_name_words \u003d [\u0027/defaults/\u0027, \u0027/group_vars/\u0027, \u0027/vars/\u0027, \u0027/user_\u0027]"},{"line_number":337,"context_line":"        file_name_words.extend(REQUIREMENTS_FILE_TYPES)"},{"line_number":338,"context_line":"        for file_name in file_names:"},{"line_number":339,"context_line":"            if file_name.endswith(ext):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_5f2e91fa","line":336,"in_reply_to":"ff0f0b1f_245f5073","updated":"2017-05-19 14:57:22.000000000","message":"@JP For that use-case the host would be overridden. This plugin is used for the repo build, so it\u0027s a global-only thing. IMO that means it\u0027s best not to include host_vars, otherwise an override for some swift host may override something for everything.","commit_id":"f8b814355d18be61188b52bdc63e940f453a8276"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"03cb2bcc489686831353d5ae6a70ecf778a3b85e","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        :returns: ``list``"},{"line_number":334,"context_line":"        \"\"\""},{"line_number":335,"context_line":"        _file_names \u003d list()"},{"line_number":336,"context_line":"        file_name_words \u003d [\u0027/defaults/\u0027, \u0027/group_vars/\u0027, \u0027/vars/\u0027, \u0027/user_\u0027]"},{"line_number":337,"context_line":"        file_name_words.extend(REQUIREMENTS_FILE_TYPES)"},{"line_number":338,"context_line":"        for file_name in file_names:"},{"line_number":339,"context_line":"            if file_name.endswith(ext):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_1f86d9dc","line":336,"in_reply_to":"ff0f0b1f_5f2e91fa","updated":"2017-05-19 14:59:46.000000000","message":"I do wonder if it might be smart to change this to /group_vars/all to ensure that we\u0027re only processing group_vars intended for everyone?","commit_id":"f8b814355d18be61188b52bdc63e940f453a8276"}]}