)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"99690311b996f1e7a7cdc104e3ef61cc5b22504d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Check \u0027binding:profile\u0027 field of the port in"},{"line_number":10,"context_line":"MinBwAllocationPlacementTest.test_qos_min_bw_allocation_basic testcase."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: https://review.opendev.org/690999"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #1849657"},{"line_number":15,"context_line":"Change-Id: Icf9bf6ab3451432a5860df6d1499bff425e001b7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_62615339","line":12,"updated":"2019-10-29 14:24:22.000000000","message":"This is the bugfix in nova on master. As tempest is branchless this Depends-On only prevent us to break the tempest run master. But as soon as this tempest change merges the modified test case will fail on stable/train and stable/stein. So this patch needs to depend on the future backports of the master bugfix.","commit_id":"33f1149aaef7899d6b810847394a6032f93ce2ad"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"8a71b63003541b4b0f3fb152caa1b1964286529c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Check \u0027binding:profile\u0027 field of the port in"},{"line_number":10,"context_line":"MinBwAllocationPlacementTest.test_qos_min_bw_allocation_basic testcase."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: https://review.opendev.org/690999"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #1849657"},{"line_number":15,"context_line":"Change-Id: Icf9bf6ab3451432a5860df6d1499bff425e001b7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_e4d2dacf","line":12,"in_reply_to":"3fa7e38b_0d06e71d","updated":"2019-11-26 09:12:07.000000000","message":"Stein backports have been merged. I think this is good to go now.","commit_id":"33f1149aaef7899d6b810847394a6032f93ce2ad"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"1219e391a14e36d6ac614a4c60f7ff5b9eed350d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Check \u0027binding:profile\u0027 field of the port in"},{"line_number":10,"context_line":"MinBwAllocationPlacementTest.test_qos_min_bw_allocation_basic testcase."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: https://review.opendev.org/690999"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #1849657"},{"line_number":15,"context_line":"Change-Id: Icf9bf6ab3451432a5860df6d1499bff425e001b7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_0d06e71d","line":12,"in_reply_to":"3fa7e38b_62615339","updated":"2019-11-13 09:26:52.000000000","message":"stable/train backports:\nhttps://review.opendev.org/#/c/694013/\nhttps://review.opendev.org/#/c/694015/\nhttps://review.opendev.org/#/c/694018/\n\nWe still have to wait for the stain backports of two of the above patches.","commit_id":"33f1149aaef7899d6b810847394a6032f93ce2ad"}],"tempest/scenario/test_minbw_allocation_placement.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"054a4f2137fe1ff8552aaed479fef49e17b41db0","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        # Check binding_profile of the port is not empty and equals with the"},{"line_number":181,"context_line":"        # rp uuid"},{"line_number":182,"context_line":"        port \u003d self.os_admin.ports_client.show_port(valid_port[\u0027id\u0027])"},{"line_number":183,"context_line":"        self.assertIn(allocation_rp, port[\u0027port\u0027][\u0027binding:profile\u0027])"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # boot another vm with max int bandwidth"},{"line_number":186,"context_line":"        not_valid_port \u003d self.create_port("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_f5e750e2","line":183,"updated":"2019-10-28 12:37:28.000000000","message":"From the tempest test result:\n\ntesttools.matchers._impl.MismatchError: u\u0027ce0db0d8-1531-5725-a78b-d4e1f3b5c9b2\u0027 not in {u\u0027allocation\u0027: u\u0027ce0db0d8-1531-5725-a78b-d4e1f3b5c9b2\u0027}\n\nSo this should be:\n  self.assertIn(allocation_rp, port[\u0027port\u0027][\u0027binding:profile\u0027][\u0027allocation\u0027])","commit_id":"da340765e0874d9d2c07a5c0e0ac2ab937fc096d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ec08b58d29d2f8b2e7f456991c33a2f8a4a239a8","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        # Check binding_profile of the port is not empty and equals with the"},{"line_number":181,"context_line":"        # rp uuid"},{"line_number":182,"context_line":"        port \u003d self.os_admin.ports_client.show_port(valid_port[\u0027id\u0027])"},{"line_number":183,"context_line":"        self.assertIn(allocation_rp, port[\u0027port\u0027][\u0027binding:profile\u0027])"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # boot another vm with max int bandwidth"},{"line_number":186,"context_line":"        not_valid_port \u003d self.create_port("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_207928fb","line":183,"in_reply_to":"3fa7e38b_350ba88e","updated":"2019-10-28 13:43:41.000000000","message":"Done","commit_id":"da340765e0874d9d2c07a5c0e0ac2ab937fc096d"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"8807b1cf19870f55e212ec2af4347c52b0486e45","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        # Check binding_profile of the port is not empty and equals with the"},{"line_number":181,"context_line":"        # rp uuid"},{"line_number":182,"context_line":"        port \u003d self.os_admin.ports_client.show_port(valid_port[\u0027id\u0027])"},{"line_number":183,"context_line":"        self.assertIn(allocation_rp, port[\u0027port\u0027][\u0027binding:profile\u0027])"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # boot another vm with max int bandwidth"},{"line_number":186,"context_line":"        not_valid_port \u003d self.create_port("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_350ba88e","line":183,"in_reply_to":"3fa7e38b_f5e750e2","updated":"2019-10-28 12:38:15.000000000","message":"\u003e From the tempest test result:\n \u003e \n \u003e testtools.matchers._impl.MismatchError: u\u0027ce0db0d8-1531-5725-a78b-d4e1f3b5c9b2\u0027\n \u003e not in {u\u0027allocation\u0027: u\u0027ce0db0d8-1531-5725-a78b-d4e1f3b5c9b2\u0027}\n \u003e \n \u003e So this should be:\n \u003e self.assertIn(allocation_rp, port[\u0027port\u0027][\u0027binding:profile\u0027][\u0027allocation\u0027])\n\n\nI mean\n\n    self.assertEqual(allocation_rp, port[\u0027port\u0027][\u0027binding:profile\u0027][\u0027allocation\u0027])","commit_id":"da340765e0874d9d2c07a5c0e0ac2ab937fc096d"}]}
