Skip to content

[24.03] Fail nicely on Netlink connection failed.#326

Merged
crypticC0der merged 1 commit intocanonical:branch-24.03from
crypticC0der:branch-24.03
Apr 15, 2026
Merged

[24.03] Fail nicely on Netlink connection failed.#326
crypticC0der merged 1 commit intocanonical:branch-24.03from
crypticC0der:branch-24.03

Conversation

@crypticC0der
Copy link
Copy Markdown
Contributor

On some releases the devlink kernel module is not enabled by default, it would be nice to handle this error nicely and let microovn continue working.

This only blocks DPU detection and not any integral functionality so in the scenario we cant use it, continuing isnt the end of the world.

(cherry picked from commit 0543ff0)

On some releases the devlink kernel module is not enabled by default, it
would be nice to handle this error nicely and let microovn continue working.

This only blocks DPU detection and not any integral functionality so in
the scenario we cant use it, continuing isnt the end of the world.

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit 0543ff0)
@crypticC0der crypticC0der requested a review from a team as a code owner April 3, 2026 16:14
@crypticC0der crypticC0der changed the title Fail nicely on Netlink connection failed. [24.03] Fail nicely on Netlink connection failed. Apr 3, 2026
Copy link
Copy Markdown

@roosterfish roosterfish left a comment

Choose a reason for hiding this comment

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

Ty!

@roosterfish
Copy link
Copy Markdown

May I ask when this change gets backported/merged? It would unblock MicroCloud pipelines running with MicroOVN edge version.

@crypticC0der
Copy link
Copy Markdown
Contributor Author

May I ask when this change gets backported/merged? It would unblock MicroCloud pipelines running with MicroOVN edge version.

when we have someone in the ovn team review it, I sadly cannot review my own code.

cc: @fnordahl @matperin

Copy link
Copy Markdown
Contributor

@matperin matperin left a comment

Choose a reason for hiding this comment

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

I do not see any issues with the backports, sorry for missing reviewing these PRs until now.

@crypticC0der crypticC0der merged commit cd2c86b into canonical:branch-24.03 Apr 15, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants