Skip to content
This repository was archived by the owner on Apr 8, 2025. It is now read-only.

Conversation

@Watcher7
Copy link
Contributor

@Watcher7 Watcher7 commented Apr 3, 2018

Any execution of show flow-accounting interface * fails to parse pmacct's output properly.
Changes have occurred in pmacct's output between now and the last modification of this script that make the current targeted fields and command invocations problematic.
pmacct's -a flag now seems to be unreliable. In normal output mode white space is among values. In CSV output mode some field values lack headers, and some completely new field headers appear.
Easiest solution is to drop the -a flag from pmacct's command invocation, and then modify the targeted field values.
Related to: T595

Any execution of `show flow-accounting interface *` fails to parse pmacct's output properly.
Changes have occurred between now and the last modification of this script in pmacct's output that make the current targeted fields and command invocations problematic.
pmacct's `-a` flag now seems to be unreliable. In normal output mode white space is among values. In CSV output mode some field values lack headers, and some completely new field headers appear.
Easiest solution is to drop  the `-a` flag from pmacct's command invocation, and then modify the targeted field values.
Related to: T595
@dmbaturin dmbaturin merged commit af326e6 into vyos-legacy:current Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants