)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":39262,"name":"Rafal Zielinski","display_name":"sq4ind","email":"sq4ind@gmail.com","username":"sq4ind"},"change_message_id":"075f9fa57445f31dff0c0b29f15177a88ae64bef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b32a3495_1fa47754","updated":"2026-06-25 12:15:29.000000000","message":"Hi Kolla team,\n\nCould I get review on this master-branch fix for Launchpad bug 2154683?\n\nContext: I initially proposed the fix against stable/2026.1, but as @mnasiadka@gmail.com pointed out, this needs to land on master first and then be backported. This patch is the master version.\n\nThe issue affects external Cinder iSCSI backends with LVM disabled, for example:\n\n  enable_cinder_backend_pure_iscsi: true\n  enable_cinder_backend_iscsi: true\n  enable_cinder_backend_lvm: false\n\nWith this configuration, the iscsid service can remain disabled for the cinder-volume host because enable_iscsid_for_cinder only evaluates the effective cinder-volume / cinder-backup groups when enable_cinder_backend_lvm is true.\n\nIn my reproduced case, Kolla skipped the iscsid setup with:\n\n  false_condition: \"iscsi_services.iscsid.enabled | bool\"\n\nCinder then reached the external iSCSI backend correctly: the Pure driver selected the expected target portals and the array mapped the volume to the generated OpenStack host object. However, os-brick/iscsiadm failed all target portal logins with exit code 20 because the local iscsid side was not running/deployed for the cinder-volume host.\n\nObserved Cinder/os-brick symptoms:\n\n  Failed to login iSCSI target ... on portal ... (exit code 20)\n  iscsiadm: No active sessions\n  VolumeDeviceNotFound\n  ImageCopyFailure\n\nThe patch keeps the existing dynamic helper groups:\n\n  iscsid_cinder_volume_group\n  iscsid_cinder_backup_group\n\nbut removes the incorrect LVM-only gating from enable_iscsid_for_cinder, so external non-LVM iSCSI backends can deploy iscsid on cinder-volume / cinder-backup hosts as expected.\n\nOnce this lands on master, I will propose the stable/2026.1 backport.","commit_id":"5afac4e2efa10e307eb300cc9cfe5692078cef5d"}]}