From c2cc226b67a70be48fcb6a213f10ac9de7a47d91 Mon Sep 17 00:00:00 2001
From: David Edler <david.edler@canonical.com>
Date: Wed, 13 Mar 2024 10:58:35 +0100
Subject: [PATCH] feat(network) use config api for network form help texts and
 defaults, unify network form key handling WD-7391

Signed-off-by: David Edler <david.edler@canonical.com>
---
 src/pages/networks/forms/NetworkForm.tsx      | 57 ++++++------
 .../networks/forms/NetworkFormBridge.tsx      |  3 -
 src/pages/networks/forms/NetworkFormDns.tsx   |  2 -
 src/pages/networks/forms/NetworkFormIpv4.tsx  |  3 -
 src/pages/networks/forms/NetworkFormIpv6.tsx  |  3 -
 .../storage/forms/StorageVolumeFormMain.tsx   |  2 -
 src/types/config.d.ts                         |  2 +
 src/util/configInheritance.tsx                | 24 +++++-
 src/util/networkForm.tsx                      | 86 +++++++------------
 src/util/networks.tsx                         | 82 +++++++++++-------
 tests/networks.spec.ts                        | 14 +--
 11 files changed, 140 insertions(+), 138 deletions(-)

diff --git a/src/pages/networks/forms/NetworkForm.tsx b/src/pages/networks/forms/NetworkForm.tsx
index 3c0e4c0166..0e3d8119f4 100644
--- a/src/pages/networks/forms/NetworkForm.tsx
+++ b/src/pages/networks/forms/NetworkForm.tsx
@@ -10,6 +10,7 @@ import {
 import {
   LxdNetwork,
   LxdNetworkBridgeDriver,
+  LxdNetworkConfig,
   LxdNetworkDnsMode,
   LxdNetworkType,
 } from "types/network";
@@ -31,9 +32,9 @@ import NetworkFormDns from "pages/networks/forms/NetworkFormDns";
 import NetworkFormIpv4 from "pages/networks/forms/NetworkFormIpv4";
 import NetworkFormIpv6 from "pages/networks/forms/NetworkFormIpv6";
 import { slugify } from "util/slugify";
-import { handleConfigKeys } from "util/networkForm";
 import { useDocs } from "context/useDocs";
 import YamlConfirmation from "components/forms/YamlConfirmation";
+import { getHandledNetworkConfigKeys, getNetworkKey } from "util/networks";
 
 export interface NetworkFormValues {
   readOnly: boolean;
@@ -88,10 +89,12 @@ export const toNetwork = (values: NetworkFormValues): Partial<LxdNetwork> => {
     ),
   );
 
-  const excludeConfigKeys = new Set(handleConfigKeys);
+  const excludeConfigKeys = getHandledNetworkConfigKeys();
   const missingConfigFields = Object.fromEntries(
     Object.entries(values.bareNetwork?.config ?? {}).filter(
-      (e) => !excludeConfigKeys.has(e[0]) && !e[0].startsWith("volatile"),
+      (e) =>
+        !excludeConfigKeys.has(e[0] as keyof LxdNetworkConfig) &&
+        !e[0].startsWith("volatile"),
     ),
   );
 
@@ -102,30 +105,30 @@ export const toNetwork = (values: NetworkFormValues): Partial<LxdNetwork> => {
     type: values.networkType,
     config: {
       ...missingConfigFields,
-      ["bridge.driver"]: values.bridge_driver,
-      ["bridge.hwaddr"]: values.bridge_hwaddr,
-      ["bridge.mtu"]: values.bridge_mtu,
-      ["dns.domain"]: values.dns_domain,
-      ["dns.mode"]: values.dns_mode,
-      ["dns.search"]: values.dns_search,
-      ["ipv4.address"]: values.ipv4_address,
-      ["ipv4.dhcp"]: values.ipv4_dhcp,
-      ["ipv4.dhcp.expiry"]: values.ipv4_dhcp_expiry,
-      ["ipv4.dhcp.ranges"]: values.ipv4_dhcp_ranges,
-      ["ipv4.l3only"]: values.ipv4_l3only,
-      ["ipv4.nat"]: values.ipv4_nat,
-      ["ipv4.nat.address"]: values.ipv4_nat_address,
-      ["ipv4.ovn.ranges"]: values.ipv4_ovn_ranges,
-      ["ipv6.address"]: values.ipv6_address,
-      ["ipv6.dhcp"]: values.ipv6_dhcp,
-      ["ipv6.dhcp.expiry"]: values.ipv6_dhcp_expiry,
-      ["ipv6.dhcp.ranges"]: values.ipv6_dhcp_ranges,
-      ["ipv6.dhcp.stateful"]: values.ipv6_dhcp_stateful,
-      ["ipv6.l3only"]: values.ipv6_l3only,
-      ["ipv6.nat"]: values.ipv6_nat,
-      ["ipv6.nat.address"]: values.ipv6_nat_address,
-      ["ipv6.ovn.ranges"]: values.ipv6_ovn_ranges,
-      ["network"]: values.network,
+      [getNetworkKey("bridge_driver")]: values.bridge_driver,
+      [getNetworkKey("bridge_hwaddr")]: values.bridge_hwaddr,
+      [getNetworkKey("bridge_mtu")]: values.bridge_mtu,
+      [getNetworkKey("dns_domain")]: values.dns_domain,
+      [getNetworkKey("dns_mode")]: values.dns_mode,
+      [getNetworkKey("dns_search")]: values.dns_search,
+      [getNetworkKey("ipv4_address")]: values.ipv4_address,
+      [getNetworkKey("ipv4_dhcp")]: values.ipv4_dhcp,
+      [getNetworkKey("ipv4_dhcp_expiry")]: values.ipv4_dhcp_expiry,
+      [getNetworkKey("ipv4_dhcp_ranges")]: values.ipv4_dhcp_ranges,
+      [getNetworkKey("ipv4_l3only")]: values.ipv4_l3only,
+      [getNetworkKey("ipv4_nat")]: values.ipv4_nat,
+      [getNetworkKey("ipv4_nat_address")]: values.ipv4_nat_address,
+      [getNetworkKey("ipv4_ovn_ranges")]: values.ipv4_ovn_ranges,
+      [getNetworkKey("ipv6_address")]: values.ipv6_address,
+      [getNetworkKey("ipv6_dhcp")]: values.ipv6_dhcp,
+      [getNetworkKey("ipv6_dhcp_expiry")]: values.ipv6_dhcp_expiry,
+      [getNetworkKey("ipv6_dhcp_ranges")]: values.ipv6_dhcp_ranges,
+      [getNetworkKey("ipv6_dhcp_stateful")]: values.ipv6_dhcp_stateful,
+      [getNetworkKey("ipv6_l3only")]: values.ipv6_l3only,
+      [getNetworkKey("ipv6_nat")]: values.ipv6_nat,
+      [getNetworkKey("ipv6_nat_address")]: values.ipv6_nat_address,
+      [getNetworkKey("ipv6_ovn_ranges")]: values.ipv6_ovn_ranges,
+      [getNetworkKey("network")]: values.network,
     },
   };
 };
