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

Enhancement: parser to detect nextline (long names) #769

Open
edudppaz opened this issue Jul 11, 2023 · 11 comments
Open

Enhancement: parser to detect nextline (long names) #769

edudppaz opened this issue Jul 11, 2023 · 11 comments
Assignees

Comments

@edudppaz
Copy link

When using the parser towards output which have long names and create a "newline" , the parsing fails. For example, when using "show l2vpn bridge-domain brief" as long as the output is exactly on a line by line basis, everything works fine:

Bridge Group:Bridge-Domain Name  ID    State          Num ACs/up   Num PWs/up    Num PBBs/up Num VNIs/up
-------------------------------- ----- -------------- ------------ ------------- ----------- -----------
D:D-w                            0     up             1/1          1/1           0/0         0/0
d:s-VLAN_20                      1     up             2/2          1/1           0/0         0/0

But if the combination of the bridge group and bridge-domain name its too long, the ios-xr output puts the id,state, etc on a newline, and the whole parsing fails:

Bridge Group:Bridge-Domain Name  ID    State          Num ACs/up   Num PWs/up    Num PBBs/up Num VNIs/up
-------------------------------- ----- -------------- ------------ ------------- ----------- -----------
Dasdnalsdnalk:D-dasdasdasdwa
                                 0     up             1/1          1/1           0/0         0/0
d:s-VLAN_20                      1     up             2/2          1/1           0/0         0/0

I´ve only tried with "show l2vpn xconnect", "show l2vpn bridge-domain brief" and a few other commands in ios-xr l2vpn and they all seem to have the same behaviour.

@iamsatyanarayan
Copy link

Hi @edudppaz
I will be working on your ticket. I'll get back to you once I have any update on it.

@iamsatyanarayan
Copy link

HI @edudppaz
PR has been created for your issue. I will be update you when it will merge.

@edudppaz
Copy link
Author

HI @edudppaz PR has been created for your issue. I will be update you when it will merge.

Great, thanks for the info

@iamsatyanarayan
Copy link

iamsatyanarayan commented Aug 22, 2023

Hi edudppaz,
PR has been merged. It will reflect in next release.

@iamsatyanarayan
Copy link

Hi edudppaz,
The latest version has been released kindly upgrade to 23.8 and confirm.

@iamsatyanarayan
Copy link

Hi @edudppaz ,
Could you please provide any update on your issue ?

@iamsatyanarayan
Copy link

Hi @edudppaz,
Could you please let me know if you have any update ?

@edudppaz
Copy link
Author

edudppaz commented Sep 6, 2023

Hi @iamsatyanarayan , the 23.8 package still has not been cleared for intern use, so i will test at my home lab later today and let you know.

@iamsatyanarayan
Copy link

Hi @edudppaz ,
Ok sure

@iamsatyanarayan
Copy link

Hi @edudppaz
Any update on this issue ?

@iamsatyanarayan
Copy link

Hi @edudppaz
As we haven't heard back from you , we will be closing this ticket. If you still require further assistance, please feel free to open a new ticket.

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

No branches or pull requests

3 participants