vtysh: only show error codes once

[qlyoung]

When using show error commands, show errors shared between multiple
daemons only once.

Signed-off-by: Quentin Young qlyoung@cumulusnetworks.com

[LabN-CI]

🚧 Basic BGPD CI results: Partial FAILURE, 1 tests failed

Results table

_	_
Result	SUCCESS git merge/4732 7be9968
Date	07/25/2019
Start	11:15:30
Finish	11:37:04
Run-Time	21:34
Total	1815
Pass	1814
Fail	1
Valgrind-Errors	0
Valgrind-Loss	0
Details	vncregress-2019-07-25-11:15:30.txt
Log	autoscript-2019-07-25-11:16:20.log.bz2
Memory	433 429 353

For details, please contact louberger

[NetDEF-CI]

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8445/

This is a comment from an automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.

Warnings Generated during build:

Checkout code: Successful with additional warnings

Report for vtysh.c | 4 issues
===============================================
< WARNING: Missing a blank line after declarations
< #2407: FILE: /tmp/f1-2455/vtysh.c:2407:
< WARNING: Missing a blank line after declarations
< #2413: FILE: /tmp/f1-2455/vtysh.c:2413:

Warnings Generated during build:

Debian 10 amd64 build: Successful with additional warnings

Debian Package lintian failed for Debian 10 amd64 build:
(see full package build log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8445/artifact/DEB10BUILD/ErrorLog/log_lintian.txt)

W: frr source: changelog-should-mention-nmu

---

CLANG Static Analyzer Summary

• Github Pull Request 4732, comparing to Git base SHA e72ce28

No Changes in Static Analysis warnings compared to base

1 Static Analyzer issues remaining.

See details at
https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8445/artifact/shared/static_analysis/index.html

[donaldsharp]

LGTM modulus the two nits

[qlyoung]

@donaldsharp fixed, please un-NACK when you can

Fixed problems

[LabN-CI]

💚 Basic BGPD CI results: SUCCESS, 0 tests failed

Results table

_	_
Result	SUCCESS git merge/4732 2d6e4d2
Date	07/29/2019
Start	13:27:16
Finish	13:48:56
Run-Time	21:40
Total	1815
Pass	1815
Fail	0
Valgrind-Errors	0
Valgrind-Loss	0
Details	vncregress-2019-07-29-13:27:16.txt
Log	autoscript-2019-07-29-13:28:05.log.bz2
Memory	414 434 361

For details, please contact louberger

[NetDEF-CI]

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8466/

This is a comment from an automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.

Warnings Generated during build:

Debian 10 amd64 build: Successful with additional warnings

Debian Package lintian failed for Debian 10 amd64 build:
(see full package build log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8466/artifact/DEB10BUILD/ErrorLog/log_lintian.txt)

W: frr source: changelog-should-mention-nmu

---

CLANG Static Analyzer Summary

• Github Pull Request 4732, comparing to Git base SHA a041a8f
• Base image data for Git a041a8f does not exist - compare skipped

1 Static Analyzer issues remaining.

See details at
https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8466/artifact/shared/static_analysis/index.html

[riw777]

looks fine