)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"e1d54dfb597158323024bb45faf49d0131fd27af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6f08725e_a199ccf5","updated":"2021-10-13 10:15:36.000000000","message":"Tested locally, it works as expected:\n\n+-------------------------+-------+\n| Field                   | Value |\n+-------------------------+-------+\n| availability_zone_hints | az-1  |\n| availability_zones      | az-1  |\n+-------------------------+-------+","commit_id":"449b7c2baf5277ef5d2d99951d285a7d857e81e9"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"190142c4ae8e0cd463d2473f133af61e29bdb7aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"49c3b71a_02cd9a71","updated":"2021-10-13 10:15:17.000000000","message":"Very nice! Thanks a lot for this","commit_id":"449b7c2baf5277ef5d2d99951d285a7d857e81e9"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"51723bad895908215920d9cde94305cc7c7a2e21","unresolved":true,"context_lines":[{"line_number":1308,"context_line":""},{"line_number":1309,"context_line":""},{"line_number":1310,"context_line":"def get_network_availability_zones(cls, network, _driver\u003dNone):"},{"line_number":1311,"context_line":"    lswitch \u003d _driver._nb_ovn.get_lswitch(network[\u0027id\u0027])"},{"line_number":1312,"context_line":"    if not lswitch:"},{"line_number":1313,"context_line":"        return []"},{"line_number":1314,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"8229c219_52c755b4","line":1311,"updated":"2021-10-14 10:31:49.000000000","message":"if _driver will be None (default value) then this line will fail. Shouldn\u0027t we either make _driver mandatory parameter or handle the case when it\u0027s None?","commit_id":"449b7c2baf5277ef5d2d99951d285a7d857e81e9"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"87dbd1974fc02d48a56ca5d51dad411d1e980168","unresolved":true,"context_lines":[{"line_number":1308,"context_line":""},{"line_number":1309,"context_line":""},{"line_number":1310,"context_line":"def get_network_availability_zones(cls, network, _driver\u003dNone):"},{"line_number":1311,"context_line":"    lswitch \u003d _driver._nb_ovn.get_lswitch(network[\u0027id\u0027])"},{"line_number":1312,"context_line":"    if not lswitch:"},{"line_number":1313,"context_line":"        return []"},{"line_number":1314,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fb7073cb_3a68d42b","line":1311,"in_reply_to":"8229c219_52c755b4","updated":"2021-10-14 10:44:45.000000000","message":"That\u0027s a good point, I don\u0027t think it will be None because this is coming from patch_plugin_choose() but, just as we do in the validate_availability_zone() let\u0027s make it mandatory.","commit_id":"449b7c2baf5277ef5d2d99951d285a7d857e81e9"},{"author":{"_account_id":34125,"name":"huanghailun","display_name":"huanghailun","email":"hailun.huang@easystack.cn","username":"lnun"},"change_message_id":"f60a9dd732298f9eaa9d359d373fb7464733cb19","unresolved":true,"context_lines":[{"line_number":1308,"context_line":""},{"line_number":1309,"context_line":""},{"line_number":1310,"context_line":"def get_network_availability_zones(cls, network, _driver\u003dNone):"},{"line_number":1311,"context_line":"    lswitch \u003d _driver._nb_ovn.get_lswitch(network[\u0027id\u0027])"},{"line_number":1312,"context_line":"    if not lswitch:"},{"line_number":1313,"context_line":"        return []"},{"line_number":1314,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bb608f83_3e7bed70","line":1311,"in_reply_to":"fb7073cb_3a68d42b","updated":"2021-10-15 02:31:16.000000000","message":"Fine","commit_id":"449b7c2baf5277ef5d2d99951d285a7d857e81e9"}]}