MGMT-17540: Remove unnecessary calls to openshift-versions API in clu…

…ster creation page [FOLLOWUP] (#2638) * Fix issues with the Openshift versions context in Day2 * Fix day2 tests
openshift-assisted · Jul 12, 2024 · 6402f88
1 parent 7267cff
commit 6402f88
Show file tree

Hide file tree

Showing 5 changed files with 48 additions and 40 deletions.
diff --git a/libs/ui-lib-tests/cypress/integration/day-2/1-first-access.cy.ts b/libs/ui-lib-tests/cypress/integration/day-2/1-first-access.cy.ts
@@ -39,8 +39,6 @@ describe(`Day2 flow`, () => {
     });
 
     it('Creates a Day2 cluster when the cluster has metrics', () => {
-      Cypress.env('OPENSHIFT_VERSIONS_DELAY', true);
-
       cy.findByRole('button', { name: 'Add hosts (With metrics)' }).click();
       cy.wait('@create-day2-cluster').then(() => {
         setLastWizardSignal('CREATED_DAY2_CLUSTER');

diff --git a/libs/ui-lib-tests/cypress/support/interceptors.ts b/libs/ui-lib-tests/cypress/support/interceptors.ts
@@ -366,7 +366,6 @@ const addPlatformFeatureIntercepts = () => {
   cy.intercept('GET', '/api/assisted-install/v2/openshift-versions*', (req) => {
     req.reply({
       body: fixtures.openShiftVersions,
-      delay: Cypress.env('OPENSHIFT_VERSIONS_DELAY') ? 3 * 1000 : 0,
     });
   }).as('openshift-versions');
 

diff --git a/libs/ui-lib/lib/ocm/components/HostsClusterDetailTab/HostsClusterDetailTabContent.tsx b/libs/ui-lib/lib/ocm/components/HostsClusterDetailTab/HostsClusterDetailTabContent.tsx
@@ -23,6 +23,7 @@ import {
 import useInfraEnv from '../../hooks/useInfraEnv';
 import { mapOcmArchToCpuArchitecture } from '../../services/CpuArchitectureService';
 import { Cluster } from '@openshift-assisted/types/assisted-installer-service';
+import { OpenshiftVersionsContextProvider } from '../clusterWizard/OpenshiftVersionsContext';
 
 export const HostsClusterDetailTabContent = ({
   cluster: ocmCluster,
@@ -155,15 +156,17 @@ export const HostsClusterDetailTabContent = ({
       ocpConsoleUrl={ocmCluster?.console?.url}
       canEdit={ocmCluster.canEdit}
     >
-      <NewFeatureSupportLevelProvider
-        loadingUi={<LoadingState />}
-        cluster={day2Cluster}
-        cpuArchitecture={infraEnv?.cpuArchitecture as CpuArchitecture}
-        openshiftVersion={day2Cluster.openshiftVersion}
-        platformType={day2Cluster.platform?.type}
-      >
-        <AddHosts />
-      </NewFeatureSupportLevelProvider>
+      <OpenshiftVersionsContextProvider>
+        <NewFeatureSupportLevelProvider
+          loadingUi={<LoadingState />}
+          cluster={day2Cluster}
+          cpuArchitecture={infraEnv?.cpuArchitecture as CpuArchitecture}
+          openshiftVersion={day2Cluster.openshiftVersion}
+          platformType={day2Cluster.platform?.type}
+        >
+          <AddHosts />
+        </NewFeatureSupportLevelProvider>
+      </OpenshiftVersionsContextProvider>
     </AddHostsContextProvider>
   );
 };
diff --git a/libs/ui-lib/lib/ocm/components/clusterDetail/AssistedInstallerDetailCard.tsx b/libs/ui-lib/lib/ocm/components/clusterDetail/AssistedInstallerDetailCard.tsx
@@ -30,6 +30,7 @@ import { usePullSecret } from '../../hooks';
 import { Cluster } from '@openshift-assisted/types/assisted-installer-service';
 import { useFeatureDetection } from '../../hooks/use-feature-detection';
 import { BrowserRouter } from 'react-router-dom';
+import { OpenshiftVersionsContextProvider } from '../clusterWizard/OpenshiftVersionsContext';
 
 type AssistedInstallerDetailCardProps = {
   aiClusterId: string;
@@ -138,28 +139,30 @@ const AssistedInstallerDetailCard = ({
   const isOutdatedClusterData = uiState === ResourceUIState.POLLING_ERROR;
   return (
     <AlertsContextProvider>
-      <SentryErrorMonitorContextProvider>
-        <ModalDialogsContextProvider>
-          <ClusterDefaultConfigurationProvider
-            loadingUI={<LoadingCard />}
-            errorUI={<LoadingDefaultConfigFailedCard />}
-          >
-            <NewFeatureSupportLevelProvider
-              loadingUi={<LoadingCard />}
-              cluster={cluster}
-              cpuArchitecture={infraEnv.cpuArchitecture as CpuArchitecture}
-              openshiftVersion={cluster.openshiftVersion}
-              platformType={cluster.platform?.type}
+      <OpenshiftVersionsContextProvider>
+        <SentryErrorMonitorContextProvider>
+          <ModalDialogsContextProvider>
+            <ClusterDefaultConfigurationProvider
+              loadingUI={<LoadingCard />}
+              errorUI={<LoadingDefaultConfigFailedCard />}
             >
-              {content}
-            </NewFeatureSupportLevelProvider>
-            {isOutdatedClusterData && <ClusterPollingErrorModal />}
-            <CancelInstallationModal />
-            <ResetClusterModal />
-            <DiscoveryImageModal />
-          </ClusterDefaultConfigurationProvider>
-        </ModalDialogsContextProvider>
-      </SentryErrorMonitorContextProvider>
+              <NewFeatureSupportLevelProvider
+                loadingUi={<LoadingCard />}
+                cluster={cluster}
+                cpuArchitecture={infraEnv.cpuArchitecture as CpuArchitecture}
+                openshiftVersion={cluster.openshiftVersion}
+                platformType={cluster.platform?.type}
+              >
+                {content}