@rancher/shell 0.3.9 → 0.3.11

package/edit/provisioning.cattle.io.cluster/__tests__/rke2.test.ts

@@ -1,8 +1,80 @@
 import { mount } from '@vue/test-utils';
 import rke2 from '@shell/edit/provisioning.cattle.io.cluster/rke2.vue';
 
+/**
+ * DISCLAIMER ***************************************************************************************
+ * Declarations should not be done outside the tests!!
+ * This component is overwhelming for test and requires too much initialization.
+ * In this way the tests are more readable and we can avoid annoying repetitions.
+ ****************************************************************************************************
+ */
+const defaultStubs = {
+  CruResource:              { template: '<div><slot></slot></div>' }, // Required to render the slot content
+  Banner:                   true,
+  LabeledSelect:            true,
+  ACE:                      true,
+  AgentEnv:                 true,
+  ArrayList:                true,
+  ArrayListGrouped:         true,
+  BadgeState:               true,
+  Checkbox:                 true,
+  ClusterMembershipEditor:  true,
+  DrainOptions:             true,
+  LabeledInput:             true,
+  Labels:                   true,
+  Loading:                  true,
+  MachinePool:              true,
+  MatchExpressions:         true,
+  NameNsDescription:        true,
+  Questions:                true,
+  RadioGroup:               true,
+  RegistryConfigs:          true,
+  RegistryMirrors:          true,
+  S3Config:                 true,
+  SelectCredential:         true,
+  SelectOrCreateAuthSecret: true,
+  Tab:                      true,
+  Tabbed:                   true,
+  UnitInput:                true,
+  YamlEditor:               true,
+};
+
+const defaultComputed = {
+  showForm() {
+    return true;
+  },
+  showk8s21LegacyWarning() {
+    return false;
+  },
+};
+
+const defaultGetters = {
+  currentStore:           () => 'current_store',
+  'management/schemaFor': jest.fn(),
+  'current_store/all':    jest.fn(),
+  'i18n/t':               jest.fn(),
+  'i18n/withFallback':    jest.fn(),
+};
+
+const defaultMocks = {
+  $fetchState: { pending: false },
+  $route:      {
+    name:  'anything',
+    query: { AS: 'yaml' },
+  },
+};
+
+const defaultSpec = {
+  rkeConfig:   { etcd: { disableSnapshots: false } },
+  chartValues: {},
+};
+
 describe('component: rke2', () => {
-  // Disable existing log to avoid pollution
+  /**
+   * DISCLAIMER ***************************************************************************************
+   * Logs are prevented to avoid polluting the test output.
+   ****************************************************************************************************
+  */
   // eslint-disable-next-line jest/no-hooks
   beforeEach(() => {
     jest.spyOn(console, 'log').mockImplementation(() => {});
@@ -23,76 +95,25 @@ describe('component: rke2', () => {
         mode:  'create',
         value: {
           spec: {
-            rkeConfig:                                            { etcd: { disableSnapshots: false } },
-            chartValues:                                          {},
+            ...defaultSpec,
             defaultPodSecurityAdmissionConfigurationTemplateName: label,
             kubernetesVersion:                                    k8s
           }
         },
         provider: 'whatever',
-        resource: {}
       },
-      computed: {
-        showForm() {
-          return true;
-        },
-        hasMachinePools() {
-          return false;
-        },
-        showk8s21LegacyWarning() {
-          return false;
-        },
-      },
-      mocks: {
-        $fetchState: { pending: false },
-        $route:      {
-          name:  'anything',
-          query: { AS: 'yaml' },
-        },
+      computed: defaultComputed,
+      mocks:    {
+        ...defaultMocks,
         $store: {
-          getters: {
-            currentStore:           () => 'current_store',
-            'management/schemaFor': jest.fn(),
-            'current_store/all':    jest.fn(),
-            'i18n/t':               jest.fn(),
-            'i18n/withFallback':    jest.fn(),
-          },
+          getters:  defaultGetters,
           dispatch: {
             'management/find':    jest.fn(),
             'management/findAll': () => ([option]),
           }
         },
       },
-      stubs: {
-        CruResource:              { template: '<div><slot></slot></div>' }, // Required to render the slot content
-        Banner:                   true,
-        LabeledSelect:            true,
-        ACE:                      true,
-        AgentEnv:                 true,
-        ArrayList:                true,
-        ArrayListGrouped:         true,
-        BadgeState:               true,
-        Checkbox:                 true,
-        ClusterMembershipEditor:  true,
-        DrainOptions:             true,
-        LabeledInput:             true,
-        Labels:                   true,
-        Loading:                  true,
-        MachinePool:              true,
-        MatchExpressions:         true,
-        NameNsDescription:        true,
-        Questions:                true,
-        RadioGroup:               true,
-        RegistryConfigs:          true,
-        RegistryMirrors:          true,
-        S3Config:                 true,
-        SelectCredential:         true,
-        SelectOrCreateAuthSecret: true,
-        Tab:                      true,
-        Tabbed:                   true,
-        UnitInput:                true,
-        YamlEditor:               true,
-      }
+      stubs: defaultStubs
     });
 
     const select = wrapper.find('[data-testid="rke2-custom-edit-psa"]');
@@ -115,76 +136,25 @@ describe('component: rke2', () => {
         mode:  'create',
         value: {
           spec: {
-            rkeConfig:                                            { etcd: { disableSnapshots: false } },
-            chartValues:                                          {},
+            ...defaultSpec,
             defaultPodSecurityAdmissionConfigurationTemplateName: label,
             kubernetesVersion:                                    k8s
           }
         },
         provider: 'whatever',
-        resource: {}
-      },
-      computed: {
-        showForm() {
-          return true;
-        },
-        hasMachinePools() {
-          return false;
-        },
-        showk8s21LegacyWarning() {
-          return false;
-        },
       },
-      mocks: {
-        $fetchState: { pending: false },
-        $route:      {
-          name:  'anything',
-          query: { AS: 'yaml' },
-        },
+      computed: defaultComputed,
+      mocks:    {
+        ...defaultMocks,
         $store: {
-          getters: {
-            currentStore:           () => 'current_store',
-            'management/schemaFor': jest.fn(),
-            'current_store/all':    jest.fn(),
-            'i18n/t':               jest.fn(),
-            'i18n/withFallback':    jest.fn(),
-          },
+          getters:  defaultGetters,
           dispatch: {
             'management/find':    jest.fn(),
             'management/findAll': () => ([option]),
           }
         },
       },
-      stubs: {
-        CruResource:              { template: '<div><slot></slot></div>' }, // Required to render the slot content
-        Banner:                   true,
-        LabeledSelect:            true,
-        ACE:                      true,
-        AgentEnv:                 true,
-        ArrayList:                true,
-        ArrayListGrouped:         true,
-        BadgeState:               true,
-        Checkbox:                 true,
-        ClusterMembershipEditor:  true,
-        DrainOptions:             true,
-        LabeledInput:             true,
-        Labels:                   true,
-        Loading:                  true,
-        MachinePool:              true,
-        MatchExpressions:         true,
-        NameNsDescription:        true,
-        Questions:                true,
-        RadioGroup:               true,
-        RegistryConfigs:          true,
-        RegistryMirrors:          true,
-        S3Config:                 true,
-        SelectCredential:         true,
-        SelectOrCreateAuthSecret: true,
-        Tab:                      true,
-        Tabbed:                   true,
-        UnitInput:                true,
-        YamlEditor:               true,
-      }
+      stubs: defaultStubs
     });
 
     const select = wrapper.find('[data-testid="rke2-custom-edit-psa"]');
@@ -206,25 +176,15 @@ describe('component: rke2', () => {
         value: {
           agentConfig: { profile: cis },
           spec:        {
-            rkeConfig:                                            { etcd: { disableSnapshots: false } },
-            chartValues:                                          {},
+            ...defaultSpec,
             defaultPodSecurityAdmissionConfigurationTemplateName: label,
             kubernetesVersion:                                    k8s
           }
         },
         provider: 'custom',
-        resource: {}
       },
       computed: {
-        showForm() {
-          return true;
-        },
-        hasMachinePools() {
-          return false;
-        },
-        showk8s21LegacyWarning() {
-          return false;
-        },
+        ...defaultComputed,
         agentArgs:      () => ({ profile: { options: [cis] } }),
         versionOptions: () => [
           {
@@ -236,55 +196,16 @@ describe('component: rke2', () => {
         ]
       },
       mocks: {
-        $fetchState: { pending: false },
-        $route:      {
-          name:  'anything',
-          query: { AS: 'yaml' },
-        },
+        ...defaultMocks,
         $store: {
-          getters: {
-            currentStore:           () => 'current_store',
-            'management/schemaFor': jest.fn(),
-            'current_store/all':    jest.fn(),
-            'i18n/t':               jest.fn(),
-            'i18n/withFallback':    jest.fn(),
-          },
+          getters:  defaultGetters,
           dispatch: {
             'management/find':    jest.fn(),
             'management/findAll': () => ([option]),
           }
         },
       },
-      stubs: {
-        CruResource:              { template: '<div><slot></slot></div>' }, // Required to render the slot content
-        Banner:                   true,
-        LabeledSelect:            true,
-        ACE:                      true,
-        AgentEnv:                 true,
-        ArrayList:                true,
-        ArrayListGrouped:         true,
-        BadgeState:               true,
-        Checkbox:                 true,
-        ClusterMembershipEditor:  true,
-        DrainOptions:             true,
-        LabeledInput:             true,
-        Labels:                   true,
-        Loading:                  true,
-        MachinePool:              true,
-        MatchExpressions:         true,
-        NameNsDescription:        true,
-        Questions:                true,
-        RadioGroup:               true,
-        RegistryConfigs:          true,
-        RegistryMirrors:          true,
-        S3Config:                 true,
-        SelectCredential:         true,
-        SelectOrCreateAuthSecret: true,
-        Tab:                      true,
-        Tabbed:                   true,
-        UnitInput:                true,
-        YamlEditor:               true,
-      }
+      stubs: defaultStubs
     });
 
     wrapper.setData({ cisOverride: override });
@@ -293,4 +214,57 @@ describe('component: rke2', () => {
 
     expect((select.vm as unknown as any).disabled).toBe(disabled);
   });
+
+  it.each([
+    ['custom', true],
+    ['anything else', false] // without proper data, machine pool is always not present
+  ])('should allow creation of RKE2 cluster with provider %p if pool machines are missing (%p)', (provider, result) => {
+    const k8s = 'v1.25.0+rke2r1';
+    const wrapper = mount(rke2, {
+      propsData: {
+        mode:  'create',
+        value: {
+          spec: {
+            ...defaultSpec,
+            kubernetesVersion: k8s,
+
+          }
+        },
+        provider,
+      },
+      computed: defaultComputed,
+      mocks:    {
+        ...defaultMocks,
+        $store: { getters: defaultGetters },
+      },
+      stubs: defaultStubs
+    });
+
+    expect(wrapper.vm.validationPassed()).toBe(result);
+  });
+
+  it('should allow creation of K3 clusters if pool machines are missing', () => {
+    const k8s = 'v1.25.0+k3s1';
+    const wrapper = mount(rke2, {
+      propsData: {
+        mode:  'create',
+        value: {
+          spec: {
+            ...defaultSpec,
+            kubernetesVersion: k8s
+          }
+        },
+        provider: 'custom'
+      },
+      data:     () => ({ credentialId: 'I am authenticated' }),
+      computed: defaultComputed,
+      mocks:    {
+        ...defaultMocks,
+        $store: { getters: defaultGetters },
+      },
+      stubs: defaultStubs
+    });
+
+    expect(wrapper.vm.validationPassed()).toBe(true);
+  });
 });

package/edit/provisioning.cattle.io.cluster/rke2.vue

@@ -718,7 +718,7 @@ export default {
     },
 
     unremovedMachinePools() {
-      return this.machinePools.filter(x => !x.remove);
+      return (this.machinePools || []).filter(x => !x.remove);
     },
 
     machineConfigSchema() {
@@ -1133,6 +1133,10 @@ export default {
       return isRequiredVersion;
     },
 
+    /**
+     * Get machine pools from the cluster configuration
+     * this.value.spec.rkeConfig.machinePools
+     */
     async initMachinePools(existing) {
       const out = [];
 
@@ -2029,6 +2033,11 @@ export default {
 
           this.previousKubernetesVersion = value;
         }
+
+        // If Harvester driver, reset cloud provider if not compatible
+        if (this.isHarvesterDriver && this.mode === _CREATE && this.isHarvesterIncompatible) {
+          this.setHarvesterDefaultCloudProvider();
+        }
       }
     },
 
@@ -2120,21 +2129,21 @@ export default {
             class="pull-right"
           >
             <BadgeState
-              v-tooltip="nodeTotals.tooltip.etcd"
+              v-clean-tooltip="nodeTotals.tooltip.etcd"
               :color="nodeTotals.color.etcd"
               :icon="nodeTotals.icon.etcd"
               :label="nodeTotals.label.etcd"
               class="mr-10"
             />
             <BadgeState
-              v-tooltip="nodeTotals.tooltip.controlPlane"
+              v-clean-tooltip="nodeTotals.tooltip.controlPlane"
               :color="nodeTotals.color.controlPlane"
               :icon="nodeTotals.icon.controlPlane"
               :label="nodeTotals.label.controlPlane"
               class="mr-10"
             />
             <BadgeState
-              v-tooltip="nodeTotals.tooltip.worker"
+              v-clean-tooltip="nodeTotals.tooltip.worker"
               :color="nodeTotals.color.worker"
               :icon="nodeTotals.icon.worker"
               :label="nodeTotals.label.worker"
@@ -2551,7 +2560,7 @@ export default {
           <h3>
             {{ t('cluster.rke2.address.header') }}
             <i
-              v-tooltip="t('cluster.rke2.address.tooltip')"
+              v-clean-tooltip="t('cluster.rke2.address.tooltip')"
               class="icon icon-info"
             />
           </h3>
@@ -2843,7 +2852,7 @@ export default {
             <h3>
               {{ t('cluster.addOns.additionalManifest.title') }}
               <i
-                v-tooltip="t('cluster.addOns.additionalManifest.tooltip')"
+                v-clean-tooltip="t('cluster.addOns.additionalManifest.tooltip')"
                 class="icon icon-info"
               />
             </h3>

package/edit/resources.cattle.io.restore.vue

@@ -274,7 +274,7 @@ export default {
               >
                 <template #label>
                   <span
-                    v-tooltip="t('backupRestoreOperator.prune.tip')"
+                    v-clean-tooltip="t('backupRestoreOperator.prune.tip')"
                     class="text-label"
                   >
                     {{ t('backupRestoreOperator.prune.label') }} <i class="icon icon-info" />
@@ -290,7 +290,7 @@ export default {
               >
                 <template #label>
                   <label
-                    v-tooltip="t('backupRestoreOperator.deleteTimeout.tip')"
+                    v-clean-tooltip="t('backupRestoreOperator.deleteTimeout.tip')"
                     class="has-tooltip"
                   >
                     {{ t('backupRestoreOperator.deleteTimeout.label') }} <i class="icon icon-info" />

package/edit/service.vue

@@ -20,12 +20,13 @@ import { Banner } from '@components/Banner';
 import Labels from '@shell/components/form/Labels';
 import HarvesterServiceAddOnConfig from '@shell/components/HarvesterServiceAddOnConfig';
 import { clone } from '@shell/utils/object';
-import { POD, CAPI } from '@shell/config/types';
+import { POD, CAPI, HCI } from '@shell/config/types';
 import { matching } from '@shell/utils/selector';
 import { HARVESTER_NAME as HARVESTER } from '@shell/config/features';
 import { allHash } from '@shell/utils/promise';
 import { isHarvesterSatisfiesVersion } from '@shell/utils/cluster';
 import { Port } from '@shell/utils/validators/formRules';
+import { HCI as HCI_LABELS_ANNOTATIONS } from '@shell/config/labels-annotations';
 
 const SESSION_AFFINITY_ACTION_VALUES = {
   NONE:     'None',
@@ -268,8 +269,9 @@ export default {
         const inStore = this.$store.getters['currentStore'](POD);
 
         const hash = {
-          provClusters: this.$store.dispatch('management/findAll', { type: CAPI.RANCHER_CLUSTER }),
-          pods:         this.$store.dispatch(`${ inStore }/findAll`, { type: POD }),
+          provClusters:     this.$store.dispatch('management/findAll', { type: CAPI.RANCHER_CLUSTER }),
+          pods:             this.$store.dispatch(`${ inStore }/findAll`, { type: POD }),
+          harvesterConfigs: this.$store.dispatch(`management/findAll`, { type: HCI.HARVESTER_CONFIG }),
         };
 
         const res = await allHash(hash);
@@ -313,6 +315,23 @@ export default {
       if (ports && ports.length > 0) {
         this.value.spec.ports = this.targetPortsStrOrInt(this.value.spec.ports);
       }
+
+      if (this.showHarvesterAddOnConfig) {
+        const clusters = this.$store.getters['management/all'](CAPI.RANCHER_CLUSTER);
+        const configs = this.$store.getters['management/all'](HCI.HARVESTER_CONFIG);
+        const cluster = clusters.find(c => c.status.clusterName === this.currentCluster.id);
+
+        const machinePools = cluster?.spec?.rkeConfig?.machinePools || [];
+        const machineConfigName = machinePools[0]?.machineConfigRef?.name;
+        const config = configs.find(c => c.id === `fleet-default/${ machineConfigName }`);
+
+        if (config) {
+          const { vmNamespace, networkName } = config;
+
+          this.value.metadata.annotations[HCI_LABELS_ANNOTATIONS.CLOUD_PROVIDER_NAMESPACE] = vmNamespace;
+          this.value.metadata.annotations[HCI_LABELS_ANNOTATIONS.CLOUD_PROVIDER_NETWORK] = networkName;
+        }
+      }
     },
   },
 };

package/edit/storage.k8s.io.storageclass/index.vue

@@ -125,7 +125,7 @@ export default {
         if (driver.metadata.name === LONGHORN_DRIVER || provisionerOptionsDrivers.includes(driver.metadata.name)) {
           return;
         }
-        const fallback = `${ driver.metadata.name }  ${ this.t('persistentVolume.csi.drivers.suffix') }`;
+        const fallback = `${ driver.metadata.name }  ${ this.t('persistentVolume.csi.suffix') }`;
 
         dropdownOptions.push({
           value: driver.metadata.name,

package/edit/token.vue

@@ -273,6 +273,7 @@ export default {
       <div class="right">
         <button
           type="button"
+          data-testid="token_done_create_button"
           class="btn role-primary"
           @click="doneCreate"
         >

package/edit/workload/Job.vue

@@ -265,7 +265,7 @@ export default {
               >
                 {{ t('workload.upgrading.terminationGracePeriodSeconds.label') }}
                 <i
-                  v-tooltip="t('workload.upgrading.terminationGracePeriodSeconds.tip')"
+                  v-clean-tooltip="t('workload.upgrading.terminationGracePeriodSeconds.tip')"
                   class="icon icon-info"
                 />
               </label>
@@ -326,7 +326,7 @@ export default {
             >
               {{ t('workload.upgrading.terminationGracePeriodSeconds.label') }}
               <i
-                v-tooltip="t('workload.upgrading.terminationGracePeriodSeconds.tip')"
+                v-clean-tooltip="t('workload.upgrading.terminationGracePeriodSeconds.tip')"
                 class="icon icon-info"
               />
             </label>

package/edit/workload/index.vue

@@ -252,7 +252,7 @@ export default {
                 <h3>
                   {{ t('workload.container.ports.expose') }}
                   <i
-                    v-tooltip="t('workload.container.ports.toolTip')"
+                    v-clean-tooltip="t('workload.container.ports.toolTip')"
                     class="icon icon-info"
                   />
                 </h3>

package/edit/workload/mixins/workload.js

@@ -12,6 +12,7 @@ import {
   CAPI,
   POD,
   LIST_WORKLOAD_TYPES,
+  HCI,
 } from '@shell/config/types';
 import Tab from '@shell/components/Tabbed/Tab';
 import CreateEditView from '@shell/mixins/create-edit-view';
@@ -49,6 +50,7 @@ import NameNsDescription from '@shell/components/form/NameNsDescription';
 import formRulesGenerator from '@shell/utils/validators/formRules';
 import { TYPES as SECRET_TYPES } from '@shell/models/secret';
 import { defaultContainer } from '@shell/models/workload';
+import { allHash } from '@shell/utils/promise';
 
 const TAB_WEIGHT_MAP = {
   general:              99,
@@ -143,7 +145,11 @@ export default {
   },
 
   async fetch() {
-    await this.$store.dispatch('management/findAll', { type: CAPI.RANCHER_CLUSTER });
+    // TODO Should remove these lines
+    await allHash({
+      rancherClusters:  this.$store.dispatch('management/findAll', { type: CAPI.RANCHER_CLUSTER }),
+      harvesterConfigs: this.$store.dispatch('management/findAll', { type: HCI.HARVESTER_CONFIG }),
+    });
 
     // don't block UI for these resources
     this.resourceManagerFetchSecondaryResources(this.secondaryResourceData);