Skip to content

ospf: T7297: fix redistribute-table - #4704

Merged
jestabro merged 2 commits into
vyos:currentfrom
hedrok:T7297-fix-ospf-redistribute-table
Sep 11, 2025
Merged

ospf: T7297: fix redistribute-table#4704
jestabro merged 2 commits into
vyos:currentfrom
hedrok:T7297-fix-ospf-redistribute-table

Conversation

@hedrok

@hedrok hedrok commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

OSPF: fix redistribute-table

Requires FRR to support redistribute table-direct for ospf (see 'Related PR(s)')

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

vyos/vyos-build#1026

How to test / Smoketest result

Described in task in detail. Fix also requires adding

set protocols ospf table 10

On both sides.

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

Requires FRR to support `redistribute table-direct` for ospf
@github-actions

github-actions Bot commented Sep 9, 2025

Copy link
Copy Markdown

👍
No issues in PR Title / Commit Title

@c-po c-po left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please extend the smoketests to also cover the code-path for the new redistribute-table vtyshcommand

@hedrok
hedrok force-pushed the T7297-fix-ospf-redistribute-table branch from 9764541 to cb91488 Compare September 10, 2025 07:54
@hedrok

hedrok commented Sep 10, 2025

Copy link
Copy Markdown
Contributor Author

@c-po Thanks, I've expanded smoketest/scripts/cli/test_protocols_ospf.py to test this.

@hedrok
hedrok requested a review from c-po September 10, 2025 07:56
@github-actions

Copy link
Copy Markdown

CI integration ❌ failed!

Details

CI logs

  • CLI Smoketests (no interfaces) ❌ failed
  • CLI Smoketests VPP 👍 passed
  • CLI Smoketests (interfaces only) 👍 passed
  • Config tests 👍 passed
  • Config tests VPP 👍 passed
  • RAID1 tests 👍 passed
  • TPM tests 👍 passed

@c-po c-po left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change looks reasonable and extended with a dedicated smoketest validating the resulting FRR configuration.

@c-po c-po added the bp/circinus Create automatic backport for circinus label Sep 11, 2025

@jestabro jestabro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Smoketest included; looks good.

@jestabro
jestabro merged commit b3daa15 into vyos:current Sep 11, 2025
18 of 19 checks passed
@vyosbot vyosbot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Sep 11, 2025
@andamasov andamasov removed the bp/circinus Create automatic backport for circinus label May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants