)]}'
{"tempest/api/volume/admin/test_volume_retype.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"9f1dfa18b6d3b59edef17f59cf5536ea844c6826","unresolved":false,"context_lines":[{"line_number":159,"context_line":"                               \u0027user_id\u0027, \u0027os-vol-tenant-attr:tenant_id\u0027,)"},{"line_number":160,"context_line":"        keys_with_change \u003d (\u0027volume_type\u0027,)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        # NOTE(vsaienko): with active-active cluster deployment volume"},{"line_number":163,"context_line":"        # services registered with different hostname."},{"line_number":164,"context_line":"        if CONF.volume_feature_enabled.cluster_active_active:"},{"line_number":165,"context_line":"            keys_with_change +\u003d (\u0027os-vol-host-attr:host\u0027,)"},{"line_number":166,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1fa4df85_3eca264f","line":163,"range":{"start_line":162,"start_character":0,"end_line":163,"end_character":54},"updated":"2020-03-12 09:37:37.000000000","message":"A silly question:\nAre volume services sure to be registered with different hostname? or just MAY be different?","commit_id":"0eaec7a123d097c525690bc991e1cff3198a5a0e"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"0769cfaf24c9a55a374b7f7c9230df2e8c3d5ff1","unresolved":false,"context_lines":[{"line_number":159,"context_line":"                               \u0027user_id\u0027, \u0027os-vol-tenant-attr:tenant_id\u0027,)"},{"line_number":160,"context_line":"        keys_with_change \u003d (\u0027volume_type\u0027,)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        # NOTE(vsaienko): with active-active cluster deployment volume"},{"line_number":163,"context_line":"        # services registered with different hostname."},{"line_number":164,"context_line":"        if CONF.volume_feature_enabled.cluster_active_active:"},{"line_number":165,"context_line":"            keys_with_change +\u003d (\u0027os-vol-host-attr:host\u0027,)"},{"line_number":166,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_d1902798","line":163,"range":{"start_line":162,"start_character":0,"end_line":163,"end_character":54},"in_reply_to":"1fa4df85_3eca264f","updated":"2020-04-29 18:22:51.000000000","message":"they have to be uniq if you are using cluster\u003dTrue (Active/Active) mode \nhttps://docs.openstack.org/cinder/latest/configuration/block-storage/samples/cinder.conf.html","commit_id":"0eaec7a123d097c525690bc991e1cff3198a5a0e"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"cfd367c0d2ed3f801d1b3914d98aa93fd81ebe97","unresolved":false,"context_lines":[{"line_number":159,"context_line":"                               \u0027user_id\u0027, \u0027os-vol-tenant-attr:tenant_id\u0027,)"},{"line_number":160,"context_line":"        keys_with_change \u003d (\u0027volume_type\u0027,)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        # NOTE(vsaienko): with active-active cluster deployment volume"},{"line_number":163,"context_line":"        # services registered with different hostname since we don\u0027t know"},{"line_number":164,"context_line":"        # which service process request host might or might not be changed."},{"line_number":165,"context_line":"        if not CONF.volume_feature_enabled.cluster_active_active:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f493fa4_c76d9605","line":162,"updated":"2020-04-23 09:54:11.000000000","message":"Please, add a TODO, that we need to check \u0027cluster\u0027 attribute didn\u0027t change in A/A once Cinder API will return it","commit_id":"aef2aead8965d557d1de4a9aa80f2d7c5de04153"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"35a7327f929475ca43e9d770a18d3709a90c4d84","unresolved":false,"context_lines":[{"line_number":159,"context_line":"                               \u0027user_id\u0027, \u0027os-vol-tenant-attr:tenant_id\u0027,)"},{"line_number":160,"context_line":"        keys_with_change \u003d (\u0027volume_type\u0027,)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        # NOTE(vsaienko): with active-active cluster deployment volume"},{"line_number":163,"context_line":"        # services registered with different hostname since we don\u0027t know"},{"line_number":164,"context_line":"        # which service process request host might or might not be changed."},{"line_number":165,"context_line":"        # Revisit logic when is fixed"},{"line_number":166,"context_line":"        # https://bugs.launchpad.net/cinder/+bug/1874414"},{"line_number":167,"context_line":"        if not CONF.volume_feature_enabled.cluster_active_active:"}],"source_content_type":"text/x-python","patch_set":6,"id":"ff570b3c_c1fefb36","line":164,"range":{"start_line":162,"start_character":0,"end_line":164,"end_character":75},"updated":"2020-05-14 00:49:16.000000000","message":"i am confused on this. so this is non-migration test ( migration_policy\u003d\u0027never\u0027) so host for existing volume should not be change in any cluster. I mean this test does not comparing the static host value here what this tests compare is source and destination volume host remain same or not.\n\nif in active active mode volume gets migrated to new host with  migration_policy\u003d\u0027never\u0027 then it is issue.","commit_id":"61dda8c0e20f245505aea3bca33ff26bce53c4b2"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"e0f4c075ce6499bdaa043ee142d2171f60688451","unresolved":false,"context_lines":[{"line_number":159,"context_line":"                               \u0027user_id\u0027, \u0027os-vol-tenant-attr:tenant_id\u0027,)"},{"line_number":160,"context_line":"        keys_with_change \u003d (\u0027volume_type\u0027,)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        # NOTE(vsaienko): with active-active cluster deployment volume"},{"line_number":163,"context_line":"        # services registered with different hostname since we don\u0027t know"},{"line_number":164,"context_line":"        # which service process request host might or might not be changed."},{"line_number":165,"context_line":"        # Revisit logic when is fixed"},{"line_number":166,"context_line":"        # https://bugs.launchpad.net/cinder/+bug/1874414"},{"line_number":167,"context_line":"        if not CONF.volume_feature_enabled.cluster_active_active:"}],"source_content_type":"text/x-python","patch_set":6,"id":"ff570b3c_5d463251","line":164,"range":{"start_line":162,"start_character":0,"end_line":164,"end_character":75},"in_reply_to":"ff570b3c_c1fefb36","updated":"2020-05-15 10:13:31.000000000","message":"with A/A mode, cluster name should be used instead of hostname. Actually, a volume won\u0027t be migrated between storages but could be managed by a different volume service in the same cinder cluster","commit_id":"61dda8c0e20f245505aea3bca33ff26bce53c4b2"}]}