)]}'
{"openstack/tests/functional/network/v2/test_qos_bandwidth_limit_rule.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"003ccc7a013fd9fb79e8d83157892439be692edb","unresolved":true,"context_lines":[{"line_number":56,"context_line":"        self.RULE_ID \u003d qos_rule.id"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def tearDown(self):"},{"line_number":59,"context_line":"        rule \u003d self.conn.network.delete_qos_minimum_bandwidth_rule("},{"line_number":60,"context_line":"            self.RULE_ID,"},{"line_number":61,"context_line":"            self.QOS_POLICY_ID)"},{"line_number":62,"context_line":"        qos_policy \u003d self.conn.network.delete_qos_policy(self.QOS_POLICY_ID)"}],"source_content_type":"text/x-python","patch_set":1,"id":"0a7e7567_8f9f17ca","line":59,"range":{"start_line":59,"start_character":33,"end_line":59,"end_character":66},"updated":"2022-01-28 16:49:27.000000000","message":"nit: I\u0027m not sure why we are calling this min-BW rule method","commit_id":"4ca7ca7ceed28b0726cc3b1eb4507452512e86a1"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d2cb6a6cb9923ea25f6300db0da1ef2eb9464d1c","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        self.RULE_ID \u003d qos_rule.id"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def tearDown(self):"},{"line_number":59,"context_line":"        rule \u003d self.conn.network.delete_qos_minimum_bandwidth_rule("},{"line_number":60,"context_line":"            self.RULE_ID,"},{"line_number":61,"context_line":"            self.QOS_POLICY_ID)"},{"line_number":62,"context_line":"        qos_policy \u003d self.conn.network.delete_qos_policy(self.QOS_POLICY_ID)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a7034d4_3509b60d","line":59,"range":{"start_line":59,"start_character":33,"end_line":59,"end_character":66},"in_reply_to":"0a7e7567_8f9f17ca","updated":"2022-01-28 20:42:24.000000000","message":"This raises some interesting questions, since it seems it was you who introduced this code a long time ago. ;)\n\nIn fact after testing locally, this should really be delete_qos_bandwidth_limit_rule(), we also should set ignore_missing\u003dFalse, so that we would actually have gotten an error due to this mismatch. I\u0027ll create a follow-up for that.\n\nAlso, this function is defined to not have a return value, so maybe we can skip checking it to be None.","commit_id":"4ca7ca7ceed28b0726cc3b1eb4507452512e86a1"}]}