)]}'
{"roles/configure-mirrors/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f140f81bb0ad3d3776778735ae44500c78c95aaa","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  become: yes"},{"line_number":28,"context_line":"  apt:"},{"line_number":29,"context_line":"    update_cache: yes"},{"line_number":30,"context_line":"  when: ansible_os_family \u003d\u003d \u0027Debian\u0027"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"7f231b9d_3a7feee0","line":30,"updated":"2017-06-16 16:48:07.000000000","message":"Why not put this in the above include instead then each distro can do the appropriate distro specific index update (if any) in their distro specific plays.","commit_id":"33b0ce06720ad62317da7d7054d850ba699ebd49"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"21ef4ab5cea347f1af6eef91a4cd088621f4576f","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  become: yes"},{"line_number":28,"context_line":"  apt:"},{"line_number":29,"context_line":"    update_cache: yes"},{"line_number":30,"context_line":"  when: ansible_os_family \u003d\u003d \u0027Debian\u0027"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"7f231b9d_5af3ca74","line":30,"in_reply_to":"7f231b9d_3a7feee0","updated":"2017-06-16 16:52:04.000000000","message":"Sure, we could (I initiall did this). Right now we only need to do it for debian / ubuntu. Yum / dnf will ensure indexes are first updated.","commit_id":"33b0ce06720ad62317da7d7054d850ba699ebd49"}],"roles/configure-mirrors/vars/ubuntu.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f140f81bb0ad3d3776778735ae44500c78c95aaa","unresolved":false,"context_lines":[{"line_number":1,"context_line":"nodepool_wheel_mirror: \"http://{{ nodepool_mirror_host }}/wheel/{{ ansible_distribution | lower }}-{{ ansible_distribution_version }}-{{ ansible_architecture | lower }}\""},{"line_number":2,"context_line":"nodepool_package_mirror: \"http://{{ nodepool_mirror_host }}/ubuntu\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"7f231b9d_3a0c8ea3","line":2,"updated":"2017-06-16 16:48:07.000000000","message":"Reading this it might be more clear to drop the nodepool prefix (we aren\u0027t mirroring nodepool) and be as specific as possible:\n\n  python_wheel_mirror\n  ubuntu_package_mirror\n\nWe don\u0027t need the distro specific var to be the same in all cases since we are using a distro specific play and file template(s).","commit_id":"33b0ce06720ad62317da7d7054d850ba699ebd49"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"21ef4ab5cea347f1af6eef91a4cd088621f4576f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"nodepool_wheel_mirror: \"http://{{ nodepool_mirror_host }}/wheel/{{ ansible_distribution | lower }}-{{ ansible_distribution_version }}-{{ ansible_architecture | lower }}\""},{"line_number":2,"context_line":"nodepool_package_mirror: \"http://{{ nodepool_mirror_host }}/ubuntu\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"7f231b9d_da8b5ac0","line":2,"in_reply_to":"7f231b9d_3a0c8ea3","updated":"2017-06-16 16:52:04.000000000","message":"Sure","commit_id":"33b0ce06720ad62317da7d7054d850ba699ebd49"}]}