Skip to content
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 junos integration zuul CI failures (#57309) #57638

Open
wants to merge 1 commit into
base: stable-2.8
from

Conversation

Projects
None yet
3 participants
@danielmellado
Copy link
Contributor

commented Jun 10, 2019

SUMMARY

Fixes junos zuul issues, backported from Devel

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

junos_facts

danielmellado added a commit to danielmellado/ansible that referenced this pull request Jun 10, 2019

Make netconf_get python3 compat (ansible#57578)
This uses to_text to ensure tostring() from lxml is decoded properly.

Depends-On: ansible#57638
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
(cherry picked from commit f709d2e)

pabelanger added a commit to pabelanger/ansible that referenced this pull request Jun 10, 2019

Remove hardcoded ansible user for junos_user integration tests (ansib…
…le#56452)

It is possible we are using a different username then 'ansible' for
testing junos. By using ansible_user_id, this should be a more dynamic
check.

Depends-On: ansible#57639
Depends-On: ansible#57638
Signed-off-by: Paul Belanger <pabelanger@redhat.com>

@danielmellado danielmellado force-pushed the danielmellado:stable-2.8-junos-fix branch from 258adb3 to a6d1983 Jun 11, 2019

@danielmellado

This comment has been minimized.

Copy link
Contributor Author

commented Jun 11, 2019

Pull-request updated, HEAD is now a6d1983

danielmellado added a commit to danielmellado/ansible that referenced this pull request Jun 11, 2019

Make netconf_get python3 compat (ansible#57578)
This uses to_text to ensure tostring() from lxml is decoded properly.

Depends-On: ansible#57638
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
(cherry picked from commit f709d2e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.