)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"c4f44eaca6c0425d18759d66e7417a9c18c8376e","unresolved":true,"context_lines":[{"line_number":9,"context_line":"PowerFlex driver does not use openstack volume id as its volume name (or"},{"line_number":10,"context_line":"part of its volume name) due to length limitation. It converts the"},{"line_number":11,"context_line":"cinder uuid to an internal name less than 32 chars and maps this name to"},{"line_number":12,"context_line":"an internal id which is a 16-byte hexdecmimal number. Most PowerFlex"},{"line_number":13,"context_line":"volume operations require the internal id, like volume/snapshot manage."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"There\u0027re 2 conversions:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dc71cb30_e4ad20b0","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":45},"updated":"2020-12-15 00:45:49.000000000","message":"typo","commit_id":"61741109506e154c1ae2876c7943a9c52b347a36"},{"author":{"_account_id":14624,"name":"Sam Wan","email":"sam.wan@emc.com","username":"samwan"},"change_message_id":"31d85e5c6f7a01bb76e6ffdb3810105d120908fd","unresolved":false,"context_lines":[{"line_number":9,"context_line":"PowerFlex driver does not use openstack volume id as its volume name (or"},{"line_number":10,"context_line":"part of its volume name) due to length limitation. It converts the"},{"line_number":11,"context_line":"cinder uuid to an internal name less than 32 chars and maps this name to"},{"line_number":12,"context_line":"an internal id which is a 16-byte hexdecmimal number. Most PowerFlex"},{"line_number":13,"context_line":"volume operations require the internal id, like volume/snapshot manage."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"There\u0027re 2 conversions:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9df8c821_edbc2b79","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":45},"in_reply_to":"dc71cb30_e4ad20b0","updated":"2020-12-15 03:12:08.000000000","message":"will fix the typo.","commit_id":"61741109506e154c1ae2876c7943a9c52b347a36"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"c4f44eaca6c0425d18759d66e7417a9c18c8376e","unresolved":true,"context_lines":[{"line_number":21,"context_line":"tempest to know the PowerFlex internal name or id, thus tempest"},{"line_number":22,"context_line":"volume/snapshot manage cases fail."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"This change is to make PowerFlex accept 3 types of source-id so that"},{"line_number":25,"context_line":"PowerFlex volume/snapshot manage is more user-friendly."},{"line_number":26,"context_line":"1. Openstack volume/snapshot uuid"},{"line_number":27,"context_line":"2. PowerFlex internal name"},{"line_number":28,"context_line":"3. PowerFlex internal id"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Change-Id: I7b99982a5e10271f89e3f9c545bb493eae8f477a"},{"line_number":31,"context_line":"Closes-Bug: #1906380"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a1eada3_d6c31474","line":28,"range":{"start_line":24,"start_character":0,"end_line":28,"end_character":24},"updated":"2020-12-15 00:45:49.000000000","message":"need to make clear if this is powerflex specific change or is applicable to all drivers.","commit_id":"61741109506e154c1ae2876c7943a9c52b347a36"},{"author":{"_account_id":14624,"name":"Sam Wan","email":"sam.wan@emc.com","username":"samwan"},"change_message_id":"31d85e5c6f7a01bb76e6ffdb3810105d120908fd","unresolved":false,"context_lines":[{"line_number":21,"context_line":"tempest to know the PowerFlex internal name or id, thus tempest"},{"line_number":22,"context_line":"volume/snapshot manage cases fail."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"This change is to make PowerFlex accept 3 types of source-id so that"},{"line_number":25,"context_line":"PowerFlex volume/snapshot manage is more user-friendly."},{"line_number":26,"context_line":"1. Openstack volume/snapshot uuid"},{"line_number":27,"context_line":"2. PowerFlex internal name"},{"line_number":28,"context_line":"3. PowerFlex internal id"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Change-Id: I7b99982a5e10271f89e3f9c545bb493eae8f477a"},{"line_number":31,"context_line":"Closes-Bug: #1906380"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"425ae07d_33e1f95b","line":28,"range":{"start_line":24,"start_character":0,"end_line":28,"end_character":24},"in_reply_to":"3a1eada3_d6c31474","updated":"2020-12-15 03:12:08.000000000","message":"This is powerflex specific change.","commit_id":"61741109506e154c1ae2876c7943a9c52b347a36"}],"cinder/tests/unit/fake_constants.py":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"c4f44eaca6c0425d18759d66e7417a9c18c8376e","unresolved":true,"context_lines":[{"line_number":99,"context_line":"UUID4 \u003d \u00274cd72b2b-5a4f-4f24-93dc-7c0212002916\u0027"},{"line_number":100,"context_line":"UUID5 \u003d \u00270a574d83-cacf-42b9-8f9f-8f4faa6d4746\u0027"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"# Fake PowerFlex volume names"},{"line_number":103,"context_line":"# converted from VOLUME_ID using powerflex.utils.id_to_base64"},{"line_number":104,"context_line":"POWERFLEX_OBJ_NAME1 \u003d \u0027HlF355XlSg+xcORfS0afag\u003d\u003d\u0027"},{"line_number":105,"context_line":"# converted from VOLUME2_ID using powerflex.utils.id_to_base64"},{"line_number":106,"context_line":"POWERFLEX_OBJ_NAME2 \u003d \u0027Q6CZFOSVR1+4YgvaPIkY5A\u003d\u003d\u0027"},{"line_number":107,"context_line":"# converted from SNAPSHOT_ID using powerflex.utils.id_to_base64"},{"line_number":108,"context_line":"POWERFLEX_OBJ_NAME3 \u003d \u0027JTsoeOxgR5OtGeZUlux6qw\u003d\u003d\u0027"},{"line_number":109,"context_line":"# converted from SNAPSHOT2_ID using powerflex.utils.id_to_base64"},{"line_number":110,"context_line":"POWERFLEX_OBJ_NAME4 \u003d \u0027wCxE+lZlSiaeZi668l5dLQ\u003d\u003d\u0027"},{"line_number":111,"context_line":"# converted from SNAPSHOT3_ID using powerflex.utils.id_to_base64"},{"line_number":112,"context_line":"POWERFLEX_OBJ_NAME5 \u003d \u0027RU+ZcB4FQZOj7Vw5DD+qGA\u003d\u003d\u0027"},{"line_number":113,"context_line":"# Fake PowerFlex volume object id corresponding to above names"},{"line_number":114,"context_line":"POWERFLEX_OBJ_ID1 \u003d \u0027550b4bf700000001\u0027"},{"line_number":115,"context_line":"POWERFLEX_OBJ_ID2 \u003d \u0027550b4bf700000002\u0027"},{"line_number":116,"context_line":"POWERFLEX_OBJ_ID3 \u003d \u0027550b4bf700000003\u0027"},{"line_number":117,"context_line":"POWERFLEX_OBJ_ID4 \u003d \u0027550b4bf700000004\u0027"},{"line_number":118,"context_line":"POWERFLEX_OBJ_ID5 \u003d \u0027550b4bf700000005\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"0c2753c2_b3a22571","line":118,"range":{"start_line":102,"start_character":0,"end_line":118,"end_character":38},"updated":"2020-12-15 00:45:49.000000000","message":"is it worth making it more generic?\nThat way drivers can specify which format they can handle.\nDefault is current UUID.","commit_id":"61741109506e154c1ae2876c7943a9c52b347a36"},{"author":{"_account_id":14624,"name":"Sam Wan","email":"sam.wan@emc.com","username":"samwan"},"change_message_id":"31d85e5c6f7a01bb76e6ffdb3810105d120908fd","unresolved":false,"context_lines":[{"line_number":99,"context_line":"UUID4 \u003d \u00274cd72b2b-5a4f-4f24-93dc-7c0212002916\u0027"},{"line_number":100,"context_line":"UUID5 \u003d \u00270a574d83-cacf-42b9-8f9f-8f4faa6d4746\u0027"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"# Fake PowerFlex volume names"},{"line_number":103,"context_line":"# converted from VOLUME_ID using powerflex.utils.id_to_base64"},{"line_number":104,"context_line":"POWERFLEX_OBJ_NAME1 \u003d \u0027HlF355XlSg+xcORfS0afag\u003d\u003d\u0027"},{"line_number":105,"context_line":"# converted from VOLUME2_ID using powerflex.utils.id_to_base64"},{"line_number":106,"context_line":"POWERFLEX_OBJ_NAME2 \u003d \u0027Q6CZFOSVR1+4YgvaPIkY5A\u003d\u003d\u0027"},{"line_number":107,"context_line":"# converted from SNAPSHOT_ID using powerflex.utils.id_to_base64"},{"line_number":108,"context_line":"POWERFLEX_OBJ_NAME3 \u003d \u0027JTsoeOxgR5OtGeZUlux6qw\u003d\u003d\u0027"},{"line_number":109,"context_line":"# converted from SNAPSHOT2_ID using powerflex.utils.id_to_base64"},{"line_number":110,"context_line":"POWERFLEX_OBJ_NAME4 \u003d \u0027wCxE+lZlSiaeZi668l5dLQ\u003d\u003d\u0027"},{"line_number":111,"context_line":"# converted from SNAPSHOT3_ID using powerflex.utils.id_to_base64"},{"line_number":112,"context_line":"POWERFLEX_OBJ_NAME5 \u003d \u0027RU+ZcB4FQZOj7Vw5DD+qGA\u003d\u003d\u0027"},{"line_number":113,"context_line":"# Fake PowerFlex volume object id corresponding to above names"},{"line_number":114,"context_line":"POWERFLEX_OBJ_ID1 \u003d \u0027550b4bf700000001\u0027"},{"line_number":115,"context_line":"POWERFLEX_OBJ_ID2 \u003d \u0027550b4bf700000002\u0027"},{"line_number":116,"context_line":"POWERFLEX_OBJ_ID3 \u003d \u0027550b4bf700000003\u0027"},{"line_number":117,"context_line":"POWERFLEX_OBJ_ID4 \u003d \u0027550b4bf700000004\u0027"},{"line_number":118,"context_line":"POWERFLEX_OBJ_ID5 \u003d \u0027550b4bf700000005\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"2c05c7ba_c47b1f3b","line":118,"range":{"start_line":102,"start_character":0,"end_line":118,"end_character":38},"in_reply_to":"0c2753c2_b3a22571","updated":"2020-12-15 03:12:08.000000000","message":"I\u0027m not sure what \u0027more generic\u0027 means.\nThe fix allows powerflex to accept 3 types of format:\n1. uuid\n2. powerflex internal name\n3. powerflex object id","commit_id":"61741109506e154c1ae2876c7943a9c52b347a36"}]}