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

tests: Add a couple simple Null0 spelling tests #4864

Merged
merged 1 commit into from
Aug 22, 2019

Conversation

donaldsharp
Copy link
Member

Add a bit of code to test different spelling of Null0 routes.
This was broken at some point in the past and with recent
changes is working again, but it would be nice to
know when this breaks again.

Signed-off-by: Donald Sharp sharpd@cumulusnetworks.com

Add a bit of code to test different spelling of Null0 routes.
This was broken at some point in the past and with recent
changes is working again, but it would be nice to
know when this breaks again.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
@polychaeta polychaeta added the tests Topotests, make check, etc label Aug 21, 2019
@LabN-CI
Copy link
Collaborator

LabN-CI commented Aug 21, 2019

💚 Basic BGPD CI results: SUCCESS, 0 tests failed

Results table
_ _
Result SUCCESS git merge/4864 a7e52bd
Date 08/21/2019
Start 12:15:24
Finish 12:37:51
Run-Time 22:27
Total 1818
Pass 1818
Fail 0
Valgrind-Errors 0
Valgrind-Loss 0
Details vncregress-2019-08-21-12:15:24.txt
Log autoscript-2019-08-21-12:16:40.log.bz2
Memory 435 417 360

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

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.

Get source / Pull Request: Successful

Building Stage: Successful

Basic Tests: Failed

Topology tests on Ubuntu 18.04 amd64: Failed (click for details) Topology tests on Ubuntu 18.04 amd64: No useful log found
Topotest tests on Ubuntu 16.04 i386: Failed (click for details) Topotest tests on Ubuntu 16.04 i386: No useful log found
Topology tests on Ubuntu 16.04 amd64: Failed (click for details) Topology tests on Ubuntu 16.04 amd64: No useful log found
Successful on other platforms
  • Ubuntu 12.04 deb pkg check
  • Fedora 29 rpm pkg check
  • Static analyzer (clang)
  • IPv6 protocols on Ubuntu 14.04
  • Ubuntu 16.04 deb pkg check
  • CentOS 7 rpm pkg check
  • Addresssanitizer topotest
  • Debian 10 deb pkg check
  • Debian 8 deb pkg check
  • Debian 9 deb pkg check
  • IPv4 protocols on Ubuntu 14.04
  • IPv4 ldp protocol on Ubuntu 16.04
  • Ubuntu 14.04 deb pkg check

CLANG Static Analyzer Summary

  • Github Pull Request 4864, comparing to Git base SHA 9316c82

No Changes in Static Analysis warnings compared to base

1 Static Analyzer issues remaining.

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

@donaldsharp
Copy link
Member Author

ci:rerun

@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-8670/

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.


CLANG Static Analyzer Summary

  • Github Pull Request 4864, comparing to Git base SHA 9316c82

No Changes in Static Analysis warnings compared to base

1 Static Analyzer issues remaining.

See details at
https://ci1.netdef.org/browse/FRR-FRRPULLREQ-8670/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-8673/

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.


CLANG Static Analyzer Summary

  • Github Pull Request 4864, comparing to Git base SHA 9316c82

No Changes in Static Analysis warnings compared to base

1 Static Analyzer issues remaining.

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

@srimohans srimohans merged commit 401ba2a into FRRouting:master Aug 22, 2019
@donaldsharp donaldsharp deleted the nullzero branch December 9, 2019 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Topotests, make check, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants