)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"8191564d8f4a98614c06ba4584ca270632691378","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4e88ca48_a78de987","updated":"2022-09-09 20:13:00.000000000","message":"I discover this patch today.\nI think it will probably easier for this patch to be rebased on top of 855647.\n\nI touch a little bit the user management (and user path) because it was a bit buggy with podman.","commit_id":"3ae3e8c4e95897b5a57b0e745c290be6a1eda833"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"202eaa3666585fe292013594ad100bad19ed79ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e48af773_72da5a81","in_reply_to":"4e88ca48_a78de987","updated":"2022-09-21 07:09:33.000000000","message":"Done, I\u0027ll see how CI handles it and adjust accordingly.","commit_id":"3ae3e8c4e95897b5a57b0e745c290be6a1eda833"},{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"8282361ec0af6849852667ade2f0809582855a9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"2a3a0583_9948374f","updated":"2022-10-03 11:11:57.000000000","message":"Generally great.  Good work Jiri :-)\n\nOne niggle I\u0027d like amended, and one showstopper that we cannot let through.\n\nBoth commented.","commit_id":"267d345f969714e09082be0ec36562c5157848fb"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"1da141b2e5965f320e2a359b6f679c5c8af6443d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"fb2c1626_2d9ad984","updated":"2022-10-03 07:05:22.000000000","message":"recheck - tempest issue","commit_id":"267d345f969714e09082be0ec36562c5157848fb"}],"container/validation":[{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"8282361ec0af6849852667ade2f0809582855a9b","unresolved":true,"context_lines":[{"line_number":52,"context_line":"#Setting up the default directory structure for both ansible,"},{"line_number":53,"context_line":"#and the VF"},{"line_number":54,"context_line":"RUN ln -s /usr/local/share/ansible /usr/share/ansible \u0026\u0026\\"},{"line_number":55,"context_line":"    mkdir -m 755 /home/%(user)s/validations \u0026\u0026\\"},{"line_number":56,"context_line":"    mkdir -m 755 /home/%(user)s/community-validations \u0026\u0026\\"},{"line_number":57,"context_line":"    chown -R %(uid)s:%(gid)s /home/%(user)s"},{"line_number":58,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":25,"id":"adf21c9e_ae0ed0f0","line":55,"updated":"2022-10-03 11:11:57.000000000","message":"This should probably be more generic to cater to wider variety of Unix systems.  Not all have /home, so let\u0027s make use of $HOME, and other such variables.","commit_id":"267d345f969714e09082be0ec36562c5157848fb"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"206f65cab8e96d30ffc843221cf9bbd6a7cdca77","unresolved":false,"context_lines":[{"line_number":52,"context_line":"#Setting up the default directory structure for both ansible,"},{"line_number":53,"context_line":"#and the VF"},{"line_number":54,"context_line":"RUN ln -s /usr/local/share/ansible /usr/share/ansible \u0026\u0026\\"},{"line_number":55,"context_line":"    mkdir -m 755 /home/%(user)s/validations \u0026\u0026\\"},{"line_number":56,"context_line":"    mkdir -m 755 /home/%(user)s/community-validations \u0026\u0026\\"},{"line_number":57,"context_line":"    chown -R %(uid)s:%(gid)s /home/%(user)s"},{"line_number":58,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":25,"id":"6b7a06be_41e76adf","line":55,"in_reply_to":"84085288_ead802a3","updated":"2022-10-03 12:00:25.000000000","message":"Done","commit_id":"267d345f969714e09082be0ec36562c5157848fb"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"3f2b330f729a6c06e9ae715f47f804c1925abaf1","unresolved":true,"context_lines":[{"line_number":52,"context_line":"#Setting up the default directory structure for both ansible,"},{"line_number":53,"context_line":"#and the VF"},{"line_number":54,"context_line":"RUN ln -s /usr/local/share/ansible /usr/share/ansible \u0026\u0026\\"},{"line_number":55,"context_line":"    mkdir -m 755 /home/%(user)s/validations \u0026\u0026\\"},{"line_number":56,"context_line":"    mkdir -m 755 /home/%(user)s/community-validations \u0026\u0026\\"},{"line_number":57,"context_line":"    chown -R %(uid)s:%(gid)s /home/%(user)s"},{"line_number":58,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":25,"id":"84085288_ead802a3","line":55,"in_reply_to":"adf21c9e_ae0ed0f0","updated":"2022-10-03 11:59:30.000000000","message":"Another good catch.","commit_id":"267d345f969714e09082be0ec36562c5157848fb"},{"author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"change_message_id":"8282361ec0af6849852667ade2f0809582855a9b","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                            help\u003d(\u0027User UID in container. \u0027))"},{"line_number":106,"context_line":"        parser.add_argument(\u0027--gid\u0027, \u0027-G\u0027, type\u003dint, default\u003duser_entry.pw_gid,"},{"line_number":107,"context_line":"                            help\u003d(\u0027Group UID in container. \u0027))"},{"line_number":108,"context_line":"        parser.add_argument(\u0027--image\u0027, type\u003dstr, default\u003d\u0027docker.io/redhat/ubi9\u0027,"},{"line_number":109,"context_line":"                            help\u003d\u0027Container base image. Defaults to ubi9\u0027)"},{"line_number":110,"context_line":"        parser.add_argument(\u0027--extra-pkgs\u0027, type\u003dstr, default\u003d\u0027\u0027,"},{"line_number":111,"context_line":"                            help\u003d(\u0027Extra packages to install in the container.\u0027"}],"source_content_type":"application/octet-stream","patch_set":25,"id":"54353b42_906136cb","line":108,"updated":"2022-10-03 11:11:57.000000000","message":"We can\u0027t do this.  For historical and reasons to be discussed outside of this review, we cannot link to docker.io.","commit_id":"267d345f969714e09082be0ec36562c5157848fb"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"206f65cab8e96d30ffc843221cf9bbd6a7cdca77","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                            help\u003d(\u0027User UID in container. \u0027))"},{"line_number":106,"context_line":"        parser.add_argument(\u0027--gid\u0027, \u0027-G\u0027, type\u003dint, default\u003duser_entry.pw_gid,"},{"line_number":107,"context_line":"                            help\u003d(\u0027Group UID in container. \u0027))"},{"line_number":108,"context_line":"        parser.add_argument(\u0027--image\u0027, type\u003dstr, default\u003d\u0027docker.io/redhat/ubi9\u0027,"},{"line_number":109,"context_line":"                            help\u003d\u0027Container base image. Defaults to ubi9\u0027)"},{"line_number":110,"context_line":"        parser.add_argument(\u0027--extra-pkgs\u0027, type\u003dstr, default\u003d\u0027\u0027,"},{"line_number":111,"context_line":"                            help\u003d(\u0027Extra packages to install in the container.\u0027"}],"source_content_type":"application/octet-stream","patch_set":25,"id":"967e1ab6_56a5adc5","line":108,"in_reply_to":"18cba353_fc456530","updated":"2022-10-03 12:00:25.000000000","message":"Done","commit_id":"267d345f969714e09082be0ec36562c5157848fb"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"3f2b330f729a6c06e9ae715f47f804c1925abaf1","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                            help\u003d(\u0027User UID in container. \u0027))"},{"line_number":106,"context_line":"        parser.add_argument(\u0027--gid\u0027, \u0027-G\u0027, type\u003dint, default\u003duser_entry.pw_gid,"},{"line_number":107,"context_line":"                            help\u003d(\u0027Group UID in container. \u0027))"},{"line_number":108,"context_line":"        parser.add_argument(\u0027--image\u0027, type\u003dstr, default\u003d\u0027docker.io/redhat/ubi9\u0027,"},{"line_number":109,"context_line":"                            help\u003d\u0027Container base image. Defaults to ubi9\u0027)"},{"line_number":110,"context_line":"        parser.add_argument(\u0027--extra-pkgs\u0027, type\u003dstr, default\u003d\u0027\u0027,"},{"line_number":111,"context_line":"                            help\u003d(\u0027Extra packages to install in the container.\u0027"}],"source_content_type":"application/octet-stream","patch_set":25,"id":"18cba353_fc456530","line":108,"in_reply_to":"54353b42_906136cb","updated":"2022-10-03 11:59:30.000000000","message":"Good eye. That was a leftover from development.","commit_id":"267d345f969714e09082be0ec36562c5157848fb"}]}