@rancher/shell 0.3.9 → 0.3.10

package/store/index.js

@@ -27,11 +27,13 @@ import {
   NAMESPACE_FILTER_NAMESPACED_PREFIX as NAMESPACED_PREFIX,
   NAMESPACE_FILTER_NAMESPACED_YES as NAMESPACED_YES,
   splitNamespaceFilterKey,
+  NAMESPACE_FILTER_NS_FULL_PREFIX,
 } from '@shell/utils/namespace-filter';
 import { allHash, allHashSettled } from '@shell/utils/promise';
 import { sortBy } from '@shell/utils/sort';
 import { addParam } from '@shell/utils/url';
 import semver from 'semver';
+import { STORE } from '@shell/store/store-types';
 
 // Disables strict mode for all store instances to prevent warning about changing state outside of mutations
 // because it's more efficient to do that sometimes.
@@ -41,19 +43,19 @@ export const BLANK_CLUSTER = '_';
 
 export const plugins = [
   Steve({
-    namespace:      'management',
+    namespace:      STORE.MANAGEMENT,
     baseUrl:        '/v1',
     modelBaseClass: BY_TYPE,
     supportsStream: false, // true, -- Disabled due to report that it's sometimes much slower in Chrome
   }),
   Steve({
-    namespace:      'cluster',
+    namespace:      STORE.CLUSTER,
     baseUrl:        '', // URL is dynamically set for the selected cluster
     supportsStream: false, // true, -- Disabled due to report that it's sometimes much slower in Chrome
     supportsGc:     true, // Enable garbage collection for this store only
   }),
   Steve({
-    namespace:      'rancher',
+    namespace:      STORE.RANCHER,
     baseUrl:        '/v3',
     supportsStream: false, // The norman API doesn't support streaming
     modelBaseClass: STEVE_MODEL_TYPES.NORMAN,
@@ -225,7 +227,6 @@ export const state = () => {
   return {
     managementReady:         false,
     clusterReady:            false,
-    isMultiCluster:          false,
     isRancher:               false,
     namespaceFilters:        [],
     activeNamespaceCache:    {}, // Used to efficiently check if a resource should be displayed
@@ -241,7 +242,6 @@ export const state = () => {
     serverVersion:           null,
     systemNamespaces:        [],
     isSingleProduct:         undefined,
-    namespaceFilterMode:     null,
   };
 };
 
@@ -250,8 +250,14 @@ export const getters = {
     return state.clusterReady === true;
   },
 
-  isMultiCluster(state) {
-    return state.isMultiCluster === true;
+  isMultiCluster(state, getters) {
+    const clusters = getters['management/all'](MANAGEMENT.CLUSTER);
+
+    if (clusters.length === 1 && clusters[0].metadata?.name === 'local') {
+      return false;
+    } else {
+      return true;
+    }
   },
 
   isRancher(state) {
@@ -278,15 +284,6 @@ export const getters = {
     return state.systemNamespaces;
   },
 
-  /**
-   * Namespace Filter Mode supplies a resource type to the NamespaceFilter.
-   *
-   * Only one of the resource type is allowed to be selected
-   */
-  namespaceFilterMode(state) {
-    return state.namespaceFilterMode;
-  },
-
   currentCluster(state, getters) {
     return getters['management/byId'](MANAGEMENT.CLUSTER, state.clusterId);
   },
@@ -372,34 +369,6 @@ export const getters = {
     return state.namespaceFilters.filter(x => !`${ x }`.startsWith(NAMESPACED_PREFIX)).length === 0;
   },
 
-  isSingleNamespace(state, getters) {
-    const product = getters['currentProduct'];
-
-    if ( !product ) {
-      return false;
-    }
-
-    if ( product.showWorkspaceSwitcher ) {
-      return false;
-    }
-
-    if ( getters.isAllNamespaces ) {
-      return false;
-    }
-
-    const filters = state.namespaceFilters;
-
-    if ( filters.length !== 1 ) {
-      return false;
-    }
-
-    if (filters[0].startsWith('ns://')) {
-      return filters[0];
-    }
-
-    return false;
-  },
-
   isMultipleNamespaces(state, getters) {
     const product = getters['currentProduct'];
 
@@ -421,7 +390,7 @@ export const getters = {
       return true;
     }
 
-    return !filters[0].startsWith('ns://');
+    return !filters[0].startsWith(NAMESPACE_FILTER_NS_FULL_PREFIX);
   },
 
   namespaceFilters(state) {
@@ -591,9 +560,8 @@ export const getters = {
 };
 
 export const mutations = {
-  managementChanged(state, { ready, isMultiCluster, isRancher }) {
+  managementChanged(state, { ready, isRancher }) {
     state.managementReady = ready;
-    state.isMultiCluster = isMultiCluster;
     state.isRancher = isRancher;
   },
   clusterReady(state, ready) {
@@ -621,10 +589,6 @@ export const mutations = {
     state.allNamespaces = namespace;
   },
 
-  setNamespaceFilterMode(state, mode) {
-    state.namespaceFilterMode = mode;
-  },
-
   pageActions(state, pageActions) {
     state.pageActions = pageActions;
   },
@@ -747,11 +711,7 @@ export const actions = {
 
     res = await allHash(promises);
     dispatch('i18n/init');
-    let isMultiCluster = true;
-
-    if ( res.clusters.length === 1 && res.clusters[0].metadata?.name === 'local' ) {
-      isMultiCluster = false;
-    }
+    const isMultiCluster = getters['isMultiCluster'];
 
     const pl = res.settings?.find(x => x.id === 'ui-pl')?.value;
     const brand = res.settings?.find(x => x.id === SETTING.BRAND)?.value;
@@ -773,7 +733,6 @@ export const actions = {
 
     commit('managementChanged', {
       ready: true,
-      isMultiCluster,
       isRancher,
     });
 
@@ -967,10 +926,6 @@ export const actions = {
     commit('updateNamespaces', { filters: ids });
   },
 
-  setNamespaceFilterMode({ commit }, mode) {
-    commit('setNamespaceFilterMode', mode);
-  },
-
   async cleanNamespaces({ getters, dispatch }) {
     // Initialise / Remove any filters that the user no-longer has access to
     await dispatch('management/findAll', { type: MANAGEMENT.CLUSTER }); // So they can be got byId below

package/store/store-types.js

@@ -0,0 +1,5 @@
+export const STORE = {
+  CLUSTER:    'cluster',
+  RANCHER:    'rancher',
+  MANAGEMENT: 'management',
+};

package/store/type-map.js

@@ -971,7 +971,7 @@ export const getters = {
 
   groupByFor(state) {
     return (schema) => {
-      return state.groupBy[schema.id];
+      return state.groupBy[schema?.id];
     };
   },
 

package/types/shell/index.d.ts

@@ -1738,6 +1738,7 @@ export namespace MANAGEMENT {
     const GLOBAL_ROLE_BINDING_1: string;
     export { GLOBAL_ROLE_BINDING_1 as GLOBAL_ROLE_BINDING };
     export const POD_SECURITY_POLICY_TEMPLATE: string;
+    export const PSP_TEMPLATE_BINDING: string;
     export const PSA: string;
     export const MANAGED_CHART: string;
     export const USER_NOTIFICATION: string;
@@ -1840,6 +1841,7 @@ export namespace HCI {
     export const IMAGE: string;
     const SETTING_1: string;
     export { SETTING_1 as SETTING };
+    export const HARVESTER_CONFIG: string;
 }
 export const VIRTUAL_HARVESTER_PROVIDER: "harvester";
 export namespace ADDRESSES {
@@ -2493,6 +2495,10 @@ export default _default;
 declare module '@shell/utils/create-yaml' {
 export function createYamlWithOptions(schemas: any, type: any, data: any, options: any): string;
 export function createYaml(schemas: any, type: any, data: any, processAlwaysAdd?: boolean, depth?: number, path?: string, rootType?: any, dataOptions?: {}): string;
+export function getBlockDescriptor(value: any, key: any): {
+    header: string;
+    indentation: string;
+};
 export function typeRef(type: any, str: any): any;
 export function typeMunge(type: any): any;
 export function saferDump(obj: any): any;
@@ -2509,7 +2515,7 @@ export function saferDump(obj: any): any;
  *     one of '|', '>'
  *     default '|'
  * - chomping:
- *     one of: null, '-', '+'
+ *     one of: null, '', '-', '+'
  *     default: null
  * @returns the result of jsyaml.dump with the addition of multiline indicators
  */
@@ -2674,14 +2680,24 @@ export function haveV1MonitoringWorkloads(store: any): Promise<boolean>;
 // @shell/utils/namespace-filter
 
 declare module '@shell/utils/namespace-filter' {
-export const NAMESPACE_FILTER_SPECIAL: "special";
+export const NAMESPACE_FILTER_ALL_PREFIX: "all";
+export const NAMESPACE_FILTER_NS_PREFIX: "ns";
+export const NAMESPACE_FILTER_P_PREFIX: "project";
+export const NAMESPACE_FILTER_NS_FULL_PREFIX: string;
+export const NAMESPACE_FILTER_P_FULL_PREFIX: string;
 export const NAMESPACE_FILTER_ALL: "all";
-export const NAMESPACE_FILTER_ALL_SYSTEM: "all://system";
-export const NAMESPACE_FILTER_ALL_USER: "all://user";
-export const NAMESPACE_FILTER_ALL_ORPHANS: "all://orphans";
+export const NAMESPACE_FILTER_ALL_SYSTEM: string;
+export const NAMESPACE_FILTER_ALL_USER: string;
+export const NAMESPACE_FILTER_ALL_ORPHANS: string;
 export const NAMESPACE_FILTER_NAMESPACED_PREFIX: "namespaced://";
 export const NAMESPACE_FILTER_NAMESPACED_YES: "namespaced://true";
 export const NAMESPACE_FILTER_NAMESPACED_NO: "namespaced://false";
+export namespace NAMESPACE_FILTER_KINDS {
+    const DIVIDER: string;
+    const PROJECT: string;
+    const NAMESPACE: string;
+    const SPECIAL: string;
+}
 export function createNamespaceFilterKey(clusterId: any, product: any): any;
 export function createNamespaceFilterKeyWithId(clusterId: any, productId: any): string;
 export function splitNamespaceFilterKey(key: any): {
@@ -2945,8 +2961,8 @@ export function simplify(matchExpressionsInput: any): {
     matchLabels: {};
     matchExpressions: any[];
 };
-export function matches(obj: any, selector: any): boolean;
-export function matching(ary: any, selector: any): any;
+export function matches(obj: any, selector: any, labelKey?: string): boolean;
+export function matching(ary: any, selector: any, labelKey: any): any;
 }
 
 // @shell/utils/socket

package/utils/__tests__/create-yaml.test.ts

@@ -0,0 +1,63 @@
+import {
+  getBlockDescriptor,
+  dumpBlock,
+} from '@shell/utils/create-yaml';
+
+const key = 'example';
+const randomData = '\n      foo\n      bar\n';
+
+const scalarStyles = ['>', '|'];
+const chomping = ['+', '-', ''];
+const indentations = ['4', '2', ''];
+
+describe('fx: getBlockDescriptor', () => {
+  describe('should parse blocks header for all block indicators combo', () => {
+    scalarStyles.forEach((scalar) => {
+      chomping.forEach((chomping) => {
+        indentations.forEach((indentation) => {
+          const combo = `${ scalar }${ indentation }${ chomping }`;
+
+          it(`combo: ${ combo }`, () => {
+            const toParse = `${ key }: ${ combo }${ randomData }`;
+
+            const desc = getBlockDescriptor(toParse, key);
+
+            expect(desc?.header).toBe(`${ key }: ${ combo }`);
+            expect(desc?.indentation).toBe(indentation);
+          });
+        });
+      });
+    });
+  });
+});
+
+describe('fx: dumpBlock', () => {
+  describe('should create a data block replacing indicators with blocks indicator from options', () => {
+    const key = 'example';
+
+    scalarStyles.forEach((scalarStyle) => {
+      chomping.forEach((chomping) => {
+        const options = {
+          [key]: {
+            chomping,
+            scalarStyle
+          }
+        };
+
+        it(`options: { scalarStyle: ${ scalarStyle }, chomping: ${ chomping } } with indentation`, () => {
+          const data = { [key]: ' foo  \n bar   \n   \n   foo\n   bar\n   ' };
+          const block = dumpBlock(data, options);
+
+          expect(block.includes(`example: ${ scalarStyle }${ chomping }2`)).toBeTruthy();
+        });
+
+        it(`options: { scalarStyle: ${ scalarStyle }, chomping: ${ chomping } } without indentation`, () => {
+          const data = { [key]: 'foo  \nbar   \n\nfoo\nbar\n   ' };
+          const block = dumpBlock(data, options);
+
+          expect(block.includes(`example: ${ scalarStyle }${ chomping }`)).toBeTruthy();
+        });
+      });
+    });
+  });
+});

package/utils/array.ts

@@ -188,6 +188,10 @@ export function uniq<T>(ary: T[]): T[] {
   return out;
 }
 
+export function concatStrings(a: string[], b: string[]) {
+  return [...a.map(aa => b.map(bb => aa.concat(bb)))].reduce((acc, arr) => [...arr, ...acc], []);
+}
+
 interface KubeResource { metadata: { labels: { [name: string]: string} } } // Migrate to central kube types resource when those are brought in
 export function getUniqueLabelKeys<T extends KubeResource>(aryResources: T[]): string[] {
   const uniqueObj = aryResources.reduce((res, r) => {

package/utils/create-yaml.js

@@ -370,7 +370,7 @@ function serializeSimpleValue(data) {
   return jsyaml.dump(data).trim();
 }
 
-function getBlockDescriptor(value, key) {
+export function getBlockDescriptor(value, key) {
   const header = getBlockHeader(value, key);
 
   return {
@@ -451,7 +451,7 @@ export function saferDump(obj) {
  *     one of '|', '>'
  *     default '|'
  * - chomping:
- *     one of: null, '-', '+'
+ *     one of: null, '', '-', '+'
  *     default: null
  * @returns the result of jsyaml.dump with the addition of multiline indicators
  */
@@ -464,15 +464,15 @@ export function dumpBlock(data, options = {}) {
 
   if (blockFields.length) {
     for (const key of blockFields) {
+      const { header, indentation } = getBlockDescriptor(out, key);
+
       const scalarStyle = options[key]?.scalarStyle ?? '|';
       const chomping = options[key]?.chomping ?? '';
 
-      const desc = getBlockDescriptor(out, key);
-
       /**
        * Replace the original block indicators with the ones provided in the options param
        */
-      out = out.replace(desc.header, `${ key }: ${ scalarStyle }${ chomping }${ desc.indentation }`);
+      out = out.replace(header, `${ key }: ${ scalarStyle }${ chomping }${ indentation }`);
     }
   }
 

package/utils/namespace-filter.js

@@ -1,13 +1,25 @@
-export const NAMESPACE_FILTER_SPECIAL = 'special';
+export const NAMESPACE_FILTER_ALL_PREFIX = 'all';
+export const NAMESPACE_FILTER_NS_PREFIX = 'ns';
+export const NAMESPACE_FILTER_P_PREFIX = 'project';
 
-export const NAMESPACE_FILTER_ALL = 'all';
-export const NAMESPACE_FILTER_ALL_SYSTEM = 'all://system';
-export const NAMESPACE_FILTER_ALL_USER = 'all://user';
-export const NAMESPACE_FILTER_ALL_ORPHANS = 'all://orphans';
+export const NAMESPACE_FILTER_NS_FULL_PREFIX = `${ NAMESPACE_FILTER_NS_PREFIX }://`;
+export const NAMESPACE_FILTER_P_FULL_PREFIX = `${ NAMESPACE_FILTER_P_PREFIX }://`;
+
+export const NAMESPACE_FILTER_ALL = NAMESPACE_FILTER_ALL_PREFIX;
+export const NAMESPACE_FILTER_ALL_SYSTEM = `${ NAMESPACE_FILTER_ALL_PREFIX }://system`;
+export const NAMESPACE_FILTER_ALL_USER = `${ NAMESPACE_FILTER_ALL_PREFIX }://user`;
+export const NAMESPACE_FILTER_ALL_ORPHANS = `${ NAMESPACE_FILTER_ALL_PREFIX }://orphans`;
 export const NAMESPACE_FILTER_NAMESPACED_PREFIX = 'namespaced://';
 export const NAMESPACE_FILTER_NAMESPACED_YES = 'namespaced://true';
 export const NAMESPACE_FILTER_NAMESPACED_NO = 'namespaced://false';
 
+export const NAMESPACE_FILTER_KINDS = {
+  DIVIDER:   'divider',
+  PROJECT:   'project',
+  NAMESPACE: 'namespace',
+  SPECIAL:   'special'
+};
+
 const SEPARATOR = '__%%__';
 
 export const createNamespaceFilterKey = (clusterId, product) => {

package/utils/projectAndNamespaceFiltering.utils.ts

@@ -0,0 +1,62 @@
+import { NAMESPACE_FILTER_NS_FULL_PREFIX, NAMESPACE_FILTER_P_FULL_PREFIX } from '@shell/utils/namespace-filter';
+import { getPerformanceSetting } from '@shell/utils/settings';
+
+type Opt = { [key: string]: any, namespaced?: string[]}
+
+class ProjectAndNamespaceFiltering {
+  static param = 'projectsornamespaces'
+
+  /**
+   * Does the request `opt` definition require resources are fetched from a specific set namespaces/projects?
+   */
+  isApplicable(opt: Opt): boolean {
+    return Array.isArray(opt.namespaced);
+  }
+
+  isEnabled(rootGetters: any): boolean {
+    const currentProduct = rootGetters['currentProduct'];
+
+    // 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 (currentProduct?.inStore !== 'cluster') {
+      return false;
+    }
+
+    if (currentProduct?.showWorkspaceSwitcher) {
+      return false;
+    }
+
+    const perfConfig = getPerformanceSetting(rootGetters);
+
+    if (!perfConfig.forceNsFilterV2?.enabled) {
+      return false;
+    }
+
+    return true;
+  }
+
+  /**
+   * Check if `opt` requires resources from specific ns/projects, if so return the required query param (x=y)
+   */
+  checkAndCreateParam(opt: Opt): string {
+    if (!this.isApplicable(opt)) {
+      return '';
+    }
+
+    return this.createParam(opt.namespaced);
+  }
+
+  private createParam(namespaceFilter: string[] | undefined): string {
+    if (!namespaceFilter || !namespaceFilter.length) {
+      return '';
+    }
+
+    const projectsOrNamespaces = namespaceFilter
+      .map(f => f.replace(NAMESPACE_FILTER_NS_FULL_PREFIX, '')
+        .replace(NAMESPACE_FILTER_P_FULL_PREFIX, ''))
+      .join(',');
+
+    return `${ ProjectAndNamespaceFiltering.param }=${ projectsOrNamespaces }`;
+  }
+}
+
+export default new ProjectAndNamespaceFiltering();

package/utils/selector.js

@@ -1,4 +1,5 @@
 import { isArray, addObject, findBy } from '@shell/utils/array';
+import { get } from '@shell/utils/object';
 
 const parseCache = {};
 
@@ -121,7 +122,7 @@ export function convert(matchLabelsObj, matchExpressions) {
       out.push({
         key,
         operator: 'In',
-        values:   [value],
+        values:   isArray(value) ? value : [value],
       });
     }
   }
@@ -162,7 +163,7 @@ export function simplify(matchExpressionsInput) {
   return { matchLabels, matchExpressions };
 }
 
-export function matches(obj, selector) {
+export function matches(obj, selector, labelKey = 'metadata.labels') {
   let rules = [];
 
   if ( typeof selector === 'string' ) {
@@ -178,7 +179,7 @@ export function matches(obj, selector) {
     return false;
   }
 
-  const labels = obj?.metadata?.labels || {};
+  const labels = get(obj, labelKey) || {};
 
   for ( const rule of rules ) {
     const value = labels[rule.key];
@@ -222,6 +223,6 @@ export function matches(obj, selector) {
   return true;
 }
 
-export function matching(ary, selector) {
-  return ary.filter(obj => matches(obj, selector));
+export function matching(ary, selector, labelKey) {
+  return ary.filter(obj => matches(obj, selector, labelKey));
 }

package/utils/settings.ts

@@ -44,19 +44,17 @@ export const setSetting = async(store: Store<any>, id: string, val: string): Pro
 };
 
 export const getPerformanceSetting = (rootGetters: Record<string, (arg0: string, arg1: string) => any>) => {
-  const perfSetting = rootGetters['management/byId'](MANAGEMENT.SETTING, SETTING.UI_PERFORMANCE);
-  let perfConfig = {};
+  const perfSettingResource = rootGetters['management/byId'](MANAGEMENT.SETTING, SETTING.UI_PERFORMANCE);
+  let perfSetting = {};
 
-  if (perfSetting && perfSetting.value) {
+  if (perfSettingResource?.value) {
     try {
-      perfConfig = JSON.parse(perfSetting.value);
+      perfSetting = JSON.parse(perfSettingResource.value);
     } catch (e) {
       console.warn('ui-performance setting contains invalid data'); // eslint-disable-line no-console
     }
   }
 
   // Start with the default and overwrite the values from the setting - ensures we have defaults for newly added options
-  perfConfig = Object.assign(DEFAULT_PERF_SETTING, perfConfig);
-
-  return perfConfig;
+  return Object.assign(DEFAULT_PERF_SETTING, perfSetting || {});
 };

package/models/k8s.cni.cncf.io.networkattachmentdefinition.js

@@ -1,93 +0,0 @@
-import Vue from 'vue';
-import SteveModel from '@shell/plugins/steve/steve-class';
-import { HCI } from '@shell/config/labels-annotations';
-
-export default class NetworkAttachmentDef extends SteveModel {
-  get _availableActions() {
-    let out = super._availableActions;
-    const toFilter = ['goToClone', 'cloneYaml', 'goToViewConfig', 'goToEditYaml', 'goToEdit'];
-
-    out = out.filter((action) => {
-      if (!toFilter.includes(action.action)) {
-        return action;
-      }
-    });
-
-    return out;
-  }
-
-  applyDefaults() {
-    const spec = this.spec || {
-      config: JSON.stringify({
-        cniVersion:  '0.3.1',
-        name:        '',
-        type:        'bridge',
-        bridge:      'harvester-br0',
-        promiscMode: true,
-        vlan:        '',
-        ipam:        {}
-      })
-    };
-
-    Vue.set(this, 'spec', spec);
-  }
-
-  get parseConfig() {
-    try {
-      return JSON.parse(this.spec.config) || {};
-    } catch (err) {
-      return {};
-    }
-  }
-
-  get isIpamStatic() {
-    return this.parseConfig.ipam?.type === 'static';
-  }
-
-  get vlanType() {
-    const type = this.parseConfig.type;
-
-    return type === 'bridge' ? 'L2VlanNetwork' : type;
-  }
-
-  get vlanId() {
-    return this.parseConfig.vlan;
-  }
-
-  get customValidationRules() {
-    const rules = [
-      {
-        nullable:       false,
-        path:           'metadata.name',
-        required:       true,
-        minLength:      1,
-        maxLength:      63,
-        translationKey: 'harvester.fields.name'
-      }
-    ];
-
-    return rules;
-  }
-
-  get connectivity() {
-    const annotations = this.metadata?.annotations || {};
-    const route = annotations[HCI.NETWORK_ROUTE];
-    let config = {};
-
-    try {
-      config = JSON.parse(route || '{}');
-    } catch {
-      return 'invalid';
-    }
-
-    const connectivity = config.connectivity;
-
-    if (connectivity === 'false') {
-      return 'inactive';
-    } else if (connectivity === 'true') {
-      return 'active';
-    } else {
-      return connectivity;
-    }
-  }
-}