Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flatpak module: incorrect task status reported for updating depending on language #8883

Closed
1 task done
kevonfernando opened this issue Sep 19, 2024 · 3 comments · Fixed by #9187
Closed
1 task done
Labels
bug This issue/PR relates to a bug module module plugins plugin (any type)

Comments

@kevonfernando
Copy link

Summary

Installing/Updating apps with the flatpak module always reports changed as task status when language is not english. See flatpak.py line 250

Issue Type

Bug Report

Component Name

flatpak

Ansible Version

$ ansible --version

Community.general Version

$ ansible-galaxy collection list community.general
ansible [core 2.16.10]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/var/home/kevonfernando/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3.12/site-packages/ansible
  ansible collection location = /var/home/kevonfernando/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/bin/ansible
  python version = 3.12.6 (main, Sep  9 2024, 00:00:00) [GCC 14.2.1 20240801 (Red Hat 14.2.1-1)] (/usr/bin/python3)
  jinja version = 3.1.4
  libyaml = True

Configuration

$ ansible-config dump --only-changed

OS / Environment

Control node:

  • Fedora 40 toolbox container
  • Ansible installed from Fedora 40 repos
    Target node:
  • Fedora Silverblue 40

Steps to Reproduce

- name: Install flatpaks from the Flathub remote to the system installation
  community.general.flatpak:
    method: system
    remote: flathub
    name:
      - page.kramo.Cartridges
    state: latest

Expected Results

I expected the reported task status to be ok and not changed.

Actual Results

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot
Copy link
Collaborator

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added bug This issue/PR relates to a bug module module plugins plugin (any type) labels Sep 19, 2024
@felixfontein
Copy link
Collaborator

I guess something like module.run_command_environ_update = dict(LANG='C', LC_ALL='C', LC_MESSAGES='C', LC_CTYPE='C') (or using the environ_update parameter of module.run_command() with a similar dict) would help here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants