Merge pull request #619 from yaacov/ova-plan-support

[MTV-567] Add plan support for OVA providers
kubev2v · Jul 27, 2023 · 9497fda · 9497fda
2 parents 0bcaa5d + a2e1f45
commit 9497fda
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/packages/legacy/src/Plans/components/Wizard/TypeForm.tsx b/packages/legacy/src/Plans/components/Wizard/TypeForm.tsx
@@ -32,6 +32,8 @@ export const TypeForm: React.FunctionComponent<ITypeFormProps> = ({
   const isSourceOvirtInsecure = checkIfOvirtInsecureProvider(sourceProvider, secretsQuery.data);
   const isSourceOpenstack = sourceProvider?.type === 'openstack';
   const isSourceOCP = sourceProvider?.type === 'openshift';
+  const isSourceOVA = sourceProvider?.type === 'ova';
+  const isDisabled = isSourceOvirtInsecure || isSourceOpenstack || isSourceOCP || isSourceOVA;
 
   return (
     <ResolvedQueries results={[secretsQuery]} errorTitles={['Cannot load provider secrets']}>
@@ -60,18 +62,11 @@ export const TypeForm: React.FunctionComponent<ITypeFormProps> = ({
           </StackItem>
         )}
 
-        {isSourceOpenstack && (
+        {!isSourceOvirtInsecure && isDisabled && (
           <StackItem>
             <Alert variant="warning" isInline title="Warm migration is not currently available.">
-              Warm migrations from {PROVIDER_TYPE_NAMES.openstack} source providers are unsupported.
-            </Alert>
-          </StackItem>
-        )}
-
-        {isSourceOCP && (
-          <StackItem>
-            <Alert variant="warning" isInline title="Warm migration is not currently available.">
-              Warm migrations from {PROVIDER_TYPE_NAMES.openshift} source providers are unsupported.
+              Warm migrations from {PROVIDER_TYPE_NAMES?.[sourceProvider?.type]} source providers
+              are unsupported.
             </Alert>
           </StackItem>
         )}
@@ -81,7 +76,7 @@ export const TypeForm: React.FunctionComponent<ITypeFormProps> = ({
             id="migration-type-warm"
             name="migration-type"
             label="Warm migration"
-            isDisabled={isSourceOvirtInsecure || isSourceOpenstack || isSourceOCP}
+            isDisabled={isDisabled}
             description={
               <List>
                 <ListItem>VM data is incrementally copied, leaving source VMs running.</ListItem>
@@ -92,7 +87,7 @@ export const TypeForm: React.FunctionComponent<ITypeFormProps> = ({
               </List>
             }
             body={
-              !(isSourceOvirtInsecure || isSourceOpenstack || isSourceOCP) && (
+              !isDisabled && (
                 <>
                   {isAnalyzingVms && (
                     <div className={`${spacing.mtMd} ${spacing.mlXs}`}>

diff --git a/packages/legacy/src/queries/storages.ts b/packages/legacy/src/queries/storages.ts
@@ -32,6 +32,9 @@ export const useSourceStoragesQuery = (
     if (provider?.type === 'openshift') {
       return '/storageclasses?detail=1';
     }
+    if (provider?.type === 'ova') {
+      return '/disks?detail=1';
+    }
     return '/storagedomains';
   };
   const mockStorage = (provider: SourceInventoryProvider) => {