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 #57755

Open
wants to merge 2 commits into
base: stable-2.7
from

Conversation

Projects
None yet
4 participants
@ganeshrn
Copy link
Member

commented Jun 12, 2019

SUMMARY
  • Convert the ElementTree object to string
    before dumping the configuration in file.

  • Fix netconf plugin to return data_xml object
    if present for the device handler object or else return raw xml.

  • Add check to run lldp integration test cases only if lldp is
    supported on the target host

Merged PR to devel
#57309
#57578
#56452

(cherry picked from commit 9c5745a)
(cherry picked from commit 0957835)
(cherry picked from commit 2ca324b)

ISSUE TYPE
  • Bugfix Pull Request
  • Test Pull Request
COMPONENT NAME

plugins/netconf/init.py
netconf_config
netconf_get

ADDITIONAL INFORMATION

Fix junos integration zuul CI failures
*  Convert the ElementTree object to string
   before dumping the configuration in file.

* Fix netconf plugin to return data_xml object
  if present for the device handler object or else return raw xml.

* Add check to run lldp integration test cases only if lldp is
  supported on the target host

Merged PR to devel
#57309
#57578
#56452

(cherry picked from commit 9c5745a)
(cherry picked from commit 0957835)
(cherry picked from commit 2ca324b)

@ganeshrn ganeshrn force-pushed the ganeshrn:backport/stable_2.7/57309 branch from 9dde176 to 6c37fbd Jun 12, 2019

@ganeshrn ganeshrn requested review from pabelanger and danielmellado Jun 12, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 12, 2019

@ansibot ansibot removed the needs_triage label Jun 12, 2019

@pabelanger

This comment has been minimized.

Copy link
Contributor

commented Jun 15, 2019

recheck

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.