Merge pull request #2281 from upalatucci/disable-delete

CNV-51418: disable delete on running vm

locales/en/plugin__kubevirt-plugin.json

@@ -1248,6 +1248,7 @@
   "The SCSI reservation for disk makes the disk attached to the VirtualMachine as a SCSI LUN. This option should only be used for cluster-aware applications": "The SCSI reservation for disk makes the disk attached to the VirtualMachine as a SCSI LUN. This option should only be used for cluster-aware applications",
   "The template has": "The template has",
   "The virtctl client is a supplemental command-line utility for managing virtualization resources from the command line.": "The virtctl client is a supplemental command-line utility for managing virtualization resources from the command line.",
+  "The VirtualMachine is running": "The VirtualMachine is running",
   "The VirtualMachine will be bridged to the selected network, ideal for L2 devices": "The VirtualMachine will be bridged to the selected network, ideal for L2 devices",
   "The VirtualMachine will have": "The VirtualMachine will have",
   "The VirtualMachine you are creating does not have an available boot source. We recommended that you select a boot source to create the VirtualMachine.": "The VirtualMachine you are creating does not have an available boot source. We recommended that you select a boot source to create the VirtualMachine.",

src/views/virtualmachines/actions/VirtualMachineActionFactory.tsx

@@ -22,7 +22,13 @@ import { Action, k8sPatch } from '@openshift-console/dynamic-plugin-sdk';
 import { CopyIcon } from '@patternfly/react-icons';
 import VirtualMachineMigrateModal from '@virtualmachines/actions/components/VirtualMachineMigration/VirtualMachineMigrationModal';
 
-import { isLiveMigratable, isRestoring, isSnapshotting, printableVMStatus } from '../utils';
+import {
+  isLiveMigratable,
+  isRestoring,
+  isRunning,
+  isSnapshotting,
+  printableVMStatus,
+} from '../utils';
 
 import DeleteVMModal from './components/DeleteVMModal/DeleteVMModal';
 import {
@@ -96,7 +102,8 @@ export const VirtualMachineActionFactory = {
         createModal(({ isOpen, onClose }) => (
           <DeleteVMModal isOpen={isOpen} onClose={onClose} vm={vm} />
         )),
-      disabled: false,
+      description: isRunning(vm) && t('The VirtualMachine is running'),
+      disabled: isRunning(vm),
       id: 'vm-action-delete',
       label: t('Delete'),
     };