diff options
| author | Fernando Fernandez Mancera <ffmancera@riseup.net> | 2022-04-19 18:39:37 +0200 |
|---|---|---|
| committer | Fernando Fernandez Mancera <ffmancera@riseup.net> | 2022-04-28 10:43:36 +0200 |
| commit | 1722220eed1619dd64adfb5a053246cad00be959 (patch) | |
| tree | 78292bb2246019e33762e1e97d5045c0b86dacc7 /src/libnm-platform/nm-platform.h | |
| parent | c3b7ec5b4258b9dd932fd85d3dc0f90400097146 (diff) | |
| download | NetworkManager-1722220eed1619dd64adfb5a053246cad00be959.tar.gz | |
l3cfg: drop NM_L3_CFG_COMMIT_TYPE_ASSUME and assume_config_onceff/asusme
ASSUME is causing more troubles than benefits it provides. This patch is
dropping NM_L3_CFG_COMMIT_TYPE_ASSUME and assume_config_once. NM3LCfg
will commit as if the sys-iface-state is MANAGED.
This patch is part of the effort to remove ASSUME from NetworkManager.
After ASSUME is dropped when starting NetworkManager it will take full
control of the interface, re-configuring it. The interface will be
managed from the start instead of assumed and then managed.
This will solve the situations where an interface is half-up and then a
restart happens. When NetworkManager is back it won't add the missing
addresses (which is what assume does) so the interface will fail during
the activation and will require a full activation.
https://bugzilla.redhat.com/show_bug.cgi?id=2050216
https://bugzilla.redhat.com/show_bug.cgi?id=2077605
Diffstat (limited to 'src/libnm-platform/nm-platform.h')
| -rw-r--r-- | src/libnm-platform/nm-platform.h | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/libnm-platform/nm-platform.h b/src/libnm-platform/nm-platform.h index 1329c90e61..414f15999a 100644 --- a/src/libnm-platform/nm-platform.h +++ b/src/libnm-platform/nm-platform.h @@ -330,9 +330,7 @@ typedef enum { \ /* Meta flags not honored by NMPlatform (netlink code). Instead, they can be * used by the upper layers which use NMPlatformIPRoute to track addresses that - * should be configured. */ \ - /* Whether the address is should be configured once during assume. */ \ - bool a_assume_config_once : 1; \ + * should be configured. */ \ bool a_force_commit : 1; \ \ guint8 plen; \ @@ -476,12 +474,9 @@ typedef union { * This field overrides "table_coerced" field. If "table_any" is true, then * the "table_coerced" field is ignored (unlike for the metric). */ \ bool table_any : 1; \ - \ /* Meta flags not honored by NMPlatform (netlink code). Instead, they can be * used by the upper layers which use NMPlatformIPRoute to track routes that - * should be configured. */ \ - /* Whether the route is should be configured once during assume. */ \ - bool r_assume_config_once : 1; \ + * should be configured. */ \ /* Whether the route should be committed even if it was removed externally. */ \ bool r_force_commit : 1; \ \ |