Skip to content
This repository was archived by the owner on Oct 8, 2020. It is now read-only.

Lookup plugin yang_json2xml python3 support #48

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

rawat-he
Copy link

@rawat-he rawat-he commented Aug 20, 2020

When using lookup plugin yang_json2xml with python3 getting below error

msg: 'An unhandled exception occurred while running the lookup plugin ''yang_json2xml''.
Error was a <class ''ansible.errors.AnsibleError''>,
original message: Error while reading xml document: cannot use a string pattern on a bytes-like object'

Make this plugin python3 compatible.

getting below error

msg: 'An unhandled exception occurred while running the lookup plugin ''yang_json2xml''.
Error was a <class ''ansible.errors.AnsibleError''>,
original message: Error while reading xml document: cannot use a string pattern on a bytes-like object'

Make this plugin python3 compatible.

Signed-off-by: rawat-he <himanshu.rawat@gmail.com>
Copy link
Member

@ganeshrn ganeshrn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution. FYI the future development work is moved to https://github.com/ansible-collections/community.yang collection

@rawat-he
Copy link
Author

@ganeshrn Thank you for approving the changes.

Some of the required checks are failing, can you please let me know how to fix them so that this PR can be merged?

@GomathiselviS
Copy link
Member

recheck

1 similar comment
@GomathiselviS
Copy link
Member

recheck

@ganeshrn ganeshrn added the gate To make Zuul automerge after CI check are passing label Oct 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
gate To make Zuul automerge after CI check are passing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants