Merge pull request #1012 from yaacov/clean-hooks

🧼 Remove unused hooks directory

package.json

@@ -89,10 +89,6 @@
     "type-fest": "^3.10.0",
     "typescript": "^4.7.4",
     "webpack": "^5.79.0",
-    "webpack-merge": "^5.8.0",
-    "react-syntax-highlighter": "^15.5.0"
-  },
-  "engines": {
-    "node": ">=16"
+    "webpack-merge": "^5.8.0"
   }
 }

packages/forklift-console-plugin/locales/en/plugin__forklift-console-plugin.json

@@ -26,7 +26,6 @@
   "Actions": "Actions",
   "Add mapping": "Add mapping",
   "Add source and target providers for the migration.": "Add source and target providers for the migration.",
-  "Adding hooks to the plan is optional. Hooks are contained in Ansible playbooks that can be run before or after the migration.": "Adding hooks to the plan is optional. Hooks are contained in Ansible playbooks that can be run before or after the migration.",
   "All discovered networks have been mapped to the default network.": "All discovered networks have been mapped to the default network.",
   "All discovered storages have been mapped to the default storage.": "All discovered storages have been mapped to the default storage.",
   "All networks detected on the selected VMs require a mapping.": "All networks detected on the selected VMs require a mapping.",
@@ -57,7 +56,6 @@
   "Category": "Category",
   "Certificate change detected": "Certificate change detected",
   "Clear all filters": "Clear all filters",
-  "Click the relevant buttons within the table for managing the plan hooks.": "Click the relevant buttons within the table for managing the plan hooks.",
   "Click the update credentials button to save your changes, button is disabled until a change is detected.": "Click the update credentials button to save your changes, button is disabled until a change is detected.",
   "Click the update mappings button to save your changes, button is disabled until a change is detected.": "Click the update mappings button to save your changes, button is disabled until a change is detected.",
   "Cluster": "Cluster",
@@ -119,7 +117,6 @@
   "Edit Controller Memory limit": "Edit Controller Memory limit",
   "Edit credentials": "Edit credentials",
   "Edit Default Transfer Network": "Edit Default Transfer Network",
-  "Edit hooks": "Edit hooks",
   "Edit mappings": "Edit mappings",
   "Edit Maximum concurrent VM migrations": "Edit Maximum concurrent VM migrations",
   "Edit migration plan target namespace": "Edit migration plan target namespace",
@@ -174,7 +171,6 @@
   "Hide from view": "Hide from view",
   "Hide values": "Hide values",
   "Hooks": "Hooks",
-  "Hooks data is loading, please wait.": "Hooks data is loading, please wait.",
   "Hooks for virtualization": "Hooks for virtualization",
   "Host": "Host",
   "Host cluster": "Host cluster",
@@ -249,7 +245,6 @@
   "New name was generated for the Storage Map due to naming conflict.": "New name was generated for the Storage Map due to naming conflict.",
   "NICs with empty NIC profile": "NICs with empty NIC profile",
   "No credentials found.": "No credentials found.",
-  "No hooks have been added to this migration plan.": "No hooks have been added to this migration plan.",
   "No inventory data available.": "No inventory data available.",
   "No Mapping found.": "No Mapping found.",
   "No NetworkMaps found.": "No NetworkMaps found.",
@@ -320,12 +315,10 @@
   "Pods": "Pods",
   "Pods not found": "Pods not found",
   "Post migration hook": "Post migration hook",
-  "Post-migration": "Post-migration",
   "Power state": "Power state",
   "Powered off": "Powered off",
   "Powered on": "Powered on",
   "Pre migration hook": "Pre migration hook",
-  "Pre-migration": "Pre-migration",
   "Precopy interval (minutes)": "Precopy interval (minutes)",
   "Preserve CPU model": "Preserve CPU model",
   "Preserve the CPU model and flags the VM runs with in its oVirt cluster.": "Preserve the CPU model and flags the VM runs with in its oVirt cluster.",
@@ -381,7 +374,6 @@
   "Skip certificate validation": "Skip certificate validation",
   "Snapshot polling interval (seconds)": "Snapshot polling interval (seconds)",
   "Something is wrong, the data was not loaded due to an error, please try to reload the page.": "Something is wrong, the data was not loaded due to an error, please try to reload the page.",
-  "Something is wrong, the hooks data was not loaded due to an error, please try to reload the page.": "Something is wrong, the hooks data was not loaded due to an error, please try to reload the page.",
   "Something is wrong, the secret was not loaded, please try to reload the page.": "Something is wrong, the secret was not loaded, please try to reload the page.",
   "source": "source",
   "Source Only": "Source Only",
@@ -415,7 +407,6 @@
   "The certificate is not a valid PEM-encoded X.509 certificate": "The certificate is not a valid PEM-encoded X.509 certificate",
   "The chosen provider is no longer available.": "The chosen provider is no longer available.",
   "The current certificate does not match the certificate fetched from URL. Manually validate the fingerprint before proceeding.": "The current certificate does not match the certificate fetched from URL. Manually validate the fingerprint before proceeding.",
