@rancher/shell 0.3.23 → 0.3.25

package/mixins/fetch.client.js

@@ -75,9 +75,9 @@ async function $_fetch() { // eslint-disable-line camelcase
   try {
     await this.$options.fetch.call(this);
   } catch (err) {
-    if (process.dev) {
-      console.error('Error in fetch():', err); // eslint-disable-line no-console
-    }
+    // In most cases we don't handle errors at all in `fetch`es. Lets always log to help in production
+    console.error('Error in fetch():', err); // eslint-disable-line no-console
+
     error = normalizeError(err);
   }
 

package/models/__tests__/management.cattle.io.node.ts

@@ -0,0 +1,96 @@
+import MgmtNode from '@shell/models/management.cattle.io.node';
+
+describe('class MgmtNode', () => {
+  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, the internalNodeStatus, or the rkeNode key in status', () => {
+    const mgmtNode = new MgmtNode({ status: {} });
+
+    expect(mgmtNode.addresses).toStrictEqual([]);
+    resetMocks();
+  });
+
+  describe('should return addresses', () => {
+    const addresses = [foo];
+
+    it('if they are present directly on the resource status', () => {
+      const mgmtNode = new MgmtNode({ status: { addresses } });
+
+      expect(mgmtNode.addresses).toStrictEqual(addresses);
+    });
+    it('if they are not present directly on the resource status but are on "status.internalNodeStatus"', () => {
+      const mgmtNode = new MgmtNode({ status: { internalNodeStatus: { addresses } } });
+
+      expect(mgmtNode.addresses).toStrictEqual(addresses);
+    });
+  });
+
+  describe('should return an internalIp', () => {
+    const addresses = [{ type: 'InternalIP', address: foo }];
+    const internalAddress = foo;
+
+    it('if addresses includes an object with an appropriate type and address', () => {
+      const mgmtNode = new MgmtNode({ status: { addresses } });
+
+      expect(mgmtNode.internalIp).toStrictEqual(foo);
+    });
+    it('if internalNodeStatus.addresses includes an object with an appropriate type and address', () => {
+      const mgmtNode = new MgmtNode({ status: { internalNodeStatus: { addresses } } });
+
+      expect(mgmtNode.internalIp).toStrictEqual(foo);
+    });
+    it('if addresses and internalNodeStatus.addresses do not provide an internal ip and the status includes an rkeNode key with an appropriate type and address', () => {
+      const mgmtNode = new MgmtNode({ status: { rkeNode: { internalAddress } } });
+
+      expect(mgmtNode.internalIp).toStrictEqual(internalAddress);
+    });
+  });
+
+  describe('should return an externalIp', () => {
+    const addresses = [{ type: 'ExternalIP', address: foo }];
+    const address = foo;
+
+    it('if addresses includes an object with an appropriate type and address', () => {
+      const mgmtNode = new MgmtNode({ status: { addresses } });
+
+      expect(mgmtNode.externalIp).toStrictEqual(foo);
+    });
+    it('if internalNodeStatus.addresses includes an object with an appropriate type and address', () => {
+      const mgmtNode = new MgmtNode({ status: { internalNodeStatus: { addresses } } });
+
+      expect(mgmtNode.externalIp).toStrictEqual(foo);
+    });
+    it('if addresses and internalNodeStatus.addresses do not provide an external ip and the status includes an rkeNode key with an appropriate type and address', () => {
+      const mgmtNode = new MgmtNode({ status: { rkeNode: { address } } });
+
+      expect(mgmtNode.externalIp).toStrictEqual(address);
+    });
+  });
+
+  describe('should return an appropriate message', () => {
+    it('if there is no internalIp to display', () => {
+      const mgmtNode = new MgmtNode({ status: {} }, ctx);
+
+      expect(mgmtNode.internalIp).toStrictEqual(bar);
+      expect(t).toHaveBeenCalledTimes(1);
+      expect(t).toHaveBeenCalledWith('generic.none');
+      resetMocks();
+    });
+    it('if there is no externalIp to display', () => {
+      const mgmtNode = new MgmtNode({ status: {} }, ctx);
+
+      expect(mgmtNode.externalIp).toStrictEqual(bar);
+      expect(t).toHaveBeenCalledTimes(1);
+      expect(t).toHaveBeenCalledWith('generic.none');
+      resetMocks();
+    });
+  });
+});

package/models/__tests__/node.ts

@@ -0,0 +1,74 @@
+import Node from '@shell/models/management.cattle.io.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: {} });
+
+    expect(node.addresses).toStrictEqual([]);
+    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/cluster/node.js

@@ -92,16 +92,17 @@ export default class ClusterNode extends SteveModel {
     return this.metadata.name;
   }
 
-  get internalIp() {
-    const addresses = this.status?.addresses || [];
+  get addresses() {
+    return this.status?.addresses || [];
+  }
 
-    return findLast(addresses, (address) => address.type === 'InternalIP')?.address;
+  get internalIp() {
+    return findLast(this.addresses, (address) => address.type === 'InternalIP')?.address;
   }
 
   get externalIp() {
-    const addresses = this.status?.addresses || [];
     const annotationAddress = this.metadata.annotations[RKE.EXTERNAL_IP];
-    const statusAddress = findLast(addresses, (address) => address.type === 'ExternalIP')?.address;
+    const statusAddress = findLast(this.addresses, (address) => address.type === 'ExternalIP')?.address;
 
     return statusAddress || annotationAddress;
   }

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

@@ -127,11 +127,11 @@ export default class CapiMachineDeployment extends SteveModel {
     }
 
     this.scaleTimer = setTimeout(() => {
-      this.cluster.save().catch((err) => {
+      this.cluster.save().catch(async(err) => {
         let errors = exceptionToErrorsArray(err);
 
         if ( err.status === 409 && depth < 2 ) {
-          const conflicts = handleConflict(initialValue, value, liveModel, this.$rootGetters, this.$store);
+          const conflicts = await handleConflict(initialValue, value, liveModel, this.$rootGetters, { dispatch: this.$dispatch }, 'management');
 
           if ( conflicts === false ) {
             // It was automatically figured out, save again

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

@@ -1,7 +1,7 @@
 import Vue from 'vue';
 import { CATALOG, CLUSTER_BADGE } from '@shell/config/labels-annotations';
 import { NODE, FLEET, MANAGEMENT, CAPI } from '@shell/config/types';
-import { insertAt } from '@shell/utils/array';
+import { insertAt, addObject, removeObject } from '@shell/utils/array';
 import { downloadFile } from '@shell/utils/download';
 import { parseSi } from '@shell/utils/units';
 import { parseColor, textColor } from '@shell/utils/color';
@@ -14,6 +14,7 @@ import { isHarvesterCluster } from '@shell/utils/cluster';
 import HybridModel from '@shell/plugins/steve/hybrid-class';
 import { LINUX, WINDOWS } from '@shell/store/catalog';
 import { KONTAINER_TO_DRIVER } from './management.cattle.io.kontainerdriver';
+import { PINNED_CLUSTERS } from '@shell/store/prefs';
 
 // See translation file cluster.providers for list of providers
 // If the logo is not named with the provider name, add an override here
@@ -455,4 +456,24 @@ export default class MgmtCluster extends HybridModel {
 
     return findRelationship(verb === 'to' ? 'from' : 'to', CAPI.RANCHER_CLUSTER, this.metadata?.relationships);
   }
+
+  get pinned() {
+    return this.$rootGetters['prefs/get'](PINNED_CLUSTERS).includes(this.id);
+  }
+
+  pin() {
+    const types = this.$rootGetters['prefs/get'](PINNED_CLUSTERS) || [];
+
+    addObject(types, this.id);
+
+    this.$dispatch('prefs/set', { key: PINNED_CLUSTERS, value: types }, { root: true });
+  }
+
+  unpin() {
+    const types = this.$rootGetters['prefs/get'](PINNED_CLUSTERS) || [];
+
+    removeObject(types, this.id);
+
+    this.$dispatch('prefs/set', { key: PINNED_CLUSTERS, value: types }, { root: true });
+  }
 }

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

@@ -43,7 +43,7 @@ export default class CRTB extends HybridModel {
 
   get principalId() {
     // We've either set it ourselves or it's comes from native properties
-    return this.principalName || this.userPrincipalName || this.groupPrincipalName;
+    return this.principalName || this.userPrincipalName || this.groupPrincipalName || '';
   }
 
   get nameDisplay() {
@@ -117,12 +117,12 @@ export default class CRTB extends HybridModel {
   get norman() {
     return (async() => {
       const principal = await this.principal;
-      const principalProperty = principal.principalType === 'group' ? 'groupPrincipalId' : 'userPrincipalId';
+      const principalProperty = principal?.principalType === 'group' ? 'groupPrincipalId' : 'userPrincipalId';
 
       return this.$dispatch(`rancher/create`, {
         type:                NORMAN.CLUSTER_ROLE_TEMPLATE_BINDING,
         roleTemplateId:      this.roleTemplateName,
-        [principalProperty]: principal.id,
+        [principalProperty]: principal?.id,
         clusterId:           this.clusterName,
         id:                  this.id?.replace('/', ':')
       }, { root: true });

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

@@ -4,7 +4,6 @@ import { CATTLE_API_GROUP, SUBTYPE_MAPPING, CREATE_VERBS } from '@shell/models/m
 import { uniq } from '@shell/utils/array';
 import { get } from '@shell/utils/object';
 import SteveDescriptionModel from '@shell/plugins/steve/steve-description-class';
-import Role from './rbac.authorization.k8s.io.role';
 import { AS, MODE, _CLONE, _UNFLAG } from '@shell/config/query-params';
 
 const BASE = 'user-base';
@@ -16,7 +15,14 @@ const GLOBAL = SUBTYPE_MAPPING.GLOBAL.key;
 
 export default class GlobalRole extends SteveDescriptionModel {
   get customValidationRules() {
-    return Role.customValidationRules();
+    return [
+      {
+        path:       'rules',
+        validators: [`roleTemplateRules:${ this.type }`],
+        nullable:   false,
+        type:       'array',
+      },
+    ];
   }
 
   get details() {
@@ -137,6 +143,15 @@ export default class GlobalRole extends SteveDescriptionModel {
   async save() {
     const norman = await this.norman;
 
+    for (const rule of norman.rules) {
+      if (rule.nonResourceURLs.length) {
+        delete rule.resources;
+        delete rule.apiGroups;
+      } else {
+        delete rule.nonResourceURLs;
+      }
+    }
+
     return norman.save();
   }
 

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

@@ -125,11 +125,14 @@ export default class MgmtNode extends HybridModel {
     return false;
   }
 
+  get addresses() {
+    return this.status?.addresses || this.status?.internalNodeStatus?.addresses || [];
+  }
+
   get internalIp() {
     // This shows in the IP address column for RKE1 nodes in the
     // list of nodes in the cluster detail page of Cluster Management.
-
-    const internal = this.status?.addresses?.find(({ type }) => {
+    const internal = this.addresses.find(({ type }) => {
       return type === ADDRESSES.INTERNAL_IP;
     });
 
@@ -147,8 +150,7 @@ export default class MgmtNode extends HybridModel {
   }
 
   get externalIp() {
-    const addresses = this.status?.addresses || [];
-    const statusAddress = findLast(addresses, (address) => address.type === 'ExternalIP')?.address;
+    const statusAddress = findLast(this.addresses, (address) => address.type === 'ExternalIP')?.address;
 
     if (statusAddress) {
       return statusAddress;

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

@@ -31,7 +31,7 @@ export default class PRTB extends HybridModel {
 
   get principalId() {
     // We've either set it ourselves or it's comes from native properties
-    return this.principalName || this.userPrincipalName || this.groupPrincipalName;
+    return this.principalName || this.userPrincipalName || this.groupPrincipalName || '';
   }
 
   get nameDisplay() {
@@ -123,12 +123,12 @@ export default class PRTB extends HybridModel {
   get norman() {
     return (async() => {
       const principal = await this.principal;
-      const principalProperty = principal.principalType === 'group' ? 'groupPrincipalId' : 'userPrincipalId';
+      const principalProperty = principal?.principalType === 'group' ? 'groupPrincipalId' : 'userPrincipalId';
 
       return this.$dispatch(`rancher/create`, {
         type:                  NORMAN.PROJECT_ROLE_TEMPLATE_BINDING,
         roleTemplateId:        this.roleTemplateName,
-        [principalProperty]:   principal.id,
+        [principalProperty]:   principal?.id,
         projectId:             this.projectName,
         projectRoleTemplateId: '',
         id:                    this.id?.replace('/', ':')

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

@@ -3,7 +3,6 @@ import { get } from '@shell/utils/object';
 import { DESCRIPTION } from '@shell/config/labels-annotations';
 import { NORMAN } from '@shell/config/types';
 import SteveDescriptionModel from '@shell/plugins/steve/steve-description-class';
-import Role from './rbac.authorization.k8s.io.role';
 import { AS, MODE, _CLONE, _UNFLAG } from '@shell/config/query-params';
 
 export const CATTLE_API_GROUP = '.cattle.io';
@@ -60,7 +59,14 @@ export const CREATE_VERBS = new Set(['PUT', 'blocked-PUT']);
 
 export default class RoleTemplate extends SteveDescriptionModel {
   get customValidationRules() {
-    return Role.customValidationRules();
+    return [
+      {
+        path:       'rules',
+        validators: [`roleTemplateRules:${ this.type }`],
+        nullable:   false,
+        type:       'array',
+      },
+    ];
   }
 
   get details() {
@@ -185,6 +191,15 @@ export default class RoleTemplate extends SteveDescriptionModel {
   async save() {
     const norman = await this.norman;
 
+    for (const rule of norman.rules) {
+      if (rule.nonResourceURLs.length) {
+        delete rule.resources;
+        delete rule.apiGroups;
+      } else {
+        delete rule.nonResourceURLs;
+      }
+    }
+
     return norman.save();
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rancher/shell",
-  "version": "0.3.23",
+  "version": "0.3.25",
   "description": "Rancher Dashboard Shell",
   "repository": "https://github.com/rancherlabs/dashboard",
   "license": "Apache-2.0",
@@ -17,7 +17,6 @@
     "lint": "./node_modules/.bin/eslint --max-warnings 0 --ext .ts,.js,.vue .",
     "test": "./node_modules/.bin/nyc ava --serial --verbose",
     "dev": "./node_modules/.bin/vue-cli-service dev",
-    "docker-dev": "docker run --rm --name dashboard-dev -p 8005:8005 -e API=$API -v $(pwd):/src -v dashboard_node:/src/node_modules rancher/dashboard:dev",
     "build": "./node_modules/.bin/vue-cli-service build",
     "analyze": "./node_modules/.bin/vue-cli-service build --report",
     "start": "./node_modules/.bin/vue-cli-service start",
@@ -38,7 +37,7 @@
     "@novnc/novnc": "1.2.0",
     "@nuxt/types": "2.14.6",
     "@nuxt/typescript-build": "2.1.0",
-    "@nuxtjs/axios": "5.12.0",
+    "@nuxtjs/axios": "5.13.6",
     "@nuxtjs/eslint-config-typescript": "6.0.1",
     "@nuxtjs/webpack-profile": "0.1.0",
     "@popperjs/core": "2.4.4",
@@ -107,9 +106,6 @@
     "papaparse": "5.3.0",
     "portal-vue": "2.1.7",
     "rancher-icons": "rancher/icons#v2.0.16",
-    "require-extension-hooks": "0.3.3",
-    "require-extension-hooks-babel": "1.0.0",
-    "require-extension-hooks-vue": "3.0.0",
     "sass": "1.51.0",
     "sass-loader": "10.2.1",
     "serve-static": "1.14.1",

package/pages/__tests__/prefs.test.ts

@@ -1,5 +1,5 @@
-import { shallowMount } from '@vue/test-utils';
 import Preferences from '@shell/pages/prefs.vue';
+import { shallowMount } from '@vue/test-utils';
 
 describe('page: prefs should', () => {
   it.each([

package/pages/about.vue

@@ -101,6 +101,7 @@ export default {
       <n-link
         :to="{ name: 'diagnostic' }"
         class="btn role-primary"
+        data-testid="about__diagnostics_button"
       >
         {{ t('about.diagnostic.title') }}
       </n-link>
@@ -195,6 +196,7 @@ export default {
           <td>
             <a
               v-if="d.imageList"
+              :data-testid="`image_list_download_link__${d.label}`"
               @click="d.imageList"
             >
               {{ t('asyncButton.download.action') }}

package/pages/auth/setup.vue

@@ -17,6 +17,7 @@ import { exceptionToErrorsArray } from '@shell/utils/error';
 import Password from '@shell/components/form/Password';
 import { applyProducts } from '@shell/store/type-map';
 import BrandImage from '@shell/components/BrandImage';
+import { waitFor } from '@shell/utils/async';
 
 const calcIsFirstLogin = (store) => {
   const firstLoginSetting = store.getters['management/byId'](MANAGEMENT.SETTING, SETTING.FIRST_LOGIN);
@@ -250,10 +251,10 @@ export default {
 
         await Promise.all(promises);
 
-        setTimeout(() => {
-          buttonCb(true);
-          this.done();
-        }, 2000);
+        await waitFor(() => !calcIsFirstLogin(this.$store), 'first login to be completed', 10000, 1000, true);
+
+        buttonCb(true);
+        this.done();
       } catch (err) {
         console.error(err) ; // eslint-disable-line no-console
         buttonCb(false);

package/pages/c/_cluster/explorer/ConfigBadge.vue

@@ -27,6 +27,7 @@ export default {
     <div>
       <a
         class="badge-install"
+        data-testid="add-custom-cluster-badge"
         @click="customBadgeDialog"
       >
         <i class="icon icon-cluster" />

package/pages/c/_cluster/monitoring/index.vue

@@ -94,12 +94,11 @@ export default {
   methods: {
     async fetchDeps() {
       const { $store, externalLinks } = this;
-      const currentCluster = this.$store.getters['currentCluster'];
 
       this.v1Installed = await haveV1MonitoringWorkloads($store);
       const hash = await allHash({
+        apps:      $store.dispatch('cluster/findAll', { type: CATALOG.APP }),
         endpoints: $store.dispatch('cluster/findAll', { type: ENDPOINTS }),
-        app:       $store.dispatch(`cluster/find`, { type: CATALOG.APP, id: 'cattle-monitoring-system/rancher-monitoring' })
       });
 
       if (!isEmpty(hash.endpoints)) {
@@ -109,7 +108,13 @@ export default {
           (el) => el.group === 'prometheus'
         );
 
-        grafanaMatch.link = `${ getClusterPrefix(hash.app?.currentVersion || '', currentCluster.id) }/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy/`;
+        // Generate Grafana link
+        const currentCluster = this.$store.getters['currentCluster'];
+        const rancherMonitoring = !isEmpty(hash.apps) ? findBy(hash.apps, 'id', 'cattle-monitoring-system/rancher-monitoring') : '';
+        const clusterPrefix = getClusterPrefix(rancherMonitoring?.currentVersion || '', currentCluster.id);
+
+        grafanaMatch.link = `${ clusterPrefix }/api/v1/namespaces/cattle-monitoring-system/services/http:rancher-monitoring-grafana:80/proxy/`;
+
         const alertmanager = findBy(
           hash.endpoints,
           'id',

package/pages/c/_cluster/uiplugins/CatalogList/CatalogLoadDialog.vue

@@ -2,9 +2,7 @@
 import { mapGetters } from 'vuex';
 import isEmpty from 'lodash/isEmpty';
 
-import {
-  CATALOG, SECRET, SERVICE, UI_PLUGIN, WORKLOAD_TYPES
-} from '@shell/config/types';
+import { CATALOG, SECRET, SERVICE, WORKLOAD_TYPES } from '@shell/config/types';
 import { UI_PLUGIN_LABELS, UI_PLUGIN_NAMESPACE } from '@shell/config/uiplugins';
 import { TYPES as SECRET_TYPES } from '@shell/models/secret';
 import { allHash } from '@shell/utils/promise';
@@ -195,11 +193,15 @@ export default {
             }
 
             if (this.extensionRepo) {
-              // Create uiplugin crd
-              await this.loadPlugin(name, this.extensionUrl, image);
+              btnCb(true);
+              this.closeDialog();
+              this.$store.dispatch('growl/success', {
+                title:   this.t('plugins.manageCatalog.imageLoad.success.title', { name }),
+                message: this.t('plugins.manageCatalog.imageLoad.success.message'),
+                timeout: 4000,
+              }, { root: true });
+              this.$emit('refresh');
             }
-
-            btnCb(true);
           } else {
             throw new Error('Unable to determine image name');
           }
@@ -313,62 +315,6 @@ export default {
       }
     },
 
-    async loadPlugin(name, url, image, btnCb) {
-      // Try and parse version number from the image
-      const version = this.extractImageVersion(image) || 'latest';
-
-      if (!this.extractImageVersion(image)) {
-        this.$store.dispatch('growl/warning', {
-          title:   this.t('plugins.manageCatalog.imageLoad.imageVersion.title'),
-          message: this.t('plugins.manageCatalog.imageLoad.imageVersion.message', { image }),
-          timeout: 4000,
-        }, { root: true });
-      }
-
-      let crdName = name;
-
-      const parts = name.split('-');
-
-      if (parts.length >= 2) {
-        crdName = parts.join('-');
-      }
-
-      this.extensionCrd = await this.$store.dispatch('management/create', {
-        type:     UI_PLUGIN,
-        metadata: {
-          name,
-          namespace: UI_PLUGIN_NAMESPACE,
-          labels:    {
-            [UI_PLUGIN_LABELS.CATALOG_IMAGE]: name,
-            [UI_PLUGIN_LABELS.REPOSITORY]:    this.extensionRepo.metadata.name
-          }
-        },
-        spec: {
-          plugin: {
-            name:     crdName,
-            version,
-            endpoint: url,
-            noCache:  false,
-            metadata: { [UI_PLUGIN_LABELS.CATALOG]: 'true' }
-          }
-        }
-      });
-
-      try {
-        await this.extensionCrd.save({ url: `/v1/${ UI_PLUGIN }`, method: 'POST' });
-
-        this.closeDialog();
-        this.$store.dispatch('growl/success', {
-          title:   this.t('plugins.manageCatalog.imageLoad.success.title', { name }),
-          message: this.t('plugins.manageCatalog.imageLoad.success.message'),
-          timeout: 4000,
-        }, { root: true });
-      } catch (e) {
-        this.handleGrowlError(e, true);
-        btnCb(false);
-      }
-    },
-
     parseDeploymentValues(name) {
       let out = {};
 
@@ -459,9 +405,6 @@ export default {
       if (this.extensionRepo) {
         this.extensionRepo.remove();
       }
-      if (this.extensionCrd) {
-        this.extensionCrd.remove();
-      }
     },
 
     handleGrowlError(e, clean = false) {