)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"5fccf9972a9870a1d41172c5223f7217e35a4403","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8d973798_ae9a20b1","updated":"2023-05-22 08:49:26.000000000","message":"recheck node timeout","commit_id":"1bda41a8e86899debb89b74e91f4c8213978b74a"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"127393894f048e5637f660e402306f7ea0de69c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"c8b74096_25d0470d","updated":"2023-05-26 16:15:59.000000000","message":"I left a nit regarding the README.est file, but it looks good to me","commit_id":"4e6e8f0c0e6db892f5f60437541531f84aabc7cb"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6f46a531d8b87680637aafa005bd51bc01b5cb3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"6d220c85_d7ad2d35","updated":"2023-08-22 20:37:21.000000000","message":"LGTM, just a question about a periodic image build job","commit_id":"4e6e8f0c0e6db892f5f60437541531f84aabc7cb"}],"devstack/plugin.sh":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"9d6d7203fec101a1d7902ad8dd028e2190fb4671","unresolved":true,"context_lines":[{"line_number":19,"context_line":"            if [[ ${DISTRO} \u003d~ (rhel|centos) ]]; then"},{"line_number":20,"context_line":"                install_package qemu-kvm"},{"line_number":21,"context_line":"                if [[ \"$OCTAVIA_AMP_BASE_OS\" \u003d\u003d \"rocky\" ]]; then"},{"line_number":22,"context_line":"                    install_package podman"},{"line_number":23,"context_line":"                fi"},{"line_number":24,"context_line":"            else"},{"line_number":25,"context_line":"                install_package qemu"}],"source_content_type":"text/x-sh","patch_set":9,"id":"06afb00e_74d79a6b","line":22,"range":{"start_line":22,"start_character":20,"end_line":22,"end_character":42},"updated":"2023-05-22 09:03:33.000000000","message":"It would be good to add an explaination why this is needed for rocky. Octavia doesn\u0027t require podman on the amp image or does it?","commit_id":"1bda41a8e86899debb89b74e91f4c8213978b74a"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9e9b0997787d4904b76aaa3c7368601e91ba4f2d","unresolved":true,"context_lines":[{"line_number":19,"context_line":"            if [[ ${DISTRO} \u003d~ (rhel|centos) ]]; then"},{"line_number":20,"context_line":"                install_package qemu-kvm"},{"line_number":21,"context_line":"                if [[ \"$OCTAVIA_AMP_BASE_OS\" \u003d\u003d \"rocky\" ]]; then"},{"line_number":22,"context_line":"                    install_package podman"},{"line_number":23,"context_line":"                fi"},{"line_number":24,"context_line":"            else"},{"line_number":25,"context_line":"                install_package qemu"}],"source_content_type":"text/x-sh","patch_set":9,"id":"66d6959d_75a2654f","line":22,"range":{"start_line":22,"start_character":20,"end_line":22,"end_character":42},"in_reply_to":"06afb00e_74d79a6b","updated":"2023-05-23 07:50:45.000000000","message":"done (note: podman is not required in the amphora, but it is required on the host that builds the image)","commit_id":"1bda41a8e86899debb89b74e91f4c8213978b74a"}],"diskimage-create/README.rst":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"127393894f048e5637f660e402306f7ea0de69c4","unresolved":true,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"   $ sudo apt install qemu-utils git kpartx debootstrap"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"Fedora, CentOS and Red Hat Enterprise Linux"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":".. code:: bash"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"06a184f8_5d1dbb05","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":43},"updated":"2023-05-26 16:15:59.000000000","message":"nit: rocky?\nJust saying they appear a few times in this file without \"rocky\"","commit_id":"4e6e8f0c0e6db892f5f60437541531f84aabc7cb"}],"diskimage-create/diskimage-create.sh":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"17fb8587c51fcaf8e80e52a83afb6a0ef2f38c10","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    echo \"            [-f]\""},{"line_number":29,"context_line":"    echo \"            [-g **repository branch** | stable/train | stable/stein | ... ]\""},{"line_number":30,"context_line":"    echo \"            [-h]\""},{"line_number":31,"context_line":"    echo \"            [-i **ubuntu-minimal** | fedora | centos-minimal | rhel | rocky ]\""},{"line_number":32,"context_line":"    echo \"            [-k \u003ckernel package name\u003e ]\""},{"line_number":33,"context_line":"    echo \"            [-l \u003clog file\u003e ]\""},{"line_number":34,"context_line":"    echo \"            [-m]\""}],"source_content_type":"text/x-sh","patch_set":9,"id":"ed7a21f9_a94ec2da","line":31,"range":{"start_line":31,"start_character":80,"end_line":31,"end_character":85},"updated":"2023-05-22 09:14:30.000000000","message":"We should add release notes for this and maybe also mention that OCTAVIA_AMP_BASE_OS\u003d\"rocky\" and OCTAVIA_AMP_DISTRIBUTION_RELEASE_ID\u003d\"9\" in local.conf will build a rocky amp image in devstack.","commit_id":"1bda41a8e86899debb89b74e91f4c8213978b74a"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9e9b0997787d4904b76aaa3c7368601e91ba4f2d","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    echo \"            [-f]\""},{"line_number":29,"context_line":"    echo \"            [-g **repository branch** | stable/train | stable/stein | ... ]\""},{"line_number":30,"context_line":"    echo \"            [-h]\""},{"line_number":31,"context_line":"    echo \"            [-i **ubuntu-minimal** | fedora | centos-minimal | rhel | rocky ]\""},{"line_number":32,"context_line":"    echo \"            [-k \u003ckernel package name\u003e ]\""},{"line_number":33,"context_line":"    echo \"            [-l \u003clog file\u003e ]\""},{"line_number":34,"context_line":"    echo \"            [-m]\""}],"source_content_type":"text/x-sh","patch_set":9,"id":"3ac06d7d_73fa1ae3","line":31,"range":{"start_line":31,"start_character":80,"end_line":31,"end_character":85},"in_reply_to":"ed7a21f9_a94ec2da","updated":"2023-05-23 07:50:45.000000000","message":"done, thanks","commit_id":"1bda41a8e86899debb89b74e91f4c8213978b74a"}],"zuul.d/jobs.yaml":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6f46a531d8b87680637aafa005bd51bc01b5cb3a","unresolved":true,"context_lines":[{"line_number":183,"context_line":"        amphora_os_release: 9-stream"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"- job:"},{"line_number":186,"context_line":"    name: octavia-amphora-image-build-live-rocky-9"},{"line_number":187,"context_line":"    parent: octavia-amphora-image-build"},{"line_number":188,"context_line":"    nodeset: centos-9-stream"},{"line_number":189,"context_line":"    description: |"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"58ba26b2_9ca31f67","line":186,"updated":"2023-08-22 20:37:21.000000000","message":"Should we have a periodic job for this?","commit_id":"4e6e8f0c0e6db892f5f60437541531f84aabc7cb"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"50d8d8d0151b55d77b1fa8d56d82d8dcac1dd7ee","unresolved":true,"context_lines":[{"line_number":183,"context_line":"        amphora_os_release: 9-stream"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"- job:"},{"line_number":186,"context_line":"    name: octavia-amphora-image-build-live-rocky-9"},{"line_number":187,"context_line":"    parent: octavia-amphora-image-build"},{"line_number":188,"context_line":"    nodeset: centos-9-stream"},{"line_number":189,"context_line":"    description: |"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"858f7a0f_02578f1c","line":186,"in_reply_to":"58ba26b2_9ca31f67","updated":"2023-08-24 12:09:49.000000000","message":"and publishing the image to https://tarballs.opendev.org/openstack/octavia/test-images/?","commit_id":"4e6e8f0c0e6db892f5f60437541531f84aabc7cb"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"5d997d498246eccfe92eee7259df96985c10aa8a","unresolved":true,"context_lines":[{"line_number":183,"context_line":"        amphora_os_release: 9-stream"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"- job:"},{"line_number":186,"context_line":"    name: octavia-amphora-image-build-live-rocky-9"},{"line_number":187,"context_line":"    parent: octavia-amphora-image-build"},{"line_number":188,"context_line":"    nodeset: centos-9-stream"},{"line_number":189,"context_line":"    description: |"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"07d8fc1d_bb0ebe6d","line":186,"in_reply_to":"858f7a0f_02578f1c","updated":"2023-08-24 12:13:51.000000000","message":"I will add that in a follo-wup commit\n\nI just noticed that octavia-amphora-image-build-live-rocky-9 uses the centos-9-stream nodeset, but it cannot use the rocky-linux nodeset until the associated octavia-tempest-plugin patch is merged... but the o-t-p patch depends on this octavia patch...\n\nhere\u0027s my plan:\n- merge the octavia patch\n- merge the octavia-tempest-plugin patch\n- add the publish-...-rocky-9 job and update the octavia-amphora-image-build-live-rocky-9 to use the correct nodeset","commit_id":"4e6e8f0c0e6db892f5f60437541531f84aabc7cb"}]}