@rancher/shell 3.0.5-rc.6 → 3.0.5-rc.8

package/models/__tests__/node.test.ts

@@ -1,74 +1,18 @@
-import Node from '@shell/models/management.cattle.io.node';
+import Node from '@shell/models/cluster/node';
 
 describe('class Node', () => {
-  const foo = 'foo';
-  const bar = 'bar';
-  const t = jest.fn(() => bar);
-  const ctx = { rootGetters: { 'i18n/t': t } };
-
   const resetMocks = () => {
     // Clear all mock function calls:
     jest.clearAllMocks();
   };
 
-  it('should not return addresses if they are not present in the resource status', () => {
-    const node = new Node({ status: {} });
+  it.each([
+    ['1200', 1200],
+    ['1k', 1000]
+  ])('given %p status pod capacity value from the backend, should parse the value correctly as %p', (value, result) => {
+    const node = new Node({ status: { capacity: { pods: value } } });
 
-    expect(node.addresses).toStrictEqual([]);
+    expect(node.podCapacity).toStrictEqual(result);
     resetMocks();
   });
-
-  describe('should return addresses', () => {
-    const addresses = [foo];
-
-    it('if they are present directly on the resource status', () => {
-      const node = new Node({ status: { addresses } });
-
-      expect(node.addresses).toStrictEqual(addresses);
-    });
-  });
-
-  describe('should return an internalIp', () => {
-    const addresses = [{ type: 'InternalIP', address: foo }];
-
-    it('if addresses includes an object with an appropriate type and address', () => {
-      const node = new Node({ status: { addresses } });
-
-      expect(node.internalIp).toStrictEqual(foo);
-    });
-  });
-
-  describe('should return an externalIp', () => {
-    const addresses = [{ type: 'ExternalIP', address: foo }];
-
-    it('if addresses includes an object with an appropriate type and address', () => {
-      const node = new Node({ status: { addresses } });
-
-      expect(node.externalIp).toStrictEqual(foo);
-    });
-    it('if internalNodeStatus.addresses includes an object with an appropriate type and address', () => {
-      const node = new Node({ status: { internalNodeStatus: { addresses } } });
-
-      expect(node.externalIp).toStrictEqual(foo);
-    });
-  });
-
-  describe('should return an appropriate message', () => {
-    it('if there is no internalIp to display', () => {
-      const node = new Node({ status: {} }, ctx);
-
-      expect(node.internalIp).toStrictEqual(bar);
-      expect(t).toHaveBeenCalledTimes(1);
-      expect(t).toHaveBeenCalledWith('generic.none');
-      resetMocks();
-    });
-    it('if there is no externalIp to display', () => {
-      const node = new Node({ status: {} }, ctx);
-
-      expect(node.externalIp).toStrictEqual(bar);
-      expect(t).toHaveBeenCalledTimes(1);
-      expect(t).toHaveBeenCalledWith('generic.none');
-      resetMocks();
-    });
-  });
 });

package/models/catalog.cattle.io.app.js

@@ -31,7 +31,7 @@ export default class CatalogApp extends SteveModel {
     const upgrade = {
       action:  'goToUpgrade',
       enabled: true,
-      icon:    'icon icon-fw icon-edit',
+      icon:    'icon icon-edit',
       label:   this.t('catalog.install.action.goToUpgrade'),
     };
 

package/models/catalog.cattle.io.operation.js

@@ -17,7 +17,7 @@ export default class CatalogOperation extends SteveModel {
     const openLogs = {
       action:  'openLogs',
       enabled: true,
-      icon:    'icon icon-fw icon-chevron-right',
+      icon:    'icon icon-chevron-right',
       label:   this.t('action.openLogs'),
       total:   1,
     };

package/models/chart.js

@@ -4,7 +4,7 @@ import {
 } from '@shell/config/query-params';
 import { BLANK_CLUSTER } from '@shell/store/store-types.js';
 import SteveModel from '@shell/plugins/steve/steve-class';
-import { CATALOG } from '@shell/config/types';
+import { CATALOG, ZERO_TIME } from '@shell/config/types';
 import { CATALOG as CATALOG_ANNOTATIONS } from '@shell/config/labels-annotations';
 import day from 'dayjs';
 
@@ -121,42 +121,58 @@ export default class Chart extends SteveModel {
    */
   get cardContent() {
     if (!this._cardContent) {
-      const subHeaderItems = [
-        {
+      const latestVersion = this.versions?.[0] || {};
+      const subHeaderItems = [];
+
+      if (latestVersion) {
+        const hasZeroTime = latestVersion.created === ZERO_TIME;
+
+        subHeaderItems.push({
           icon:        'icon-version-alt',
           iconTooltip: { key: 'tableHeaders.version' },
-          label:       this.versions[0].version
-        },
-        {
+          label:       latestVersion.version
+        });
+
+        const lastUpdatedItem = {
           icon:        'icon-refresh-alt',
           iconTooltip: { key: 'tableHeaders.lastUpdated' },
-          label:       day(this.versions[0].created).format('MMM D, YYYY')
+          label:       hasZeroTime ? this.t('generic.na') : day(latestVersion.created).format('MMM D, YYYY')
+        };
+
+        if (hasZeroTime) {
+          lastUpdatedItem.labelTooltip = this.t('catalog.charts.appChartCard.subHeaderItem.missingVersionDate');
         }
-      ];
+
+        subHeaderItems.push(lastUpdatedItem);
+      }
+
       const footerItems = [
         {
-          type:        REPO,
-          icon:        'icon-repository-alt',
-          iconTooltip: { key: 'tableHeaders.repoName' },
-          labels:      [this.repoNameDisplay]
+          type:         REPO,
+          icon:         'icon-repository-alt',
+          iconTooltip:  { key: 'tableHeaders.repoName' },
+          labels:       [this.repoNameDisplay],
+          labelTooltip: this.t('catalog.charts.findSimilar.message', { type: this.t('catalog.charts.findSimilar.types.repo') }, true)
         }
       ];
 
       if (this.categories.length) {
         footerItems.push( {
-          type:        CATEGORY,
-          icon:        'icon-category-alt',
-          iconTooltip: { key: 'generic.category' },
-          labels:      this.categories
+          type:         CATEGORY,
+          icon:         'icon-category-alt',
+          iconTooltip:  { key: 'generic.category' },
+          labels:       this.categories,
+          labelTooltip: this.t('catalog.charts.findSimilar.message', { type: this.t('catalog.charts.findSimilar.types.category') }, true)
         });
       }
 
       if (this.tags.length) {
         footerItems.push({
-          type:        TAG,
-          icon:        'icon-tag-alt',
-          iconTooltip: { key: 'generic.tags' },
-          labels:      this.tags
+          type:         TAG,
+          icon:         'icon-tag-alt',
+          iconTooltip:  { key: 'generic.tags' },
+          labels:       this.tags,
+          labelTooltip: this.t('catalog.charts.findSimilar.message', { type: this.t('catalog.charts.findSimilar.types.tag') }, true)
         });
       }
 

package/models/cloudcredential.js

@@ -1,65 +1,11 @@
-import { CAPI, CLOUD_CREDENTIALS } from '@shell/config/labels-annotations';
+import { CAPI } from '@shell/config/labels-annotations';
 import { fullFields, prefixFields, simplify, suffixFields } from '@shell/store/plugins';
 import { isEmpty, set } from '@shell/utils/object';
-import { MANAGEMENT, SECRET } from '@shell/config/types';
+import { SECRET } from '@shell/config/types';
 import { escapeHtml } from '@shell/utils/string';
 import NormanModel from '@shell/plugins/steve/norman-class';
-import { DATE_FORMAT, TIME_FORMAT } from '@shell/store/prefs';
-import day from 'dayjs';
-
-const harvesterProvider = 'harvester';
-
-const renew = {
-  [harvesterProvider]: {
-    renew: ({ cloudCredential, $ctx }) => {
-      return renew[harvesterProvider].renewBulk(
-        { cloudCredentials: [cloudCredential], $ctx }
-      );
-    },
-    renewBulk: async({ cloudCredentials, $ctx }) => {
-      // A harvester cloud credential (at the moment) is a kubeconfig complete with expiring token
-      // So to renew we just need to generate a new kubeconfig and save it to the cc (similar to shell/cloud-credential/harvester.vue)
-      await Promise.all(cloudCredentials.map(async(cc) => {
-        try {
-          if (!cc.harvestercredentialConfig?.clusterId) {
-            throw new Error(`credential has no matching harvester cluster`);
-          }
-          const mgmtCluster = $ctx.rootGetters['management/byId'](MANAGEMENT.CLUSTER, cc.harvestercredentialConfig.clusterId);
-
-          if (!mgmtCluster) {
-            throw new Error(`cannot find harvester cluster`);
-          }
-
-          const kubeconfigContent = await mgmtCluster.generateKubeConfig();
-
-          cc.setData('kubeconfigContent', kubeconfigContent);
-
-          await cc.save();
-        } catch (error) {
-          console.error(`Unable to refresh harvester cloud credential '${ cc.id }'`, error); // eslint-disable-line no-console
-        }
-      }));
-    }
-  }
-};
 
 export default class CloudCredential extends NormanModel {
-  get _availableActions() {
-    const out = super._availableActions;
-
-    out.splice(0, 0, { divider: true });
-    out.splice(0, 0, {
-      action:     'renew',
-      enabled:    this.canRenew,
-      bulkable:   this.canBulkRenew,
-      bulkAction: 'renewBulk',
-      icon:       'icon icon-fw icon-refresh',
-      label:      this.t('manager.cloudCredentials.renew'),
-    });
-
-    return out;
-  }
-
   get hasSensitiveData() {
     return true;
   }
@@ -231,111 +177,4 @@ export default class CloudCredential extends NormanModel {
   get doneRoute() {
     return 'c-cluster-manager-secret';
   }
-
-  get canRenew() {
-    return !!renew[this.provider]?.renew && this.expires !== undefined && this.canUpdate;
-  }
-
-  get canBulkRenew() {
-    return !!renew[this.provider]?.renewBulk;
-  }
-
-  get expiresForSort() {
-    // Why not just `expires`? Ensures the correct sort order of expired --> expiring --> never expires
-    // (instead of 'never expired' --> 'expired' --> 'expiring')
-    return this.expires !== undefined ? this.expires : Number.MAX_SAFE_INTEGER;
-  }
-
-  get expires() {
-    const expires = this.annotations[CLOUD_CREDENTIALS.EXPIRATION];
-
-    if (typeof expires === 'string') {
-      return parseInt(expires);
-    } else if (typeof expires === 'number') {
-      return expires;
-    }
-
-    return undefined; // Weird things happen if this isn't a number
-  }
-
-  get expireData() {
-    if (typeof this.expiresIn !== 'number') {
-      return null;
-    }
-
-    const sevenDays = 1000 * 60 * 60 * 24 * 7;
-
-    if (this.expiresIn === 0) {
-      return {
-        expired:  true,
-        expiring: false,
-      };
-    } else if (this.expiresIn < sevenDays) {
-      return {
-        expired:  false,
-        expiring: true,
-      };
-    } else if (this.expiresIn) {
-      return {
-        expired:  false,
-        expiring: false,
-      };
-    }
-
-    return null;
-  }
-
-  get expiresString() {
-    if (this.expires === undefined) {
-      return '';
-    }
-
-    if (this.expireData.expired) {
-      return this.t('manager.cloudCredentials.expired');
-    }
-
-    const dateFormat = escapeHtml( this.$rootGetters['prefs/get'](DATE_FORMAT));
-    const timeFormat = escapeHtml( this.$rootGetters['prefs/get'](TIME_FORMAT));
-
-    return day(this.expires).format(`${ dateFormat } ${ timeFormat }`);
-  }
-
-  get expiresIn() {
-    if (this.expires === undefined) {
-      return null;
-    }
-
-    const timeThen = this.expires;
-    const timeNow = Date.now();
-
-    const expiresIn = timeThen - timeNow;
-
-    return expiresIn < 0 ? 0 : expiresIn;
-  }
-
-  renew() {
-    const renewFn = renew[this.provider]?.renew;
-
-    if (!renewFn) {
-      console.error('No fn renew function for ', this.provider); // eslint-disable-line no-console
-    }
-
-    return renewFn({
-      cloudCredential: this,
-      $ctx:            this.$ctx
-    });
-  }
-
-  async renewBulk(cloudCredentials = []) {
-    const renewBulkFn = renew[this.provider]?.renewBulk;
-
-    if (!renewBulkFn) {
-      console.error('No fn renew bulk function for ', this.provider); // eslint-disable-line no-console
-    }
-
-    return renewBulkFn({
-      cloudCredentials,
-      $ctx: this.$ctx
-    });
-  }
 }

package/models/cluster/node.js

@@ -16,7 +16,7 @@ export default class ClusterNode extends SteveModel {
     const cordon = {
       action:   'cordon',
       enabled:  !!normanAction.cordon,
-      icon:     'icon icon-fw icon-pause',
+      icon:     'icon icon-pause',
       label:    'Cordon',
       total:    1,
       bulkable: true
@@ -25,7 +25,7 @@ export default class ClusterNode extends SteveModel {
     const uncordon = {
       action:   'uncordon',
       enabled:  !!normanAction.uncordon,
-      icon:     'icon icon-fw icon-play',
+      icon:     'icon icon-play',
       label:    'Uncordon',
       total:    1,
       bulkable: true
@@ -34,7 +34,7 @@ export default class ClusterNode extends SteveModel {
     const drain = {
       action:     'drain',
       enabled:    !!normanAction.drain,
-      icon:       'icon icon-fw icon-dot-open',
+      icon:       'icon icon-dot-open',
       label:      this.t('drainNode.action'),
       bulkable:   true,
       bulkAction: 'drain'
@@ -43,7 +43,7 @@ export default class ClusterNode extends SteveModel {
     const stopDrain = {
       action:   'stopDrain',
       enabled:  !!normanAction.stopDrain,
-      icon:     'icon icon-fw icon-x',
+      icon:     'icon icon-x',
       label:    this.t('drainNode.actionStop'),
       bulkable: true,
     };
@@ -51,14 +51,14 @@ export default class ClusterNode extends SteveModel {
     const openSsh = {
       action:  'openSsh',
       enabled: !!this.provisionedMachine?.links?.shell,
-      icon:    'icon icon-fw icon-chevron-right',
+      icon:    'icon icon-chevron-right',
       label:   'SSH Shell',
     };
 
     const downloadKeys = {
       action:  'downloadKeys',
       enabled: !!this.provisionedMachine?.links?.sshkeys,
-      icon:    'icon icon-fw icon-download',
+      icon:    'icon icon-download',
       label:   this.t('node.actions.downloadSSHKey'),
     };
 
@@ -227,7 +227,7 @@ export default class ClusterNode extends SteveModel {
   }
 
   get podCapacity() {
-    return Number.parseInt(this.status.capacity?.pods);
+    return parseSi(this.status.capacity?.pods);
   }
 
   get podConsumed() {

package/models/cluster.x-k8s.io.machine.js

@@ -60,13 +60,13 @@ export default class CapiMachine extends SteveModel {
     const openSsh = {
       action:  'openSsh',
       enabled: !!this.links.shell && this.isRunning,
-      icon:    'icon icon-fw icon-chevron-right',
+      icon:    'icon icon-chevron-right',
       label:   'SSH Shell',
     };
     const downloadKeys = {
       action:  'downloadKeys',
       enabled: !!this.links.sshkeys,
-      icon:    'icon icon-fw icon-download',
+      icon:    'icon icon-download',
       label:   this.t('node.actions.downloadSSHKey'),
     };
     const forceRemove = {
@@ -80,7 +80,7 @@ export default class CapiMachine extends SteveModel {
       action:     'toggleScaleDownModal',
       bulkAction: 'toggleScaleDownModal',
       enabled:    !!this.canScaleDown,
-      icon:       'icon icon-minus icon-fw',
+      icon:       'icon icon-minus',
       label:      this.t('node.actions.scaleDown'),
       bulkable:   true
     };

package/models/cluster.x-k8s.io.machinedeployment.js

@@ -125,7 +125,7 @@ export default class CapiMachineDeployment extends SteveModel {
       return;
     }
 
-    const initialValue = this.cluster.toJSON();
+    const initialValue = this.cluster;
 
     this.inClusterSpec.quantity += delta;
 
@@ -145,7 +145,16 @@ export default class CapiMachineDeployment extends SteveModel {
         let errors = exceptionToErrorsArray(err);
 
         if ( err.status === 409 && depth < 2 ) {
-          const conflicts = await handleConflict(initialValue, value, liveModel, this.$rootGetters, { dispatch: this.$dispatch }, 'management');
+          const conflicts = await handleConflict(
+            initialValue,
+            value,
+            liveModel,
+            {
+              dispatch: this.$dispatch,
+              getters:  this.$rootGetters
+            },
+            'management'
+          );
 
           if ( conflicts === false ) {
             // It was automatically figured out, save again

package/models/compliance.cattle.io.clusterscan.js

@@ -52,7 +52,7 @@ export default class ClusterScan extends SteveModel {
     const downloadReport = {
       action:  'downloadLatestReport',
       enabled: this.hasReport,
-      icon:    'icon icon-fw icon-download',
+      icon:    'icon icon-download',
       label:   t('compliance.downloadReport'),
       total:   1,
     };
@@ -60,7 +60,7 @@ export default class ClusterScan extends SteveModel {
     const downloadAllReports = {
       action:  'downloadAllReports',
       enabled: this.hasReport,
-      icon:    'icon icon-fw icon-download',
+      icon:    'icon icon-download',
       label:   t('compliance.downloadAllReports'),
       total:   1,
     };

package/models/configmap.js

@@ -17,4 +17,8 @@ export default class ConfigMap extends SteveModel {
 
     return keys.join(', ');
   }
+
+  get fullDetailPageOverride() {
+    return true;
+  }
 }

package/models/constraints.gatekeeper.sh.constraint.js

@@ -15,7 +15,7 @@ export default class GateKeeperConstraint extends SteveModel {
 
     const downloadViolations = {
       action: 'downloadViolations',
-      icon:   'icon icon-fw icon-download',
+      icon:   'icon icon-download',
       label:  t('gatekeeperConstraint.downloadViolations'),
       total:  1,
     };

package/models/fleet-application.js

@@ -8,9 +8,6 @@ import SteveModel from '@shell/plugins/steve/steve-class';
 import { mapStateToEnum, primaryDisplayStatusFromCount, STATES_ENUM } from '@shell/plugins/dashboard-store/resource-class';
 import FleetUtils from '@shell/utils/fleet';
 
-export const MINIMUM_POLLING_INTERVAL = 15;
-export const DEFAULT_POLLING_INTERVAL = 60;
-
 function normalizeStateCounts(data) {
   if (isEmpty(data)) {
     return {
@@ -46,16 +43,6 @@ export default class FleetApplication extends SteveModel {
     this.save();
   }
 
-  enablePollingAction() {
-    this.spec.disablePolling = false;
-    this.save();
-  }
-
-  disablePollingAction() {
-    this.spec.disablePolling = true;
-    this.save();
-  }
-
   goToClone() {
     if (this.metadata?.labels?.[FLEET_ANNOTATIONS.CREATED_BY_USER_ID]) {
       delete this.metadata.labels[FLEET_ANNOTATIONS.CREATED_BY_USER_ID];
@@ -68,10 +55,6 @@ export default class FleetApplication extends SteveModel {
     super.goToClone();
   }
 
-  get isPollingEnabled() {
-    return !this.spec.disablePolling;
-  }
-
   get state() {
     if (this.spec?.paused === true) {
       return 'paused';

package/models/fleet.cattle.io.cluster.js

@@ -223,8 +223,8 @@ export default class FleetCluster extends SteveModel {
     return parsedLabels;
   }
 
-  async saveYaml(yaml) {
-    await this._saveYaml(yaml);
+  async saveYaml(yaml, initialYaml) {
+    await this._saveYaml(yaml, initialYaml);
 
     const parsed = jsyaml.load(yaml);
 

package/models/fleet.cattle.io.gitrepo.js

@@ -76,6 +76,16 @@ export default class GitRepo extends FleetApplication {
     return out;
   }
 
+  enablePollingAction() {
+    this.spec.disablePolling = false;
+    this.save();
+  }
+
+  disablePollingAction() {
+    this.spec.disablePolling = true;
+    this.save();
+  }
+
   forceUpdate(resources = [this]) {
     this.$dispatch('promptModal', {
       componentProps: { repositories: resources },
@@ -90,6 +100,10 @@ export default class GitRepo extends FleetApplication {
     });
   }
 
+  get isPollingEnabled() {
+    return !this.spec.disablePolling;
+  }
+
   get isWebhookConfigured() {
     return !!this.status?.webhookCommit;
   }
@@ -185,7 +199,7 @@ export default class GitRepo extends FleetApplication {
       value,
       display:  this.repoDisplay,
       icon:     this.resourceIcon,
-      showLink: matchHttps || matchSSH
+      showLink: !!(matchHttps || matchSSH)
     };
   }