Merge pull request #1255 from yaacov/dont-use-namespace-for-owner-refs

🐞 Don't use namespace for owner refs
kubev2v · Jun 30, 2024 · b504688 · b504688
2 parents 35a2a55 + 4fbf084
commit b504688
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/...console-plugin/src/modules/Providers/views/details/tabs/Hosts/utils/helpers/onSaveHost.ts b/...console-plugin/src/modules/Providers/views/details/tabs/Hosts/utils/helpers/onSaveHost.ts
@@ -137,7 +137,6 @@ async function processHostSecretPair(
             apiVersion: 'forklift.konveyor.io/v1beta1',
             kind: 'Provider',
             name: provider.metadata.name,
-            namespace: provider.metadata.namespace,
             uid: provider.metadata.uid,
           },
         ],

diff --git a/packages/forklift-console-plugin/src/modules/Providers/views/migrate/useSaveEffect.ts b/packages/forklift-console-plugin/src/modules/Providers/views/migrate/useSaveEffect.ts
@@ -47,14 +47,19 @@ const createPlan = async (
 };
 
 const addOwnerRef = async (model: K8sModel, resource, ownerReferences) => {
+  const cleanOwnerReferences = ownerReferences.map((ref) => ({
+    ...ref,
+    namespace: undefined,
+  }));
+
   return await k8sPatch({
     model,
     resource,
     data: [
       {
         op: 'add',
         path: '/metadata/ownerReferences',
-        value: ownerReferences,
+        value: cleanOwnerReferences,
       },
     ],
   });