diff --git a/src/pages/networks/forms/NetworkFormBridge.tsx b/src/pages/networks/forms/NetworkFormBridge.tsx
index 628d93d9d8..51ab30d105 100644
--- a/src/pages/networks/forms/NetworkFormBridge.tsx
+++ b/src/pages/networks/forms/NetworkFormBridge.tsx
@@ -17,7 +17,6 @@ const NetworkFormBridge: FC<Props> = ({ formik }) => {
           formik,
           name: "bridge_mtu",
           label: "MTU",
-          help: "Bridge MTU (default varies if tunnel or fan setup)",
           defaultValue: "",
           children: <Input type="text" />,
         }),
@@ -26,7 +25,6 @@ const NetworkFormBridge: FC<Props> = ({ formik }) => {
           formik,
           name: "bridge_hwaddr",
           label: "Hardware address",
-          help: "MAC address for the bridge",
           defaultValue: "",
           children: <Input type="text" />,
         }),
@@ -37,7 +35,6 @@ const NetworkFormBridge: FC<Props> = ({ formik }) => {
                 formik,
                 name: "bridge_driver",
                 label: "Bridge driver",
-                help: "Native or openvswitch",
                 defaultValue: "",
                 children: (
                   <Select
diff --git a/src/pages/networks/forms/NetworkFormDns.tsx b/src/pages/networks/forms/NetworkFormDns.tsx
index f3e03924fb..afbf67356f 100644
--- a/src/pages/networks/forms/NetworkFormDns.tsx
+++ b/src/pages/networks/forms/NetworkFormDns.tsx
@@ -17,7 +17,6 @@ const NetworkFormDns: FC<Props> = ({ formik }) => {
           formik,
           name: "dns_domain",
           label: "DNS domain",
-          help: "Domain to advertise to DHCP clients and use for DNS resolution",
           defaultValue: "",
           children: <Input type="text" />,
         }),
@@ -60,7 +59,6 @@ const NetworkFormDns: FC<Props> = ({ formik }) => {
           formik,
           name: "dns_search",
           label: "DNS search",
-          help: "Full comma-separated domain search list, defaulting to DNS domain value",
           defaultValue: "",
           children: <Textarea />,
         }),
