)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"6ffd617a28328de7e42ea1c05c0f65885371beba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bbbd50e7_c1f9f03d","updated":"2024-07-03 05:38:25.000000000","message":"Test patch fails[1] with:-\n2024-06-27 06:54:36.802746 | controller | + lib/neutron:_enable_ovn_maintenance:374  :   enable neutron-ovn-maintenance-worker\n2024-06-27 06:54:36.802791 | controller | /opt/stack/devstack/lib/neutron: line 374: enable: neutron-ovn-maintenance-worker: not a shell builtin\n\n[1] https://c9f898dfe8b0839741c7-39f5f8dab729bdab706fbbfdb548a0f1.ssl.cf5.rackcdn.com/922613/2/experimental/neutron-tempest-plugin-api-ovn-wsgi/69547de/job-output.txt","commit_id":"399668825bb18e3749179cea0a27ba1b1a3f935e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f04241938f8a36bed6255c232ea490e83a80fcc9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5f51cc99_8d7b4165","updated":"2024-06-24 12:41:03.000000000","message":"recheck Neutron patch updated","commit_id":"399668825bb18e3749179cea0a27ba1b1a3f935e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8908189bc90dbba9fde5c5f5b724c36c55f676dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"59ee4d39_5c2ed776","updated":"2024-06-25 05:49:37.000000000","message":"recheck updated neutron patch","commit_id":"399668825bb18e3749179cea0a27ba1b1a3f935e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"06adca0275af3522561fd9d6ac5370c23c8c36f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3185cf01_9a0b1e31","in_reply_to":"bbbd50e7_c1f9f03d","updated":"2024-07-03 08:30:26.000000000","message":"Right, the method \"_enable_ovn_maintenance\" is incorrectly implemented.","commit_id":"399668825bb18e3749179cea0a27ba1b1a3f935e"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"62edb34de2e064ba60c8c1269eafe45b08f0f07c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5caa8c27_eead150b","updated":"2024-07-03 11:18:53.000000000","message":"Yes! Now the \"neutron-tempest-plugin-api-ovn-wsgi\" is working. With all the needed services (API, RPC, maintenance worker, periodic worker) running.\n\nThe controller has logs for the following services (as expected):\n* screen-neutron-api.txt\n* screen-neutron-ovn-maintenance-worker.txt\n* screen-neutron-periodic-workers.txt\n* screen-neutron-rpc-server.txt\n\nTesting patch: https://review.opendev.org/c/openstack/neutron/+/922613\nJob result: https://zuul.opendev.org/t/openstack/build/6e9bb1fe0d974cfca83588fdd7d1ed16","commit_id":"3a0c0b9ff4bb3568efc471e1bf98fc273e8bc767"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"a7d22e05d06e84e51731b43d32bfc6f794d20ffb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"65aa08c5_89039b5c","updated":"2024-07-09 14:29:45.000000000","message":"lgtm and the parent is already merged","commit_id":"3a0c0b9ff4bb3568efc471e1bf98fc273e8bc767"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3dfbb2222649fcf9d3b8f5e595cf5ee091418a12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"22850cfe_33d9abc9","updated":"2024-07-10 05:42:25.000000000","message":"recheck unrelated error ironic-tempest-bios-ipmi-direct-tinyipa","commit_id":"3a0c0b9ff4bb3568efc471e1bf98fc273e8bc767"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"27e2a99930d4037a80ae6c119d7a5d8a86087678","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c307abb5_37aa5a7f","updated":"2024-07-08 07:21:54.000000000","message":"same comment here as for the parent: From an operator and kolla developer perspective, I\u0027d rather see a unified periodic+rpc worker service, as each service will need a new container, so more overhead","commit_id":"3a0c0b9ff4bb3568efc471e1bf98fc273e8bc767"}],"lib/neutron":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"06adca0275af3522561fd9d6ac5370c23c8c36f9","unresolved":false,"context_lines":[{"line_number":371,"context_line":""},{"line_number":372,"context_line":"function _enable_ovn_maintenance {"},{"line_number":373,"context_line":"    if [[ $Q_AGENT \u003d\u003d \"ovn\" ]]; then"},{"line_number":374,"context_line":"        enable neutron-ovn-maintenance-worker"},{"line_number":375,"context_line":"    fi"},{"line_number":376,"context_line":"}"},{"line_number":377,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"1c7d522a_01cbe456","line":374,"range":{"start_line":374,"start_character":8,"end_line":374,"end_character":14},"updated":"2024-07-03 08:30:26.000000000","message":"my bad! this should be \"enable_service neutron-ovn-maintenance-worker\"","commit_id":"399668825bb18e3749179cea0a27ba1b1a3f935e"}]}