@rancher/shell 3.0.9-rc.5 → 3.0.9-rc.6

package/edit/workload/__tests__/index.test.ts

@@ -68,6 +68,8 @@ describe('component: Workload', () => {
             $store:      {
               getters: {
                 'cluster/schemaFor': jest.fn(),
+                'cluster/canList':   jest.fn(),
+                currentStore:        () => 'cluster',
                 'type-map/labelFor': jest.fn(),
                 'i18n/t':            (text: string, v: {[key:string]: string}) => {
                   return `${ text }, ${ Object.values(v || {}) }`;
@@ -99,42 +101,5 @@ describe('component: Workload', () => {
 
       expect(result).toStrictEqual(newMessage);
     });
-
-    describe('secondaryResourceDataConfig', () => {
-      it('should filter out nodes with control-plane or etcd taints from workerNodes parsingFunc', () => {
-        const allNodeObjects = [
-          {
-            id:   'node-1',
-            spec: { taints: [{ key: 'node-role.kubernetes.io/control-plane', effect: 'NoSchedule' }] }
-          },
-          {
-            id:   'node-2',
-            spec: { taints: [{ key: 'node-role.kubernetes.io/etcd', effect: 'NoSchedule' }] }
-          },
-          {
-            id:   'node-3',
-            spec: { taints: [{ key: 'node-role.kubernetes.io/worker', effect: 'NoSchedule' }] }
-          },
-          {
-            id:   'node-4',
-            spec: { taints: [] }
-          },
-          {
-            id:   'node-5',
-            spec: {}
-          },
-          {
-            id:   'node-6',
-            spec: null
-          }
-        ];
-
-        const { data } = (Workload.mixins[2] as any).methods.secondaryResourceDataConfig.apply({ value: { metadata: { namespace: 'test' } } });
-        const workerNodesParsingFunc = data.node.applyTo.find((r: any) => r.var === 'workerNodes').parsingFunc;
-        const result = workerNodesParsingFunc(allNodeObjects);
-
-        expect(result).toStrictEqual(['node-3', 'node-4', 'node-5', 'node-6']);
-      });
-    });
   });
 });

package/edit/workload/index.vue

@@ -4,6 +4,7 @@ import FormValidation from '@shell/mixins/form-validation';
 import WorkLoadMixin from '@shell/edit/workload/mixins/workload';
 import { mapGetters } from 'vuex';
 import { FORM_TYPES } from '@shell/components/form/Security';
+import { NODE } from '@shell/config/types';
 
 export default {
   name:   'Workload',
@@ -21,9 +22,13 @@ export default {
     },
   },
   data() {
+    const inStore = this.$store.getters['currentStore'](NODE);
+    const canNode = this.$store.getters[`${ inStore }/canList`](NODE);
+
     return {
       selectedName: null,
       errors:       [],
+      canNode,
       FORM_TYPES
     };
   },
@@ -506,11 +511,11 @@ export default {
               <PodAffinity
                 :mode="mode"
                 :value="podTemplateSpec"
-                :nodes="allNodeObjects"
                 :loading="isLoadingSecondaryResources"
               />
             </Tab>
             <Tab
+              v-if="canNode"
               :label="t('workload.container.titles.nodeScheduling')"
               name="nodeScheduling-pod"
               :weight="tabWeightMap['nodeScheduling']"
@@ -518,7 +523,6 @@ export default {
               <NodeScheduling
                 :mode="mode"
                 :value="podTemplateSpec"
-                :nodes="workerNodes"
                 :loading="isLoadingSecondaryResources"
               />
             </Tab>

package/edit/workload/mixins/workload.js

@@ -5,7 +5,6 @@ import {
   CONFIG_MAP,
   SECRET,
   WORKLOAD_TYPES,
-  NODE,
   SERVICE,
   PVC,
   SERVICE_ACCOUNT,
@@ -263,9 +262,6 @@ export default {
     return {
       secondaryResourceData:      this.secondaryResourceDataConfig(),
       namespacedConfigMaps:       [],
-      allNodes:                   null,
-      workerNodes:                null,
-      allNodeObjects:             [],
       namespacedSecrets:          [],
       imagePullNamespacedSecrets: [],
       allServices:                [],
@@ -690,34 +686,6 @@ export default {
               }
             ]
           },
-          [NODE]: {
-            applyTo: [
-              { var: 'allNodeObjects' },
-              {
-                var:         'allNodes',
-                parsingFunc: (data) => {
-                  return data.map((node) => node.id);
-                }
-              },
-              {
-                var:         'workerNodes',
-                parsingFunc: (data) => {
-                  const keys = [
-                    `node-role.kubernetes.io/control-plane`,
-                    `node-role.kubernetes.io/etcd`
-                  ];
-
-                  return data
-                    .filter((node) => {
-                      const taints = node?.spec?.taints || [];
-
-                      return taints.every((taint) => !keys.includes(taint.key));
-                    })
-                    .map((node) => node.id);
-                }
-              },
-            ]
-          },
           [SERVICE]: {
             applyTo: [
               { var: 'allServices' },

package/list/__tests__/management.cattle.io.setting.test.ts

@@ -0,0 +1,198 @@
+import { mount } from '@vue/test-utils';
+import { nextTick } from 'vue';
+import ManagementSetting from '@shell/list/management.cattle.io.setting.vue';
+import { SETTING } from '@shell/config/settings';
+
+const mockStore = {
+  getters: {
+    'prefs/get': () => false,
+    'i18n/t':    (key: string) => key,
+  },
+  dispatch: jest.fn()
+};
+
+const mockRoute = {
+  hash:   '',
+  params: {}
+};
+
+const mockRouter = {
+  push:    jest.fn(),
+  replace: jest.fn()
+};
+
+// Mock scrollIntoView
+const mockScrollIntoView = jest.fn();
+
+Object.defineProperty(Element.prototype, 'scrollIntoView', {
+  value:    mockScrollIntoView,
+  writable: true
+});
+
+// Mock querySelector
+const mockQuerySelector = jest.fn();
+
+Object.defineProperty(document, 'querySelector', {
+  value:    mockQuerySelector,
+  writable: true
+});
+
+describe('managementSetting - Scroll Behavior', () => {
+  let wrapper: any;
+
+  const createWrapper = (routeHash = '') => {
+    const route = { ...mockRoute, hash: routeHash };
+
+    return mount(ManagementSetting, {
+      global: {
+        mocks: {
+          $store:      mockStore,
+          $route:      route,
+          $router:     mockRouter,
+          $fetchState: { pending: false }
+        },
+        stubs: {
+          Loading: true,
+          Banner:  true,
+          Setting: true
+        }
+      }
+    });
+  };
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+    mockScrollIntoView.mockClear();
+    mockQuerySelector.mockClear();
+
+    // Mock successful API response with cluster agent setting
+    mockStore.dispatch.mockResolvedValue([
+      {
+        id:               SETTING.CLUSTER_AGENT_DEFAULT_PRIORITY_CLASS,
+        value:            '{"enabled": true}',
+        default:          '{"enabled": false}',
+        availableActions: ['edit']
+      },
+      {
+        id:               SETTING.FLEET_AGENT_DEFAULT_PRIORITY_CLASS,
+        value:            '{"enabled": true}',
+        default:          '{"enabled": false}',
+        availableActions: ['edit']
+      }
+    ]);
+  });
+
+  afterEach(() => {
+    if (wrapper) {
+      wrapper.unmount();
+    }
+  });
+
+  it('should scroll to hash anchor after data is loaded', async() => {
+    const mockElement = document.createElement('div');
+
+    mockQuerySelector.mockReturnValue(mockElement);
+
+    wrapper = createWrapper('#cluster-agent-default-priority-class');
+
+    // Trigger fetch manually since we can't easily test the async fetch hook
+    await wrapper.vm.$options.fetch.call(wrapper.vm);
+    await nextTick();
+
+    expect(mockQuerySelector).toHaveBeenCalledWith('#cluster-agent-default-priority-class');
+    expect(mockScrollIntoView).toHaveBeenCalledWith({
+      behavior: 'smooth',
+      block:    'start'
+    });
+  });
+
+  it('should not scroll if hash is empty', async() => {
+    wrapper = createWrapper('');
+
+    await wrapper.vm.$options.fetch.call(wrapper.vm);
+    await nextTick();
+
+    expect(mockQuerySelector).not.toHaveBeenCalled();
+    expect(mockScrollIntoView).not.toHaveBeenCalled();
+  });
+
+  it('should not scroll if element is not found', async() => {
+    mockQuerySelector.mockReturnValue(null);
+
+    wrapper = createWrapper('#non-existent-element');
+
+    await wrapper.vm.$options.fetch.call(wrapper.vm);
+    await nextTick();
+
+    expect(mockQuerySelector).toHaveBeenCalledWith('#non-existent-element');
+    expect(mockScrollIntoView).not.toHaveBeenCalled();
+  });
+
+  it('should scroll to fleet agent setting hash', async() => {
+    const mockElement = document.createElement('div');
+
+    mockQuerySelector.mockReturnValue(mockElement);
+
+    wrapper = createWrapper('#fleet-agent-default-priority-class');
+
+    await wrapper.vm.$options.fetch.call(wrapper.vm);
+    await nextTick();
+
+    expect(mockQuerySelector).toHaveBeenCalledWith('#fleet-agent-default-priority-class');
+    expect(mockScrollIntoView).toHaveBeenCalledWith({
+      behavior: 'smooth',
+      block:    'start'
+    });
+  });
+
+  it('should handle hash with special characters', async() => {
+    const mockElement = document.createElement('div');
+
+    mockQuerySelector.mockReturnValue(mockElement);
+
+    wrapper = createWrapper('#setting-with-special-chars_123');
+
+    await wrapper.vm.$options.fetch.call(wrapper.vm);
+    await nextTick();
+
+    expect(mockQuerySelector).toHaveBeenCalledWith('#setting-with-special-chars_123');
+    expect(mockScrollIntoView).toHaveBeenCalledWith({
+      behavior: 'smooth',
+      block:    'start'
+    });
+  });
+
+  it('should render settings with correct IDs for anchors', async() => {
+    wrapper = createWrapper();
+
+    // Manually set the data to simulate successful fetch
+    wrapper.vm.settings = [
+      {
+        id:          SETTING.CLUSTER_AGENT_DEFAULT_PRIORITY_CLASS,
+        description: 'test-description',
+        data:        { value: 'test' },
+        customized:  false,
+        fromEnv:     false,
+        hasActions:  true
+      }
+    ];
+    wrapper.vm.provisioningSettings = [
+      {
+        id:          SETTING.FLEET_AGENT_DEFAULT_PRIORITY_CLASS,
+        description: 'test-description',
+        data:        { value: 'test' },
+        customized:  false,
+        fromEnv:     false,
+        hasActions:  true
+      }
+    ];
+
+    await nextTick();
+
+    const settingDivs = wrapper.findAll('div[id]');
+    const ids = settingDivs.map((div: any) => div.attributes('id'));
+
+    expect(ids).toContain(SETTING.CLUSTER_AGENT_DEFAULT_PRIORITY_CLASS);
+    expect(ids).toContain(SETTING.FLEET_AGENT_DEFAULT_PRIORITY_CLASS);
+  });
+});

package/list/management.cattle.io.setting.vue

@@ -66,6 +66,17 @@ export default {
 
     this.settings = settings;
     this.provisioningSettings = provisioningSettings;
+
+    this.$nextTick(() => {
+      // Handle scrolling to hash anchor after data is loaded
+      if (this.$route.hash) {
+        const element = document.querySelector(this.$route.hash);
+
+        if (element) {
+          element.scrollIntoView({ behavior: 'smooth', block: 'start' });
+        }
+      }
+    });
   },
 
   data() {
@@ -89,6 +100,7 @@ export default {
     </Banner>
     <div
       v-for="(setting) in settings"
+      :id="setting.id"
       :key="setting.id"
     >
       <Setting
@@ -101,6 +113,7 @@ export default {
     </h2>
     <div
       v-for="(setting) in provisioningSettings"
+      :id="setting.id"
       :key="setting.id"
     >
       <Setting

package/list/provisioning.cattle.io.cluster.vue

@@ -167,6 +167,12 @@ export default {
 
   },
 
+  methods: {
+    getCustomDetailLink(cluster) {
+      return cluster.isCapiHybrid ? null : cluster.detailLocation;
+    }
+  },
+
   $loadingResources() {
     // results are filtered so we wouldn't get the correct count on indicator...
     return { loadIndeterminate: true };
@@ -211,7 +217,6 @@ export default {
         </router-link>
       </template>
     </Masthead>
-
     <ResourceTable
       :headers="headers"
       :table-actions="true"
@@ -221,6 +226,8 @@ export default {
       :use-query-params-for-simple-filtering="useQueryParamsForSimpleFiltering"
       :data-testid="'cluster-list'"
       :force-update-live-and-delayed="forceUpdateLiveAndDelayed"
+      :get-custom-detail-link="getCustomDetailLink"
+      :sub-rows="true"
     >
       <!-- Why are state column and subrow overwritten here? -->
       <!-- for rke1 clusters, where they try to use the mgmt cluster stateObj instead of prov cluster stateObj,  -->
@@ -266,6 +273,48 @@ export default {
           {{ t('cluster.explore') }}
         </button>
       </template>
+      <template #additional-sub-row="{row, fullColspan, tableActions}">
+        <tr
+          class="capi-unsupported"
+          :class="{'has-description': !!row.stateDescription}"
+        >
+          <td
+            v-if="row.isCapiHybrid"
+            class="row-check"
+          />
+          <td
+            v-if="row.isCapiHybrid"
+            :data-testid="`capi-unsupported-warning-${row?.metadata?.name}`"
+            :colspan="fullColspan - (tableActions ? 1: 0)"
+          >
+            <div
+              class="text-warning"
+              :class="{'mt-5': !!row.stateDescription.trim()}"
+            >
+              <i class="icon icon-warning" />{{ t('cluster.capi.notSupported') }}
+            </div>
+          </td>
+        </tr>
+      </template>
     </ResourceTable>
   </div>
 </template>
+
+<stye scoped lang="scss">
+  .capi-unsupported {
+    &.has-description {
+      border-bottom: none;
+      padding: 0px;
+      td {
+        padding-top: 0px;
+      }
+    }
+
+    & div {
+      & i {
+        margin-right: 0.1em;
+      }
+      display: flex;
+    }
+  }
+</stye>

package/list/secret.vue

@@ -7,8 +7,8 @@ import PaginatedResourceTable from '@shell/components/PaginatedResourceTable';
 import { TableColumn } from '@shell/types/store/type-map';
 import ResourceFetch from '@shell/mixins/resource-fetch';
 import { mapGetters } from 'vuex';
-import { SECRET_CLONE, SECRET_PROJECT_SCOPED } from '@shell/config/table-headers';
-import { STEVE_SECRET_CLONE } from '@shell/config/pagination-table-headers';
+import { SECRET_ORIGIN } from '@shell/config/table-headers';
+import { STEVE_SECRET_ORIGIN } from '@shell/config/pagination-table-headers';
 
 export default {
   name: 'ListSecret',
@@ -61,13 +61,8 @@ export default {
 
     if (this.canViewProjects) {
       // if the user can see projects, add a column to let them know if it's a secret from a project scoped secret
-      headers.push(SECRET_CLONE);
-      headersSSP.push(STEVE_SECRET_CLONE);
-      if (this.currentCluster.isLocal) {
-        // if the user is on the local cluster, add a column to let them know if it's a project scoped secret (from another cluster)
-        headers.push(SECRET_PROJECT_SCOPED);
-        headersSSP.push(SECRET_PROJECT_SCOPED);
-      }
+      headers.push(SECRET_ORIGIN);
+      headersSSP.push(STEVE_SECRET_ORIGIN);
     }
 
     headers.push(this.namespacedHeaders[this.namespacedHeaders.length - 1]);

package/list/service.vue

@@ -1,6 +1,6 @@
 <script>
 import PaginatedResourceTable from '@shell/components/PaginatedResourceTable';
-import { NODE } from '@shell/config/types';
+import { fetchNodesForServiceTargets } from '@shell/models/service';
 
 export default {
   name:       'ListService',
@@ -27,13 +27,11 @@ export default {
      * of type PagTableFetchSecondaryResources
      */
     async fetchSecondaryResources(opts) {
-      const inStore = this.$store.getters['currentStore']();
-
-      if (this.$store.getters[`${ inStore }/schemaFor`](NODE)) {
-        // fetch nodes before loading this page, as they may be referenced in the Target table column
-        // shell/components/formatter/ServiceTargets.vue --> shell/components/formatter/Endpoints.vue --> Picks the first one that has a model's externalIp
-        await this.$store.dispatch(`${ inStore }/findAll`, { type: NODE });
-      }
+      // Nodes should be fetched because they may be referenced in the target column of a service list item.
+      await fetchNodesForServiceTargets({
+        $store:  this.$store,
+        inStore: this.$store.getters['currentStore']()
+      });
     }
   }
 };

package/machine-config/amazonec2.vue

@@ -24,7 +24,7 @@ export default {
 
   mixins: [CreateEditView],
 
-  emits: ['validationChanged', 'update:hasIpv6'],
+  emits: ['validationChanged', 'update:isIpv6', 'update:isDualStack'],
 
   props: {
     uuid: {
@@ -47,7 +47,12 @@ export default {
       default: false
     },
 
-    hasIpv6: {
+    isIpv6: {
+      type:    Boolean,
+      default: false
+    },
+
+    isDualStack: {
       type:    Boolean,
       default: false
     },
@@ -386,10 +391,12 @@ export default {
           :zone="value.zone"
           :region="value.region"
           :machine-pools="machinePools"
-          :has-ipv6="hasIpv6"
+          :is-ipv6="isIpv6"
+          :is-dual-stack="isDualStack"
           :disabled="disabled"
           :is-new="poolCreateMode"
-          @update:has-ipv6="e=>$emit('update:hasIpv6', e)"
+          @update:is-ipv6="e=>$emit('update:isIpv6', e)"
+          @update:is-dual-stack="e=>$emit('update:isDualStack', e)"
           @validation-changed="e=>$emit('validationChanged',e)"
         />