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 parser if vlan config is on multiple lines #57

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

Conversation

lagrip
Copy link

@lagrip lagrip commented Jan 16, 2023

SUMMARY

Bring multiple vlan config lines to one single line.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

os6.py: os6_parse()

ADDITIONAL INFORMATION

The dell switch with OS6 wrap the vlan config to multiple lines if there are a lot of vlans configured. Ansible leads always into a changed status because the comparison between running-config an the new config will not work well.

The fix brings the code in the variable "line" from

!System Software Version 6.7.1.17
!
configure
vlan 2,5,11-13,18,24,35-36,38,40,42,44,46-48,53-55,101,240,1902-1903,1909
vlan 2001-2003
vlan 2007,2300
exit
vlan 2

to

!System Software Version 6.7.1.17
!
configure
vlan 2,5,11-13,18,24,35-36,38,40,42,44,46-48,53-55,101,240,1902-1903,1909,2001-2003,2007,2300
exit
vlan 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant