@rancher/shell 3.0.5 → 3.0.7

package/plugins/dashboard-store/resource-class.js

@@ -1,4 +1,7 @@
 import { NORMAN_NAME } from '@shell/config/labels-annotations';
+import { getVersionData } from '@shell/config/version';
+import { parseRancherVersion } from '@shell/config/uiplugins';
+import semver from 'semver';
 import {
   _CLONE,
   _CONFIG,
@@ -59,7 +62,7 @@ const DEFAULT_COLOR = 'warning';
 const DEFAULT_ICON = 'x';
 
 const DEFAULT_WAIT_INTERVAL = 1000;
-const DEFAULT_WAIT_TMIMEOUT = 30000;
+const DEFAULT_WAIT_TIMEOUT = 30000;
 
 export const STATES_ENUM = {
   IN_USE:           'in-use',
@@ -803,7 +806,7 @@ export default class Resource {
   // ------------------------------------------------------------------
 
   waitForTestFn(fn, msg, timeoutMs, intervalMs) {
-    return waitFor(() => fn.apply(this), msg, timeoutMs || DEFAULT_WAIT_TMIMEOUT, intervalMs || DEFAULT_WAIT_INTERVAL, true);
+    return waitFor(() => fn.apply(this), msg, timeoutMs || DEFAULT_WAIT_TIMEOUT, intervalMs || DEFAULT_WAIT_INTERVAL, true);
   }
 
   waitForState(state, timeout, interval) {
@@ -852,7 +855,7 @@ export default class Resource {
     return (entry.status || '').toLowerCase() === `${ withStatus }`.toLowerCase();
   }
 
-  waitForCondition(name, withStatus = 'True', timeoutMs = DEFAULT_WAIT_TMIMEOUT, intervalMs = DEFAULT_WAIT_INTERVAL) {
+  waitForCondition(name, withStatus = 'True', timeoutMs = DEFAULT_WAIT_TIMEOUT, intervalMs = DEFAULT_WAIT_INTERVAL) {
     return this.waitForTestFn(() => {
       return this.isCondition(name, withStatus);
     }, `condition ${ name }=${ withStatus }`, timeoutMs, intervalMs);
@@ -929,12 +932,20 @@ export default class Resource {
     const currentRoute = this.currentRouter().currentRoute.value;
     const extensionMenuActions = getApplicableExtensionEnhancements(this.$rootState, ExtensionPoint.ACTION, ActionLocation.TABLE, currentRoute, this);
 
+    const currRancherVersionData = getVersionData();
+    const parsedRancherVersion = parseRancherVersion(currRancherVersionData.Version);
+
+    // "showConfiguration" table action is only compatible with Rancher 2.13 and onwards
+    // defence against extension issue https://github.com/rancher/dashboard/issues/15564
+    // where mostly likely extension CRD model is extending from resource-class
+    const isResourceDetailDrawerCompatibleWithRancherSystem = semver.satisfies(parsedRancherVersion, '>= 2.13.0');
+
     const all = [
       {
         action:  'showConfiguration',
         label:   this.t('action.showConfiguration'),
         icon:    'icon icon-document',
-        enabled: this.disableResourceDetailDrawer !== true && (this.canCustomEdit || this.canYaml), // If the resource can't show an edit or a yaml we don't want to show the configuration drawer
+        enabled: isResourceDetailDrawerCompatibleWithRancherSystem && this.disableResourceDetailDrawer !== true && (this.canCustomEdit || this.canYaml), // If the resource can't show an edit or a yaml we don't want to show the configuration drawer
       },
       { divider: true },
       {

package/plugins/steve/__tests__/subscribe.spec.ts

@@ -1,17 +1,19 @@
 import { actions, getters, mutations } from '../subscribe';
 import { REVISION_TOO_OLD } from '../../../utils/socket';
-import { STEVE_WATCH_EVENT } from '../../../types/store/subscribe.types';
+import { STEVE_WATCH_MODE } from '../../../types/store/subscribe.types';
 import backOff from '../../../utils/back-off';
+import { SteveWatchEventListenerManager } from '../../subscribe-events';
 
 describe('steve: subscribe', () => {
   describe('actions', () => {
     describe('watch', () => {
-      const state = {};
+      const state = { listenerManager: new SteveWatchEventListenerManager() };
       const getters = {
-        normalizeType: (type: string) => type,
-        schemaFor:     () => null,
-        inError:       () => false,
-        watchStarted:  () => false,
+        normalizeType:   (type: string) => type,
+        schemaFor:       () => null,
+        inError:         () => false,
+        watchStarted:    () => false,
+        listenerManager: state.listenerManager
       };
       const rootGetters = {
         'type-map/isSpoofed': () => false,
@@ -211,15 +213,11 @@ describe('steve: subscribe', () => {
         }, {
           ...obj,
           revision,
-          mode:  STEVE_WATCH_EVENT.CHANGES,
+          mode:  STEVE_WATCH_MODE.RESOURCE_CHANGES,
           force: true,
         });
 
-        expect(dispatch).toHaveBeenNthCalledWith(1, 'unwatchIncompatible', {
-          id: undefined, mode: STEVE_WATCH_EVENT.CHANGES, namespace: undefined, selector: undefined, type: obj.type
-        });
-
-        expect(dispatch).toHaveBeenNthCalledWith(2, 'send', {
+        expect(dispatch).toHaveBeenNthCalledWith(1, 'send', {
           debounceMs:      4000,
           mode:            'resource.changes',
           resourceType:    obj.type,
@@ -231,7 +229,7 @@ describe('steve: subscribe', () => {
           state, dispatch, getters, commit
         }, { ...msg });
 
-        expect(dispatch).toHaveBeenCalledTimes(2);
+        expect(dispatch).toHaveBeenCalledTimes(1);
         dispatch.mockClear();
       };
 
@@ -252,7 +250,7 @@ describe('steve: subscribe', () => {
         });
         expect(state.inError).toStrictEqual(
           {
-            'type=abc,namespace=,id=,selector=': {
+            'type=abc,namespace=,id=,selector=,mode=resource.changes': {
               obj: {
                 type: msg.resourceType,
                 mode: msg.mode,
@@ -268,7 +266,7 @@ describe('steve: subscribe', () => {
         }, { ...msg });
         // stop tries to watch again, however we're in error so will be ignored
         expect(dispatch).toHaveBeenNthCalledWith(1, 'watch', {
-          id: undefined, mode: STEVE_WATCH_EVENT.CHANGES, namespace: undefined, selector: undefined, type: obj.type
+          id: undefined, mode: STEVE_WATCH_MODE.RESOURCE_CHANGES, namespace: undefined, selector: undefined, standardWatch: true, type: obj.type
         });
 
         dispatch.mockClear();
@@ -349,19 +347,24 @@ describe('steve: subscribe', () => {
       const obj = { type: 'abc' };
       const msg = {
         resourceType: obj.type,
-        mode:         STEVE_WATCH_EVENT.CHANGES,
+        mode:         STEVE_WATCH_MODE.RESOURCE_CHANGES,
       };
 
       const initStore = () => {
-        const state = { started: [], inError: {} };
+        const state = {
+          started:         [],
+          inError:         {},
+          listenerManager: new SteveWatchEventListenerManager()
+        };
         const _getters = {
-          normalizeType: (type: string) => type,
-          schemaFor:     () => ({}),
-          storeName:     'test',
-          inError:       (...args) => getters.inError(state)(...args),
-          watchStarted:  (...args) => getters.watchStarted(state)(...args),
-          backOffId:     (...args) => getters.backOffId()(...args),
-          canBackoff:    () => true,
+          normalizeType:   (type: string) => type,
+          schemaFor:       () => ({}),
+          storeName:       'test',
+          inError:         (...args) => getters.inError(state)(...args),
+          watchStarted:    (...args) => getters.watchStarted(state)(...args),
+          backOffId:       (...args) => getters.backOffId()(...args),
+          canBackoff:      () => true,
+          listenerManager: state.listenerManager
         };
         const commit = (type, ...args) => mutations[type](state, ...args);
 

package/plugins/steve/index.js

@@ -9,6 +9,8 @@ import {
 import getters, { STEVE_MODEL_TYPES } from './getters';
 import mutations from './mutations';
 import actions from './actions';
+import { SteveWatchEventListenerManager } from '@shell/plugins/subscribe-events';
+import { markRaw } from 'vue';
 
 export function SteveFactory(namespace, baseUrl) {
   return {
@@ -17,16 +19,22 @@ export function SteveFactory(namespace, baseUrl) {
     state() {
       return {
         ...coreStoreState(namespace, baseUrl),
-        socket:           null,
-        queue:            [], // For change event coalescing
-        wantSocket:       false,
-        debugSocket:      false,
-        allowStreaming:   true,
-        pendingFrames:    [],
-        deferredRequests: {},
-        started:          [],
-        inError:          {},
-        podsByNamespace:  {}, // Cache of pods by namespace
+        socket:                null,
+        queue:                 [], // For change event coalescing
+        wantSocket:            false,
+        debugSocket:           false,
+        allowStreaming:        true,
+        pendingFrames:         [],
+        deferredRequests:      {},
+        started:               [],
+        inError:               {},
+        /**
+         * Socket listener manager for this store
+         *
+         * Instance of @SteveWatchEventListenerManager . See it's description for more info
+         */
+        socketListenerManager: markRaw(new SteveWatchEventListenerManager()),
+        podsByNamespace:       {}, // Cache of pods by namespace
       };
     },
 

package/plugins/steve/mutations.js

@@ -164,10 +164,10 @@ export default {
   },
 
   reset(state) {
-    // Reset generic store things.... then steve specific things
-
+    // 1. Reset generic store things
     resetStore(state, this.commit);
 
+    // 2. Reset steve specific store things
     this.commit(`${ state.config.namespace }/resetSubscriptions`);
 
     // Clear the podsByNamespace cache

package/plugins/steve/resourceWatcher.js

@@ -40,10 +40,10 @@ export const WATCH_STATUSES = {
  * Create a unique key for a specific resource watch's params
  */
 export const keyForSubscribe = ({
-  resourceType, type, namespace, id, selector
+  resourceType, type, namespace, id, selector, mode
 } = {}) => {
   const keyMap = {
-    type: resourceType || type, namespace, id, selector
+    type: resourceType || type, namespace, id, selector, mode
   };
 
   return Object.entries(keyMap)

package/plugins/steve/steve-pagination-utils.ts

@@ -15,13 +15,12 @@ import {
   HPA,
   SECRET
 } from '@shell/config/types';
-import {
-  CAPI as CAPI_LAB_AND_ANO, CATTLE_PUBLIC_ENDPOINTS, STORAGE, UI_PROJECT_SECRET, UI_PROJECT_SECRET_COPY
-} from '@shell/config/labels-annotations';
+import { CAPI as CAPI_LAB_AND_ANO, CATTLE_PUBLIC_ENDPOINTS, STORAGE, UI_PROJECT_SECRET_COPY } from '@shell/config/labels-annotations';
 import { Schema } from '@shell/plugins/steve/schema';
-import { PaginationSettingsStore } from '@shell/types/resources/settings';
+import { PaginationSettingsStores } from '@shell/types/resources/settings';
 import paginationUtils from '@shell/utils/pagination-utils';
 import { KubeLabelSelector, KubeLabelSelectorExpression } from '@shell/types/kube/kube-api';
+import { parseField } from '@shell/utils/sort';
 
 /**
  * This is a workaround for a ts build issue found in check-plugins-build.
@@ -158,6 +157,7 @@ class StevePaginationUtils extends NamespaceProjectFilters {
       { field: 'id' },
       { field: 'metadata.state.name' },
       { field: 'metadata.creationTimestamp' },
+      { field: 'metadata.labels', startsWith: true },
     ],
     [NODE]: [
       { field: 'status.nodeInfo.kubeletVersion' },
@@ -180,18 +180,12 @@ class StevePaginationUtils extends NamespaceProjectFilters {
       { field: 'spec.internal' },
       { field: 'spec.displayName' },
       { field: `status.provider` },
-      { field: `metadata.labels["${ CAPI_LAB_AND_ANO.PROVIDER }]` },
       { field: `status.connected` },
     ],
-    [CONFIG_MAP]: [
-      { field: 'metadata.labels[harvesterhci.io/cloud-init-template]' }
-    ],
     [SECRET]: [
-      { field: `metadata.labels[${ UI_PROJECT_SECRET }]` },
       { field: `metadata.annotations[${ UI_PROJECT_SECRET_COPY }]` },
     ],
     [NAMESPACE]: [
-      { field: 'metadata.labels[field.cattle.io/projectId]' }
     ],
     [CAPI.MACHINE]: [
       { field: 'spec.clusterName' }
@@ -214,7 +208,6 @@ class StevePaginationUtils extends NamespaceProjectFilters {
       { field: 'status.releaseName' },
     ],
     [CAPI.RANCHER_CLUSTER]: [
-      { field: `metadata.labels[${ CAPI_LAB_AND_ANO.PROVIDER }]` },
       { field: `status.provider` },
       { field: 'status.clusterName' },
       { field: `metadata.annotations[${ CAPI_LAB_AND_ANO.HUMAN_NAME }]` }
@@ -224,14 +217,14 @@ class StevePaginationUtils extends NamespaceProjectFilters {
       { field: 'spec.clusterIP' },
     ],
     [INGRESS]: [
-      { field: 'spec.rules.host' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.rules.host' },
       { field: 'spec.ingressClassName' },
     ],
     [HPA]: [
-      { field: 'spec.scaleTargetRef.name' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50527
-      { field: 'spec.minReplicas' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50527
-      { field: 'spec.maxReplicas' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50527
-      { field: 'spec.currentReplicas' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50527
+      { field: 'spec.scaleTargetRef.name' },
+      { field: 'spec.minReplicas' },
+      { field: 'spec.maxReplicas' },
+      { field: 'spec.currentReplicas' },
     ],
     [PVC]: [
       { field: 'spec.volumeName' },
@@ -249,29 +242,29 @@ class StevePaginationUtils extends NamespaceProjectFilters {
     ],
     [WORKLOAD_TYPES.CRON_JOB]: [
       { field: `metadata.annotations[${ CATTLE_PUBLIC_ENDPOINTS }]` },
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
     [WORKLOAD_TYPES.DAEMON_SET]: [
       { field: `metadata.annotations[${ CATTLE_PUBLIC_ENDPOINTS }]` },
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
     [WORKLOAD_TYPES.DEPLOYMENT]: [
       { field: `metadata.annotations[${ CATTLE_PUBLIC_ENDPOINTS }]` },
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
     [WORKLOAD_TYPES.JOB]: [
       { field: `metadata.annotations[${ CATTLE_PUBLIC_ENDPOINTS }]` },
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
     [WORKLOAD_TYPES.STATEFUL_SET]: [
       { field: `metadata.annotations[${ CATTLE_PUBLIC_ENDPOINTS }]` },
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
     [WORKLOAD_TYPES.REPLICA_SET]: [
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
     [WORKLOAD_TYPES.REPLICATION_CONTROLLER]: [
-      { field: 'spec.template.spec.containers.image' }, // Pending API Support - BUG - https://github.com/rancher/rancher/issues/50526
+      { field: 'spec.template.spec.containers.image' },
     ],
   }
 
@@ -409,9 +402,13 @@ class StevePaginationUtils extends NamespaceProjectFilters {
 
       const joined = opt.pagination.sort
         .map((s) => {
-          this.validateField(validateFields, schema, s.field);
+          // Use the same mechanism as local sorting to flip logic for asc/des
+          const { field, reverse } = parseField(s.field);
+          const asc = reverse ? !s.asc : s.asc;
+
+          this.validateField(validateFields, schema, field);
 
-          return `${ s.asc ? '' : '-' }${ this.convertArrayPath(s.field) }`;
+          return `${ asc ? '' : '-' }${ this.convertArrayPath(field) }`;
         })
         .join(',');
 
@@ -623,7 +620,6 @@ class StevePaginationUtils extends NamespaceProjectFilters {
         res.push(`filter=!${ labelKey }`);
         break;
       case 'Gt':
-        // Currently broken - see https://github.com/rancher/rancher/issues/50057
         // Only applicable to node affinity (atm) - https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#operators
 
         if (typeof exp.values !== 'string') {
@@ -636,7 +632,6 @@ class StevePaginationUtils extends NamespaceProjectFilters {
         res.push(`filter=${ labelKey } > (${ exp.values })`);
         break;
       case 'Lt':
-        // Currently broken - see https://github.com/rancher/rancher/issues/50057
         // Only applicable to node affinity (atm) - https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#operators
         if (typeof exp.values !== 'string') {
           console.error(`Skipping labelSelector to API filter param conversion for ${ exp.key }(Lt) as no value was supplied`); // eslint-disable-line no-console
@@ -657,7 +652,7 @@ class StevePaginationUtils extends NamespaceProjectFilters {
   }
 }
 
-export const PAGINATION_SETTINGS_STORE_DEFAULTS: PaginationSettingsStore = {
+export const PAGINATION_SETTINGS_STORE_DEFAULTS: PaginationSettingsStores = {
   cluster: {
     resources: {
       enableAll:  false,
@@ -670,7 +665,7 @@ export const PAGINATION_SETTINGS_STORE_DEFAULTS: PaginationSettingsStore = {
           CATALOG.APP, CATALOG.OPERATION,
           HPA, INGRESS, SERVICE,
           PV, CONFIG_MAP, STORAGE_CLASS, PVC, SECRET,
-          WORKLOAD_TYPES.REPLICA_SET, WORKLOAD_TYPES.REPLICATION_CONTROLLER
+          WORKLOAD_TYPES.REPLICA_SET, WORKLOAD_TYPES.REPLICATION_CONTROLLER,
         ],
         generic: true,
       }
@@ -681,8 +676,8 @@ export const PAGINATION_SETTINGS_STORE_DEFAULTS: PaginationSettingsStore = {
       enableAll:  false,
       enableSome: {
         enabled: [
-          // { resource: CAPI.RANCHER_CLUSTER, context: ['home', 'side-bar'] }, // Disabled due to https://github.com/rancher/dashboard/issues/14493
-          // { resource: MANAGEMENT.CLUSTER, context: ['side-bar'] }, // Disabled due to https://github.com/rancher/dashboard/issues/14493
+          // { resource: CAPI.RANCHER_CLUSTER, context: ['home', 'side-bar'] },
+          // { resource: MANAGEMENT.CLUSTER, context: ['side-bar'] },
           { resource: CATALOG.APP, context: ['branding'] },
           SECRET
         ],