)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"4cd063ef544bbd644d5de47f3ed61d42c7630d48","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Flag to enable listener reconcilation for stable branches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Listener reconciliation needs to be enabled for all stable and"},{"line_number":10,"context_line":"newer branches but disabled for older branches. This flag would"},{"line_number":11,"context_line":"be used to disable listener reconciliation for Xena"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c986fef1_468c8fe5","line":9,"updated":"2022-08-23 16:25:05.000000000","message":"it\u0027s not all stable branches right? Only from the release that listener reconciliation was checked.\nIs this patch https://review.opendev.org/c/openstack/kuryr-kubernetes/+/853848 needed?","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"b56470ea9d01fccb0698407542bb0664efd295ad","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Flag to enable listener reconcilation for stable branches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Listener reconciliation needs to be enabled for all stable and"},{"line_number":10,"context_line":"newer branches but disabled for older branches. This flag would"},{"line_number":11,"context_line":"be used to disable listener reconciliation for Xena"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"785a53a5_7c22484d","line":9,"in_reply_to":"be1a914a_4fe295dd","updated":"2022-08-24 10:31:46.000000000","message":"That configuration needs to be defined in the kuryr-tempest-plugin and only switched to True here in the branches needed.\n\nNote that Kuryr has multiple stable branches https://github.com/openstack/kuryr-kubernetes/branches. The commit title and message should be updated to describe that config is meant only for master and yoga.","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"2d222e33f149fbc8df9f915bf5fab189464d96fd","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Flag to enable listener reconcilation for stable branches"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Listener reconciliation needs to be enabled for all stable and"},{"line_number":10,"context_line":"newer branches but disabled for older branches. This flag would"},{"line_number":11,"context_line":"be used to disable listener reconciliation for Xena"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"be1a914a_4fe295dd","line":9,"in_reply_to":"c986fef1_468c8fe5","updated":"2022-08-24 07:59:35.000000000","message":"The idea is to skip the listener reconciliation for Xena. So, I am trying to find a way to enable it for Master and Yoga. But as Michal suggested, I am needing to default the configuration to \u0027False\u0027 then flip it to `True` on Master and Yoga branches. I am thinking the default could be set somewhere around line #19?","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"9512785a509f2811cc87d07dfb47443afac5aa93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cb7114f8_dafd9d34","updated":"2022-08-24 12:43:52.000000000","message":"We shouldn\u0027t need this, should be abandoned.","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"0691013054e7a30b35982b85074c3872e4e66a05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"77357b1e_d443b136","updated":"2022-08-20 10:34:42.000000000","message":"recheck","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"94b403619426432dc7755034bdccb681ca5efec1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d85a4200_0a58c62b","updated":"2022-08-20 13:42:30.000000000","message":"recheck","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"},{"author":{"_account_id":33240,"name":"Sunday Mgbogu","email":"digitalsimboja@gmail.com","username":"digitalsimboja"},"change_message_id":"f52735672848900326ec08ae8eda969465b84efc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f682d39f_3143adf3","updated":"2022-08-20 18:07:22.000000000","message":"recheck","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"}],"devstack/lib/kuryr_kubernetes":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"4cd063ef544bbd644d5de47f3ed61d42c7630d48","unresolved":true,"context_lines":[{"line_number":1583,"context_line":"     # so only enable them for OVN as it\u0027s faster when creating LBs"},{"line_number":1584,"context_line":"    if [[ \"$KURYR_EP_DRIVER_OCTAVIA_PROVIDER\" \u003d\u003d \"ovn\" ]]; then"},{"line_number":1585,"context_line":"        iniset \"$TEMPEST_CONFIG\" kuryr_kubernetes enable_reconciliation True"},{"line_number":1586,"context_line":"\tiniset \"$TEMPEST_CONFIG\" kuryr_kubernetes listener_reconciliation False"},{"line_number":1587,"context_line":"    fi"},{"line_number":1588,"context_line":"    iniset \"$TEMPEST_CONFIG\" kuryr_kubernetes validate_crd True"},{"line_number":1589,"context_line":"    iniset \"$TEMPEST_CONFIG\" kuryr_kubernetes kuryrports True"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"c3c4ef52_4f9a7469","line":1586,"updated":"2022-08-23 16:25:05.000000000","message":"trailing whitespace in the beginning of the line.\nShould this be True instead as this patch was created against master?","commit_id":"fefe2f78e814688a58618eb761f5456ad3b6daf4"}]}