)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"8a05a490c133d21b243596f3c9b9fd7189088a31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bfbd821c_35a523ba","updated":"2023-05-18 00:50:03.000000000","message":"A comment and a question.","commit_id":"0c6127d7008665e1726806de89fa3d51f499ea6d"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"b279dd9ab63571fc168f14c43118ec8f9be812e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b0bc064a_4bfdc77a","updated":"2023-05-31 15:52:12.000000000","message":"Looks good to me.","commit_id":"3ae56e2d91b5fdbc70d13c43a05392c13bb512ca"}],"playbooks/roles/install-podman/tasks/default.yaml":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"8a05a490c133d21b243596f3c9b9fd7189088a31","unresolved":true,"context_lines":[{"line_number":5,"context_line":"      - skopeo"},{"line_number":6,"context_line":"      - uidmap"},{"line_number":7,"context_line":"      - containernetworking-plugins"},{"line_number":8,"context_line":"      # TODO are these required? They seem related to unprivilged containers"},{"line_number":9,"context_line":"      # - slirp4netns"},{"line_number":10,"context_line":"      # - fuse-overlayfs"},{"line_number":11,"context_line":"    state: present"},{"line_number":12,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"e3d55179_4e644abb","line":10,"range":{"start_line":8,"start_character":0,"end_line":10,"end_character":24},"updated":"2023-05-18 00:50:03.000000000","message":"They are for rootless-containers.\n\nI admit I don\u0027t know enough to know if we need them or not","commit_id":"0c6127d7008665e1726806de89fa3d51f499ea6d"}],"playbooks/roles/install-podman/tasks/main.yaml":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"8a05a490c133d21b243596f3c9b9fd7189088a31","unresolved":true,"context_lines":[{"line_number":1,"context_line":"- name: Find distribution installation"},{"line_number":2,"context_line":"  include_tasks: \"{{ zj_distro_os }}\""},{"line_number":3,"context_line":"  with_first_found:"},{"line_number":4,"context_line":"    - \"{{ ansible_distribution }}-{{ ansible_distribution_version }}.yaml\""},{"line_number":5,"context_line":"    - \"{{ ansible_distribution }}-{{ ansible_distribution_major_version }}.yaml\""},{"line_number":6,"context_line":"    - \"{{ ansible_distribution }}.yaml\""},{"line_number":7,"context_line":"    - \"{{ ansible_os_family }}.yaml\""},{"line_number":8,"context_line":"    - \"default.yaml\""},{"line_number":9,"context_line":"  loop_control:"},{"line_number":10,"context_line":"    loop_var: zj_distro_os"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Install podman-compose"},{"line_number":13,"context_line":"  pip:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"935764a7_e151226d","line":10,"range":{"start_line":1,"start_character":0,"end_line":10,"end_character":26},"updated":"2023-05-18 00:50:03.000000000","message":"I don\u0027t know if there is a style guideline you\u0027re following here but the two supplied files for 18.04 and 20.04 seem very repetitive.\n\nIs there a reason you don\u0027t use include_vars (instead of include_tasks) and have the actual install commands here?","commit_id":"0c6127d7008665e1726806de89fa3d51f499ea6d"}]}