HARVESTER: fixed create multiple vm instance (1504) & migration list filter(1501)

components/dialog/harvester/MigrationDialog.vue

@@ -48,7 +48,7 @@ export default {
 
       return nodes.filter((n) => {
         // do not allow to migrate to self node
-        return n.id !== this.vmi?.status?.nodeName && !n.metadata?.annotations?.[HCI_ANNOTATIONS.MAINTENANCE_STATUS];
+        return n.id !== this.vmi?.status?.nodeName && !n.metadata?.annotations?.[HCI_ANNOTATIONS.MAINTENANCE_STATUS] && !n.isUnSchedulable;
       }).map((n) => {
         let label = n?.metadata?.name;
         const value = n?.metadata?.name;

edit/harvesterhci.io.virtualmachinetemplateversion.vue

@@ -144,13 +144,10 @@ export default {
       const templates = await this.$store.dispatch('harvester/findAll', { type: HCI.VM_TEMPLATE });
       const template = templates.find( O => O.metadata.name === this.templateValue.metadata.name);
 
+      try {
         if (!this.templateId) {
           if (this.templateValue?.metadata?.name) {
-          try {
             await this.templateValue.save();
-          } catch (err) {
-            this.errors = [err];
-          }
           } else {
             this.errors = ['"Name" is required'];
             buttonCb(false);
@@ -180,6 +177,10 @@ export default {
 
         await this.saveSecret(res);
         this.done();
+      } catch (e) {
+        this.errors = [e];
+        buttonCb(false);
+      }
     },
 
     onTabChanged({ tab }) {

edit/kubevirt.io.virtualmachine/index.vue

@@ -19,10 +19,13 @@ import CpuMemory from '@/edit/kubevirt.io.virtualmachine/VirtualMachineCpuMemory
 import CloudConfig from '@/edit/kubevirt.io.virtualmachine/VirtualMachineCloudConfig';
 import NodeScheduling from '@/components/form/NodeScheduling';
 
+import { clear } from '@/utils/array';
 import { clone } from '@/utils/object';
 import { HCI } from '@/config/types';
+import { exceptionToErrorsArray } from '@/utils/error';
 import { cleanForNew } from '@/plugins/steve/normalize';
 import { HCI as HCI_ANNOTATIONS } from '@/config/labels-annotations';
+import { BEFORE_SAVE_HOOKS, AFTER_SAVE_HOOKS } from '@/mixins/child-hook';
 
 import VM_MIXIN from '@/mixins/harvester-vm';
 import CreateEditView from '@/mixins/create-edit-view';
@@ -219,11 +222,15 @@ export default {
         this.$set(this.value.spec.template.spec, 'hostname', this.value.metadata.name);
       }
 
-      await this.save(buttonCb);
+      try {
+        await this._save(this.value, buttonCb);
+        buttonCb(true);
 
-      const res = this.$store.getters['harvester/byId'](HCI.VM, `${ this.value.metadata.namespace }/${ this.value.metadata.name }`);
+        this.done();
-
+      } catch (e) {
-      await this.saveSecret(res);
+        this.errors = exceptionToErrorsArray(e);
+        buttonCb(false);
+      }
     },
 
     async saveMultiple(buttonCb) {
@@ -248,23 +255,36 @@ export default {
         const hostname = `${ baseHostname }${ join }${ suffix }`;
 
         this.$set(this.value.spec.template.spec, 'hostname', hostname);
-
+        this.secretName = '';
         await this.parseVM();
-        const basicValue = await this.$store.dispatch('harvester/create', clone(this.value));
+        const basicValue = await this.$store.dispatch('harvester/clone', { resource: this.value });
 
         try {
-          if (i === 1) {
+          await this._save(basicValue);
-            await this.save();
+
-          } else {
+          if (i === this.count) {
-            basicValue.save();
+            buttonCb(true);
+            this.done();
           }
-          const res = this.$store.getters['harvester/byId'](HCI.VM, `${ this.value.metadata.namespace }/${ this.value.metadata.name }`);
+        } catch (e) {
+          this.errors = exceptionToErrorsArray(e);
+          buttonCb(false);
+        }
+      }
+    },
+
+    async _save(value) {
+      if ( this.errors ) {
+        clear(this.errors);
+      }
+
+      await this.applyHooks(BEFORE_SAVE_HOOKS);
+
+      const res = await value.save();
+
+      await this.applyHooks(AFTER_SAVE_HOOKS);
 
       await this.saveSecret(res);
-        } catch (err) {
-          return Promise.reject(new Error(err));
-        }
-      }
     },
 
     restartVM() {

mixins/harvester-vm/index.js

@@ -882,6 +882,8 @@ export default {
         }
       } catch (e) {
         new Error(`Function(saveSecret) error ${ e }`);
+
+        return Promise.reject(e);
       }
     },
 

models/harvester/node.js

@@ -146,7 +146,7 @@ export default class HciNode extends SteveModel {
   }
 
   get isCordoned() {
-    return !!this.spec.unschedulable;
+    return this.isUnSchedulable;
   }
 
   get isEnteringMaintenance() {