Skip to content

Branch 25.03 backports#321

Merged
matperin merged 8 commits intocanonical:branch-25.03from
crypticC0der:branch-25.03
Apr 2, 2026
Merged

Branch 25.03 backports#321
matperin merged 8 commits intocanonical:branch-25.03from
crypticC0der:branch-25.03

Conversation

@crypticC0der
Copy link
Copy Markdown
Contributor

Adds DPU support to 25.03

@crypticC0der crypticC0der requested a review from a team as a code owner March 31, 2026 11:18
As recently discovered, newly applied permissions are not updated
on already running processes, the MicroOVN install starts the daemon and
then gives it the needed permissions, so we must restart the daemon to
be able to use them.

This only affects things that require specific linux capabilities to be
enabled compared to just file permissions, such as netlink connections
for devlink in the upcoming dpu work.

This will not affect microovn installed from store as connections are
applied before processes start.

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit ac1a0cc)
Setup Build MicroOVN workflow to be used in test.yaml and future
workflows that build the MicroOVN snap.

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit 906578f)
Adds DPU setup to join and bootstrap, this will detect if its running on
a DPU and return the PCI address, then it will extract the serial number
and properly insert it into the ovn-cms-option.

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit ef8e289)
Iterating through maps is non deterministic, so this unit test only
passes half the time, which means builds fail half the time, so we need
to remove it.

In the real world we likely shouldnt have multiple valid ports, so this
wont be an issue, and even if it comes up just using the first one we
get should work fine.

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit 7589627)
In recent research with real hardware we have found our previous
assumptions to be incorrect, so we are updating it to be controller = 1
and pcipf OR pcivf in the devlink port output[0].

[0]: https://docs.kernel.org/networking/devlink/devlink-port.html

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit eecd48c)
Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit 61f3938)
MicroOVN will now check for a file, which if it finds it will allow the
user to force install microovn alongside system OVS. The file should
only exist if the user *really* knows what they are doing and
intentionally creates it.

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit 0392316)
I have no idea how this test is passing in main, genuinely

Signed-off-by: MJ Ponsonby <mj.ponsonby@canonical.com>
(cherry picked from commit bc4bdf8)
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.

Everything seems in place, thank you for working on this!

@matperin matperin merged commit 3648dc1 into canonical:branch-25.03 Apr 2, 2026
32 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.

2 participants