diff --git a/src/pages/networks/forms/NetworkFormIpv4.tsx b/src/pages/networks/forms/NetworkFormIpv4.tsx
index bc388beff8..e86272cd86 100644
--- a/src/pages/networks/forms/NetworkFormIpv4.tsx
+++ b/src/pages/networks/forms/NetworkFormIpv4.tsx
@@ -30,7 +30,6 @@ const NetworkFormIpv4: FC<Props> = ({ formik }) => {
                 formik,
                 name: "ipv4_dhcp_expiry",
                 label: "IPv4 DHCP expiry",
-                help: "When to expire DHCP leases",
                 defaultValue: "",
                 children: <Input type="text" />,
               }),
@@ -39,7 +38,6 @@ const NetworkFormIpv4: FC<Props> = ({ formik }) => {
                 formik,
                 name: "ipv4_dhcp_ranges",
                 label: "IPv4 DHCP ranges",
-                help: "Comma-separated list of IP ranges to use for DHCP (FIRST-LAST format)",
                 defaultValue: "",
                 children: <Textarea />,
               }),
@@ -64,7 +62,6 @@ const NetworkFormIpv4: FC<Props> = ({ formik }) => {
                 formik,
                 name: "ipv4_ovn_ranges",
                 label: "IPv4 OVN ranges",
-                help: "Comma-separated list of IPv4 ranges to use for child OVN network routers (FIRST-LAST format)",
                 defaultValue: "",
                 children: <Textarea />,
               }),
diff --git a/src/pages/networks/forms/NetworkFormIpv6.tsx b/src/pages/networks/forms/NetworkFormIpv6.tsx
index 59d75fb98f..55c3767614 100644
--- a/src/pages/networks/forms/NetworkFormIpv6.tsx
+++ b/src/pages/networks/forms/NetworkFormIpv6.tsx
@@ -30,7 +30,6 @@ const NetworkFormIpv6: FC<Props> = ({ formik }) => {
                 formik,
                 name: "ipv6_dhcp_expiry",
                 label: "IPv6 DHCP expiry",
-                help: "When to expire DHCP leases",
                 defaultValue: "",
                 children: <Input type="text" />,
               }),
@@ -39,7 +38,6 @@ const NetworkFormIpv6: FC<Props> = ({ formik }) => {
                 formik,
                 name: "ipv6_dhcp_ranges",
                 label: "IPv6 DHCP ranges",
-                help: "Comma-separated list of IPv6 ranges to use for DHCP (FIRST-LAST format)",
                 defaultValue: "",
                 children: <Textarea />,
               }),
@@ -76,7 +74,6 @@ const NetworkFormIpv6: FC<Props> = ({ formik }) => {
                 formik,
                 name: "ipv6_ovn_ranges",
                 label: "IPv6 OVN ranges",
-                help: "Comma-separated list of IPv6 ranges to use for child OVN network routers (FIRST-LAST format)",
                 defaultValue: "",
                 children: <Textarea />,
               }),
