)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ea0dd6afac969aa5f78f2a1109a1484d664e7d52","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove timestamp assertions from volume retype test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The use of a timestamp written to the volume is slow, error prone, and"},{"line_number":10,"context_line":"not enough to ensure the successful retype of a volume."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change replaces this assertions with specific state checks"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_eb9135f9","line":9,"range":{"start_line":9,"start_character":54,"end_line":9,"end_character":65},"updated":"2019-03-04 17:07:39.000000000","message":"how so?","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ea0dd6afac969aa5f78f2a1109a1484d664e7d52","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove timestamp assertions from volume retype test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The use of a timestamp written to the volume is slow, error prone, and"},{"line_number":10,"context_line":"not enough to ensure the successful retype of a volume."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change replaces this assertions with specific state checks"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_4bf7215b","line":9,"range":{"start_line":9,"start_character":48,"end_line":9,"end_character":52},"updated":"2019-03-04 17:07:39.000000000","message":"why?","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ea0dd6afac969aa5f78f2a1109a1484d664e7d52","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove timestamp assertions from volume retype test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The use of a timestamp written to the volume is slow, error prone, and"},{"line_number":10,"context_line":"not enough to ensure the successful retype of a volume."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change replaces this assertions with specific state checks"},{"line_number":13,"context_line":"regarding the volume, the migration and the instance."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_0b97d9ed","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":55},"updated":"2019-03-04 17:07:39.000000000","message":"why?","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"8b6f43f9f595712e5610f74f35de74f58e3ad5db","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove timestamp assertions from volume retype test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The use of a timestamp written to the volume is slow, error prone, and"},{"line_number":10,"context_line":"not enough to ensure the successful retype of a volume."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change replaces this assertions with specific state checks"},{"line_number":13,"context_line":"regarding the volume, the migration and the instance."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_2348b485","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":55},"in_reply_to":"9fdfeff1_0b97d9ed","updated":"2019-03-05 10:39:02.000000000","message":"Yup awful commit message and pointless to remove it if it\u0027s passing in the gate. The slowness and instability appear to be localised to my Fedora devstack env. I\u0027ll try to dig into that separately but for now I\u0027ll just add some additional assertions here as requested by mdbooth.","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ea0dd6afac969aa5f78f2a1109a1484d664e7d52","unresolved":false,"context_lines":[{"line_number":10,"context_line":"not enough to ensure the successful retype of a volume."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change replaces this assertions with specific state checks"},{"line_number":13,"context_line":"regarding the volume, the migration and the instance."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I32f0611bdfd2ccede73e7ab774286f5315ff92c3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_ab9b2d15","line":13,"updated":"2019-03-04 17:07:39.000000000","message":"Do you have a bug for this?","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"8b6f43f9f595712e5610f74f35de74f58e3ad5db","unresolved":false,"context_lines":[{"line_number":10,"context_line":"not enough to ensure the successful retype of a volume."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This change replaces this assertions with specific state checks"},{"line_number":13,"context_line":"regarding the volume, the migration and the instance."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I32f0611bdfd2ccede73e7ab774286f5315ff92c3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fdfeff1_c38528b0","line":13,"in_reply_to":"9fdfeff1_ab9b2d15","updated":"2019-03-05 10:39:02.000000000","message":"I\u0027ll mark it as related to the volume migration bugfix for bug #1803961.","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"}],"tempest/scenario/test_volume_migrate_attached.py":[{"author":{"_account_id":9555,"name":"Matthew Booth","email":"mbooth@redhat.com","username":"MatthewBooth"},"change_message_id":"84951126b6ab334eced42be1b6e1b92d8b12eff3","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        # Assert that the volume is in-use and migration_status is success"},{"line_number":119,"context_line":"        volume \u003d self.admin_volumes_client.show_volume(volume_id)[\u0027volume\u0027]"},{"line_number":120,"context_line":"        self.assertEqual(\u0027in-use\u0027, volume[\u0027status\u0027])"},{"line_number":121,"context_line":"        self.assertEqual(\u0027success\u0027, volume[\u0027migration_status\u0027])"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        # Assert that the same volume id is attached to the instance, ensuring"},{"line_number":124,"context_line":"        # the os-migrate_volume_completion Cinder API has been called."}],"source_content_type":"text/x-python","patch_set":3,"id":"9fdfeff1_c0d9b528","line":121,"updated":"2019-03-04 13:13:44.000000000","message":"Is it worth asserting here that the volume\u0027s host changed?","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"1d8ba28d182cbd77cbc018326bcc8be5e80b7bde","unresolved":false,"context_lines":[{"line_number":118,"context_line":"        # Assert that the volume is in-use and migration_status is success"},{"line_number":119,"context_line":"        volume \u003d self.admin_volumes_client.show_volume(volume_id)[\u0027volume\u0027]"},{"line_number":120,"context_line":"        self.assertEqual(\u0027in-use\u0027, volume[\u0027status\u0027])"},{"line_number":121,"context_line":"        self.assertEqual(\u0027success\u0027, volume[\u0027migration_status\u0027])"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        # Assert that the same volume id is attached to the instance, ensuring"},{"line_number":124,"context_line":"        # the os-migrate_volume_completion Cinder API has been called."}],"source_content_type":"text/x-python","patch_set":3,"id":"9fdfeff1_e04dd999","line":121,"in_reply_to":"9fdfeff1_c0d9b528","updated":"2019-03-04 13:25:18.000000000","message":"Maybe, the type change is already checked by the above waiter but I can always pull back the host check from the following migration change.","commit_id":"09708438f08a7c64deba2758fbd7c429197d1be3"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"0420368e1372359eddbf2f1868c1e31066fa7804","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        # migration_status of success"},{"line_number":141,"context_line":"        volume \u003d self.admin_volumes_client.show_volume(volume_id)[\u0027volume\u0027]"},{"line_number":142,"context_line":"        # dest_type is host@backend, os-vol-host-attr:host is host@backend#type"},{"line_number":143,"context_line":"        self.assertTrue(dest_type[\u0027host\u0027] in volume[\u0027os-vol-host-attr:host\u0027])"},{"line_number":144,"context_line":"        self.assertEqual(\u0027in-use\u0027, volume[\u0027status\u0027])"},{"line_number":145,"context_line":"        self.assertEqual(\u0027success\u0027, volume[\u0027migration_status\u0027])"},{"line_number":146,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"5fc1f717_873afa95","line":143,"range":{"start_line":143,"start_character":8,"end_line":143,"end_character":77},"updated":"2019-03-19 08:59:03.000000000","message":"self.assertIn(dest_type[\u0027host\u0027], volume[\u0027os-vol-host-attr:host\u0027])","commit_id":"52a5096179b0735084d61dc86d05de1ac53f711a"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"794fde7050f912599019e4778fbb3e3e45b9286e","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":98,"context_line":"    @decorators.idempotent_id(\u0027deadd2c2-beef-4dce-98be-f86765ff311b\u0027)"},{"line_number":99,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":100,"context_line":"    def test_volume_migrate_attached(self):"},{"line_number":101,"context_line":"        LOG.info(\"Creating keypair and security group\")"},{"line_number":102,"context_line":"        keypair \u003d self.create_keypair()"}],"source_content_type":"text/x-python","patch_set":9,"id":"3fce034c_495bc62d","line":99,"updated":"2019-04-11 08:21:33.000000000","message":"Using `@decorators.related_bug()` could be helpful to understand the additional tests?","commit_id":"67ae340460211a15cf05aa5458fcc9f4f8135035"}]}