)]}'
{"cinder/volume/drivers/dell_emc/powerstore/adapter.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"7a17d97be7c007dd6b1158aac216b06a9a73e82e","unresolved":true,"context_lines":[{"line_number":654,"context_line":"            )"},{"line_number":655,"context_line":"            connection_properties[\"data\"][\"auth_password\"] \u003d ("},{"line_number":656,"context_line":"                chap_credentials.get(\"chap_single_password\")"},{"line_number":657,"context_line":"            )"},{"line_number":658,"context_line":"        return connection_properties"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"    def _detach_volume_from_hosts(self, volume, hosts_to_detach\u003dNone):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bce21370_c5f5818d","line":657,"updated":"2020-12-07 21:50:03.000000000","message":"Some unit test should validate that auth_username and auth_password are returned here, but I don\u0027t see this in the tests.","commit_id":"7f17a46c5c4579340a6a7ad02731137050f37c52"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"25122cb68a96c106c058b8bd5c00c9151aa851d2","unresolved":false,"context_lines":[{"line_number":654,"context_line":"            )"},{"line_number":655,"context_line":"            connection_properties[\"data\"][\"auth_password\"] \u003d ("},{"line_number":656,"context_line":"                chap_credentials.get(\"chap_single_password\")"},{"line_number":657,"context_line":"            )"},{"line_number":658,"context_line":"        return connection_properties"},{"line_number":659,"context_line":""},{"line_number":660,"context_line":"    def _detach_volume_from_hosts(self, volume, hosts_to_detach\u003dNone):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab8193a_59a70545","line":657,"in_reply_to":"bce21370_c5f5818d","updated":"2020-12-08 11:52:02.000000000","message":"Corresponding tests were added.","commit_id":"7f17a46c5c4579340a6a7ad02731137050f37c52"}],"cinder/volume/drivers/dell_emc/powerstore/utils.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5e87cc0468bc55cc8e668663e228a79aae763baf","unresolved":true,"context_lines":[{"line_number":146,"context_line":"    \"\"\""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"    return {"},{"line_number":149,"context_line":"        \"chap_single_username\": CHAP_DEFAULT_USERNAME,"},{"line_number":150,"context_line":"        \"chap_single_password\": volume_utils.generate_password("},{"line_number":151,"context_line":"            CHAP_DEFAULT_SECRET_LENGTH"},{"line_number":152,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"a26fb080_bb91acc5","line":149,"range":{"start_line":149,"start_character":32,"end_line":149,"end_character":53},"updated":"2020-12-10 22:12:13.000000000","message":"I guess it doesn\u0027t matter that the username is always the same as long as the passwords keep changing.","commit_id":"7d50d58b634e9821dbd680303f25c0eee2116598"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"e76ad2f618b2f77ddb2e12b03036353324d7d8c1","unresolved":false,"context_lines":[{"line_number":146,"context_line":"    \"\"\""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"    return {"},{"line_number":149,"context_line":"        \"chap_single_username\": CHAP_DEFAULT_USERNAME,"},{"line_number":150,"context_line":"        \"chap_single_password\": volume_utils.generate_password("},{"line_number":151,"context_line":"            CHAP_DEFAULT_SECRET_LENGTH"},{"line_number":152,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"6e81f5dd_e76dd2a0","line":149,"range":{"start_line":149,"start_character":32,"end_line":149,"end_character":53},"in_reply_to":"a26fb080_bb91acc5","updated":"2020-12-11 06:59:14.000000000","message":"Yes, NetApp, for example, uses the same approach.","commit_id":"7d50d58b634e9821dbd680303f25c0eee2116598"}],"doc/source/configuration/block-storage/drivers/dell-emc-powerstore-driver.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5e87cc0468bc55cc8e668663e228a79aae763baf","unresolved":true,"context_lines":[{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The driver supports one-way (Single mode) CHAP authentication."},{"line_number":85,"context_line":"To use CHAP authentication CHAP Single mode has to be enabled on the storage"},{"line_number":86,"context_line":"side."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":".. note:: When enabling CHAP, any previously added hosts will need to be updated"},{"line_number":89,"context_line":"          with CHAP configuration since there will be I/O disruption for those hosts."}],"source_content_type":"text/x-rst","patch_set":4,"id":"bbdf880f_6fc924c4","line":86,"updated":"2020-12-10 22:12:13.000000000","message":"You might want to mention that the driver checks to see if CHAP is enabled on the backend and acts appropriately (because i think with PowerMax, it has to be enabled in the cinder config file).","commit_id":"7d50d58b634e9821dbd680303f25c0eee2116598"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"e76ad2f618b2f77ddb2e12b03036353324d7d8c1","unresolved":false,"context_lines":[{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The driver supports one-way (Single mode) CHAP authentication."},{"line_number":85,"context_line":"To use CHAP authentication CHAP Single mode has to be enabled on the storage"},{"line_number":86,"context_line":"side."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":".. note:: When enabling CHAP, any previously added hosts will need to be updated"},{"line_number":89,"context_line":"          with CHAP configuration since there will be I/O disruption for those hosts."}],"source_content_type":"text/x-rst","patch_set":4,"id":"c71b62d7_2a376dcc","line":86,"in_reply_to":"bbdf880f_6fc924c4","updated":"2020-12-11 06:59:14.000000000","message":"I think it is clearly described below.","commit_id":"7d50d58b634e9821dbd680303f25c0eee2116598"}]}