-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
community.general.one_vm: creating fails with "Parse error: syntax error" #6225
Comments
Files identified in the description: If these files are incorrect, please update the |
nilsding
added a commit
to nilsding/community.general
that referenced
this issue
Apr 6, 2023
with more complex templates that make use of quoted strings the new "render" method fails to produce a template that is accepted by OpenNebula. ==> escape double quotes in strings to make OpenNebula happy again. I also tested whether newlines need to be escaped, looks like they are fine as they are. Fixes ansible-collections#6225
nilsding
added a commit
to nilsding/community.general
that referenced
this issue
Apr 11, 2023
with more complex templates that make use of quoted strings the new "render" method fails to produce a template that is accepted by OpenNebula. ==> escape double quotes in strings to make OpenNebula happy again. I also tested whether newlines need to be escaped, looks like they are fine as they are. Fixes ansible-collections#6225
felixfontein
pushed a commit
that referenced
this issue
Apr 16, 2023
…te (#6294) * one_vm: fix syntax error when creating VMs with a more complex template with more complex templates that make use of quoted strings the new "render" method fails to produce a template that is accepted by OpenNebula. ==> escape double quotes in strings to make OpenNebula happy again. I also tested whether newlines need to be escaped, looks like they are fine as they are. Fixes #6225 * module_utils/opennebula: skip empty values in render
patchback bot
pushed a commit
that referenced
this issue
Apr 16, 2023
…te (#6294) * one_vm: fix syntax error when creating VMs with a more complex template with more complex templates that make use of quoted strings the new "render" method fails to produce a template that is accepted by OpenNebula. ==> escape double quotes in strings to make OpenNebula happy again. I also tested whether newlines need to be escaped, looks like they are fine as they are. Fixes #6225 * module_utils/opennebula: skip empty values in render (cherry picked from commit cb3ca05)
felixfontein
pushed a commit
that referenced
this issue
Apr 16, 2023
…creating VMs with a more complex template (#6342) one_vm: fix syntax error when creating VMs with a more complex template (#6294) * one_vm: fix syntax error when creating VMs with a more complex template with more complex templates that make use of quoted strings the new "render" method fails to produce a template that is accepted by OpenNebula. ==> escape double quotes in strings to make OpenNebula happy again. I also tested whether newlines need to be escaped, looks like they are fine as they are. Fixes #6225 * module_utils/opennebula: skip empty values in render (cherry picked from commit cb3ca05) Co-authored-by: Georg Gadinger <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
When we try to create a VM in our OpenNebula environment using an Ansible task it fails with an error message.
Notably this worked before on
community.general
6.2.0 or lower and thus is a regression.We suspect that the bug appeared in 8818a6f, especially since it relates to new ONE features (we are still on an old ONE version).
Issue Type
Bug Report
Component Name
one_vm
Ansible Version
Community.general Version
Configuration
no special configuration
OS / Environment
OpenNebula installation: version 6.0.0.2
Steps to Reproduce
Expected Results
VM in OpenNebula successfully created
Actual Results
Depending on which vm template is used, there is sometimes a different error:
In either case the exact same playbook worked before.
Code of Conduct
The text was updated successfully, but these errors were encountered: