)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"7fa1e071bb84ac31b842fa00f46c60e3fdc7cd00","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Disable gzipping of text logs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Log collection should not attempt to gzip files as these can no longer"},{"line_number":10,"context_line":"be browsed when uploaded swift."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Compression at rest and in-transit is from now on fully handled by"},{"line_number":13,"context_line":"the log servers."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Non text files that can be big should continue to be archived."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_7e790e2e","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":16},"updated":"2020-01-21 15:26:45.000000000","message":"Note that this is only happening when logs are uploaded to swift using the upload-logs-swift [1] role from zuul-jobs.\n\nIf logs are uploaded to a logserver using the upload-logs role [2], such as what would happen in review.rdoproject.org, there is no such compression (only the console log compression is managed by the role).\n\n[1] - https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/upload-logs-swift\n[2] - https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/upload-logs","commit_id":"99ae04df7291e8dde36c842040714f18dec34b6b"}],"molecule/default/molecule.yml":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"483fcdf48a5162dbd4b57037ebafa2194786c91b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"lint:"},{"line_number":8,"context_line":"  name: yamllint"},{"line_number":9,"context_line":"platforms:"},{"line_number":10,"context_line":"  - name: centos8"},{"line_number":11,"context_line":"    image: docker.io/pycontribs/centos:8"},{"line_number":12,"context_line":"    pre_build_image: true"},{"line_number":13,"context_line":"provisioner:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_69515d63","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":17},"updated":"2020-01-21 14:15:23.000000000","message":"what about c7? we still run jobs on it","commit_id":"99ae04df7291e8dde36c842040714f18dec34b6b"},{"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":"88c71583d4788629a25fbfcb7a747c092057e888","unresolved":false,"context_lines":[{"line_number":7,"context_line":"lint:"},{"line_number":8,"context_line":"  name: yamllint"},{"line_number":9,"context_line":"platforms:"},{"line_number":10,"context_line":"  - name: centos8"},{"line_number":11,"context_line":"    image: docker.io/pycontribs/centos:8"},{"line_number":12,"context_line":"    pre_build_image: true"},{"line_number":13,"context_line":"provisioner:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_1b858c59","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":17},"in_reply_to":"3fa7e38b_69515d63","updated":"2020-01-21 14:33:34.000000000","message":"We can run both, that is not an issue but we should do this in a follow up. I used c8 because is more important than c7, and more likely to break.","commit_id":"99ae04df7291e8dde36c842040714f18dec34b6b"}],"molecule/default/prepare.yml":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"483fcdf48a5162dbd4b57037ebafa2194786c91b","unresolved":false,"context_lines":[{"line_number":26,"context_line":"      # local_working_dir is not available here"},{"line_number":27,"context_line":"      shell:"},{"line_number":28,"context_line":"        cmd: |"},{"line_number":29,"context_line":"          rm -rf ~/.quickstart/* || true"},{"line_number":30,"context_line":"        warn: false"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_69d63dc7","line":29,"range":{"start_line":29,"start_character":10,"end_line":29,"end_character":40},"updated":"2020-01-21 14:15:23.000000000","message":"this looks creepy to run on localhost, can we do it little bit more safe?\nrm doesn\u0027t require ||true","commit_id":"99ae04df7291e8dde36c842040714f18dec34b6b"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"604ebe01c5ea6fba6c07b5c1a0ee4f5527bbcfc1","unresolved":false,"context_lines":[{"line_number":26,"context_line":"      # local_working_dir is not available here"},{"line_number":27,"context_line":"      shell:"},{"line_number":28,"context_line":"        cmd: |"},{"line_number":29,"context_line":"          rm -rf ~/.quickstart/* || true"},{"line_number":30,"context_line":"        warn: false"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_74f4cdd0","line":29,"range":{"start_line":29,"start_character":10,"end_line":29,"end_character":40},"in_reply_to":"3fa7e38b_3bb08841","updated":"2020-01-21 16:46:03.000000000","message":"I mean \"rm -rf\" stuff, sova doesn\u0027t have it.","commit_id":"99ae04df7291e8dde36c842040714f18dec34b6b"},{"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":"88c71583d4788629a25fbfcb7a747c092057e888","unresolved":false,"context_lines":[{"line_number":26,"context_line":"      # local_working_dir is not available here"},{"line_number":27,"context_line":"      shell:"},{"line_number":28,"context_line":"        cmd: |"},{"line_number":29,"context_line":"          rm -rf ~/.quickstart/* || true"},{"line_number":30,"context_line":"        warn: false"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_3bb08841","line":29,"range":{"start_line":29,"start_character":10,"end_line":29,"end_character":40},"in_reply_to":"3fa7e38b_69d63dc7","updated":"2020-01-21 14:33:34.000000000","message":"Creepy to run on localhost? How about previous sova scenario that was running delegate fully on localhost?\n\nI do find the wipind of ~/.quickstart a perfectly acceptable tradeoff for testing. it is considered to be a temp directory.\n\nYep, I could remove this task but we will not be able to confirm that we ever collected something if we do not wipe it.","commit_id":"99ae04df7291e8dde36c842040714f18dec34b6b"}],"tasks/collect.yml":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"b3203ececaef408de066cbc7f07d9dc3c139ccd9","unresolved":false,"context_lines":[{"line_number":68,"context_line":"    find /tmp/{{ inventory_hostname }} -not -type f -not -type d -delete;"},{"line_number":69,"context_line":"    chown -R {{ ansible_user }}: /tmp/{{ inventory_hostname }};"},{"line_number":70,"context_line":"    {# chown can fail with: chown: invalid spec: \u00270:\u0027 #}"},{"line_number":71,"context_line":"    chown -R {{ ansible_user | default(ansible_effective_user_id) }}: /tmp/{{ inventory_hostname }} || true;"},{"line_number":72,"context_line":"  args:"},{"line_number":73,"context_line":"    executable: /bin/bash"},{"line_number":74,"context_line":"  changed_when: true"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"3fa7e38b_2abd90d6","line":71,"range":{"start_line":71,"start_character":4,"end_line":71,"end_character":108},"updated":"2020-01-21 18:26:12.000000000","message":"is it a dup of line 69?","commit_id":"9a7860ff053049bdcc4bb17baea8f71b571bda12"}]}