cronvar: Handle empty value string properly (#10445)

* Fix empty  value issue  in cronvar

* Update changelog

* Update plugins/modules/cronvar.py

Co-authored-by: Felix Fontein <felix@fontein.de>

* Update changelogs/fragments/10445-cronvar-reject-empty-values.yml

Co-authored-by: Felix Fontein <felix@fontein.de>

* Update tests/integration/targets/cronvar/tasks/main.yml

Co-authored-by: Felix Fontein <felix@fontein.de>

* Update tests/integration/targets/cronvar/tasks/main.yml

Co-authored-by: Felix Fontein <felix@fontein.de>

* Accept empty strings on cronvar

* Update plugins/modules/cronvar.py

Co-authored-by: Felix Fontein <felix@fontein.de>

* Update main.yml

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
This commit is contained in:
Giorgos Drosos 2025-07-28 07:31:51 +03:00 committed by GitHub
commit 1f8b5eea4c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 21 additions and 0 deletions

View file

@ -123,6 +123,23 @@
- custom_varcheck2.stdout == '1'
- custom_varcheck3.stdout == '0'
- name: Add variable with empty string
community.general.cronvar:
name: EMPTY_VAR
value: ""
state: present
- name: Assert empty var present
ansible.builtin.shell: crontab -l
register: result
changed_when: false
- name: Assert line is quoted
ansible.builtin.assert:
that: >-
'EMPTY_VAR=""' in result.stdout
- name: Attempt to add cron variable to non-existent parent directory
cronvar:
name: NOPARENT_VAR