-
Notifications
You must be signed in to change notification settings - Fork 194
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
libnetwork: add function to get the default network backend #872
libnetwork: add function to get the default network backend #872
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Luap99 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
1df9945
to
2fce8d4
Compare
I also added this to containers/podman#12642 but it looks like there is some not networking related regression in c/common which is causing failures in podman. |
When the network backend is unset in the config we have to to figure out if we need CNI or netavark. New installs should use netavark while existing installs should continue to use CNI to prevent breaking systems. We use the following conditions to determine what backend to use: 1. read ${graphroot}/defaultNetworkBackend 2. find netavark binary (if not installed use CNI) 3. check containers, images and CNI networks and if there are some we have an existing install and should continue to use CNI Signed-off-by: Paul Holzinger <pholzing@redhat.com>
If the caller can decide the lockfile path it is just more likely that there is a lock missmatch due different lock names. We should always just use the config dir and use a lock with a static name. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
We should not use XDG_RUNTIME_DIR for the ipam db. This directory is shared but we need the ipam db per storage. We can use the runroot for this. It should be a tmpfs to reset the ipam assignments after a reboot. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2fce8d4
to
34e2806
Compare
LGTM |
The HardwareAddr type must bbe the actual value and not a pointer otherwise the marshal logic will not use this function. Added a test to confirm that this works. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
/lgtm |
When the network backend is unset in the config we have to to figure out
if we need CNI or netavark. New installs should use netavark while
existing installs should continue to use CNI to prevent breaking
systems.
We use the following conditions to determine what backend to use:
we have an existing install and should continue to use CNI