From 6e0bc4f45c4cd4a7e375f0a213c95efd7ce0da54 Mon Sep 17 00:00:00 2001 From: Svenum <43136984+Svenum@users.noreply.github.com> Date: Thu, 20 Apr 2023 06:15:46 +0200 Subject: [PATCH] =?UTF-8?q?Remove=20--app=20by=20flatpak=20check=20if=20al?= =?UTF-8?q?ready=20installed=20(ansible-collectio=E2=80=A6=20(#6289)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Remove --app by flatpak check if already installed (ansible-collections#6265) * Add Changelogfragment * Fix syntax * Update changelogs/fragments/6289-bugfix-flatpak-check-if-already-installed.yml Co-authored-by: Felix Fontein --------- Co-authored-by: Felix Fontein --- .../6289-bugfix-flatpak-check-if-already-installed.yml | 2 ++ plugins/modules/flatpak.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) create mode 100644 changelogs/fragments/6289-bugfix-flatpak-check-if-already-installed.yml diff --git a/changelogs/fragments/6289-bugfix-flatpak-check-if-already-installed.yml b/changelogs/fragments/6289-bugfix-flatpak-check-if-already-installed.yml new file mode 100644 index 0000000000..e331250035 --- /dev/null +++ b/changelogs/fragments/6289-bugfix-flatpak-check-if-already-installed.yml @@ -0,0 +1,2 @@ +bugfixes: + - flatpak - fixes idempotency detection issues. In some cases the module could fail to properly detect already existing Flatpaks because of a parameter witch only checks the installed apps (https://github.com/ansible-collections/community.general/pull/6289). diff --git a/plugins/modules/flatpak.py b/plugins/modules/flatpak.py index feda6ed5b4..40a13736f0 100644 --- a/plugins/modules/flatpak.py +++ b/plugins/modules/flatpak.py @@ -215,7 +215,7 @@ def uninstall_flat(module, binary, names, method): def flatpak_exists(module, binary, names, method): """Check if the flatpaks are installed.""" - command = [binary, "list", "--{0}".format(method), "--app"] + command = [binary, "list", "--{0}".format(method)] output = _flatpak_command(module, False, command) installed = [] not_installed = []