)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"b23407b23fcca91ecee381c40e74c9e7f568a1ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"06df9146_86bfd085","updated":"2026-04-02 07:58:16.000000000","message":"A few suggestions to prevent errors when re-running devstack setup\n\nReviewed with assistance from Claude Code","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"e086dfb669837e3de156852e4c2f52e788127043","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c3c8de2a_96a969d4","updated":"2026-04-02 09:02:17.000000000","message":"Fair point, I missed that it was a backport and not to master\n\nChanging to +1 since it matches the original change","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"8f0c652f4b797f1bafdc6a70b74b844ab99d8a74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff1859aa_65216b55","updated":"2026-04-02 09:02:45.000000000","message":"Marking issues as done since they can be fixed with a different backport","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"76ea2ed85e692c2086e17aa7feb2ed1accd7e85a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d4a349c5_8377df9a","updated":"2026-04-02 08:05:57.000000000","message":"we can backport I675b48404e3b81602d792d653db348df0968e99c that addresses these issues, but it\u0027s not really useful for the CI\n\nand I think we should not use AI to review backports, when reviewing backports we need to consider:\n- if the backport doesn\u0027t break the policy (features, db changes, etc)\n- if the backport is relevant on stable branches\n- if the patch hasn\u0027t been altered in a way that could break the stable branch","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"}],"devstack/plugin.sh":[{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"b23407b23fcca91ecee381c40e74c9e7f568a1ed","unresolved":true,"context_lines":[{"line_number":612,"context_line":"}"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"function configure_rsyslog {"},{"line_number":615,"context_line":"    sudo mkdir -m 775 /var/log/octavia"},{"line_number":616,"context_line":"    sudo chgrp syslog /var/log/octavia"},{"line_number":617,"context_line":""},{"line_number":618,"context_line":"    sudo cp ${OCTAVIA_DIR}/devstack/etc/rsyslog/10-octavia-log-offloading.conf /etc/rsyslog.d/"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3204e0a1_ae443192","line":615,"updated":"2026-04-02 07:58:16.000000000","message":"Will fail if directory already exists, the -p option prevents this\n\n```suggestion\n    sudo mkdir -p -m 775 /var/log/octavia\n```","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"8f0c652f4b797f1bafdc6a70b74b844ab99d8a74","unresolved":false,"context_lines":[{"line_number":612,"context_line":"}"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"function configure_rsyslog {"},{"line_number":615,"context_line":"    sudo mkdir -m 775 /var/log/octavia"},{"line_number":616,"context_line":"    sudo chgrp syslog /var/log/octavia"},{"line_number":617,"context_line":""},{"line_number":618,"context_line":"    sudo cp ${OCTAVIA_DIR}/devstack/etc/rsyslog/10-octavia-log-offloading.conf /etc/rsyslog.d/"}],"source_content_type":"text/x-sh","patch_set":1,"id":"77ff1ad5_e3ff1d1a","line":615,"in_reply_to":"3204e0a1_ae443192","updated":"2026-04-02 09:02:45.000000000","message":"Done","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"b23407b23fcca91ecee381c40e74c9e7f568a1ed","unresolved":true,"context_lines":[{"line_number":626,"context_line":"    sudo touch /var/log/octavia/octavia-tenant-traffic.log"},{"line_number":627,"context_line":"    sudo chmod 664 /var/log/octavia/octavia-tenant-traffic.log"},{"line_number":628,"context_line":"    sudo chgrp syslog /var/log/octavia/octavia-tenant-traffic.log"},{"line_number":629,"context_line":"    sudo ln -s /var/log/octavia/octavia-tenant-traffic.log /var/log/octavia-tenant-traffic.log"},{"line_number":630,"context_line":""},{"line_number":631,"context_line":"    sudo touch /var/log/octavia/octavia-amphora.log"},{"line_number":632,"context_line":"    sudo chmod 664 /var/log/octavia/octavia-amphora.log"}],"source_content_type":"text/x-sh","patch_set":1,"id":"d8e53452_b7ebe705","line":629,"updated":"2026-04-02 07:58:16.000000000","message":"This will fail if the symlink already exists, use the -f option to force recreation\n\n```suggestion\n    sudo ln -fs /var/log/octavia/octavia-tenant-traffic.log /var/log/octavia-tenant-traffic.log\n```","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"8f0c652f4b797f1bafdc6a70b74b844ab99d8a74","unresolved":false,"context_lines":[{"line_number":626,"context_line":"    sudo touch /var/log/octavia/octavia-tenant-traffic.log"},{"line_number":627,"context_line":"    sudo chmod 664 /var/log/octavia/octavia-tenant-traffic.log"},{"line_number":628,"context_line":"    sudo chgrp syslog /var/log/octavia/octavia-tenant-traffic.log"},{"line_number":629,"context_line":"    sudo ln -s /var/log/octavia/octavia-tenant-traffic.log /var/log/octavia-tenant-traffic.log"},{"line_number":630,"context_line":""},{"line_number":631,"context_line":"    sudo touch /var/log/octavia/octavia-amphora.log"},{"line_number":632,"context_line":"    sudo chmod 664 /var/log/octavia/octavia-amphora.log"}],"source_content_type":"text/x-sh","patch_set":1,"id":"8afbc7fe_da2ae8d1","line":629,"in_reply_to":"d8e53452_b7ebe705","updated":"2026-04-02 09:02:45.000000000","message":"Done","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"b23407b23fcca91ecee381c40e74c9e7f568a1ed","unresolved":true,"context_lines":[{"line_number":631,"context_line":"    sudo touch /var/log/octavia/octavia-amphora.log"},{"line_number":632,"context_line":"    sudo chmod 664 /var/log/octavia/octavia-amphora.log"},{"line_number":633,"context_line":"    sudo chgrp syslog /var/log/octavia/octavia-amphora.log"},{"line_number":634,"context_line":"    sudo ln -s /var/log/octavia/octavia-amphora.log /var/log/octavia-amphora.log"},{"line_number":635,"context_line":"}"},{"line_number":636,"context_line":""},{"line_number":637,"context_line":"function octavia_start {"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5727e794_129ebb6a","line":634,"updated":"2026-04-02 07:58:16.000000000","message":"Same symlink issue\n\n```suggestion\n    sudo ln -fs /var/log/octavia/octavia-amphora.log /var/log/octavia-amphora.log\n```","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"8f0c652f4b797f1bafdc6a70b74b844ab99d8a74","unresolved":false,"context_lines":[{"line_number":631,"context_line":"    sudo touch /var/log/octavia/octavia-amphora.log"},{"line_number":632,"context_line":"    sudo chmod 664 /var/log/octavia/octavia-amphora.log"},{"line_number":633,"context_line":"    sudo chgrp syslog /var/log/octavia/octavia-amphora.log"},{"line_number":634,"context_line":"    sudo ln -s /var/log/octavia/octavia-amphora.log /var/log/octavia-amphora.log"},{"line_number":635,"context_line":"}"},{"line_number":636,"context_line":""},{"line_number":637,"context_line":"function octavia_start {"}],"source_content_type":"text/x-sh","patch_set":1,"id":"8682dada_5d35c628","line":634,"in_reply_to":"5727e794_129ebb6a","updated":"2026-04-02 09:02:45.000000000","message":"Done","commit_id":"5c3d31d47cf1146ad14543113dc4d5d4cd2fa819"}]}
