Cisco IOS-XE additional transceiver data (CURRENT)

[paineta]

Existing template did not properly parse the additional "current" value in the "show interface transceiver" output on Cisco IOS-XE. I added support to collect the Current value or leave it blank, as to not break legacy output, and I also fleshed out the template to include an Error directive.

[paineta]

Since the message output from the networking device would be string literals, I feel like matching (a space) with regex adds unnecessary ambiguity.

[mjbear]

Since the message output from the networking device would be string literals, I feel like matching (a space) with regex adds unnecessary ambiguity.

I can see where you're coming from.

While not an absolute, there's a general consensus going back years to flexibly match white space so the possibility of breakage on vendor white space changes is minimized. This minimizes the possibility for issues and maintenance on the templates (or at least that's the hope! ☺️).

For those couple error messages, tucking \s+ for a literal space doesn't make them terribly long in line length and the convention with whitespace regexes is widespread across the templates I've reviewed over several years. This is an open discussion and I wouldn't want it either way. 🙂

I'd like to leave this up for discussion so some templating veterans can comment here.

Thank you for the contributions and discussion @paineta!

[jmcgill298]

Thanks @paineta, as @mjbear mentioned, we do prefer \s+ over a literal character to avoid a silly change in whitespace to affect the template

[paineta]

No problem. Thanks.