Tadej Borovšak

Results 24 comments of Tadej Borovšak

The problematic part seems to be the `AnsibleMapping([('b', 'c')])`. When the `safe_eval` function tries to convert the string into a list of dictionaries, it sees the mapping constructor and bails...

I switched jobs and I do not use Ansible anymore. So I have no way of testing this.

I did not look at the code, but I can confirm that combining the `black` code formatting tool and the pep8 sanity test is not doable because they have different...

I am not sure if this has anything to do with Python 3.9. The docstring that the error points to contains an invalid escape sequence (`\*`) and that error is...

I cannot replicate the error on my Linux machines, but it looks like the original error was produced on a mac, so maybe that makes the difference? But in any...

@funkyfuture Are you running `flake8` from the repo? The invalid escape sequence will not be reported there because the `tox.ini` file instructs `flake8` to ignore that error: https://github.com/pyeve/cerberus/blob/a70b404e5bf164bb6eaf1d7f264957db4ab7d55c/tox.ini#L28 That change...

The invalid escape sequence warning is silenced by default because it would annoy people maintaining ancient codebases written when those escape sequences were valid. Python interpreted them as literal backslash...

@adn1107 Ansible 2.9 does not support ignore stuff. You will need to update to at least ansible-base-2.10 or ansible-core.

> We can address this with either @flowerysong filter plugin, or a new `use_contrib_script_compatible_expand_csv_tags` for the inventory plugin that would reproduce the same behaviour. I would probably go with the...

Just removing that entry is probably not the best idea. Tho core engine still [redirects to the netbox.netbox.netbox_interface](https://github.com/ansible/ansible/blob/c56d98e738637a23df807bf72dd5fe6cf10a93fd/lib/ansible/config/ansible_builtin_runtime.yml#L5039-L5040). Without the corresponding entry in the netbox.netbox collection, users will get a...