)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"d1b7ba0b82f7abe755f47a78094908dd3f591c7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6199a542_373e7c8e","updated":"2024-01-17 08:29:38.000000000","message":"I know this is WIP, but looks great!!\n\nIn my opinion is good to check neutron tempest to ensure we don\u0027t break anything there... but should we also add some specific tempest-like coverage? And would we need an ovn-bgp-agent tempest repo for it? can we make it part of the neutron?","commit_id":"81f081e62356481b18dfa9295d9e6b8614582de8"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f0940611cd29ee9c54064ecc01abf72f42120da1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b94a47bb_3f81bb23","in_reply_to":"4b29cc15_0a62f41f","updated":"2024-01-18 07:20:05.000000000","message":"I agree, if we can have it on neutron tempest, it will be awesome","commit_id":"81f081e62356481b18dfa9295d9e6b8614582de8"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"4273104a92652893ec3968155ca0ac9ee014861a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4b29cc15_0a62f41f","in_reply_to":"6199a542_373e7c8e","updated":"2024-01-17 16:56:16.000000000","message":"I frankly didn\u0027t expect it to pass this soon :) Yes, I think we should improve the testing coverage in general, not sure about the tempest repo unless it\u0027s very bgp specific given that tempest is rather blackbox testing. I think it would make sense to have it in Neutron too if folks are not against. All great points, thanks :)","commit_id":"81f081e62356481b18dfa9295d9e6b8614582de8"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"2febee2c87d2f679664525ecb70d09fd2c6e8014","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"574e1b1e_5d14fb52","updated":"2024-02-09 16:11:48.000000000","message":"Look good to me. Wondering if we should filter out some tests not needed from neutron to make it lighter though","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"f2b2ae282da8619d581fcaf17edcbe13a0ee34ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"63143a42_1e5d9947","updated":"2024-02-29 08:09:47.000000000","message":"given that we don\u0027t have specific test on tempest... should this job only be enabled on the gate (before final merge) and not in the initial checkings to save time and resources? We can also add it to the experimental gate so that we can run it manually if/when needed. Thoughts?","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"317043643ab96173ea051861dfe2a1e2a2508a73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"633305f9_541583f6","updated":"2024-03-04 06:53:28.000000000","message":"it is green again","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"e327520baec3a93e95e88ca21df372e115d86305","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"920a78c4_9d79e553","updated":"2024-02-29 08:08:09.000000000","message":"recheck","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"bba8d7e8b080801ec76a4508693ab5777bf53f3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a069bc3b_c8607541","updated":"2024-03-01 15:01:04.000000000","message":"recheck","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"3c51d5145126ba809b33a61c1fc8f8701f486123","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5c78a738_5bc2e8ad","in_reply_to":"2493e814_4b4b8cb7","updated":"2024-03-01 15:03:48.000000000","message":"my doubt here is because we are going from 5 minutes gate to 2 hours, without adding bgp specific testing (though it is nice to know it is deploy-able)","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"3778bb677eca29776b17e581d68bfc82d6cb5401","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2493e814_4b4b8cb7","in_reply_to":"63143a42_1e5d9947","updated":"2024-02-29 13:04:16.000000000","message":"I would rather have it in the check queue and run the tempest twice than having the benefit of quick results.","commit_id":"9bf72c14e3ba855c42a01d08c85c571cee263676"}]}