Branch 25.03 backports#321
Merged
matperin merged 8 commits intocanonical:branch-25.03from Apr 2, 2026
Merged
Conversation
5bf4a92 to
e22b4a3
Compare
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)
e22b4a3 to
9e81c7d
Compare
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)
matperin
approved these changes
Apr 2, 2026
Contributor
matperin
left a comment
There was a problem hiding this comment.
Everything seems in place, thank you for working on this!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds DPU support to 25.03