@rancher/shell 0.3.9 → 0.3.11

package/edit/workload/storage/__tests__/Storage.test.ts

@@ -2,20 +2,99 @@ import { mount } from '@vue/test-utils';
 import Storage from '@shell/edit//workload/storage/index.vue';
 
 describe('component: Storage', () => {
-  // TODO: Complete test after integrating #5631
+  describe.each([
+    'awsElasticBlockStore',
+    'azureDisk',
+    'azureFile',
+    'configMap',
+    // 'createPVC',
+    'csi',
+    'emptyDir',
+    'gcePersistentDisk',
+    'gcePersistentDisk',
+    'hostPath',
+    'secret',
+    'vsphereVolume'
+  ])('given volume type %p', (volumeType) => {
+    it('should display the volume name as first input of the form array', () => {
+      const name = 'whatever';
+      const wrapper = mount(Storage, {
+        propsData: {
+          savePvcHookName: '',
+          value:           {
+            volumes: [{
+              _type:        volumeType,
+              [volumeType]: {},
+              name
+            }]
+          },
+        },
+        mocks: {
+          t:      (text: string) => text, // Mock i18n global function used as alternative to the getter
+          $store: {
+            getters: {
+              'i18n/t':      jest.fn(),
+              'i18n/exists': jest.fn()
+            }
+          }
+        },
+      });
+
+      const input = wrapper.find('input').element as HTMLInputElement;
+
+      expect(input.value).toStrictEqual(name);
+    });
+
+    it('should edit the volume name', () => {
+      const name = 'whatever';
+      const newName = 'new whatever';
+      const wrapper = mount(Storage, {
+        propsData: {
+          savePvcHookName: '',
+          value:           {
+            volumes: [{
+              _type:        volumeType,
+              [volumeType]: {},
+              name
+            }]
+          },
+        },
+        mocks: {
+          t:      (text: string) => text, // Mock i18n global function used as alternative to the getter
+          $store: {
+            getters: {
+              'i18n/t':      jest.fn(),
+              'i18n/exists': jest.fn()
+            }
+          }
+        },
+      });
+
+      wrapper.find('input').setValue(newName);
+
+      expect(wrapper.vm.value.volumes[0].name).toStrictEqual(newName);
+    });
+  });
+
+  // TODO: find how to interact with the tooltip selection outside of the component
   // eslint-disable-next-line jest/no-disabled-tests
   it.skip('should allow to add a new volume', async() => {
     const wrapper = mount(Storage, {
       propsData: { savePvcHookName: '' },
       mocks:     {
-        t:      (text: string) => text, // Fixes another issue with another i18n logic not from the getters
+        t:      (text: string) => text, // Mock i18n global function used as alternative to the getter
         $store: { getters: { 'i18n/t': jest.fn() } }
       },
     });
 
-    await wrapper.find('#add-volume').trigger('click');
-    const title = wrapper.find('h3');
+    await wrapper.find('#select-volume').trigger('click');
+    await wrapper.trigger('keydown.down');
+    await wrapper.trigger('keydown.enter');
 
-    expect(title.isVisible).toBe(true);
+    expect(wrapper.vm.value.volumes[0]).toStrictEqual({
+      _type:                'awsElasticBlockStore',
+      awsElasticBlockStore: {},
+      name:                 ''
+    });
   });
 });

package/initialize/index.js

@@ -19,6 +19,7 @@ import plugins from '../core/plugins.js';
 import pluginsLoader from '../core/plugins-loader.js';
 import axiosShell from '../plugins/axios';
 import '../plugins/tooltip';
+import '../plugins/clean-tooltip-directive';
 import '../plugins/vue-clipboard2';
 import '../plugins/v-select';
 import '../plugins/directives';

package/layouts/default.vue

