ansible

Commit Graph

Author	SHA1	Message	Date
Feanil Patel	892e230514	Don't convert nulls to strings. This change is similar to https://github.com/ansible/ansible/pull/10465 It extends the logic there to also support none types. Right now if you have a '!!null' in yaml, and that var gets passed around, it will get converted to a string. eg. defaults/main.yml ``` ENABLE_AWESOME_FEATURE: !!null # Yaml Null OTHER_CONFIG: secret1: "so_secret" secret2: "even_more_secret" CONFIG: hostname: "some_hostname" features: awesame_feature: "{{ ENABLE_AWESOME_FEATURE}}" secrets: "{{ OTHER_CONFIG }}" ``` If you output `CONFIG` to json or yaml, the feature flag would get represented in the output as a string instead of as a null, but secrets would get represented as a dictionary. This is a mis-match in behaviour where some "types" are retained and others are not. This change should fix the issue. I also updated the template test to test for this and made the changes to v2. Added a changelog entry specifically for the change from empty string to null as the default. Made the null representation configurable. It still defaults to the python NoneType but can be overriden to be an emptystring by updating the DEFAULT_NULL_REPRESENTATION config.	9 years ago
James Cammarata	27fcf1a4b5	Fix bug in registered variables related to delegate_to changes	9 years ago
James Cammarata	64a1b1e043	Fix first_available_file: support for copy and template actions	9 years ago
Toshio Kuratomi	974731bec0	Fix filename of output file	10 years ago
Toshio Kuratomi	8d0ceeca91	Make template test work with both py2.6 and 2.7+	10 years ago
Toshio Kuratomi	fb96173d10	to_nice_json filter no longer has a trailing space when formatting dicts	10 years ago
Feanil Patel	0abcebf1e4	Don't convert numbers and booleans to strings. Before this change if a variable was of type int or bool and the variable was referenced by another variable, the type would change to string. eg. defaults/main.yml ``` PORT: 4567 OTHER_CONFIG: secret1: "so_secret" secret2: "even_more_secret" CONFIG: hostname: "some_hostname" port: "{{ PORT }}" secrets: "{{ OTHER_CONFIG }}" ``` If you output `CONFIG` to json or yaml, the port would get represented in the output as a string instead of as a number, but secrets would get represented as a dictionary. This is a mis-match in behaviour where some "types" are retained and others are not. This change should fix the issue. Update template test to also test var retainment. Make the template changes in v2. Update to only short-circuit for booleans and numbers. Added an entry to the changelog.	10 years ago
Toshio Kuratomi	a8c02b7049	Rework the shell quoting of remote checksumming Instead of getting rid of pipes.quote, use pipes.quote and get rid of the manually entered toplevel quotes. This should properly escape backslashes, quotes, and other characters. Also fix the new checksumming python "one-liner" for csh. ansible_shell_type needs to be set to csh. Fixes #10363 Fixes #10353	10 years ago
Toshio Kuratomi	e146245a0d	Integration test for #10300	10 years ago
Toshio Kuratomi	10df1b898c	Fix template module broken when modifying symlinks Needed to clear unwanted parameters from both args and complex args when calling file module. Fixes #10208	10 years ago
Toshio Kuratomi	f1267c0b05	Move from md5 to sha1 to work on fips-140 enabled systems	10 years ago
Toshio Kuratomi	43b3eecf52	Debian/Ubuntu doesn't have group nobody so remove the group portion of the new template tests as owner and mode will exercise the problematic code path	10 years ago
Toshio Kuratomi	feb9ed1de8	Fix template module incorrectly handling mode when dest is a directory Fixes #9350	10 years ago
James Cammarata	0e5b0a2e97	Revert "Set Jinja2's keep_trailing_newline=True in template_from_string." This reverts commit `7a8eb0b670`.	10 years ago
Jacob Leverich	7a8eb0b670	Set Jinja2's keep_trailing_newline=True in template_from_string. Without this setting, the newline idempotence of scalar strings changes depending on whether or not they contain any Jinja2 template blocks.	10 years ago
Michael DeHaan	2177b773c8	Rename tests to test, use old directory name.	11 years ago

16 Commits (a27c5741a1a52f78a7ee25dddbf30e78ecce6a4f)