)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"5f4505975e4a3b46c38c0d11500db3d8cb150bdb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"08d28492_73ef02ca","updated":"2026-02-20 14:54:59.000000000","message":"Agree that this necessary","commit_id":"66c0cace80900a95bc871d82e05b55b4df16b63b"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"b12b5e308d9d9bda1dba41f00c6bc86ba4517c2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e758f666_d05cf1a9","updated":"2026-02-21 23:25:14.000000000","message":"I found a few more occurrences of the logging:\n\nos_brick/initiator/windows/fibre_channel.py: 91\nos_brick/initiator/windows/iscsi.py: 143\nos_brick/initiator/connectors/storpool.py: 192\n\nbut there are many others. Can you please run a grep search and see if there are moreee if there are more?","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"23418735899ef9135c4f26deffacf0c02a837314","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"82d37727_6e9a16bd","updated":"2026-02-21 04:14:11.000000000","message":"Important fix that we need to get in as CHAP creds are visible in logs.\n\nJan 22 06:48:37.443822 np34a274cd238c4 nova-compute[89280]: DEBUG os_brick.initiator.connectors.base [None req-3bb843f4-ed26-4334-a2d5-9b46ffceec6f tempest-VolumesAssistedSnapshotsTest-759639528 tempest-VolumesAssistedSnapshotsTest-759639528-project-admin] Connection properties {\u0027target_discovered\u0027: False, \u0027target_portal\u0027: \u0027158.69.73.31:3260\u0027, \u0027target_iqn\u0027: \u0027iqn.2010-10.org.openstack:volume-6aa194ac-84a2-481f-a97e-689524ef023c\u0027, \u0027target_lun\u0027: 0, \u0027volume_id\u0027: \u00276aa194ac-84a2-481f-a97e-689524ef023c\u0027, \u0027auth_method\u0027: \u0027CHAP\u0027, \u0027auth_username\u0027: \u0027AsWWMsGcThFbQsdqLBg7\u0027, \u0027auth_password\u0027: \u0027DhKUpaVf5CRCrS84\u0027, \u0027encrypted\u0027: False, \u0027qos_specs\u0027: None, \u0027access_mode\u0027: \u0027rw\u0027, \u0027cacheable\u0027: False, \u0027enforce_multipath\u0027: True} {{(pid\u003d89280) check_multipath /opt/stack/data/venv/lib/python3.12/site-packages/os_brick/initiator/connectors/base.py:133}}","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"facd619a4d0617aa0f0498b44c334497dace87a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1bbae95a_03795b0e","updated":"2026-02-21 04:34:13.000000000","message":"Important patch. LGTM","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6836f5ea1a75d5f8a8cebf51203260481723429b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"521edfcc_32bed6d4","updated":"2026-02-22 22:54:26.000000000","message":"Important to get this in before the gazpacho release.","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6251f6fa1ff28184d19b267bd01b324665ebe29c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6341845c_e21d140b","in_reply_to":"82d37727_6e9a16bd","updated":"2026-02-21 04:15:02.000000000","message":"After this fix, passwords are masked\n\nFeb 21 01:18:06.857220 npc27ce0e0147f4 nova-compute[93643]: DEBUG os_brick.initiator.connectors.base [None req-deb3240d-5682-4314-9f9c-d19773d0dfbf tempest-VolumesAssistedSnapshotsTest-819524431 tempest-VolumesAssistedSnapshotsTest-819524431-project-admin] Connection properties {\u0027target_discovered\u0027: False, \u0027target_portal\u0027: \u002710.209.96.213:3260\u0027, \u0027target_iqn\u0027: \u0027iqn.2010-10.org.openstack:volume-19bd228f-1aab-4a36-88c7-58ccb55e5489\u0027, \u0027target_lun\u0027: 0, \u0027volume_id\u0027: \u002719bd228f-1aab-4a36-88c7-58ccb55e5489\u0027, \u0027auth_method\u0027: \u0027CHAP\u0027, \u0027auth_username\u0027: \u0027Uac2gEQp3HRAVJ9w4qFn\u0027, \u0027auth_password\u0027: \u0027***\u0027, \u0027encrypted\u0027: False, \u0027qos_specs\u0027: None, \u0027access_mode\u0027: \u0027rw\u0027, \u0027cacheable\u0027: False, \u0027enforce_multipath\u0027: True} {{(pid\u003d93643) check_multipath /opt/stack/os-brick/os_brick/initiator/connectors/base.py:132}}","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"222a876204ed2ddb62d6f75ed87b3a15ebe64d2f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cbbc912f_0b261a2b","in_reply_to":"ba7a0c2f_dc6d1811","updated":"2026-02-23 03:08:48.000000000","message":"I agree there are a few more left in the code (for example in fiberchannel.py ) not all of these actually may get properties with credentials.\nWhile we probably want to audit these all I avoided extending the scope of this change too much at this moment given we are quite close to the release deadline.","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6836f5ea1a75d5f8a8cebf51203260481723429b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ba7a0c2f_dc6d1811","in_reply_to":"e758f666_d05cf1a9","updated":"2026-02-22 22:54:26.000000000","message":"I think you are looking at an outdated code repo.  Windows support was removed with change Iff55251ce399 and the storpool log message doesn\u0027t include the actual connection_properties.\n\nIt would be worthwhile for someone to look for more instances, but I don\u0027t want to hold up this patch.","commit_id":"0a3b639014adf83dcce76ef7d4de4ea7134defcf"}]}