fix(networks) ensure volatile keys are preserved, when saving a netwo…

…rk. see canonical/lxd#14531

Signed-off-by: David Edler <[email protected]>

src/pages/networks/forms/NetworkForm.tsx

@@ -99,9 +99,7 @@ export const toNetwork = (values: NetworkFormValues): Partial<LxdNetwork> => {
   const excludeConfigKeys = getHandledNetworkConfigKeys();
   const missingConfigFields = Object.fromEntries(
     Object.entries(values.bareNetwork?.config ?? {}).filter(
-      (e) =>
-        !excludeConfigKeys.has(e[0] as keyof LxdNetworkConfig) &&
-        !e[0].startsWith("volatile"),
+      (e) => !excludeConfigKeys.has(e[0] as keyof LxdNetworkConfig),
     ),
   );
 

src/types/network.d.ts

@@ -57,6 +57,7 @@ export interface LxdNetworkConfig {
   network?: string;
   parent?: string;
   [key: `user.${string}`]: string;
+  [key: `volatile.${string}`]: string;
 }
 
 export interface LxdNetwork {

src/util/networkForm.spec.ts

@@ -29,4 +29,19 @@ describe("conversion to form values and back with toNetworkFormValues and toNetw
 
     expect(payload.config?.["user.key"]).toBe("custom-config-value");
   });
+
+  it("preserves volatile keys in network config", () => {
+    const network = {
+      devices: {},
+      config: {
+        "user.key": "custom-config-value",
+        "volatile.network.ipv4.address": "1.2.3.4",
+      },
+    } as unknown as LxdNetwork;
+
+    const formValues = toNetworkFormValues(network);
+    const payload = toNetwork(formValues);
+
+    expect(payload.config?.["volatile.network.ipv4.address"]).toBe("1.2.3.4");
+  });
 });