From 8491bf7b49c42aca0b49a86ed31dba0a4d90dfcb Mon Sep 17 00:00:00 2001 From: Peter Shen Date: Tue, 9 Apr 2024 13:44:21 +0800 Subject: [PATCH] fix(aix_filesystem): remove extra param from running lsvg (#8176) * fix(aix_filesystem): remove extra param from running lsvg * chore: add new line to changlog file * Update 8151-fix-lsvg_cmd-failed.yml Co-authored-by: Felix Fontein --------- Co-authored-by: Felix Fontein --- changelogs/fragments/8151-fix-lsvg_cmd-failed.yml | 2 ++ plugins/modules/aix_filesystem.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) create mode 100644 changelogs/fragments/8151-fix-lsvg_cmd-failed.yml diff --git a/changelogs/fragments/8151-fix-lsvg_cmd-failed.yml b/changelogs/fragments/8151-fix-lsvg_cmd-failed.yml new file mode 100644 index 0000000000..0eeee752df --- /dev/null +++ b/changelogs/fragments/8151-fix-lsvg_cmd-failed.yml @@ -0,0 +1,2 @@ +bugfixes: + - aix_filesystem - fix ``_validate_vg`` not passing VG name to ``lsvg_cmd`` (https://github.com/ansible-collections/community.general/issues/8151). diff --git a/plugins/modules/aix_filesystem.py b/plugins/modules/aix_filesystem.py index 6abf6317f2..4a3775c672 100644 --- a/plugins/modules/aix_filesystem.py +++ b/plugins/modules/aix_filesystem.py @@ -242,7 +242,7 @@ def _validate_vg(module, vg): if rc != 0: module.fail_json(msg="Failed executing %s command." % lsvg_cmd) - rc, current_all_vgs, err = module.run_command([lsvg_cmd, "%s"]) + rc, current_all_vgs, err = module.run_command([lsvg_cmd]) if rc != 0: module.fail_json(msg="Failed executing %s command." % lsvg_cmd)