diff --git a/src/pages/storage/forms/StorageVolumeFormMain.tsx b/src/pages/storage/forms/StorageVolumeFormMain.tsx
index 8c21246a10..4dc000c923 100644
--- a/src/pages/storage/forms/StorageVolumeFormMain.tsx
+++ b/src/pages/storage/forms/StorageVolumeFormMain.tsx
@@ -101,7 +101,6 @@ const StorageVolumeFormMain: FC<Props> = ({ formik, project }) => {
               label: "Security shifted",
               name: "security_shifted",
               defaultValue: "",
-              help: "Enable id shifting overlay (allows attach to multiple isolated instances)",
               disabled: formik.values.security_unmapped === "true",
               disabledReason:
                 "This setting can't be changed while security unmapped is set to true",
@@ -113,7 +112,6 @@ const StorageVolumeFormMain: FC<Props> = ({ formik, project }) => {
               label: "Security unmapped",
               name: "security_unmapped",
               defaultValue: "",
-              help: "Disable id mapping for the volume",
               disabled: formik.values.security_shifted === "true",
               disabledReason:
                 "This setting can't be changed while security shifted is set to true",
diff --git a/src/types/config.d.ts b/src/types/config.d.ts
index 960b6a0166..70ad5674c4 100644
--- a/src/types/config.d.ts
+++ b/src/types/config.d.ts
@@ -27,6 +27,8 @@ export interface LxdConfigOptions {
   configs: {
     cluster: LxcConfigOptionCategories;
     instance: LxcConfigOptionCategories;
+    "network-bridge": LxcConfigOptionCategories;
+    "network-ovn": LxcConfigOptionCategories;
     project: LxcConfigOptionCategories;
     server: LxcConfigOptionCategories;
     "storage-btrfs": LxcConfigOptionCategories;
diff --git a/src/util/configInheritance.tsx b/src/util/configInheritance.tsx
index 1914673b44..a9972b6160 100644
--- a/src/util/configInheritance.tsx
+++ b/src/util/configInheritance.tsx
@@ -18,10 +18,11 @@ import { getProjectKey } from "util/projectConfigFields";
 import { StorageVolumeFormValues } from "pages/storage/forms/StorageVolumeForm";
 import { fetchStoragePool } from "api/storage-pools";
 import { getVolumeKey } from "util/storageVolume";
-import { getNetworkDefault } from "util/networks";
+import { getNetworkKey, networkFormTypeToOptionKey } from "util/networks";
 import { getPoolKey, storagePoolFormDriverToOptionKey } from "./storagePool";
 import { StoragePoolFormValues } from "pages/storage/forms/StoragePoolForm";
 import { useSupportedFeatures } from "context/useSupportedFeatures";
+import { NetworkFormValues } from "pages/networks/forms/NetworkForm";
 
 export interface ConfigRowMetadata {
   value?: string;
@@ -42,7 +43,7 @@ export const getConfigRowMetadata = (
     case "storageVolume":
       return getStorageVolumeRowMetadata(values, name);
     case "network":
-      return getNetworkRowMetadata(name);
+      return getNetworkRowMetadata(values, name);
     case "storagePool":
       return getStoragePoolRowMetadata(values, name);
   }
@@ -135,8 +136,23 @@ const getStorageVolumeRowMetadata = (
   return { value: lxdDefault, source: "LXD", configField };
 };
 
-const getNetworkRowMetadata = (name: string): ConfigRowMetadata => {
-  return getNetworkDefault(name);
+const getNetworkRowMetadata = (
+  values: NetworkFormValues,
+  name: string,
+): ConfigRowMetadata => {
+  const configOptions = getConfigOptions();
+
+  const optionKey = networkFormTypeToOptionKey(values.networkType);
+  const configFields = toConfigFields(configOptions?.configs[optionKey] ?? {});
+  const configKey = getNetworkKey(name);
+  const configField = configFields.find((item) => item.key === configKey);
+
+  const lxdDefault =
+    configField?.default && configField?.default.length > 0
+      ? configField?.default
+      : "-";
+
+  return { value: lxdDefault, source: "LXD", configField };
 };
 
 // NOTE: this is only relevant for Ceph RBD storage pools at the moment
diff --git a/src/util/networkForm.tsx b/src/util/networkForm.tsx
index 2549b75e6b..5a5f1f5336 100644
--- a/src/util/networkForm.tsx
+++ b/src/util/networkForm.tsx
@@ -1,5 +1,10 @@
-import { LxdNetwork } from "types/network";
+import {
+  LxdNetwork,
+  LxdNetworkBridgeDriver,
+  LxdNetworkDnsMode,
+} from "types/network";
 import { NetworkFormValues } from "pages/networks/forms/NetworkForm";
+import { getNetworkKey } from "util/networks";
 
 export const toNetworkFormValues = (network: LxdNetwork): NetworkFormValues => {
   return {
@@ -8,62 +13,33 @@ export const toNetworkFormValues = (network: LxdNetwork): NetworkFormValues => {
     name: network.name,
     description: network.description,
     networkType: network.type,
-    bridge_driver: network.config["bridge.driver"],
-    bridge_hwaddr: network.config["bridge.hwaddr"],
-    bridge_mtu: network.config["bridge.mtu"],
-    dns_domain: network.config["dns.domain"],
-    dns_mode: network.config["dns.mode"],
-    dns_search: network.config["dns.search"],
-    ipv4_address: network.config["ipv4.address"],
-    ipv4_dhcp: network.config["ipv4.dhcp"],
-    ipv4_dhcp_expiry: network.config["ipv4.dhcp.expiry"],
-    ipv4_dhcp_ranges: network.config["ipv4.dhcp.ranges"],
-    ipv4_l3only: network.config["ipv4.l3only"],
-    ipv4_nat: network.config["ipv4.nat"],
-    ipv4_nat_address: network.config["ipv4.nat.address"],
-    ipv4_ovn_ranges: network.config["ipv4.ovn.ranges"],
-    ipv6_address: network.config["ipv6.address"],
-    ipv6_dhcp: network.config["ipv6.dhcp"],
-    ipv6_dhcp_expiry: network.config["ipv6.dhcp.expiry"],
-    ipv6_dhcp_ranges: network.config["ipv6.dhcp.ranges"],
-    ipv6_dhcp_stateful: network.config["ipv6.dhcp.stateful"],
-    ipv6_l3only: network.config["ipv6.l3only"],
-    ipv6_nat: network.config["ipv6.nat"],
-    ipv6_nat_address: network.config["ipv6.nat.address"],
-    ipv6_ovn_ranges: network.config["ipv6.ovn.ranges"],
+    bridge_driver: network.config[
+      getNetworkKey("bridge_driver")
+    ] as LxdNetworkBridgeDriver,
+    bridge_hwaddr: network.config[getNetworkKey("bridge_hwaddr")],
+    bridge_mtu: network.config[getNetworkKey("bridge_mtu")],
+    dns_domain: network.config[getNetworkKey("dns_domain")],
+    dns_mode: network.config[getNetworkKey("dns_mode")] as LxdNetworkDnsMode,
+    dns_search: network.config[getNetworkKey("dns_search")],
+    ipv4_address: network.config[getNetworkKey("ipv4_address")],
+    ipv4_dhcp: network.config[getNetworkKey("ipv4_dhcp")],
+    ipv4_dhcp_expiry: network.config[getNetworkKey("ipv4_dhcp_expiry")],
+    ipv4_dhcp_ranges: network.config[getNetworkKey("ipv4_dhcp_ranges")],
+    ipv4_l3only: network.config[getNetworkKey("ipv4_l3only")],
+    ipv4_nat: network.config[getNetworkKey("ipv4_nat")],
+    ipv4_nat_address: network.config[getNetworkKey("ipv4_nat_address")],
+    ipv4_ovn_ranges: network.config[getNetworkKey("ipv4_ovn_ranges")],
+    ipv6_address: network.config[getNetworkKey("ipv6_address")],
+    ipv6_dhcp: network.config[getNetworkKey("ipv6_dhcp")],
+    ipv6_dhcp_expiry: network.config[getNetworkKey("ipv6_dhcp_expiry")],
+    ipv6_dhcp_ranges: network.config[getNetworkKey("ipv6_dhcp_ranges")],
+    ipv6_dhcp_stateful: network.config[getNetworkKey("ipv6_dhcp_stateful")],
+    ipv6_l3only: network.config[getNetworkKey("ipv6_l3only")],
+    ipv6_nat: network.config[getNetworkKey("ipv6_nat")],
+    ipv6_nat_address: network.config[getNetworkKey("ipv6_nat_address")],
+    ipv6_ovn_ranges: network.config[getNetworkKey("ipv6_ovn_ranges")],
     network: network.config.network,
     entityType: "network",
     bareNetwork: network,
   };
 };
-
-export const handleConfigKeys = [
-  "bridge.driver",
-  "bridge.hwaddr",
-  "bridge.mode",
-  "bridge.mtu",
-  "dns.domain",
-  "dns.mode",
-  "dns.search",
-  "fan.type",
-  "fan.overlay_subnet",
-  "fan.underlay_subnet",
-  "ipv4.address",
-  "ipv4.dhcp",
-  "ipv4.dhcp.expiry",
-  "ipv4.dhcp.ranges",
-  "ipv4.l3only",
-  "ipv4.nat",
-  "ipv4.nat.address",
-  "ipv4.ovn.ranges",
-  "ipv6.address",
-  "ipv6.dhcp",
-  "ipv6.dhcpexpiry",
-  "ipv6.dhcp.ranges",
-  "ipv6.dhcp.stateful",
-  "ipv6.l3only",
-  "ipv6.nat",
-  "ipv6.nat.address",
-  "ipv6.ovn.ranges",
-  "network",
-];
diff --git a/src/util/networks.tsx b/src/util/networks.tsx
index d9ff88b7fd..ea4b52d8ca 100644
--- a/src/util/networks.tsx
+++ b/src/util/networks.tsx
@@ -1,6 +1,6 @@
 import { LxdInstance } from "types/instance";
 import { LxdNetwork, LxdNetworkConfig } from "types/network";
-import { ConfigRowMetadata } from "util/configInheritance";
+import { LxdConfigOptionsKeys } from "types/config";
 
 export const getIpAddresses = (
   instance: LxdInstance,
@@ -17,37 +17,61 @@ export const getIpAddresses = (
     .filter((item) => item.family === family);
 };
 
-const networkDefaults: Record<string, string> = {
-  bridge_driver: "native",
-  bridge_hwaddr: "-",
-  bridge_mtu: "1500",
-  dns_domain: "lxd",
-  dns_mode: "managed",
-  dns_search: "-",
-  ipv4_address: "auto",
-  ipv4_dhcp: "true",
-  ipv4_dhcp_expiry: "1h",
-  ipv4_dhcp_ranges: "all addresses",
-  ipv4_l3only: "false",
-  ipv4_nat: "true",
-  ipv4_nat_address: "-",
-  ipv4_ovn_ranges: "-",
-  ipv6_address: "auto",
-  ipv6_dhcp: "true",
-  ipv6_dhcp_expiry: "1h",
-  ipv6_dhcp_ranges: "all addresses",
-  ipv6_dhcp_stateful: "false",
-  ipv6_l3only: "false",
-  ipv6_nat: "true",
-  ipv6_nat_address: ".",
-  ipv6_ovn_ranges: "-",
+export const networkFormFieldToPayloadName: Record<
+  string,
+  keyof LxdNetworkConfig
+> = {
+  bridge_driver: "bridge.driver",
+  bridge_hwaddr: "bridge.hwaddr",
+  bridge_mtu: "bridge.mtu",
+  dns_domain: "dns.domain",
+  dns_mode: "dns.mode",
+  dns_search: "dns.search",
+  ipv4_address: "ipv4.address",
+  ipv4_dhcp: "ipv4.dhcp",
+  ipv4_dhcp_expiry: "ipv4.dhcp.expiry",
+  ipv4_dhcp_ranges: "ipv4.dhcp.ranges",
+  ipv4_l3only: "ipv4.l3only",
+  ipv4_nat: "ipv4.nat",
+  ipv4_nat_address: "ipv4.nat.address",
+  ipv4_ovn_ranges: "ipv4.ovn.ranges",
+  ipv6_address: "ipv6.address",
+  ipv6_dhcp: "ipv6.dhcp",
+  ipv6_dhcp_expiry: "ipv6.dhcp.expiry",
+  ipv6_dhcp_ranges: "ipv6.dhcp.ranges",
+  ipv6_dhcp_stateful: "ipv6.dhcp.stateful",
+  ipv6_l3only: "ipv6.l3only",
+  ipv6_nat: "ipv6.nat",
+  ipv6_nat_address: "ipv6.nat.address",
+  ipv6_ovn_ranges: "ipv6.ovn.ranges",
+  network: "network",
 };
 
-export const getNetworkDefault = (formField: string): ConfigRowMetadata => {
-  if (Object.keys(networkDefaults).includes(formField)) {
-    return { value: networkDefaults[formField], source: "LXD" };
+export const getHandledNetworkConfigKeys = () => {
+  return new Set(Object.values(networkFormFieldToPayloadName));
+};
+
+export const getNetworkKey = (formField: string): keyof LxdNetworkConfig => {
+  if (!(formField in networkFormFieldToPayloadName)) {
+    throw new Error(
+      `Could not find ${formField} in networkFormFieldToPayloadName`,
+    );
+  }
+  return networkFormFieldToPayloadName[formField];
+};
+
+const networkTypeToOptionKey: Record<string, LxdConfigOptionsKeys> = {
+  bridge: "network-bridge",
+  ovn: "network-ovn",
+};
+
+export const networkFormTypeToOptionKey = (
+  type: string,
+): LxdConfigOptionsKeys => {
+  if (!(type in networkTypeToOptionKey)) {
+    throw new Error(`Could not find ${type} in networkTypeToOptionKey`);
   }
-  return { value: "", source: "LXD" };
+  return networkTypeToOptionKey[type];
 };
 
 const hasNetworkConfigDiff = (
diff --git a/tests/networks.spec.ts b/tests/networks.spec.ts
index 9a36e1e818..e3d2d7b5d5 100644
--- a/tests/networks.spec.ts
+++ b/tests/networks.spec.ts
@@ -40,7 +40,7 @@ test("network edit basic details", async ({ page }) => {
     .getByLabel("Network form navigation")
     .getByText("Bridge", { exact: true })
     .click();
-  await activateOverride(page, "MTU Bridge MTU");
+  await activateOverride(page, "MTU");
   await page.getByLabel("MTU").fill("1300");
   await activateOverride(page, "Bridge driver");
   await page
@@ -48,7 +48,7 @@ test("network edit basic details", async ({ page }) => {
     .selectOption("native");
 
   await page.getByText("DNS").click();
-  await activateOverride(page, "DNS domain Domain");
+  await activateOverride(page, "DNS domain");
   await page.getByLabel("DNS domain").fill("abc");
   await activateOverride(page, "DNS mode");
   await page.getByRole("combobox", { name: "DNS mode" }).selectOption("none");
@@ -56,14 +56,10 @@ test("network edit basic details", async ({ page }) => {
   await page.getByLabel("DNS search").fill("abc");
 
   await page.getByText("IPv4").click();
-  await activateOverride(page, "IPv4 DHCP true");
-  await setOption(page, "IPv4 DHCP", "true");
   await activateOverride(page, "IPv4 DHCP expiry");
   await page.getByLabel("IPv4 DHCP expiry").fill("2h");
 
   await page.getByText("IPv6").click();
-  await activateOverride(page, "IPv6 DHCP true");
-  await setOption(page, "IPv6 DHCP", "true");
   await activateOverride(page, "IPv6 DHCP expiry");
   await page.getByLabel("IPv6 DHCP expiry").fill("3h");
   await activateOverride(page, "IPv6 DHCP stateful");
@@ -82,20 +78,18 @@ test("network edit basic details", async ({ page }) => {
     .getByLabel("Network form navigation")
     .getByText("Bridge", { exact: true })
     .click();
-  await assertReadMode(page, "MTU Bridge MTU", "1300");
+  await assertReadMode(page, "MTU", "1300");
   await assertReadMode(page, "Bridge driver", "native");
 
   await page.getByText("DNS", { exact: true }).click();
-  await assertReadMode(page, "DNS domain Domain", "abc");
+  await assertReadMode(page, "DNS domain", "abc");
   await assertReadMode(page, "DNS mode", "none");
   await assertReadMode(page, "DNS search", "abc");
 
   await page.getByText("IPv4", { exact: true }).click();
-  await assertReadMode(page, "IPv4 DHCP", "true");
   await assertReadMode(page, "IPv4 DHCP expiry", "2h");
 
   await page.getByText("IPv6", { exact: true }).click();
-  await assertReadMode(page, "IPv6 DHCP true", "true");
   await assertReadMode(page, "IPv6 DHCP expiry", "3h");
   await assertReadMode(page, "IPv6 DHCP stateful", "true");
 });