)]}'
{"roles/prep-zanata/tasks/main.yaml":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"329bfc780648c72aae7a067fc741e11f44966938","unresolved":false,"context_lines":[{"line_number":27,"context_line":"- name: Extract cached client tarball"},{"line_number":28,"context_line":"  unarchive:"},{"line_number":29,"context_line":"    src: \"/opt/cache/files/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\""},{"line_number":30,"context_line":"    dest: \"/opt/zanata\""},{"line_number":31,"context_line":"    creates: \"/opt/zanata/zanata-cli-{{ zanata_client_version }}/bin/zanata-cli\""},{"line_number":32,"context_line":"  when: cached_client.exists"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df87a7cf_ade6bf43","line":30,"range":{"start_line":30,"start_character":10,"end_line":30,"end_character":23},"updated":"2017-12-05 15:01:30.000000000","message":"No need for double quotes here","commit_id":"3466674297c4a76af39ed3020e3903089ae4974a"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"329bfc780648c72aae7a067fc741e11f44966938","unresolved":false,"context_lines":[{"line_number":35,"context_line":"  unarchive:"},{"line_number":36,"context_line":"    src: \"https://search.maven.org/remotecontent?filepath\u003dorg/zanata/zanata-cli/{{ zanata_client_version }}/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\""},{"line_number":37,"context_line":"    remote_src: yes"},{"line_number":38,"context_line":"    dest: \"/opt/zanata/\""},{"line_number":39,"context_line":"    creates: \"/opt/zanata/zanata-cli-{{ zanata_client_version }}/bin/zanata-cli\""},{"line_number":40,"context_line":"  when: not cached_client.exists"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df87a7cf_6dd2f75e","line":38,"range":{"start_line":38,"start_character":10,"end_line":38,"end_character":24},"updated":"2017-12-05 15:01:30.000000000","message":"ditto","commit_id":"3466674297c4a76af39ed3020e3903089ae4974a"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"329bfc780648c72aae7a067fc741e11f44966938","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    remote_src: yes"},{"line_number":38,"context_line":"    dest: \"/opt/zanata/\""},{"line_number":39,"context_line":"    creates: \"/opt/zanata/zanata-cli-{{ zanata_client_version }}/bin/zanata-cli\""},{"line_number":40,"context_line":"  when: not cached_client.exists"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- name: ensure zanata-cli perms"},{"line_number":43,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df87a7cf_2df38f0c","line":40,"range":{"start_line":40,"start_character":0,"end_line":40,"end_character":32},"updated":"2017-12-05 15:01:30.000000000","message":"This is a bit of a nit and could be improved in a follow up patch..\nBut in order to be consistent with our cached sha256sum approach, we should do something like this instead:\n\n    \n    - name: Download Zanata client archive\n      get_url:\n        url: \"https://search.maven.org/remotecontent?filepath\u003dorg/zanata/zanata-cli/{{ zanata_client_version }}/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\"\n        dest: \"/tmp/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\"\n        checksum: \"sha256:{{ zanata_client_checksum }}\"\n    \n    - name: Extract Zanata client archive\n      unarchive:\n        src: \"/tmp/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\"\n        dest: /opt/zanata\n        creates: \"/opt/zanata/zanata-cli-{{ zanata_client_version }}/bin/zanata-cli\"","commit_id":"3466674297c4a76af39ed3020e3903089ae4974a"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"7c001ad4a34d0f5428cc4b4224ef3b38d97f4750","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    remote_src: yes"},{"line_number":38,"context_line":"    dest: \"/opt/zanata/\""},{"line_number":39,"context_line":"    creates: \"/opt/zanata/zanata-cli-{{ zanata_client_version }}/bin/zanata-cli\""},{"line_number":40,"context_line":"  when: not cached_client.exists"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- name: ensure zanata-cli perms"},{"line_number":43,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df87a7cf_0d1d6b0e","line":40,"range":{"start_line":40,"start_character":0,"end_line":40,"end_character":32},"in_reply_to":"df87a7cf_2df38f0c","updated":"2017-12-05 15:04:43.000000000","message":"Oh, and if we use get_url, we should have a \"retries/delay\", like this:\n\n    - name: Download Zanata client archive\n      get_url:\n        url: \"https://search.maven.org/remotecontent?filepath\u003dorg/zanata/zanata-cli/{{ zanata_client_version }}/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\"\n        dest: \"/tmp/zanata-cli-{{ zanata_client_version }}-dist.tar.gz\"\n        checksum: \"sha256:{{ zanata_client_checksum }}\"\n      register: result\n      until: result | success\n      retries: 5\n      delay: 5","commit_id":"3466674297c4a76af39ed3020e3903089ae4974a"}]}