)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"03a27aa66986541c752e160448fc90a6cf36ceb4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it should contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules also) and appending of such rule type was missing"},{"line_number":11,"context_line":"in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_e240eda7","line":9,"range":{"start_line":9,"start_character":50,"end_line":9,"end_character":65},"updated":"2017-04-10 14:27:57.000000000","message":"Well, maybe is better \"could contain\". But I\u0027m not english native speaker","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8dbdfa40ed7465782721cc410118415f180d33fc","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it should contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules also) and appending of such rule type was missing"},{"line_number":11,"context_line":"in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_81d8f47a","line":9,"range":{"start_line":9,"start_character":50,"end_line":9,"end_character":65},"in_reply_to":"bff0334d_e240eda7","updated":"2017-04-10 20:18:02.000000000","message":"I changed it. I hope it\u0027s better now (I\u0027m also not english speaker :))","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"03a27aa66986541c752e160448fc90a6cf36ceb4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it should contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules also) and appending of such rule type was missing"},{"line_number":11,"context_line":"in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_420d599f","line":10,"updated":"2017-04-10 14:27:57.000000000","message":"Max line length should be 72","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"03a27aa66986541c752e160448fc90a6cf36ceb4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it should contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules also) and appending of such rule type was missing"},{"line_number":11,"context_line":"in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_e2fb4d76","line":10,"range":{"start_line":10,"start_character":34,"end_line":10,"end_character":35},"updated":"2017-04-10 14:27:57.000000000","message":"Remove","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8dbdfa40ed7465782721cc410118415f180d33fc","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it should contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules also) and appending of such rule type was missing"},{"line_number":11,"context_line":"in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_21c7e898","line":10,"in_reply_to":"bff0334d_420d599f","updated":"2017-04-10 20:18:02.000000000","message":"Done","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8dbdfa40ed7465782721cc410118415f180d33fc","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it should contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules also) and appending of such rule type was missing"},{"line_number":11,"context_line":"in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_01c4e48f","line":10,"range":{"start_line":10,"start_character":34,"end_line":10,"end_character":35},"in_reply_to":"bff0334d_e2fb4d76","updated":"2017-04-10 20:18:02.000000000","message":"Done","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8e2a30861670abd1b0017bfe29f411267c8854bf","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it can contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules and appending of such rule type"},{"line_number":11,"context_line":"was missing in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7ffa3b31_a45b3f52","line":9,"range":{"start_line":9,"start_character":53,"end_line":9,"end_character":62},"updated":"2017-04-18 23:18:19.000000000","message":"contain","commit_id":"33034eb45a3459c9be6877d6c30501f3bcea258b"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"c48c5fd48b82e46f5a54040ef6f732a37089849c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make QoS policy object compatible with versions 1.2 and higher"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"For version 1.2 or higher of QoS policy object it can contains"},{"line_number":10,"context_line":"QoSMinumumBandwidtLimit rules and appending of such rule type"},{"line_number":11,"context_line":"was missing in make_obj_compatible function."},{"line_number":12,"context_line":"Now such rules are appended to QoS policy."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7ffa3b31_485009d5","line":9,"range":{"start_line":9,"start_character":53,"end_line":9,"end_character":62},"in_reply_to":"7ffa3b31_a45b3f52","updated":"2017-04-19 05:59:37.000000000","message":"Done","commit_id":"33034eb45a3459c9be6877d6c30501f3bcea258b"}],"neutron/tests/unit/objects/qos/test_policy.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"03a27aa66986541c752e160448fc90a6cf36ceb4","unresolved":false,"context_lines":[{"line_number":397,"context_line":"        #                 local version on the class definition"},{"line_number":398,"context_line":"        policy_obj, rule_objs \u003d self._create_test_policy_with_rules("},{"line_number":399,"context_line":"            [qos_consts.RULE_TYPE_BANDWIDTH_LIMIT,"},{"line_number":400,"context_line":"             qos_consts.RULE_TYPE_DSCP_MARKING], reload_rules\u003dTrue)"},{"line_number":401,"context_line":""},{"line_number":402,"context_line":"        policy_obj_v1_0 \u003d self._policy_through_version(policy_obj, \u00271.0\u0027)"},{"line_number":403,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_a2c30538","line":400,"updated":"2017-04-10 14:27:57.000000000","message":"Can you add min_bw to this check??","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8dbdfa40ed7465782721cc410118415f180d33fc","unresolved":false,"context_lines":[{"line_number":397,"context_line":"        #                 local version on the class definition"},{"line_number":398,"context_line":"        policy_obj, rule_objs \u003d self._create_test_policy_with_rules("},{"line_number":399,"context_line":"            [qos_consts.RULE_TYPE_BANDWIDTH_LIMIT,"},{"line_number":400,"context_line":"             qos_consts.RULE_TYPE_DSCP_MARKING], reload_rules\u003dTrue)"},{"line_number":401,"context_line":""},{"line_number":402,"context_line":"        policy_obj_v1_0 \u003d self._policy_through_version(policy_obj, \u00271.0\u0027)"},{"line_number":403,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_e656b62a","line":400,"in_reply_to":"bff0334d_a2c30538","updated":"2017-04-10 20:18:02.000000000","message":"Done","commit_id":"3b9dadee01d0765f8354043bf2b324a0a2433de1"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8e2a30861670abd1b0017bfe29f411267c8854bf","unresolved":false,"context_lines":[{"line_number":421,"context_line":"        self.assertNotIn(rule_objs[2], policy_obj_v1_1.rules)"},{"line_number":422,"context_line":""},{"line_number":423,"context_line":"    def test_object_version_degradation_1_3_to_1_2(self):"},{"line_number":424,"context_line":"        #NOTE(mangelajo): we should not check .VERSION, since that\u0027s the"},{"line_number":425,"context_line":"        #                 local version on the class definition"},{"line_number":426,"context_line":"        policy_obj, rule_objs \u003d self._create_test_policy_with_rules("},{"line_number":427,"context_line":"            [qos_consts.RULE_TYPE_BANDWIDTH_LIMIT,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_2435eff7","line":424,"updated":"2017-04-18 23:18:19.000000000","message":"is that ajo\u0027s comment really?","commit_id":"33034eb45a3459c9be6877d6c30501f3bcea258b"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"c48c5fd48b82e46f5a54040ef6f732a37089849c","unresolved":false,"context_lines":[{"line_number":421,"context_line":"        self.assertNotIn(rule_objs[2], policy_obj_v1_1.rules)"},{"line_number":422,"context_line":""},{"line_number":423,"context_line":"    def test_object_version_degradation_1_3_to_1_2(self):"},{"line_number":424,"context_line":"        #NOTE(mangelajo): we should not check .VERSION, since that\u0027s the"},{"line_number":425,"context_line":"        #                 local version on the class definition"},{"line_number":426,"context_line":"        policy_obj, rule_objs \u003d self._create_test_policy_with_rules("},{"line_number":427,"context_line":"            [qos_consts.RULE_TYPE_BANDWIDTH_LIMIT,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_e82235a4","line":424,"in_reply_to":"7ffa3b31_2435eff7","updated":"2017-04-19 05:59:37.000000000","message":"This comment is copied from other similar tests. I didn\u0027t want to change author of it here to be \"consistent\" with other tests. In fact real author of this comment is Ajo  IMHO :)\nShould I change it?","commit_id":"33034eb45a3459c9be6877d6c30501f3bcea258b"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8e2a30861670abd1b0017bfe29f411267c8854bf","unresolved":false,"context_lines":[{"line_number":430,"context_line":""},{"line_number":431,"context_line":"        policy_obj_v1_2 \u003d self._policy_through_version(policy_obj, \u00271.2\u0027)"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"        self.assertIn(rule_objs[0], policy_obj_v1_2.rules)"},{"line_number":434,"context_line":"        self.assertIn(rule_objs[1], policy_obj_v1_2.rules)"},{"line_number":435,"context_line":"        self.assertIn(rule_objs[2], policy_obj_v1_2.rules)"},{"line_number":436,"context_line":""},{"line_number":437,"context_line":"    def test_v1_4_to_v1_3_drops_project_id(self):"},{"line_number":438,"context_line":"        policy_new \u003d self._create_test_policy()"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_e43ee7d8","line":435,"range":{"start_line":433,"start_character":8,"end_line":435,"end_character":58},"updated":"2017-04-18 23:18:19.000000000","message":"for i in ...","commit_id":"33034eb45a3459c9be6877d6c30501f3bcea258b"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"c48c5fd48b82e46f5a54040ef6f732a37089849c","unresolved":false,"context_lines":[{"line_number":430,"context_line":""},{"line_number":431,"context_line":"        policy_obj_v1_2 \u003d self._policy_through_version(policy_obj, \u00271.2\u0027)"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"        self.assertIn(rule_objs[0], policy_obj_v1_2.rules)"},{"line_number":434,"context_line":"        self.assertIn(rule_objs[1], policy_obj_v1_2.rules)"},{"line_number":435,"context_line":"        self.assertIn(rule_objs[2], policy_obj_v1_2.rules)"},{"line_number":436,"context_line":""},{"line_number":437,"context_line":"    def test_v1_4_to_v1_3_drops_project_id(self):"},{"line_number":438,"context_line":"        policy_new \u003d self._create_test_policy()"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_c82539ad","line":435,"range":{"start_line":433,"start_character":8,"end_line":435,"end_character":58},"in_reply_to":"7ffa3b31_e43ee7d8","updated":"2017-04-19 05:59:37.000000000","message":"Done","commit_id":"33034eb45a3459c9be6877d6c30501f3bcea258b"}]}