)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"d264e000d625bfe0fbadc93de58274973c308665","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3216f6fe_27ec1aff","updated":"2024-03-04 16:50:30.000000000","message":"Okay, this doesn\u0027t seem to work.","commit_id":"9aea2b3f00b5a1c03c2ce74f99b3e125ae54398d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"86da902195041ff29e96d6b7aa02cee11ab3bacd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d26b10ce_8686e87b","in_reply_to":"3216f6fe_27ec1aff","updated":"2024-03-05 04:16:56.000000000","message":"Hi, thanks for your review,\n\nWith the PS3 changes, some jobs are now successful.\n\nHowever the `kuryr-kubernetes-tempest-docker` job has been still failed because `KURYR_KUBERNETES_VERSION: 1.23.16` was set. [1]\nAlso the k8s new repository provide versions starting with `1.24.0`, as described in the official documentation [2] below:\n\n\u003e The new package repositories provide downloads for Kubernetes versions starting with v1.24.0.\n\nIs it OK to change `kuryr-kubernetes-tempest-docker` to non-voting in this patch?\nIf you need to fix it, please fix with another patch.\n\nAlso, please kindly confirm the result of other non-voting failed jobs.\n\n[1] https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.zuul.d/tempest-jobs.yaml#L186\n[2] https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/change-package-repository/#before-you-begin","commit_id":"9aea2b3f00b5a1c03c2ce74f99b3e125ae54398d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"a593fb992ba9647b4a5132f1bc6b8ff58b5fa95e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c293c2ac_674e34d7","in_reply_to":"d26b10ce_8686e87b","updated":"2024-03-05 04:23:51.000000000","message":"Sorry, `PS2` is correct,\n\u003e With the PS3 changes, some jobs are now successful.","commit_id":"9aea2b3f00b5a1c03c2ce74f99b3e125ae54398d"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"f397259305c360158e93895f927075f789ab71e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d31ba5a5_012d5dd9","updated":"2024-03-05 08:06:20.000000000","message":"Docker gate is pinned to the last supported version for docker (1.23), and seems like there is no k8s packages for this version on new repo location. 1.23 is more than 2 years old now, and unsupported by the k8s team. Perhaps it\u0027ll be worth to drop docker support or adopt it to cri-dockerd - which should happen in devstack-plugin-container.","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"f08c0c495ccacff9493c0a0beab6d73d7e88e648","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"43fa8856_0975f649","in_reply_to":"1eaa6c5c_167d6134","updated":"2024-03-05 13:06:40.000000000","message":"Just to be clear - dropping it instead of making non-voting would effect in relief for infra. Non working CI have no added value, and let\u0027s face it - more than two years for transitioning from docker to other CNI is more than enough.","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"bb54b3ee21e016d8400080b51bd7d835a0fb37b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fd059d79_73a327d4","in_reply_to":"1eaa6c5c_167d6134","updated":"2024-03-05 13:05:59.000000000","message":"Thanks! I drop the docker gate in PS4.\nPlease kindly review it.","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"4c7f92bbbb13777a6c0e46be9743fe886fc8d3f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a5032134_2f59554f","in_reply_to":"5672b8c9_7e5dfd0f","updated":"2024-03-05 12:09:26.000000000","message":"Personally I\u0027d just drop it in this commit. Roman, what do you think?","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"25b5bbd5991f67b23b7c9330788df61c004d601f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1eaa6c5c_167d6134","in_reply_to":"a5032134_2f59554f","updated":"2024-03-05 13:00:44.000000000","message":"Yup, Let\u0027s do that. Otherwise we will have chicken-egg problem.","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"837c31c58e9e3174da0006b7f2bc80734fa1b24c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dd4480c4_a10b816f","in_reply_to":"d31ba5a5_012d5dd9","updated":"2024-03-05 09:21:39.000000000","message":"Why don\u0027t we just drop the docker gate then?","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"17b8d886d168f9b9aa04ed0f02f73140baafdb51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5672b8c9_7e5dfd0f","in_reply_to":"dd4480c4_a10b816f","updated":"2024-03-05 10:09:54.000000000","message":"Thanks for your comments.\n\nFor this patch, is it better to drop the docker gate,\n\n\u003e * https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.zuul.d/project.yaml#L20\n\u003e * https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.zuul.d/tempest-jobs.yaml#L180-L188\n\nor just change `voting` to false (and drop the docker gate by the another patch)?\n\n\u003e * https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.zuul.d/tempest-jobs.yaml#L187","commit_id":"7acb2be008a93dd8cf1e19f79a31a43452aac806"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"b1711844bacfe1be980f5b2cc915f4ce8edd4aca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5e08cd33_40f38732","updated":"2024-03-05 13:30:50.000000000","message":"Let\u0027s wait for the CI.","commit_id":"0c1fe5fd3c5b441c8d0b8e36997f4293ea891a27"}]}