change of parsing of values? (in ansible.cfg)

70 views
Skip to first unread message

Adrian Sevcenco

unread,
Jan 6, 2024, 3:53:24 PM1/6/24
to ansible...@googlegroups.com
So, after updating of ansible (i do not know the versions that i had)
and in the context of setting values for redis fact caching
see https://github.com/ansible-collections/community.general/issues/7814
i notice that quotes are now interpreted literally instead of being evaluated...
so for an empty variable now i need
a_var =
instead of
a_var = ""
because the later case would just set
a_var = '""'

Is anyone aware of this change? Maybe of the reason for it?

Thanks a lot!
Adrian

Brian Coca

unread,
Jan 8, 2024, 1:42:55 PM1/8/24
to ansible...@googlegroups.com
Not really, only thing I can think of is this PR
https://github.com/ansible/ansible/pull/79837, but ini should have
kept the old behavior.


--
----------
Brian Coca (he/him/yo)

Brian Coca

unread,
Jan 8, 2024, 2:27:17 PM1/8/24
to ansible...@googlegroups.com
sorry, yes, that pr above broke it, this one should fix it
https://github.com/ansible/ansible/pull/82388

my brain just came back from vacation and reminded me ....

Adrian Sevcenco

unread,
Jan 10, 2024, 3:56:52 PM1/10/24
to ansible...@googlegroups.com
On 08.01.2024 21:26, Brian Coca wrote:
> sorry, yes, that pr above broke it, this one should fix it
> https://github.com/ansible/ansible/pull/82388
>
> my brain just came back from vacation and reminded me ....

Thanks a lot for info! I closed the community.general issue
with your information!

I'm not sure from the code of the fix: at this moment i made the things work
by removing quotes, after the fix will be apply, should i expect any kind of breakage
because of lack of quotes?

Thanks a lot!
Adrian

Brian Coca

unread,
Jan 11, 2024, 12:39:37 PM1/11/24
to ansible...@googlegroups.com
It should not break again, the code had 'ini' source detection broken,
since it was looking for 'ini' as origin, but we set
'/path/to/ansible.cfg' instead, so it was not matching.
Reply all
Reply to author
Forward
0 new messages