ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-08-12 00:54:22 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 11

[PR #10599/1bd7aac0 backport][stable-11] open_iscsi: command args as list rather than string (#10633)

Browse source 
open_iscsi: command args as list rather than string (#10599)

* open_iscsi: command args as list rather than string

* add changelog frag

(cherry picked from commit 1bd7aac07e)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>
This commit is contained in:
patchback[bot] 2025-08-10 13:50:43 +02:00 • committed by GitHub
commit 3fa1c3ac2c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/10599-open-iscsi-cmd-list.yml Normal file
View file

@ -0,0 +1,2 @@
minor_changes:
- open_iscsi - using safer mechanism to run external command (https://github.com/ansible-collections/community.general/pull/10599).

2
plugins/modules/open_iscsi.py
View file

@ -242,7 +242,7 @@ def target_login(module, target, check_rc, portal=None, port=None):
params = [('node.session.auth.username_in', node_user_in), params = [('node.session.auth.username_in', node_user_in),
('node.session.auth.password_in', node_pass_in)] ('node.session.auth.password_in', node_pass_in)]
for (name, value) in params: for (name, value) in params:
cmd = '%s --mode node --targetname %s --op=update --name %s --value %s' % (iscsiadm_cmd, target, name, value) cmd = [iscsiadm_cmd, '--mode', 'node', '--targetname', target, '--op=update', '--name', name, '--value', value]
module.run_command(cmd, check_rc=check_rc) module.run_command(cmd, check_rc=check_rc)
cmd = [iscsiadm_cmd, '--mode', 'node', '--targetname', target, '--login'] cmd = [iscsiadm_cmd, '--mode', 'node', '--targetname', target, '--login']