Skip to content

Conversation

@Fredi-raspall
Copy link
Contributor

No description provided.

Enable dependency features. Otherwise tests can't be run standalone.

Signed-off-by: Fredi Raspall <[email protected]>
@Fredi-raspall Fredi-raspall requested a review from a team as a code owner January 14, 2026 11:10
@Fredi-raspall Fredi-raspall requested review from daniel-noland and removed request for a team January 14, 2026 11:10
@Fredi-raspall Fredi-raspall added ci:-upgrade Disable VLAB upgrade tests ci:-vlab Disable VLAB tests labels Jan 14, 2026
@Fredi-raspall Fredi-raspall requested a review from qmonnet January 14, 2026 11:10
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

One nit below, looks good otherwise

The fibtable concurrency test takes more time than nearly all of
the tests, and can even take nearly a minute. Scale it down
to keep test run times low.

Signed-off-by: Fredi Raspall <[email protected]>
@Fredi-raspall Fredi-raspall force-pushed the pr/fredi/fix_routing_test_deps branch from 44b3beb to 7d52d99 Compare January 14, 2026 18:23
@Fredi-raspall Fredi-raspall added this pull request to the merge queue Jan 14, 2026
Merged via the queue into main with commit 2361466 Jan 14, 2026
22 checks passed
@Fredi-raspall Fredi-raspall deleted the pr/fredi/fix_routing_test_deps branch January 14, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:-upgrade Disable VLAB upgrade tests ci:-vlab Disable VLAB tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants