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
Fix edgeos_config: {backup: yes} #37619
Conversation
@@ -56,7 +56,7 @@ def run(self, tmp=None, task_vars=None): | |||
|
|||
# strip out any keys that have two leading and two trailing | |||
# underscore characters | |||
for key in result.keys(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Python 3 dictionary views are not copies, so you can't modify over them
from __future__ import (absolute_import, division, print_function) | ||
__metaclass__ = type | ||
|
||
from ansible.plugins.action.vyos_config import ActionModule |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awful, but it works. Will replace with proper copy.
Python3 fixes herein Look good to me. +1 on that portion of the changes. |
* Fix edgeos_config: {backup: yes} * Add actual plugin copy (cherry picked from commit 6aed74a)
SUMMARY
backup: yes
does not do the right thing inedgeos_command
because there is no action plugin to do the actual backupAlso copy some Python 3 fixes back to vyos_config's action plugin
ISSUE TYPE
COMPONENT NAME
edgeos_config
ANSIBLE VERSION