From 81ac694ceae8a0ea8e95791c9028210531c711a7 Mon Sep 17 00:00:00 2001
From: Jiri Tomasek <jiri.tomasek@suse.com>
Date: Fri, 7 Jul 2023 10:11:43 +0200
Subject: [PATCH] Clean up comments and unused code

---
 .../components/console/SerialConsole.vue      |   4 +-
 pkg/kubevirt/formatters/NodeName.vue          |  50 ---------
 pkg/kubevirt/product.ts                       | 105 +-----------------
 3 files changed, 3 insertions(+), 156 deletions(-)
 delete mode 100644 pkg/kubevirt/formatters/NodeName.vue

diff --git a/pkg/kubevirt/components/console/SerialConsole.vue b/pkg/kubevirt/components/console/SerialConsole.vue
index 2781dd5..1bdde30 100644
--- a/pkg/kubevirt/components/console/SerialConsole.vue
+++ b/pkg/kubevirt/components/console/SerialConsole.vue
@@ -244,7 +244,7 @@ export default {
 </script>
 
 <template>
-  <div class="harvester-shell-container">
+  <div class="console-shell-container">
     <div ref="xterm" class="shell-body" />
     <resize-observer @notify="fit" />
   </div>
@@ -260,7 +260,7 @@ MAIN {
   height: 100%;
 }
 
-.harvester-shell-container {
+.console-shell-container {
   height: 100%;
   overflow: hidden;
 
diff --git a/pkg/kubevirt/formatters/NodeName.vue b/pkg/kubevirt/formatters/NodeName.vue
deleted file mode 100644
index 626d598..0000000
--- a/pkg/kubevirt/formatters/NodeName.vue
+++ /dev/null
@@ -1,50 +0,0 @@
-<script>
-import { NODE } from '@shell/config/types';
-import CopyToClipboard from '@shell/components/CopyToClipboard';
-import { VMI_RESOURCE_NAME } from '../constants';
-
-export default {
-  components: { CopyToClipboard },
-  props: {
-    value: {
-      type: String,
-      default: '',
-    },
-    row: {
-      type: Object,
-      required: true,
-    },
-  },
-
-  computed: {
-    nodeName() {
-      return this.node?.nameDisplay || '';
-    },
-
-    vmi() {
-      const vmiResources = this.$store.getters['harvester/all'](VMI_RESOURCE_NAME);
-      const resource = vmiResources.find((VMI) => VMI.id === this.row.id) || null;
-
-      return resource;
-    },
-
-    node() {
-      const nodeName = this.vmi?.status?.nodeName;
-
-      return this.$store.getters['harvester/byId'](NODE, nodeName);
-    },
-  },
-};
-</script>
-
-<template>
-  <div>
-    {{ nodeName }}
-    <CopyToClipboard
-      :text="nodeName"
-      label-as="tooltip"
-      class="icon-btn"
-      action-color="bg-transparent"
-    />
-  </div>
-</template>
diff --git a/pkg/kubevirt/product.ts b/pkg/kubevirt/product.ts
index 5f7a614..f2749a2 100644
--- a/pkg/kubevirt/product.ts
+++ b/pkg/kubevirt/product.ts
@@ -1,5 +1,4 @@
 import { IPlugin } from '@shell/core/types';
-// import { STATE, NAME, NAMESPACE, AGE, STATUS } from '@shell/config/table-headers';
 import {
   KUBEVIRT_RESOURCE_NAME,
   PRODUCT_NAME,
@@ -8,115 +7,13 @@ import {
 } from './constants';
 
 export function init(plugin: IPlugin, store: any) {
-  const { basicType, product, mapGroup } = plugin.DSL(store, PRODUCT_NAME);
+  const { basicType, product } = plugin.DSL(store, PRODUCT_NAME);
 
-  // top level product
-  // product({
-  //   icon: 'gear',
-  //   inStore: 'management',
-  //   weight: 100,
-  // });
-
-  // cluster level product
   product({
     inStore: 'cluster',
     showNamespaceFilter: true,
     ifHaveType: KUBEVIRT_RESOURCE_NAME,
   });
 
-  // basicType([KUBEVIRT_RESOURCE_NAME, VM_RESOURCE_NAME, VMI_RESOURCE_NAME], 'workload');
   basicType([KUBEVIRT_RESOURCE_NAME, VM_RESOURCE_NAME, VMI_RESOURCE_NAME]);
-
-  // configureType(KUBEVIRT_RESOURCE_NAME, {
-  //   displayName: 'Virtual Machines',
-  //   isCreatable: true,
-  //   isRemovable: true,
-  //   showAge: true,
-  //   showState: true,
-  //   canYaml: true,
-  //   customRoute: {
-  //     name: `${YOUR_PRODUCT_NAME}-c-cluster-resource`,
-  //     params: {
-  //       product: YOUR_PRODUCT_NAME,
-  //       cluster: BLANK_CLUSTER,
-  //       resource: YOUR_K8S_RESOURCE_NAME,
-  //     },
-  //   },
-  // });
-
-  // const UI_PLUGIN = 'catalog.cattle.io.uiplugin';
-
-  // mapGroup('workload', 'Workloads'); // the label should be translated
-  mapGroup('kubevirt', 'Virtual Machines');
-
-  // headers(VM_RESOURCE_NAME, [
-  //   STATE,
-  //   {
-  //     ...NAME,
-  //     width: 300,
-  //   },
-  //   NAMESPACE,
-  //   {
-  //     name: 'CPU',
-  //     label: 'CPU',
-  //     sort: ['spec.template.spec.domain.cpu.cores'],
-  //     value: 'spec.template.spec.domain.cpu.cores',
-  //     align: 'center',
-  //     dashIfEmpty: true,
-  //   },
-  //   {
-  //     name: 'Memory',
-  //     value: 'displayMemory',
-  //     sort: ['memorySort'],
-  //     align: 'center',
-  //     labelKey: 'tableHeaders.memory',
-  //     formatter: 'Si',
-  //     formatterOpts: {
-  //       opts: {
-  //         increment: 1024,
-  //         addSuffix: true,
-  //         maxExponent: 3,
-  //         minExponent: 3,
-  //         suffix: 'i',
-  //       },
-  //       needParseSi: true,
-  //     },
-  //   },
-  //   {
-  //     name: 'ip',
-  //     label: 'IP Address',
-  //     value: 'id',
-  //     formatter: 'HarvesterIpAddress',
-  //     labelKey: 'tableHeaders.ipAddress',
-  //   },
-  //   {
-  //     name: 'node',
-  //     label: 'Node',
-  //     value: 'id',
-  //     sort: ['realAttachNodeName'],
-  //     formatter: 'NodeName',
-  //     labelKey: 'tableHeaders.node',
-  //   },
-  //   {
-  //     ...AGE,
-  //     sort: 'metadata.creationTimestamp:desc',
-  //   },
-  // STATE,
-  // NAME,
-  // NAMESPACE,
-  // STATUS,
-  // // READY,
-  // {
-  //   name: 'run-strategy',
-  //   label: 'Run strategy',
-  //   value: 'runStrategyLabel',
-  //   sort: ['runStrategyLabel'],
-  //   getValue: (row: VirtualMachine) => {
-  //     console.log('row', row);
-  //     // return row.runStrategyLabel;
-  //     return row.isPaused;
-  //   },
-  // },
-  // AGE,
-  // ]);
 }