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.

Sign up for GitHub

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

Jump to bottom

Ospf container #820

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Ospf container #820

wants to merge 18 commits into from

Conversation

mattiaswal
Copy link
Contributor

@mattiaswal mattiaswal commented Nov 12, 2024 •
edited
Loading

This fixes #652 (but larger 0.0.0.1 case is the same), fixes #776 and fixes #743

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

@mattiaswal mattiaswal added the ci:main Build default defconfig, not minimal label Nov 13, 2024
mattiaswal and others added 10 commits November 13, 2024 12:54
@mattiaswal
Add new test: ospf_container
17fe9c4 
Aims to test a complex setup with multiple containers, OSPF and NAT
@troglobit @mattiaswal
README: minor, grammar.
b4d845b 
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit @mattiaswal
test/case: add description for ospf container use-case
ad1a815 
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@mattiaswal
test: use_case: Update test with HTTP container and ABR
093910f 
Add ABR configuration and a HTTP Container
@troglobit @mattiaswal
test/case: update use-case internal network description
a1f3eb1 
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit @mattiaswal
test/case: minor, grammar and formatting
e8aa140 
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@mattiaswal
ospf_containers: Convert to use put_config_dicts
7a05251
@mattiaswal
test: case: ospf_container: Update test
ffd6c02 
Add "VLAN switches" between each Router, to only pass the tagged traffic
between the routers.

Add test specification and add to test suite
@troglobit @mattiaswal
test: add drawio original and update images in use-case test
61c23e8 
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit @mattiaswal
test: factor out file server from upgrade test
c3bed2b 
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@mattiaswal mattiaswal force-pushed the ospf-container branch 4 times, most recently from 7d813ce to 1955b15 Compare November 14, 2024 09:08
@mattiaswal
infamy: Stability fix for getting OSPF status
7d58db2 
Always return {} if fail to get OSPF status, it may not have started
yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci:main Build default defconfig, not minimal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

test: ospf-container-basic test: verify custom container interface Test a larger area 0.0.0.0 topology
2 participants
@mattiaswal @troglobit