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

🐞 Don't use namespace for owner refs

Author: yaacov

packages/forklift-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,
           },
         ],

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,
       },
     ],
   });