search

ManageIQ / manageiq-automation_engine

Automation engine for ManageIQ
Apache License 2.0
11 stars 74 forks source link

Unable to Pass A List As Extra Variable For Ansible Playbooks #435

Closed dmetzger57 closed 1 year ago

dmetzger57 commented 4 years ago

Related BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1659092

When passing a list as an extra variable to an ansible playbook, the list is always treated as a string. Should be able to insert a list into an ansible playbook as an extra variable.

NickLaMuro commented 4 years ago

Just taking a quick look at this as it was just assigned to me:

Initial thoughts is that we would have to see how ansible-runner allows passing of arrays as a command line arg, which I don't know if it is possible off hand. My guess is that it would/should be possible to use the existing UI to pass array based arguments without the needing to make changes to the UI, or even the backend (possibly).

As a workaround, my guess is it should also be possible to re-structure most playbooks where this feature wouldn't be necessary, but that is hard to know without concrete examples of where this might be used. The BZ provides an example array, but not an example where said array might be used in the context of a playbook.

An example of this in an unrelated subject matter is how the NFS_EXPORT_* vars in https://github.com/ehough/docker-nfs-server is handled for this docker image, where you can create vars NFS_EXPORT_1, NFS_EXPORT_2, etc. as a way of defining "array like data" or list based data, but without actually needing a defined N to do so.

Not a perfect solution, but might avoid adding unneeded complexity to our product that we have to support that can be solved in a different way outside of it.

miq-bot commented 1 year ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

miq-bot commented 1 year ago

This issue has been automatically closed because it has not been updated for at least 3 months.

Feel free to reopen this issue if this issue is still valid.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.