)]}'
{"playbooks/roles/pip3/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"25bd720e721ffa66d5c2ac90415927f48dadc177","unresolved":true,"context_lines":[{"line_number":44,"context_line":"    - pip"},{"line_number":45,"context_line":"    - virtualenv"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"# test"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"cebe4582_43a23782","line":47,"updated":"2021-01-25 15:56:46.000000000","message":"Do we need this comment?","commit_id":"ba1e0335e49f3d9790a6242e917b92462ad0eb93"},{"author":{"_account_id":31905,"name":"Guillaume Chauvel","email":"guillaume.chauvel@gmail.com","username":"gchauvel"},"change_message_id":"d52ee69fbd325ab4b3fad601126a3137c949c51f","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    - pip"},{"line_number":45,"context_line":"    - virtualenv"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"# test"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c2bd7466_5cd80993","line":47,"in_reply_to":"cebe4582_43a23782","updated":"2021-01-25 17:07:44.000000000","message":"no!","commit_id":"ba1e0335e49f3d9790a6242e917b92462ad0eb93"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"a30cffb74cb084d485326029e4e4c7ebb92a4401","unresolved":true,"context_lines":[{"line_number":16,"context_line":"    state: present"},{"line_number":17,"context_line":"  when: ansible_distribution_release !\u003d \u0027xenial\u0027"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"- name: Download get-pip.py"},{"line_number":20,"context_line":"  command: wget https://bootstrap.pypa.io/get-pip.py"},{"line_number":21,"context_line":"  args:"},{"line_number":22,"context_line":"    chdir: /var/lib"},{"line_number":23,"context_line":"    creates: /var/lib/get-pip.py"},{"line_number":24,"context_line":"  when: ansible_distribution_release !\u003d \u0027xenial\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"- name: Download get-pip.py"},{"line_number":27,"context_line":"  command: wget https://bootstrap.pypa.io/3.5/get-pip.py"},{"line_number":28,"context_line":"  args:"},{"line_number":29,"context_line":"    chdir: /var/lib"},{"line_number":30,"context_line":"    creates: /var/lib/get-pip.py"},{"line_number":31,"context_line":"  when: ansible_distribution_release \u003d\u003d \u0027xenial\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"- name: Install pip"},{"line_number":34,"context_line":"  command: python3 /var/lib/get-pip.py"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3db7d7d1_8fc54e06","line":31,"range":{"start_line":19,"start_character":0,"end_line":31,"end_character":46},"updated":"2021-01-25 17:11:07.000000000","message":"Please use variables to change the url and a single task. Creating more tasks clutters the execution logs and also affects performance.\n\nUse a var that defaults to https://bootstrap.pypa.io/get-pip.py and that changes value on specific platforms.\n\nI wrote about that pattern at https://sbarnea.com/ansible/distro-vars/ -- but even a jinja2 condition would likely be acceptable.","commit_id":"664cae5421914c4867d40a780107f33419543508"},{"author":{"_account_id":31905,"name":"Guillaume Chauvel","email":"guillaume.chauvel@gmail.com","username":"gchauvel"},"change_message_id":"e11b660b0e41e1c02688f74f52e32781148c2dcf","unresolved":true,"context_lines":[{"line_number":16,"context_line":"    state: present"},{"line_number":17,"context_line":"  when: ansible_distribution_release !\u003d \u0027xenial\u0027"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"- name: Download get-pip.py"},{"line_number":20,"context_line":"  command: wget https://bootstrap.pypa.io/get-pip.py"},{"line_number":21,"context_line":"  args:"},{"line_number":22,"context_line":"    chdir: /var/lib"},{"line_number":23,"context_line":"    creates: /var/lib/get-pip.py"},{"line_number":24,"context_line":"  when: ansible_distribution_release !\u003d \u0027xenial\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"- name: Download get-pip.py"},{"line_number":27,"context_line":"  command: wget https://bootstrap.pypa.io/3.5/get-pip.py"},{"line_number":28,"context_line":"  args:"},{"line_number":29,"context_line":"    chdir: /var/lib"},{"line_number":30,"context_line":"    creates: /var/lib/get-pip.py"},{"line_number":31,"context_line":"  when: ansible_distribution_release \u003d\u003d \u0027xenial\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"- name: Install pip"},{"line_number":34,"context_line":"  command: python3 /var/lib/get-pip.py"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"df9bf902_58884732","line":31,"range":{"start_line":19,"start_character":0,"end_line":31,"end_character":46},"in_reply_to":"3db7d7d1_8fc54e06","updated":"2021-01-25 18:02:01.000000000","message":"What about doing like [1] to also include previous task \"Ensure distutils\" ?\n\n[1] https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-pip/tasks/main.yaml#L31-L42","commit_id":"664cae5421914c4867d40a780107f33419543508"}]}
