Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RFC: Support consuming Open vSwitch from snap #425

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Dec 2, 2023

  1. tests/ctests: Build ctests separately

    At present the ctests are not properly linked with the built code,
    this causes issues when there are interdependencies between the
    modules.
    
    The meson build system does not appear to have support for
    internal build order or dependncies, so the only way I have found
    to make this work is to break out the build of the ctests as a
    separate project.
    
    This is required to avoid undefined symbols in the test code for
    the next patch in this series.
    
    Signed-off-by: Frode Nordahl <[email protected]>
    fnordahl committed Dec 2, 2023
    Configuration menu
    Copy the full SHA
    3c93eea View commit details
    Browse the repository at this point in the history

Commits on Dec 4, 2023

  1. ovs: Support consuming OVS from snap

    At present netplan hard codes the location of the `ovs-vsctl`
    binary.  This does not work well when trying to integrate with
    snaps.
    
    Determine path of `ovs-vsctl` at runtime by checking for both
    '/snap/bin/ovs-vsctl' and '/usr/bin/ovs-vsctl' with a preference
    of the former.
    
    This allows making use of the Netplan Open vSwitch integration
    with for example the MicroOVN snap (and its consumers such as
    MicroCloud).
    
    TODO: Figure out how to hanlde the ovsdb-server.service unit check
    TODO: Add test cases that checks the snap workflow
    
    Signed-off-by: Frode Nordahl <[email protected]>
    fnordahl committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    880db4e View commit details
    Browse the repository at this point in the history