-  "The edit hooks button is disabled if the plan started running and at least one virtual machine was migrated successfully.": "The edit hooks button is disabled if the plan started running and at least one virtual machine was migrated successfully.",
   "The edit mappings button is disabled if the plan started running and at least one virtual machine was migrated successfully.": "The edit mappings button is disabled if the plan started running and at least one virtual machine was migrated successfully.",
   "The interval in minutes for precopy. Default value is 60.": "The interval in minutes for precopy. Default value is 60.",
   "The interval in seconds for snapshot pooling. Default value is 10.": "The interval in seconds for snapshot pooling. Default value is 10.",

packages/forklift-console-plugin/src/modules/NetworkMaps/componenets/NetworkMapsEmptyState.tsx

@@ -3,7 +3,6 @@ import { Link } from 'react-router-dom';
 import ForkliftEmptyState from 'src/components/empty-states/ForkliftEmptyState';
 import automationIcon from 'src/components/empty-states/images/automation.svg';
 import { getResourceUrl } from 'src/modules/Providers/utils';
-import { HELP_LINK_HREF } from 'src/utils/constants';
 import { useHasSufficientProviders } from 'src/utils/fetch';
 import { ForkliftTrans, useForkliftTranslation } from 'src/utils/i18n';
 
@@ -13,6 +12,9 @@ import { Button, Flex, FlexItem } from '@patternfly/react-core';
 
 import { NetworkMapsAddButton } from './NetworkMapsAddButton';
 
+const HELP_LINK_HREF =
+  'https://access.redhat.com/documentation/en-us/migration_toolkit_for_virtualization/';
+
 const AutomationIcon = () => <img src={automationIcon} className="forklift-empty-state__icon" />;
 
 const EmptyStatePlans: React.FC<{ namespace: string }> = ({ namespace }) => {

packages/forklift-console-plugin/src/modules/NetworkMaps/utils/helpers/index.ts

@@ -1,4 +1,3 @@
 // @index(['./*', /style/g], f => `export * from '${f.path}';`)
-export * from './deepCopy';
 export * from './getNetworkMapPhase';
 // @endindex

packages/forklift-console-plugin/src/modules/NetworkMaps/views/details/components/MapsSection/state/reducer.ts

@@ -1,4 +1,4 @@
-import { deepCopy } from 'src/modules/NetworkMaps/utils';
+import { deepCopy } from 'src/utils';
 
 import { V1beta1NetworkMap, V1beta1NetworkMapSpecMap } from '@kubev2v/types';
 

packages/forklift-console-plugin/src/modules/NetworkMaps/views/details/components/ProvidersSection/state/reducer.ts

@@ -1,4 +1,4 @@
-import { deepCopy } from 'src/modules/NetworkMaps/utils';
+import { deepCopy } from 'src/utils';
 
 import { V1beta1NetworkMap, V1beta1Provider } from '@kubev2v/types';
 

packages/forklift-console-plugin/src/modules/Plans/componenets/PlansEmptyState.tsx

@@ -3,7 +3,6 @@ import { Link } from 'react-router-dom';
 import ForkliftEmptyState from 'src/components/empty-states/ForkliftEmptyState';
 import digitalTransformation from 'src/modules/Overview/images/digitalTransormation.svg';
 import { getResourceUrl } from 'src/modules/Providers/utils';
-import { HELP_LINK_HREF } from 'src/utils/constants';
 import { useHasSufficientProviders } from 'src/utils/fetch';
 import { ForkliftTrans, useForkliftTranslation } from 'src/utils/i18n';
 
@@ -13,6 +12,9 @@ import { Button, Flex, FlexItem } from '@patternfly/react-core';
 
 import PlansAddButton from './PlansAddButton';
 
+const HELP_LINK_HREF =
+  'https://access.redhat.com/documentation/en-us/migration_toolkit_for_virtualization/';
+
 const AutomationIcon = () => (
   <img src={digitalTransformation} className="forklift-empty-state__icon" />
 );

packages/forklift-console-plugin/src/modules/Plans/utils/helpers/index.ts

@@ -1,5 +1,4 @@
 // @index(['./*', /style/g], f => `export * from '${f.path}';`)
-export * from './deepCopy';
 export * from './getMigrationPhase';
 export * from './getMigrationVmsCounts';
 export * from './getPhaseLabel';

packages/forklift-console-plugin/src/modules/Plans/views/details/PlanDetailsPage.tsx

@@ -6,11 +6,11 @@ import { HorizontalNav, K8sModel } from '@openshift-console/dynamic-plugin-sdk';
 import { PlanPageHeadings } from './components';
 import {
   PlanDetails,
+  PlanHooks,
   PlanMappings,
   PlanResources,
   PlanVirtualMachines,
   PlanYAML,
-  SimpleHooks,
 } from './tabs';
 
 import './PlanDetailsPage.style.css';
@@ -55,7 +55,7 @@ const PlanDetailsPage_: React.FC<{ name: string; namespace: string }> = ({ name,
     {
       href: 'hooks',
       name: t('Hooks'),
-      component: () => <SimpleHooks name={name} namespace={namespace} />,
+      component: () => <PlanHooks name={name} namespace={namespace} />,
     },
   ];
 

packages/forklift-console-plugin/src/modules/Plans/views/details/components/ProvidersSection/state/reducer.ts

@@ -1,4 +1,4 @@
-import { deepCopy } from 'src/modules/NetworkMaps/utils';
+import { deepCopy } from 'src/utils';
 
 import { V1beta1Plan, V1beta1Provider } from '@kubev2v/types';