)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e43535d69252f7bd5e0460a643e5b9fc94b4c257","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"957d2af0_f47edecf","updated":"2023-05-22 11:09:41.000000000","message":"I\u0027m not too happy with the reordering of the when: things, maybe we can consider ignoring that rule for now? for osism we made a custom module to keep some nicer ordering, but that would be a discussion for the next PTG","commit_id":"de4298b74cda4803bbb37d24e9730de762629e28"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"d1cec4930c90cc60670eb83487efacf1719e9d2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"72ab85a0_269d10cb","updated":"2023-05-22 12:15:37.000000000","message":"ok, linters passing now","commit_id":"239b08c74d5f117e927011c2541876b442e7787d"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"43b9584cb3dcea3467539d87511a2b652d5f2404","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"72963d56_68528210","updated":"2023-05-22 14:12:40.000000000","message":"Let\u0027s clean up in a followup, this has had enough iterations","commit_id":"b4f1d3c99350a827693ea23d2709742480c04773"}],"roles/apparmor_libvirt/tasks/remove-profile.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d3aa45e0d70142f414de37fae9cb856a364f8db1","unresolved":true,"context_lines":[{"line_number":10,"context_line":"  register: apparmor_libvirtd_disable_profile"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Remove apparmor profile for libvirt"},{"line_number":13,"context_line":"  # noqa no-changed-when"},{"line_number":14,"context_line":"  shell: |"},{"line_number":15,"context_line":"    apparmor_parser -v -R /etc/apparmor.d/usr.sbin.libvirtd \u0026\u0026 \\"},{"line_number":16,"context_line":"    ln -vsf /etc/apparmor.d/usr.sbin.libvirtd /etc/apparmor.d/disable"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"b17aa68e_b4b6269e","line":13,"updated":"2023-05-22 12:54:34.000000000","message":"this one\u0027s as easy to fix as it is to ignore 😊","commit_id":"239b08c74d5f117e927011c2541876b442e7787d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"0635c4c8e1311d86a893f0d5156afb1672c4b6ba","unresolved":false,"context_lines":[{"line_number":10,"context_line":"  register: apparmor_libvirtd_disable_profile"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Remove apparmor profile for libvirt"},{"line_number":13,"context_line":"  # noqa no-changed-when"},{"line_number":14,"context_line":"  shell: |"},{"line_number":15,"context_line":"    apparmor_parser -v -R /etc/apparmor.d/usr.sbin.libvirtd \u0026\u0026 \\"},{"line_number":16,"context_line":"    ln -vsf /etc/apparmor.d/usr.sbin.libvirtd /etc/apparmor.d/disable"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"f028795f_47d27de6","line":13,"in_reply_to":"b17aa68e_b4b6269e","updated":"2023-05-22 12:55:46.000000000","message":"Done","commit_id":"239b08c74d5f117e927011c2541876b442e7787d"}],"roles/docker/tasks/install.yml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e43535d69252f7bd5e0460a643e5b9fc94b4c257","unresolved":true,"context_lines":[{"line_number":67,"context_line":"  when:"},{"line_number":68,"context_line":"    - docker_install_result is changed"},{"line_number":69,"context_line":"    - running_containers.rc \u003d\u003d 0"},{"line_number":70,"context_line":"    - running_containers.stdout !\u003d \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"cb2bb40d_70751729","side":"PARENT","line":70,"updated":"2023-05-22 11:09:41.000000000","message":"Is dropping this last condition intentional?","commit_id":"f74dd4859c9f95b555dba3ba4e41d2126ba2ec22"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"2fdbfae102ec20773dd91a69e4d6f27b2fdc7392","unresolved":false,"context_lines":[{"line_number":67,"context_line":"  when:"},{"line_number":68,"context_line":"    - docker_install_result is changed"},{"line_number":69,"context_line":"    - running_containers.rc \u003d\u003d 0"},{"line_number":70,"context_line":"    - running_containers.stdout !\u003d \u0027\u0027"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"29fcc589_5fa5e60e","side":"PARENT","line":70,"in_reply_to":"cb2bb40d_70751729","updated":"2023-05-22 11:43:36.000000000","message":"Done","commit_id":"f74dd4859c9f95b555dba3ba4e41d2126ba2ec22"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d3aa45e0d70142f414de37fae9cb856a364f8db1","unresolved":true,"context_lines":[{"line_number":62,"context_line":"      delay: 10"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    - name: Ensure containers are running after Docker upgrade"},{"line_number":65,"context_line":"      # noqa no-changed-when"},{"line_number":66,"context_line":"      command: \"docker start {{ running_containers.stdout }}\""},{"line_number":67,"context_line":"      become: true"},{"line_number":68,"context_line":"  when:"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"a9e626a8_a947f5c3","line":65,"updated":"2023-05-22 12:54:34.000000000","message":"ditto","commit_id":"239b08c74d5f117e927011c2541876b442e7787d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"0635c4c8e1311d86a893f0d5156afb1672c4b6ba","unresolved":false,"context_lines":[{"line_number":62,"context_line":"      delay: 10"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    - name: Ensure containers are running after Docker upgrade"},{"line_number":65,"context_line":"      # noqa no-changed-when"},{"line_number":66,"context_line":"      command: \"docker start {{ running_containers.stdout }}\""},{"line_number":67,"context_line":"      become: true"},{"line_number":68,"context_line":"  when:"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"4889603a_480806f8","line":65,"in_reply_to":"a9e626a8_a947f5c3","updated":"2023-05-22 12:55:46.000000000","message":"Done","commit_id":"239b08c74d5f117e927011c2541876b442e7787d"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9c548ad54200f5983f0607dfc4200173b29de8d4","unresolved":true,"context_lines":[{"line_number":62,"context_line":"      delay: 10"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    - name: Ensure containers are running after Docker upgrade"},{"line_number":65,"context_line":"      # noqa no-changed-when"},{"line_number":66,"context_line":"      command: \"docker start {{ running_containers.stdout }}\""},{"line_number":67,"context_line":"      become: true"},{"line_number":68,"context_line":"      changed_when: true"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"3c873090_a6ffe6d4","line":65,"updated":"2023-05-22 14:11:41.000000000","message":"You don\u0027t need this exclude any more","commit_id":"b4f1d3c99350a827693ea23d2709742480c04773"}]}