@@ -667,7 +667,7 @@ export default {
           </nuxt-link>
 
           <span
-            v-tooltip="{content: displayVersion, placement: 'top'}"
+            v-clean-tooltip="{content: displayVersion, placement: 'top'}"
             class="clip version text-muted"
           >
             {{ displayVersion }}

package/mixins/resource-fetch-namespaced.js

@@ -1,5 +1,7 @@
+import { NAMESPACE_FILTER_NS_PREFIX, NAMESPACE_FILTER_P_PREFIX } from '@shell/utils/namespace-filter';
 import { mapGetters } from 'vuex';
 import { ResourceListComponentName } from '../components/ResourceList/resource-list.config';
+import pAndNFiltering from '@shell/utils/projectAndNamespaceFiltering.utils';
 
 /**
  * Companion mixin used with `resource-fetch` for `ResourceList` to determine if the user needs to filter the list by a single namespace
@@ -11,46 +13,53 @@ export default {
   },
 
   computed: {
-    ...mapGetters(['currentProduct', 'currentCluster', 'isSingleNamespace']),
+    ...mapGetters(['currentProduct', 'currentCluster', 'namespaceFilters']),
 
     /**
      * Does the user need to update the filter to supply a single namespace?
      */
     namespaceFilterRequired() {
-      return this.__namespaceRequired && !this.__singleNamespaceFilter;
+      return this.__namespaceRequired && !this.__validFilter;
     },
 
     /**
      * Returns the namespace that requests should be filtered by
      */
     namespaceFilter() {
-      return this.__namespaceRequired ? this.__singleNamespaceFilter : '';
+      return this.__namespaceRequired ? this.__validFilter : '';
     },
 
     /**
-     * If the Project/Namespace filter from the header contains a single NS... return it
+     * If the Project/Namespace filter from the header contains a valid ns / project filter ... return it
      */
-    __singleNamespaceFilter() {
-      const ns = this.isSingleNamespace;
+    __validFilter() {
+      const valid = this.namespaceFilters.length && this.namespaceFilters.every(f => f.startsWith(NAMESPACE_FILTER_NS_PREFIX) || f.startsWith(NAMESPACE_FILTER_P_PREFIX));
 
-      return ns ? ns.replace('ns://', '') : '';
+      return valid ? this.namespaceFilters : null;
     },
 
     /**
-     * Do we need to filter the list by a namespace?
+     * Do we need to filter the list by a namespace? This will control whether the user is shown an error
+     *
+     * We shouldn't show an error on pages with resources that aren't namespaced
      */
     __namespaceRequired() {
-      if (!this.forceNsFilter?.enabled || this.perfConfig.forceNsFilter.threshold === undefined) {
+      if (!pAndNFiltering.isEnabled(this.$store.getters)) {
         return false;
       }
 
-      return !this.currentProduct.showWorkspaceSwitcher && this.__areResourcesNamespaced && this.__areResourcesTooMany;
+      return this.__areResourcesNamespaced;
     },
 
     /**
      * Are all core list resources namespaced?
      */
     __areResourcesNamespaced() {
+      // Only enable for the cluster store at the moment. In theory this should work in management as well, as they're both 'steve' stores
+      if (this.currentProduct.inStore !== 'cluster') {
+        return false;
+      }
+
       return (this.loadResources || []).every((type) => {
         const schema = this.$store.getters['cluster/schemaFor'](type);
 
@@ -58,26 +67,9 @@ export default {
       });
     },
 
-    /**
-     * Are there too many core list resources to show in the list?
-     */
-    __areResourcesTooMany() {
-      // __getCountForResources is defined on resource-fetch mixin...
-      const count = this.__getCountForResources(this.loadResources);
-
-      return count > this.perfConfig.forceNsFilter.threshold;
-    },
-
   },
 
   watch: {
-    __namespaceRequired: {
-      handler(neu) {
-        this.$store.dispatch('setNamespaceFilterMode', neu ? 'namespace' : null, { root: true });
-      },
-      immediate: true,
-    },
-
     async namespaceFilter(neu) {
       if (neu) {
         // When a NS filter is required and the user selects a different one, kick off a new set of API requests

package/mixins/resource-fetch.js

@@ -38,11 +38,6 @@ export default {
       // incremental loading vars
       incremental:                false,
       fetchedResourceType:        [],
-      // force ns filtering
-      forceNsFilter:              {
-        ...perfConfig.forceNsFilter,
-        threshold: parseInt(perfConfig?.forceNsFilter?.threshold || '0', 10)
-      }
     };
   },
   beforeDestroy() {

package/models/__tests__/namespace.test.ts

@@ -0,0 +1,125 @@
+import Namespace from '@shell/models/namespace';
+
+describe('class Namespace', () => {
+  describe('checking if isSystem', () => {
+    it.each([
+      ['c-whatever-system', true],
+      ['whatever', false]
+    ])('should return true if end with "-system', (name, expectation) => {
+      const namespace = new Namespace({});
+
+      namespace.metadata = { ...namespace.metadata, name };
+
+      expect(namespace.isSystem).toBe(expectation);
+    });
+  });
+
+  it.todo('should check if isFleetManaged');
+
+  describe('checking if isObscure', () => {
+    it.each([
+      ['c-whatever-system', true],
+      ['whatever', false],
+      ['', false]
+    ])('should return a value if has or not a name in the metadata', (name, expectation) => {
+      const namespace = new Namespace({});
+
+      namespace.metadata = { ...namespace.metadata, name };
+
+      expect(namespace.isObscure).toBe(expectation);
+    });
+
+    it.todo('should return a value if is or not system');
+  });
+
+  it.each([
+    ['foo:bar', 'bar'],
+    ['', null]
+  ])('given %p annotation, projectId should be %p', (value, result) => {
+    const namespace = new Namespace({});
+
+    namespace.metadata = { name: '', annotations: { 'field.cattle.io/projectId': value } };
+
+    expect(namespace.projectId).toBe(result);
+  });
+
+  it.todo('should return the project');
+  it.todo('should return the groupByLabel with i18n');
+  it.todo('should return the project name with i18n');
+  it.todo('should return the projectNameSort');
+  it.todo('should check if istioInstalled');
+  it.todo('should check if injectionEnabled');
+
+  describe('handling Istio  labels', () => {
+    const save = jest.fn();
+
+    it.each([
+      { metadata: { name: '', labels: { 'istio-injection': 'whatever' } }, save },
+      [{ metadata: { name: '', labels: { 'istio-injection': 'whatever' } }, save }],
+    ])('should handle both data as list and single object and save', (data) => {
+      const namespace = new Namespace({});
+
+      namespace.enableAutoInjection(data as unknown as Namespace);
+
+      expect(save).toHaveBeenCalledWith();
+    });
+
+    it('should add auto injection label as enable', () => {
+      const data = { metadata: { name: '', labels: { 'istio-injection': 'whatever' } }, save };
+      const namespace = new Namespace({});
+
+      namespace.enableAutoInjection(data as unknown as Namespace);
+
+      expect(data.metadata!.labels['istio-injection']).toBe('enabled');
+    });
+
+    it('should remove label on disable', () => {
+      const data = { metadata: { name: '', labels: { 'istio-injection': 'whatever' } }, save };
+      const namespace = new Namespace({});
+
+      namespace.enableAutoInjection(data as unknown as Namespace, false);
+
+      expect(data.metadata!.labels['istio-injection']).toBeUndefined();
+    });
+  });
+
+  it.todo('should disableAutoInjection');
+  it.todo('should check if confirmRemove');
+
+  describe('handling listLocation', () => {
+    it.each([
+      ['c-cluster-product-projectsnamespaces', true],
+      ['c-cluster-product-resource', false],
+    ])('should return the name %p if is Rancher (%p)', (name, isRancher) => {
+      const namespace = new Namespace({});
+
+      jest.spyOn(namespace, '$rootGetters', 'get').mockReturnValue({
+        isRancher,
+        currentProduct: { inStore: '' }
+      });
+
+      expect(namespace.listLocation.name).toBe(name);
+    });
+
+    it('should return the name and resource if Harvester', () => {
+      const namespace = new Namespace({});
+
+      jest.spyOn(namespace, '$rootGetters', 'get').mockReturnValue({
+        isRancher:      true,
+        currentProduct: { inStore: 'harvester' }
+      });
+
+      const value = {
+        name:   'harvester-c-cluster-projectsnamespaces',
+        params: { resource: 'namespace' }
+      };
+
+      expect(namespace.listLocation).toStrictEqual(value);
+    });
+  });
+
+  it.todo('should return _detailLocation with a name');
+  it.todo('should return the resourceQuota');
+  it.todo('should set the resourceQuota as reactive Vue property');
+  it.todo('should reset project with cleanForNew');
+});

package/models/management.cattle.io.project.js

@@ -128,7 +128,12 @@ export default class Project extends HybridModel {
       }
     };
 
-    await finishProjectCreation();
+    // Only update PSP template if the value changed
+    const newPSPTemplateID = this.spec.podSecurityPolicyTemplateId || null;
+
+    if (norman.podSecurityPolicyTemplateId !== newPSPTemplateID) {
+      await finishProjectCreation();
+    }
 
     return newValue;
   }

package/models/persistentvolume.js

@@ -115,7 +115,7 @@ export default class PV extends SteveModel {
   // plugin display value table
   get source() {
     const csiDriver = this.spec?.csi?.driver;
-    const fallback = `${ csiDriver } ${ this.t('persistentVolume.csi.drivers.suffix') }`;
+    const fallback = `${ csiDriver } ${ this.t('persistentVolume.csi.suffix') }`;
 
     if (csiDriver) {
       return this.$rootGetters['i18n/withFallback'](`persistentVolume.csi.drivers.${ csiDriver.replaceAll('.', '-') }`, null, fallback);

package/models/workload.service.js

@@ -1,7 +1,7 @@
 
 import { findBy } from '@shell/utils/array';
 import { TARGET_WORKLOADS, UI_MANAGED, HCI as HCI_LABELS_ANNOTATIONS } from '@shell/config/labels-annotations';
-import { WORKLOAD_TYPES, SERVICE } from '@shell/config/types';
+import { WORKLOAD_TYPES, SERVICE, CAPI, HCI } from '@shell/config/types';
 import { clone, get } from '@shell/utils/object';
 import SteveModel from '@shell/plugins/steve/steve-class';
 import { shortenedImage } from '@shell/utils/string';
@@ -304,16 +304,31 @@ export default class WorkloadService extends SteveModel {
       if (loadBalancer.id) {
         loadBalancerProxy = loadBalancer;
       } else {
-        loadBalancer = clone(loadBalancer);
+        loadBalancerProxy = await this.$dispatch(`cluster/create`, loadBalancer, { root: true });
+      }
 
-        const portsWithIpam = ports.filter(p => p._ipam) || [];
+      const portsWithIpam = ports.filter(p => p._ipam) || [];
 
-        if (portsWithIpam.length > 0) {
-          loadBalancer.metadata.annotations[HCI_LABELS_ANNOTATIONS.CLOUD_PROVIDER_IPAM] = portsWithIpam[0]._ipam;
-        }
+      if (portsWithIpam.length > 0) {
+        loadBalancerProxy.metadata.annotations[HCI_LABELS_ANNOTATIONS.CLOUD_PROVIDER_IPAM] = portsWithIpam[0]._ipam;
 
-        loadBalancerProxy = await this.$dispatch(`cluster/create`, loadBalancer, { root: true });
+        const clusters = this.$rootGetters['management/all'](CAPI.RANCHER_CLUSTER);
+        const configs = this.$rootGetters['management/all'](HCI.HARVESTER_CONFIG);
+        const currentCluster = this.$rootGetters['currentCluster'];
+        const cluster = clusters.find(c => c.status.clusterName === 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;
+
+          loadBalancerProxy.metadata.annotations[HCI_LABELS_ANNOTATIONS.CLOUD_PROVIDER_NAMESPACE] = vmNamespace;
+          loadBalancerProxy.metadata.annotations[HCI_LABELS_ANNOTATIONS.CLOUD_PROVIDER_NETWORK] = networkName;
+        }
       }
+
       toSave.push(loadBalancerProxy);
     } else if (loadBalancer.id) {
       toRemove.push(loadBalancer);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rancher/shell",
-  "version": "0.3.9",
+  "version": "0.3.11",
   "description": "Rancher Dashboard Shell",
   "repository": "https://github.com/rancherlabs/dashboard",
   "license": "Apache-2.0",
@@ -61,7 +61,7 @@
     "cron-validator": "1.2.0",
     "cronstrue": "1.95.0",
     "cross-env": "6.0.3",
-    "css-loader": "4.3.0",
+    "css-loader": "6.7.3",
     "csv-loader": "3.0.3",
     "cypress": "10.3.1",
     "d3": "7.3.0",
@@ -89,17 +89,18 @@
     "jest": "27.5.1",
     "jest-serializer-vue": "2.0.2",
     "jexl": "2.2.2",
+    "jquery": "3.5.1",
     "js-cookie": "2.2.1",
     "js-yaml": "4.1.0",
     "js-yaml-loader": "1.2.2",
     "jsdiff": "1.1.1",
     "jsdom-global": "3.0.2",
     "jsonpath-plus": "6.0.1",
-    "jsrsasign": "10.2.0",
-    "jszip": "3.7.0",
+    "jsrsasign": "10.5.25",
+    "jszip": "3.8.0",
     "lodash": "4.17.21",
     "marked": "4.0.17",
-    "nodemon": "2.0.4",
+    "nodemon": "2.0.22",
     "nuxt": "2.15.8",
     "nyc": "15.1.0",
     "papaparse": "5.3.0",
@@ -143,6 +144,17 @@
     "worker-loader": "3.0.8",
     "yarn": "1.22.18"
   },
+  "resolutions": {
+    "ejs": "^3.1.7",
+    "json5": ">=2.2.2",
+    "d3-color": ">=3.1.0",
+    "glob-parent": ">=5.1.2",
+    "node-forge": ">=1.3.0",
+    "qs": ">=6.7.3",
+    "nth-check": ">=2.0.1",
+    "follow-redirects": ">=1.14.7",
+    "merge": ">=2.1.1"
+  },
   "nyc": {
     "extension": [
       ".js",

package/pages/account/index.vue

@@ -159,6 +159,7 @@ export default {
           v-if="canChangePassword"
           type="button"
           class="btn role-primary"
+          data-testid="account_change_password"
           @click="$refs.promptChangePassword.show(true)"
         >
           {{ t("accountAndKeys.account.change") }}
@@ -179,6 +180,7 @@ export default {
       <button
         v-if="apiKeySchema"
         class="btn role-primary add mb-20"
+        data-testid="account_create_api_keys"
         @click="addKey"
       >
         {{ t('accountAndKeys.apiKeys.add.label') }}
@@ -193,6 +195,7 @@ export default {
         :rows="apiKeys"
         :headers="apiKeyheaders"
         key-field="id"
+        data-testid="api_keys_list"
         :search="true"
         :row-actions="true"
         :table-actions="true"

package/pages/auth/login.vue

@@ -326,60 +326,59 @@ export default {
         </div>
         <div
           v-if="firstLogin"
-          class="first-login-message"
+          class="first-login-message pl-10 pr-10"
+          :class="{'mt-30': !hasLoginMessage}"
           data-testid="first-login-message"
         >
-          <InfoBox color="info">
+          <t
+            k="setup.defaultPassword.intro"
+            :raw="true"
+          />
+
+          <div>
             <t
-              k="setup.defaultPassword.intro"
+              k="setup.defaultPassword.dockerPrefix"
               :raw="true"
             />
-
-            <div>
-              <t
-                k="setup.defaultPassword.dockerPrefix"
-                :raw="true"
-              />
-            </div>
-            <ul>
-              <li>
-                <t
-                  k="setup.defaultPassword.dockerPs"
-                  :raw="true"
-                />
-              </li>
-              <li>
-                <CopyCode>
-                  docker logs <u>container-id</u> 2&gt;&amp;1 | grep "Bootstrap Password:"
-                </CopyCode>
-              </li>
-            </ul>
-            <div>
+          </div>
+          <ul>
+            <li>
               <t
-                k="setup.defaultPassword.dockerSuffix"
+                k="setup.defaultPassword.dockerPs"
                 :raw="true"
               />
-            </div>
+            </li>
+            <li>
+              <CopyCode>
+                docker logs <u>container-id</u> 2&gt;&amp;1 | grep "Bootstrap Password:"
+              </CopyCode>
+            </li>
+          </ul>
+          <div>
+            <t
+              k="setup.defaultPassword.dockerSuffix"
+              :raw="true"
+            />
+          </div>
 
-            <br>
-            <div>
-              <t
-                k="setup.defaultPassword.helmPrefix"
-                :raw="true"
-              />
-            </div>
-            <br>
-            <CopyCode>
-              {{ kubectlCmd }}
-            </CopyCode>
-            <br>
-            <div>
-              <t
-                k="setup.defaultPassword.helmSuffix"
-                :raw="true"
-              />
-            </div>
-          </InfoBox>
+          <br>
+          <div>
+            <t
+              k="setup.defaultPassword.helmPrefix"
+              :raw="true"
+            />
+          </div>
+          <br>
+          <CopyCode>
+            {{ kubectlCmd }}
+          </CopyCode>
+          <br>
+          <div>
+            <t
+              k="setup.defaultPassword.helmSuffix"
+              :raw="true"
+            />
+          </div>
         </div>
 
         <div
@@ -508,6 +507,8 @@ export default {
     }
 
     .login-messages {
+      display: flex;
+      justify-content: center;
       align-items: center;
 
       .banner {
@@ -519,11 +520,7 @@ export default {
       &--hasContent {
         min-height: 70px;
       }
-    }
 
-    .login-messages, .first-login-message {
-      display: flex;
-      justify-content: center;
       .text-error, .banner {
         max-width: 80%;
       }

package/pages/c/_cluster/apps/charts/chart.vue

@@ -224,7 +224,7 @@ export default {
             <b v-if="vers.originalVersion === version.version">{{ vers.originalVersion === currentVersion ? t('catalog.install.versions.current', { ver: currentVersion }): vers.shortLabel }}</b>
             <a
               v-else
-              v-tooltip="vers.label.length > 16 ? vers.label : null"
+              v-clean-tooltip="vers.label.length > 16 ? vers.label : null"
               @click.prevent="selectVersion(vers)"
             >
               {{ vers.originalVersion === currentVersion ? t('catalog.install.versions.current', { ver: currentVersion }): vers.shortLabel }}