2.9: native types: literal_eval all the things (#68938)

[mkrizek]

SUMMARY

With pallets/jinja#1190 merged our short-circuit
is no longer valid (has it ever been?) as now data like ' True ' may go
through our ansible_native_concat function as opposed to going through
intermediate call to Jinja2's native_concat before. Now we need to always
send data through literal_eval to ensure native types are returned.

(cherry picked from commit acdc9eb)

Backport of #68938

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

lib/ansible/template/native_helpers.py

[mkrizek]

So this fails on

ansible/test/integration/targets/inventory_aws_conformance/inventory_diff.py

Line 36 in 6410570

assert value == plugin['_meta']['hostvars'][host][key], "%s != %s" % (value, plugin['_meta']['hostvars'][host][key])

which is not in devel.

Traceback (most recent call last):
  File "./inventory_diff.py", line 66, in <module>
    main()
  File "./inventory_diff.py", line 62, in main
    check_host_vars(key, value, bdata, ahost)
  File "./inventory_diff.py", line 38, in check_host_vars
    assert value == plugin['_meta']['hostvars'][host][key], "%s != %s" % (value, plugin['_meta']['hostvars'][host][key])
AssertionError: 123456789012 != 123456789012

The values that are being compared are of different types:

value=123456789012 of type=<class 'str'>
plugin['_meta']['hostvars'][host][key]=123456789012 of type=<class 'int'>

I am looking into why is that happening.