)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c3f1f5ad3791d4151d7c0fa41c31e94da1094c8f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"445ca95f_dcfa3606","updated":"2026-03-03 15:12:02.000000000","message":"-1 just for the question.\nThis new config option requires to configure each CI depending on the version of edpm-ansible to use. Isn\u0027t it better to test which directory exists and use it as base directory?","commit_id":"8f5314f978a9636b44fe1886eb0bcd8d13487ba5"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"6035ea9c0ece181fbafa18759ab68eabc8a94d2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7bbce168_3237080b","in_reply_to":"436f3141_2083760f","updated":"2026-03-03 16:32:14.000000000","message":"So, the patch where both folders are tested made sense when upstream and downstream used different folders. Now upstream and downstream are synced. I fail to see the benefit of maintaining both directories instead of adding a configurable one (In case that in the future it changes with a simple change of the job definition it will be fixed)","commit_id":"8f5314f978a9636b44fe1886eb0bcd8d13487ba5"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3fbbb0843a51959f861a09781532defbfb2d163e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"436f3141_2083760f","in_reply_to":"445ca95f_dcfa3606","updated":"2026-03-03 15:12:50.000000000","message":"Actually it is what you are using in https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/978752/1/whitebox_neutron_tempest_plugin/tests/scenario/test_metadata_rate_limiting.py","commit_id":"8f5314f978a9636b44fe1886eb0bcd8d13487ba5"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a94c9d4bccac21516886204802667e226605e58c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"85daccce_fd774557","in_reply_to":"7bbce168_3237080b","updated":"2026-03-04 14:10:24.000000000","message":"Ok, so all CI jobs are using the new path. Fine for me.","commit_id":"8f5314f978a9636b44fe1886eb0bcd8d13487ba5"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"daba2f598906bd3ae9ad67a905cbc7b84452bfbc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"92010aaa_e1dfe21d","updated":"2026-03-04 14:18:50.000000000","message":"nit: I like using os.path.join for some cases like these because it works fine regardless of adding extra \"/\" at the end/beginning of each part of the path or not, but what you did looks also good to me.\n\n-1 due to the pep8 error only","commit_id":"4d59b354c29c7368581bf05b75f9d5df7f12af78"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"a7cf5a7d54e51c74db1d5e8421bb85bef0e29edf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e70caccc_4de87297","in_reply_to":"92010aaa_e1dfe21d","updated":"2026-03-04 14:22:47.000000000","message":"Since I have to fix the pep8 issue, I\u0027ll use also os.path :)","commit_id":"4d59b354c29c7368581bf05b75f9d5df7f12af78"}],"whitebox_neutron_tempest_plugin/tests/scenario/test_dvr_ovn.py":[{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"daba2f598906bd3ae9ad67a905cbc7b84452bfbc","unresolved":true,"context_lines":[{"line_number":62,"context_line":"                    if node[\u0027is_networker\u0027] and not node[\u0027is_controller\u0027]:"},{"line_number":63,"context_line":"                        config_dir \u003d (WB_CONF.compute_root_config_dir +"},{"line_number":64,"context_line":"                            \"ovn-bgp-agent\")"},{"line_number":65,"context_line":"                        config_path \u003d( config_dir +"},{"line_number":66,"context_line":"                            \"/etc/ovn-bgp-agent/bgp-agent.conf\")"},{"line_number":67,"context_line":"                        output \u003d local_utils.conf_action("},{"line_number":68,"context_line":"                            config_path,"}],"source_content_type":"text/x-python","patch_set":3,"id":"f32efd73_af5ec0ce","line":65,"range":{"start_line":65,"start_character":36,"end_line":65,"end_character":38},"updated":"2026-03-04 14:18:50.000000000","message":"pep8: missing whitespace","commit_id":"4d59b354c29c7368581bf05b75f9d5df7f12af78"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"c8d221fcf1a1bf0d91b473d3ac27da1ae280ae3a","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                    if node[\u0027is_networker\u0027] and not node[\u0027is_controller\u0027]:"},{"line_number":63,"context_line":"                        config_dir \u003d (WB_CONF.compute_root_config_dir +"},{"line_number":64,"context_line":"                            \"ovn-bgp-agent\")"},{"line_number":65,"context_line":"                        config_path \u003d( config_dir +"},{"line_number":66,"context_line":"                            \"/etc/ovn-bgp-agent/bgp-agent.conf\")"},{"line_number":67,"context_line":"                        output \u003d local_utils.conf_action("},{"line_number":68,"context_line":"                            config_path,"}],"source_content_type":"text/x-python","patch_set":3,"id":"18d0f12b_91c5688c","line":65,"range":{"start_line":65,"start_character":36,"end_line":65,"end_character":38},"in_reply_to":"f32efd73_af5ec0ce","updated":"2026-03-05 11:08:59.000000000","message":"Done","commit_id":"4d59b354c29c7368581bf05b75f9d5df7f12af78"}]}