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

debianpkg: Add missing frr-dbg.lintian-overrides to Makefile #1550

Merged

Conversation

mwinter-osr
Copy link
Member

This fixes PR #1472 which missed adding the new lintian-overrides package to the distribution target.
Fixes all the recent lintian warnings as shown by the CI system

Signed-off-by: Martin Winter mwinter@opensourcerouting.org

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
@LabN-CI
Copy link
Collaborator

LabN-CI commented Dec 14, 2017

💚 Basic BGPD CI results: SUCCESS, 0 tests failed

Results table
_ _
Result SUCCESS git merge/1550 1ec43e9
Date 12/13/2017
Start 22:20:08
Finish 22:43:17
Run-Time 23:09
Total 1805
Pass 1805
Fail 0
Valgrind-Errors 0
Valgrind-Loss 0
Details vncregress-2017-12-13-22:20:08.txt
Log autoscript-2017-12-13-22:20:46.log.bz2

For details, please contact louberger

@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: FAILED

See below for issues.
CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2171/

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

Get source and apply patch from patchwork: Successful

Building Stage: Successful

Basic Tests: Failed

Ubuntu 12.04 deb pkg check: Successful
Static analyzer (clang): Successful
CentOS 6 rpm pkg check: Successful
IPv6 protocols on Ubuntu 14.04: Successful
Ubuntu 16.04 deb pkg check: Successful
Fedora 24 rpm pkg check: Successful
Addresssanitizer topotest: Successful
Debian 8 deb pkg check: Successful
IPv4 protocols on Ubuntu 14.04: Successful
Debian 9 deb pkg check: Successful
IPv4 ldp protocol on Ubuntu 16.04: Successful
Ubuntu 14.04 deb pkg check: Successful
CentOS 7 rpm pkg check: Successful

Topology tests on Ubuntu 16.04 amd64: Failed

Topology Test Results are at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-TOPOU1604-2171/test

Topology Tests failed for Topology tests on Ubuntu 16.04 amd64:

2017-12-13 19:41:32,496 ERROR: ******************************************************************************
2017-12-13 19:41:32,497 ERROR: Test Target Summary                                                  Pass Fail
2017-12-13 19:41:32,497 ERROR: ******************************************************************************
2017-12-13 19:41:32,497 ERROR: 6    r3     See R1s address +10.57 secs                              0    1
2017-12-13 19:41:32,497 ERROR: 17   r1     Remote Registration +10.02 secs                          0    1
2017-12-13 19:41:32,497 ERROR: 19   r4     Remote Registration +10.56 secs                          0    1
2017-12-13 19:41:32,497 ERROR: 24   ce2    Local and remote routes +10.60 secs                      0    1
2017-12-13 19:41:32,497 ERROR: 30   r1     VPN SAFI                                                 0    1
2017-12-13 19:41:32,497 ERROR: 31   r2     VPN SAFI                                                 0    1
2017-12-13 19:41:32,497 ERROR: 32   r3     VPN SAFI                                                 0    1
2017-12-13 19:41:32,497 ERROR: 33   r4     VPN SAFI                                                 0    1
2017-12-13 19:41:32,498 ERROR: ******************************************************************************
2017-12-13 19:41:32,498 ERROR: Total 49                                                             32   8
2017-12-13 19:41:32,498 ERROR: ******************************************************************************
2017-12-13 19:41:32,498 ERROR: See /tmp/topotests/bgp_direct_to_bgp_vpn.test_bgp_direct_to_bgp_vpn/output.log for details of errors
2017-12-13 19:41:32,513 ERROR: assert failed at "bgp_direct_to_bgp_vpn.test_bgp_direct_to_bgp_vpn/test_run_lu_tests": 8 tests failed
assert '8 tests failed' == 'See summary output above'
  - 8 tests failed
  + See summary output above

see full log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2171/artifact/TOPOU1604/ErrorLog/log_topotests.txt

Topology Tests memory analysis: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2171/artifact/TOPOU1604/MemoryLeaks/

CLANG Static Analyzer Summary

  • Github Pull Request 1550, comparing to Git base SHA db33b83

No Changes in Static Analysis warnings compared to base

23 Static Analyzer issues remaining.

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

@NetDEF-CI
Copy link
Collaborator

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-2171/

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


CLANG Static Analyzer Summary

  • Github Pull Request 1550, comparing to Git base SHA db33b83

No Changes in Static Analysis warnings compared to base

23 Static Analyzer issues remaining.

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

@donaldsharp donaldsharp merged commit b9bd264 into FRRouting:master Dec 14, 2017
@mwinter-osr mwinter-osr deleted the missing-lintian-file branch June 9, 2018 01:51
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.

4 participants