)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"19e4e2e96d4159ac7c750c6a88cbede315646422","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"276e4a31_c5121293","updated":"2023-11-02 10:27:14.000000000","message":"Okay, let\u0027s wait for the result, this should work.","commit_id":"ab1354512070e765734261589dfc5068b2bd33ad"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"2d2f99686054cca991bf25fb4c978e6cd8789716","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fdfd9153_a925c9a1","updated":"2023-11-02 14:45:25.000000000","message":"recheck failed podman pull","commit_id":"ab1354512070e765734261589dfc5068b2bd33ad"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"ff1d4c69a515cd7b745b77d77e163fff3f327d17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"adb41b16_7e81fd2a","updated":"2023-11-02 13:34:25.000000000","message":"recheck opendev docker mirror failed","commit_id":"ab1354512070e765734261589dfc5068b2bd33ad"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"ff1d4c69a515cd7b745b77d77e163fff3f327d17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"490efdaa_f3a08873","updated":"2023-11-02 13:34:25.000000000","message":"recheck pull from opendev docker mirror failed","commit_id":"ab1354512070e765734261589dfc5068b2bd33ad"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"317b73ac4d3231d50f8a10580c051ff64ec03559","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"f6175df9_bfb9f8bf","updated":"2023-11-03 14:17:26.000000000","message":"CI only and trivial","commit_id":"c5f3f23e05a901bf2663184f13c137e46d71f507"}],"tests/setup_gate.sh":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"72fc271ba2cef845a1d3af88469278eff21afd95","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    sudo mkdir -p /tmp/logs/build"},{"line_number":118,"context_line":"    sudo mkdir -p /opt/kolla_registry"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    if [[ \"${CONTAINER_ENGINE}\" \u003d\u003d \"podman\" ]]; then"},{"line_number":121,"context_line":"        sudo tee /etc/containers/registries.conf \u003c\u003cEOF"},{"line_number":122,"context_line":"unqualified-search-registries \u003d [\"localhost\",\"127.0.0.1\"]"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"cf4a306c_2a7a8c84","line":120,"updated":"2023-11-02 05:53:42.000000000","message":"shouldn\u0027t that be done via a-c-k?","commit_id":"cd8d73eed75e2759e9b68738876bde6f0513a680"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fc20720b529e76fc7b1c0b2b3443e9fee3c5cec8","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    sudo mkdir -p /tmp/logs/build"},{"line_number":118,"context_line":"    sudo mkdir -p /opt/kolla_registry"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    if [[ \"${CONTAINER_ENGINE}\" \u003d\u003d \"podman\" ]]; then"},{"line_number":121,"context_line":"        sudo tee /etc/containers/registries.conf \u003c\u003cEOF"},{"line_number":122,"context_line":"unqualified-search-registries \u003d [\"localhost\",\"127.0.0.1\"]"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"187bb801_2b5f1f8a","line":120,"in_reply_to":"0521277d_2db4d756","updated":"2023-11-02 08:23:58.000000000","message":"I sort of disagree it\u0027s an edge case - people might be using bootstrap-servers on a host they use for builds, so we should get that supported in a-c-k properly and just use those vars?","commit_id":"cd8d73eed75e2759e9b68738876bde6f0513a680"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"5c736f128ac87642c04465ad6d15a0f3ba6874b8","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    sudo mkdir -p /tmp/logs/build"},{"line_number":118,"context_line":"    sudo mkdir -p /opt/kolla_registry"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    if [[ \"${CONTAINER_ENGINE}\" \u003d\u003d \"podman\" ]]; then"},{"line_number":121,"context_line":"        sudo tee /etc/containers/registries.conf \u003c\u003cEOF"},{"line_number":122,"context_line":"unqualified-search-registries \u003d [\"localhost\",\"127.0.0.1\"]"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"e0cd62a7_87df93df","line":120,"in_reply_to":"187bb801_2b5f1f8a","updated":"2023-11-02 08:31:53.000000000","message":"They can use registry behind SSL ;) , then there is nothing to fix.\n\nOr should i just add localhost insecure:yes to a-c-k ?","commit_id":"cd8d73eed75e2759e9b68738876bde6f0513a680"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"bc39d0797bb1953bad4fd2862046a4e3f2ea38d9","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    sudo mkdir -p /tmp/logs/build"},{"line_number":118,"context_line":"    sudo mkdir -p /opt/kolla_registry"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    if [[ \"${CONTAINER_ENGINE}\" \u003d\u003d \"podman\" ]]; then"},{"line_number":121,"context_line":"        sudo tee /etc/containers/registries.conf \u003c\u003cEOF"},{"line_number":122,"context_line":"unqualified-search-registries \u003d [\"localhost\",\"127.0.0.1\"]"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"0521277d_2db4d756","line":120,"in_reply_to":"cf4a306c_2a7a8c84","updated":"2023-11-02 08:04:09.000000000","message":"Yep, it is but this is edge case where kolla-ansible review need kolla image from Depends-On: kolla change. You need to handle this, and i think it\u0027s not good to add such config to default a-c-k repo.\n\nBut good catch, i didn\u0027t realized before, i will leave config as it is and place it to conf.d directory.","commit_id":"cd8d73eed75e2759e9b68738876bde6f0513a680"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"95a1937f20c20ed560b62eeac03a5d51ec78a55b","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    sudo mkdir -p /tmp/logs/build"},{"line_number":118,"context_line":"    sudo mkdir -p /opt/kolla_registry"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    if [[ \"${CONTAINER_ENGINE}\" \u003d\u003d \"podman\" ]]; then"},{"line_number":121,"context_line":"        sudo tee /etc/containers/registries.conf \u003c\u003cEOF"},{"line_number":122,"context_line":"unqualified-search-registries \u003d [\"localhost\",\"127.0.0.1\"]"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"e8a8e08c_82e3204f","line":120,"in_reply_to":"e0cd62a7_87df93df","updated":"2023-11-02 10:56:56.000000000","message":"if we add an insecure localhost registry to a-c-k it should imho default to off|be configurable somehow, so people are not surprised.\n\nI know of people running properly certified internal registries (so maybe make the registry configurable as well? don\u0027t know the current state there), but there will be many users with the need for an insecure localhost registry I think as well.","commit_id":"cd8d73eed75e2759e9b68738876bde6f0513a680"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"107dfef3415385da6446b8b9668abe412fe8fa6a","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    sudo mkdir -p /tmp/logs/build"},{"line_number":118,"context_line":"    sudo mkdir -p /opt/kolla_registry"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    if [[ \"${CONTAINER_ENGINE}\" \u003d\u003d \"podman\" ]]; then"},{"line_number":121,"context_line":"        sudo tee /etc/containers/registries.conf \u003c\u003cEOF"},{"line_number":122,"context_line":"unqualified-search-registries \u003d [\"localhost\",\"127.0.0.1\"]"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"d5897422_51e20949","line":120,"in_reply_to":"e8a8e08c_82e3204f","updated":"2023-11-02 13:29:50.000000000","message":"Old patchset, not relevant , resolved.","commit_id":"cd8d73eed75e2759e9b68738876bde6f0513a680"}]}
