Skip to content

Conversation

@troglobit
Copy link
Contributor

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):

@troglobit troglobit requested a review from jovatn October 3, 2025 11:26
@troglobit troglobit added releng Release engineering (ci system, release management) ci:main Build default defconfig, not minimal labels Oct 3, 2025
Copy link
Contributor

@jovatn jovatn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
One typo, though. :-)

@troglobit troglobit removed the releng Release engineering (ci system, release management) label Oct 3, 2025
Extend OSPF basic with a regression test for issue #1169

Signed-off-by: Joachim Wiberg <[email protected]>
Also, add some documentation to lower barrier of entry/use.

Signed-off-by: Joachim Wiberg <[email protected]>
NETCONF backedn already silenced.

Signed-off-by: Joachim Wiberg <[email protected]>
It should be possible to create test reports manually, so logically
the GitHub workflow should call a make rule in test.mk

Untested: branding, or any case where Infix is used as a BR2_EXTERNAL

Signed-off-by: Joachim Wiberg <[email protected]>
@troglobit
Copy link
Contributor Author

Tests failed because of a missing patch to the Infamy helper furl.py, so I cherry-picked that one and a few other relevant fixes. Build restarted.

@troglobit troglobit merged commit 68e7167 into v25.08.x Oct 3, 2025
@troglobit troglobit deleted the releng branch October 3, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:main Build default defconfig, not minimal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants