chore: 8499 remove deprecated migrateTemplateDisablement (opendatahub…

…-io#3114) Signed-off-by: gitdallas <[email protected]>
RHEcosystemAppEng · Aug 26, 2024 · fe74438 · fe74438
1 parent b8b44ea
commit fe74438
Showing 1 changed file with 0 additions and 80 deletions.
diff --git a/backend/src/utils/resourceUtils.ts b/backend/src/utils/resourceUtils.ts
@@ -18,7 +18,6 @@ import {
   SubscriptionKind,
   SubscriptionStatusData,
   Template,
-  TemplateList,
   TolerationEffect,
   TolerationOperator,
 } from '../types';
@@ -67,7 +66,6 @@ const DASHBOARD_CONFIG = {
 
 const fetchDashboardCR = async (fastify: KubeFastifyInstance): Promise<DashboardConfig[]> => {
   return fetchOrCreateDashboardCR(fastify)
-    .then((dashboardCR) => migrateTemplateDisablement(fastify, dashboardCR))
     .then((dashboardCR) => migrateBiasTrustyForRHOAI(fastify, dashboardCR))
     .then((dashboardCR) => [dashboardCR]);
 };
@@ -862,84 +860,6 @@ export const cleanupDSPSuffix = async (fastify: KubeFastifyInstance): Promise<vo
   });
 };
 
-/**
- * @deprecated - Look to remove asap (see comments below)
- * Migrate the template enablement from a current annotation in the Template Object to a list in ODHDashboardConfig
- * We are migrating this from Dashboard v2.11.0, so we can remove this code when we no longer support that version.
- */
-export const migrateTemplateDisablement = async (
-  fastify: KubeFastifyInstance,
-  dashboardConfig: DashboardConfig,
-): Promise<DashboardConfig> => {
-  if (dashboardConfig.spec.templateDisablement) {
-    return dashboardConfig;
-  }
-
-  const namespace = fastify.kube.namespace;
-  return fastify.kube.customObjectsApi
-    .listNamespacedCustomObject(
-      'template.openshift.io',
-      'v1',
-      namespace,
-      'templates',
-      undefined,
-      undefined,
-      undefined,
-      'opendatahub.io/dashboard=true',
-    )
-    .then((response) => response.body as TemplateList)
-    .then((templateList) => {
-      const templatesDisabled = templateList.items
-        .filter(
-          (template) =>
-            template.metadata.annotations?.['opendatahub.io/template-enabled'] === 'false',
-        )
-        .map((template) => getServingRuntimeNameFromTemplate(template));
-      if (templatesDisabled.length > 0) {
-        const options = {
-          headers: { 'Content-type': PatchUtils.PATCH_FORMAT_JSON_PATCH },
-        };
-
-        return fastify.kube.customObjectsApi
-          .patchNamespacedCustomObject(
-            DASHBOARD_CONFIG.group,
-            DASHBOARD_CONFIG.version,
-            dashboardConfig.metadata.namespace,
-            DASHBOARD_CONFIG.plural,
-            dashboardConfig.metadata.name,
-            [
-              {
-                op: 'replace',
-                path: '/spec/templateDisablement',
-                value: templatesDisabled,
-              },
-            ],
-            undefined,
-            undefined,
-            undefined,
-            options,
-          )
-          .then(() => {
-            return {
-              ...dashboardConfig,
-              spec: {
-                ...dashboardConfig.spec,
-                templateDisablement: templatesDisabled,
-              },
-            };
-          });
-      } else {
-        return dashboardConfig;
-      }
-    })
-    .catch((e) => {
-      fastify.log.error(
-        `Error migrating template disablement: ${e.response?.body?.message || e.message || e}`,
-      );
-      return dashboardConfig;
-    });
-};
-
 /**
  * TODO: There should be a better way to go about this... but the namespace is unlikely to ever change
  */