)]}'
{"playbooks/repo-build.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"30c3294d4ea4488ccd407e602bb305d10e44770c","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        module: \"add_host\""},{"line_number":29,"context_line":"        name: \"{{ item }}\""},{"line_number":30,"context_line":"        groups: repo_masters"},{"line_number":31,"context_line":"      # Process all nodes that don\u0027t match the architecture of repo_all[0]"},{"line_number":32,"context_line":"      when:"},{"line_number":33,"context_line":"        - \"inventory_hostname in groups[\u0027repo_servers_\u0027 + ansible_distribution_version + \u0027_\u0027 + ansible_architecture][0]\""},{"line_number":34,"context_line":"      with_inventory_hostnames:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_1fa239af","line":31,"range":{"start_line":31,"start_character":6,"end_line":31,"end_character":74},"updated":"2017-06-01 12:22:35.000000000","message":"This doesn\u0027t match what\u0027s happening any more. I\u0027m not sure whether doing repo[0] last is actually required, but whatever is happening should match the descriptions.","commit_id":"eb32b251dbdc0376da0d5e690d0fc86563b26243"},{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"043cf4373682bcbb781b40297ebe572d28673872","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        module: \"add_host\""},{"line_number":29,"context_line":"        name: \"{{ item }}\""},{"line_number":30,"context_line":"        groups: repo_masters"},{"line_number":31,"context_line":"      # Process all nodes that don\u0027t match the architecture of repo_all[0]"},{"line_number":32,"context_line":"      when:"},{"line_number":33,"context_line":"        - \"inventory_hostname in groups[\u0027repo_servers_\u0027 + ansible_distribution_version + \u0027_\u0027 + ansible_architecture][0]\""},{"line_number":34,"context_line":"      with_inventory_hostnames:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df140735_3f9a9d6d","line":31,"range":{"start_line":31,"start_character":6,"end_line":31,"end_character":74},"in_reply_to":"df140735_1fa239af","updated":"2017-06-01 12:23:35.000000000","message":"Done","commit_id":"eb32b251dbdc0376da0d5e690d0fc86563b26243"},{"author":{"_account_id":8662,"name":"Kyle L. Henderson","email":"kyleh@us.ibm.com","username":"kyleh"},"change_message_id":"c9220a8c0acf20e41d6dbd447a400ef6f5f8c111","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    - always"},{"line_number":38,"context_line":"    - repo-build"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# repo_all[0] is built last to ensure it has all build artifacts for final indexing"},{"line_number":41,"context_line":"#  The repo_nodes group will be populated with the first node of any other system running"},{"line_number":42,"context_line":"#  different architectures. This group is a meta group created in the previous play and"},{"line_number":43,"context_line":"#  will not be found within inventory."},{"line_number":44,"context_line":"- name: Build new repo packages for a given release"},{"line_number":45,"context_line":"  hosts: repo_masters"},{"line_number":46,"context_line":"  gather_facts: \"{{ gather_facts | default(True) }}\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df140735_e368fe1d","line":43,"range":{"start_line":40,"start_character":0,"end_line":43,"end_character":38},"updated":"2017-06-02 14:27:58.000000000","message":"This isn\u0027t the case anymore.  \n\nShould the \u0027hosts:\u0027 line below be \u0027hosts: repo_masters[1:], repo_masters[0]\u0027 to enforce the semantic described in the first line of the comment?  That first line came about when Adam originally put in the multi architecture support.\n\nAnd repo_nodes has been replaced by repo_masters.","commit_id":"b360ed43e24f9238a679ab5cbf0b3c23c1dc81c6"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"13b9c29aa273cff476a3b078f64cb5b055bdc064","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    - always"},{"line_number":38,"context_line":"    - repo-build"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# repo_all[0] is built last to ensure it has all build artifacts for final indexing"},{"line_number":41,"context_line":"#  The repo_nodes group will be populated with the first node of any other system running"},{"line_number":42,"context_line":"#  different architectures. This group is a meta group created in the previous play and"},{"line_number":43,"context_line":"#  will not be found within inventory."},{"line_number":44,"context_line":"- name: Build new repo packages for a given release"},{"line_number":45,"context_line":"  hosts: repo_masters"},{"line_number":46,"context_line":"  gather_facts: \"{{ gather_facts | default(True) }}\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df140735_4ea8d38a","line":43,"range":{"start_line":40,"start_character":0,"end_line":43,"end_character":38},"in_reply_to":"df140735_e368fe1d","updated":"2017-06-02 15:17:23.000000000","message":"Well, as it turns out, ansible does not use hosts as a list but as a set - so you can try to order in the list given but it won\u0027t respect the order. You can implement an order using \u0027order\u0027 [1] but that still takes the whole set, then applies an order.\nSo the previous impression of ordering was, in fact, a bit of a red herring.\n\nYou are right, however, this comment needs to be updated to match what\u0027s happening here.\n\n[1] \u0027http://docs.ansible.com/ansible/playbooks_intro.html","commit_id":"b360ed43e24f9238a679ab5cbf0b3c23c1dc81c6"}]}