)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"b59ff344758dd18921482ef76d497c525e6c9db5","unresolved":false,"context_lines":[{"line_number":16,"context_line":"As a result, test_volume_swap_with_multiattach needs to be"},{"line_number":17,"context_line":"changed such that the volume we\u0027re swapping from has only"},{"line_number":18,"context_line":"read-only attachments."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ida387c600016b451e01118bc2c76662b46670288"},{"line_number":21,"context_line":"Related-Bug: #1775418"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_9ba4e6a4","line":19,"updated":"2018-06-12 21:37:53.000000000","message":"Why not depend on https://review.openstack.org/#/c/572790 ?\nThe Nova patch still is under review.","commit_id":"2e597ca4e0a414ba1e3a98c4cc8771e3c1f3a40f"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bcd12d3aeac93db13bc953f77951887f42f0ff11","unresolved":false,"context_lines":[{"line_number":16,"context_line":"As a result, test_volume_swap_with_multiattach needs to be"},{"line_number":17,"context_line":"changed such that the volume we\u0027re swapping from has only"},{"line_number":18,"context_line":"read-only attachments."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ida387c600016b451e01118bc2c76662b46670288"},{"line_number":21,"context_line":"Related-Bug: #1775418"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_9eb29305","line":19,"in_reply_to":"5f7c97a3_9ba4e6a4","updated":"2018-06-12 23:16:05.000000000","message":"That would create a circular dependency. We have to change tempest first.","commit_id":"2e597ca4e0a414ba1e3a98c4cc8771e3c1f3a40f"}],"tempest/api/compute/admin/test_volume_swap.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"31edb3bb80e6f00093f25ab2bfb68c04587a344a","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        # flag on an in-use volume so we have to do this before attaching"},{"line_number":166,"context_line":"        # volume1 to anything. If the compute API ever supports per-attachment"},{"line_number":167,"context_line":"        # attach modes, then we can handle this differently."},{"line_number":168,"context_line":"        self.admin_volumes_client.update_volume_readonly("},{"line_number":169,"context_line":"            volume1[\u0027id\u0027], readonly\u003dTrue)"},{"line_number":170,"context_line":"        volume2 \u003d self.create_volume(multiattach\u003dTrue)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        # Create two servers and wait for them to be ACTIVE."}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_e4ef6683","line":169,"range":{"start_line":168,"start_character":8,"end_line":169,"end_character":41},"updated":"2018-10-29 01:29:52.000000000","message":"Shall we add a check to ensure volume\u0027s readonly is really set to True?","commit_id":"c5cce217591ba05862bd04669229bfbffc8d6d96"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"38556fc5e2992e3d8283de960788dd0d96492442","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        # flag on an in-use volume so we have to do this before attaching"},{"line_number":166,"context_line":"        # volume1 to anything. If the compute API ever supports per-attachment"},{"line_number":167,"context_line":"        # attach modes, then we can handle this differently."},{"line_number":168,"context_line":"        self.admin_volumes_client.update_volume_readonly("},{"line_number":169,"context_line":"            volume1[\u0027id\u0027], readonly\u003dTrue)"},{"line_number":170,"context_line":"        volume2 \u003d self.create_volume(multiattach\u003dTrue)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        # Create two servers and wait for them to be ACTIVE."}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_f4d63bf8","line":169,"range":{"start_line":168,"start_character":8,"end_line":169,"end_character":41},"in_reply_to":"3f79a3b5_e4ef6683","updated":"2018-12-10 14:30:03.000000000","message":"It has to be, otherwise the test would be failing on this nova change which depends on it:\n\nhttps://review.openstack.org/#/c/572790/","commit_id":"c5cce217591ba05862bd04669229bfbffc8d6d96"}]}