)]}'
{"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"80b11badd3bee002b1337360a4682d89543e613f","unresolved":true,"context_lines":[{"line_number":90,"context_line":"# workaround is needed due to the fact \u0027Depends-On\u0027 tag is being ignored by"},{"line_number":91,"context_line":"# Zuul and check pipeline fails because it\u0027s missing necessary dependencies"},{"line_number":92,"context_line":"# from n-lib."},{"line_number":93,"context_line":"def _parse_rp_rate(rate_str):"},{"line_number":94,"context_line":"    \"\"\"Parse the config string value to an non-negative integer."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    :param rate_str: A decimal string representation of an non-negative"}],"source_content_type":"text/x-python","patch_set":2,"id":"89830043_710d47a1","line":93,"updated":"2021-07-15 14:32:59.000000000","message":"just to have it mentioned: unit tests for these methods are in https://review.opendev.org/c/openstack/neutron-lib/+/800441","commit_id":"e5fa263d20fc816e3b637db2d769a8045b698c97"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"def65ee69d2faed51617728331b41894101bc601","unresolved":false,"context_lines":[{"line_number":90,"context_line":"# workaround is needed due to the fact \u0027Depends-On\u0027 tag is being ignored by"},{"line_number":91,"context_line":"# Zuul and check pipeline fails because it\u0027s missing necessary dependencies"},{"line_number":92,"context_line":"# from n-lib."},{"line_number":93,"context_line":"def _parse_rp_rate(rate_str):"},{"line_number":94,"context_line":"    \"\"\"Parse the config string value to an non-negative integer."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    :param rate_str: A decimal string representation of an non-negative"}],"source_content_type":"text/x-python","patch_set":2,"id":"5c22fc67_a0713016","line":93,"in_reply_to":"89830043_710d47a1","updated":"2021-07-16 14:13:17.000000000","message":"Done","commit_id":"e5fa263d20fc816e3b637db2d769a8045b698c97"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"80b11badd3bee002b1337360a4682d89543e613f","unresolved":true,"context_lines":[{"line_number":495,"context_line":"                               n_const.RP_INVENTORY_DEFAULTS:"},{"line_number":496,"context_line":"                                   self.rp_inventory_defaults,"},{"line_number":497,"context_line":"                               constants.RP_PP_WITHOUT_DIRECTION:"},{"line_number":498,"context_line":"                               # n_const.RP_PP_WITHOUT_DIRECTION:"},{"line_number":499,"context_line":"                                   self.rp_pp_without_direction,"},{"line_number":500,"context_line":"                               constants.RP_PP_WITH_DIRECTION:"},{"line_number":501,"context_line":"                               # n_const.RP_PP_WITH_DIRECTION:"}],"source_content_type":"text/x-python","patch_set":2,"id":"b8874fea_a4e95686","line":498,"range":{"start_line":498,"start_character":0,"end_line":498,"end_character":65},"updated":"2021-07-15 14:32:59.000000000","message":"nit: I think these commented references to n_const can be left out now and can be done with the other todos","commit_id":"e5fa263d20fc816e3b637db2d769a8045b698c97"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"def65ee69d2faed51617728331b41894101bc601","unresolved":false,"context_lines":[{"line_number":495,"context_line":"                               n_const.RP_INVENTORY_DEFAULTS:"},{"line_number":496,"context_line":"                                   self.rp_inventory_defaults,"},{"line_number":497,"context_line":"                               constants.RP_PP_WITHOUT_DIRECTION:"},{"line_number":498,"context_line":"                               # n_const.RP_PP_WITHOUT_DIRECTION:"},{"line_number":499,"context_line":"                                   self.rp_pp_without_direction,"},{"line_number":500,"context_line":"                               constants.RP_PP_WITH_DIRECTION:"},{"line_number":501,"context_line":"                               # n_const.RP_PP_WITH_DIRECTION:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfe2db77_ab867ead","line":498,"range":{"start_line":498,"start_character":0,"end_line":498,"end_character":65},"in_reply_to":"b8874fea_a4e95686","updated":"2021-07-16 14:13:17.000000000","message":"Done","commit_id":"e5fa263d20fc816e3b637db2d769a8045b698c97"}],"neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"80b11badd3bee002b1337360a4682d89543e613f","unresolved":true,"context_lines":[{"line_number":2568,"context_line":""},{"line_number":2569,"context_line":"    def test_configurations_has_rp_pp_without_direction(self):"},{"line_number":2570,"context_line":"        self.assertIn(constants.RP_PP_WITHOUT_DIRECTION,"},{"line_number":2571,"context_line":"                      # n_const.RP_PP_WITHOUT_DIRECTION,"},{"line_number":2572,"context_line":"                      self.agent.agent_state[\u0027configurations\u0027])"},{"line_number":2573,"context_line":""},{"line_number":2574,"context_line":"    def test_configurations_has_rp_pp_with_direction(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"04a772ef_a58267b1","line":2571,"range":{"start_line":2571,"start_character":22,"end_line":2571,"end_character":56},"updated":"2021-07-15 14:32:59.000000000","message":"nit: I don\u0027t think it is necessary to have these commented lines here, when the todo is done this can be updated as well","commit_id":"e5fa263d20fc816e3b637db2d769a8045b698c97"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"def65ee69d2faed51617728331b41894101bc601","unresolved":false,"context_lines":[{"line_number":2568,"context_line":""},{"line_number":2569,"context_line":"    def test_configurations_has_rp_pp_without_direction(self):"},{"line_number":2570,"context_line":"        self.assertIn(constants.RP_PP_WITHOUT_DIRECTION,"},{"line_number":2571,"context_line":"                      # n_const.RP_PP_WITHOUT_DIRECTION,"},{"line_number":2572,"context_line":"                      self.agent.agent_state[\u0027configurations\u0027])"},{"line_number":2573,"context_line":""},{"line_number":2574,"context_line":"    def test_configurations_has_rp_pp_with_direction(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"83221619_e672327a","line":2571,"range":{"start_line":2571,"start_character":22,"end_line":2571,"end_character":56},"in_reply_to":"04a772ef_a58267b1","updated":"2021-07-16 14:13:17.000000000","message":"Done","commit_id":"e5fa263d20fc816e3b637db2d769a8045b698c97"}]}