@openshift-assisted/ui-lib 2.16.5-cim → 2.17.0-cim

package/lib/cim/components/modals/CimConfiguration/persist.ts

@@ -1,17 +1,21 @@
-import { K8sResourceCommon } from '@openshift-console/dynamic-plugin-sdk';
+import {
+  k8sCreate,
+  k8sGet,
+  k8sListItems,
+  k8sPatch,
+  K8sResourceCommon,
+} from '@openshift-console/dynamic-plugin-sdk';
 import { AlertVariant } from '@patternfly/react-core';
 import { TFunction } from 'i18next';
 import { getErrorMessage } from '../../../../common/utils';
 
 import {
   AgentServiceConfigK8sResource,
-  convertOCPtoCIMResourceHeader,
-  CreateResourceFuncType,
-  GetResourceFuncType,
-  ListResourcesFuncType,
-  PatchResourceFuncType,
-  ResourcePatch,
+  AgentServiceConfigModel,
+  IngressControllerModel,
+  ProvisioningModel,
   RouteK8sResource,
+  RouteModel,
 } from '../../../types';
 import { LOCAL_STORAGE_ID_LAST_UPDATE_TIMESTAMP } from './utils';
 
@@ -32,14 +36,10 @@ const ASSISTED_IMAGE_SERVICE_ROUTE_PREFIX = 'assisted-image-service-multicluster
 const getAssistedImageServiceRoute = async (
   t: TFunction,
   setError: SetErrorFuncType,
-  listResources: ListResourcesFuncType,
 ): Promise<RouteK8sResource | undefined> => {
-  let allRoutes;
+  let allRoutes: RouteK8sResource[] = [];
   try {
-    allRoutes = (await listResources({
-      kind: 'Route',
-      apiVersion: 'route.openshift.io/v1',
-    })) as RouteK8sResource[];
+    allRoutes = await k8sListItems({ model: RouteModel, queryParams: {} });
   } catch (e) {
     // console.error('Failed to read all routes: ', allRoutes);
     setError({
@@ -50,7 +50,7 @@ const getAssistedImageServiceRoute = async (
   }
 
   const assistedImageServiceRoute = allRoutes?.find(
-    (r: RouteK8sResource) => r.metadata?.name === 'assisted-image-service',
+    (r) => r.metadata?.name === 'assisted-image-service',
   );
 
   if (!assistedImageServiceRoute?.spec?.host) {
@@ -92,7 +92,6 @@ const getClusterDomain = (
 const patchAssistedImageServiceRoute = async (
   t: TFunction,
   setError: SetErrorFuncType,
-  patchResource: PatchResourceFuncType,
 
   assistedImageServiceRoute: RouteK8sResource,
   domain: string,
@@ -102,7 +101,7 @@ const patchAssistedImageServiceRoute = async (
   const labels = assistedImageServiceRoute.metadata?.labels || {};
   labels['router-type'] = 'nlb';
 
-  const patches: ResourcePatch[] = [
+  const patches = [
     {
       op: 'replace',
       path: '/spec/host',
@@ -116,7 +115,11 @@ const patchAssistedImageServiceRoute = async (
   ];
 
   try {
-    await patchResource(convertOCPtoCIMResourceHeader(assistedImageServiceRoute), patches);
+    await k8sPatch({
+      model: RouteModel,
+      resource: assistedImageServiceRoute,
+      data: patches,
+    });
   } catch (e) {
     // console.error('Failed to patch assisted-image-service route: ', e, patches);
     setError({
@@ -128,15 +131,12 @@ const patchAssistedImageServiceRoute = async (
   return true;
 };
 
-export const isIngressController = async (getResource: GetResourceFuncType): Promise<boolean> => {
+export const isIngressController = async (): Promise<boolean> => {
   try {
-    await getResource({
-      apiVersion: 'operator.openshift.io/v1',
-      kind: 'IngressController',
-      metadata: {
-        name: 'ingress-controller-with-nlb',
-        namespace: 'openshift-ingress-operator',
-      },
+    await k8sGet({
+      name: 'ingress-controller-with-nlb',
+      ns: 'openshift-ingress-operator',
+      model: IngressControllerModel,
     });
 
     return true;
@@ -148,7 +148,6 @@ export const isIngressController = async (getResource: GetResourceFuncType): Pro
 const createIngressController = async (
   t: TFunction,
   setError: SetErrorFuncType,
-  createResource: CreateResourceFuncType,
 
   domain: string,
 ): Promise<boolean> => {
@@ -184,7 +183,7 @@ const createIngressController = async (
   };
 
   try {
-    await createResource(ingressController);
+    await k8sCreate({ model: IngressControllerModel, data: ingressController });
     return true;
   } catch (e) {
     // console.error('Create IngressController error: ', e);
@@ -199,24 +198,19 @@ const createIngressController = async (
 const patchProvisioningConfiguration = async ({
   t,
   setError,
-  patchResource,
-  getResource,
 }: {
   t: TFunction;
   setError: SetErrorFuncType;
-  patchResource: PatchResourceFuncType;
-  getResource: GetResourceFuncType;
 }) => {
   try {
-    const provisioning = (await getResource({
-      kind: 'Provisioning',
-      apiVersion: 'metal3.io/v1alpha1',
-      metadata: {
-        name: 'provisioning-configuration',
-      },
-    })) as K8sResourceCommon & { spec?: { watchAllNamespaces?: boolean } };
+    const provisioning = await k8sGet<
+      K8sResourceCommon & { spec?: { watchAllNamespaces?: boolean } }
+    >({
+      model: ProvisioningModel,
+      name: 'provisioning-configuration',
+    });
 
-    const patches: ResourcePatch[] = [
+    const patches = [
       {
         op: provisioning.spec?.watchAllNamespaces ? 'replace' : 'add',
         path: '/spec/watchAllNamespaces',
@@ -224,7 +218,11 @@ const patchProvisioningConfiguration = async ({
       },
     ];
 
-    await patchResource(convertOCPtoCIMResourceHeader(provisioning), patches);
+    await k8sPatch({
+      model: ProvisioningModel,
+      resource: provisioning,
+      data: patches,
+    });
   } catch (e) {
     // console.error('Failed to patch provisioning-configuration: ', e);
     setError({
@@ -238,18 +236,18 @@ const patchProvisioningConfiguration = async ({
 const createAgentServiceConfig = async ({
   t,
   setError,
-  createResource,
   dbVolSizeGiB,
   fsVolSizeGiB,
   imgVolSizeGiB,
+  ciscoIntersightURL,
 }: {
   t: TFunction;
   setError: SetErrorFuncType;
-  createResource: CreateResourceFuncType;
 
   dbVolSizeGiB: number;
   fsVolSizeGiB: number;
   imgVolSizeGiB: number;
+  ciscoIntersightURL?: string;
 }): Promise<boolean> => {
   try {
     const agentServiceConfig = {
@@ -257,6 +255,7 @@ const createAgentServiceConfig = async ({
       kind: 'AgentServiceConfig',
       metadata: {
         name: 'agent',
+        annotations: {},
       },
       spec: {
         databaseStorage: {
@@ -286,79 +285,32 @@ const createAgentServiceConfig = async ({
       },
     };
 
-    await createResource(agentServiceConfig);
-    return true;
-  } catch (e) {
-    // console.error('Failed to create AgentServiceConfig: ', e);
-    setError({
-      title: t('ai:Failed to create AgentServiceConfig'),
-      message: getErrorMessage(e),
-    });
-    return false;
-  }
-};
+    if (ciscoIntersightURL) {
+      agentServiceConfig.metadata = {
+        ...agentServiceConfig.metadata,
+        annotations: { ciscoIntersightURL },
+      };
+    }
 
-/* Following functions are tested but recently not used.
-const patchAgentServiceConfig = async ({
-  t,
-  setError,
-  patchResource,
-  agentServiceConfig,
-  imgVolSizeGB,
-}: {
-  t: TFunction;
-  setError: SetErrorFuncType;
-  patchResource: PatchResourceFuncType;
-  agentServiceConfig: AgentServiceConfigK8sResource;
+    await k8sCreate({
+      model: AgentServiceConfigModel,
+      data: agentServiceConfig,
+    });
 
-  imgVolSizeGB: number;
-}): Promise<boolean> => {
-  try {
-    const patches: ResourcePatch[] = [
-      {
-        op: 'replace',
-        path: '/spec/imageStorage/resources/requests/storage',
-        value: `${imgVolSizeGB}G`,
-      },
-    ];
-    await patchResource(agentServiceConfig, patches);
     return true;
   } catch (e) {
-    console.error('Failed to patch AgentServiceConfig: ', e);
     setError({
-      title: t('ai:Failed to update the AgentServiceConfig'),
+      title: t('ai:Failed to create AgentServiceConfig'),
+      message: getErrorMessage(e),
     });
     return false;
   }
 };
 
-export const onDeleteCimConfig = async ({
-  deleteResource,
-}: {
-  deleteResource: DeleteResourceFuncType;
-}) => {
-  try {
-    await deleteResource({
-      apiVersion: 'agent-install.openshift.io/v1beta1',
-      kind: 'AgentServiceConfig',
-      metadata: {
-        name: 'agent',
-        // cluster-scoped resource
-      },
-    });
-  } catch (e) {
-    console.error('Failed to delete AgentServiceConfig: ', e);
-  }
-};
-*/
 // https://access.redhat.com/documentation/en-us/red_hat_advanced_cluster_management_for_kubernetes/2.6/html/multicluster_engine/multicluster_engine_overview#enable-cim
 export const onEnableCIM = async ({
   t,
   setError,
-  createResource,
-  getResource,
-  listResources,
-  patchResource,
 
   agentServiceConfig,
   platform,
@@ -368,13 +320,10 @@ export const onEnableCIM = async ({
   imgVolSizeGiB,
 
   configureLoadBalancer,
+  ciscoIntersightURL,
 }: {
   t: TFunction;
   setError: SetErrorFuncType;
-  createResource: CreateResourceFuncType;
-  getResource: GetResourceFuncType;
-  listResources: ListResourcesFuncType;
-  patchResource: PatchResourceFuncType;
 
   agentServiceConfig?: AgentServiceConfigK8sResource;
   platform: string;
@@ -384,22 +333,21 @@ export const onEnableCIM = async ({
   imgVolSizeGiB: number;
 
   configureLoadBalancer: boolean;
+  ciscoIntersightURL?: string;
 }) => {
   if (['none', 'baremetal', 'openstack', 'vsphere'].includes(platform.toLocaleLowerCase())) {
-    await patchProvisioningConfiguration({ t, setError, patchResource, getResource });
+    await patchProvisioningConfiguration({ t, setError });
   }
 
-  if (agentServiceConfig) {
-    // console.log('The AgentServiceConfig recently can not be patched. Delete and create instead.');
-  } else {
+  if (!agentServiceConfig) {
     if (
       !(await createAgentServiceConfig({
         t,
         setError,
-        createResource,
         dbVolSizeGiB,
         fsVolSizeGiB,
         imgVolSizeGiB,
+        ciscoIntersightURL,
       }))
     ) {
       return false;
@@ -411,16 +359,11 @@ export const onEnableCIM = async ({
 
   if (configureLoadBalancer) {
     // Recently No to Yes only (since we do not delete the ingress controller)
-    if (await isIngressController(getResource)) {
-      // console.log('IngressController already present, we do not patch it.');
+    if (await isIngressController()) {
       return true /* Not an error */;
     }
 
-    const assistedImageServiceRoute = await getAssistedImageServiceRoute(
-      t,
-      setError,
-      listResources,
-    );
+    const assistedImageServiceRoute = await getAssistedImageServiceRoute(t, setError);
     if (!assistedImageServiceRoute) {
       return false;
     }
@@ -431,14 +374,8 @@ export const onEnableCIM = async ({
     }
 
     if (
-      !(await createIngressController(t, setError, createResource, domain)) ||
-      !(await patchAssistedImageServiceRoute(
-        t,
-        setError,
-        patchResource,
-        assistedImageServiceRoute,
-        domain,
-      ))
+      !(await createIngressController(t, setError, domain)) ||
+      !(await patchAssistedImageServiceRoute(t, setError, assistedImageServiceRoute, domain))
     ) {
       return false;
     }

package/lib/cim/components/modals/CimConfiguration/types.ts

@@ -1,24 +1,12 @@
-import {
-  CreateResourceFuncType,
-  GetResourceFuncType,
-  ListResourcesFuncType,
-  PatchResourceFuncType,
-} from '../../../types';
 import { AgentServiceConfigK8sResource } from '../../../types/k8s/agent-service-config';
 
-export type CimConfiguratioProps = {
+export type CimConfigurationValues = {
   dbVolSize: number;
-  dbVolSizeValidation?: string;
-  setDbVolSize: (v: number) => void;
   fsVolSize: number;
-  fsVolSizeValidation?: string;
-  setFsVolSize: (v: number) => void;
   imgVolSize: number;
-  imgVolSizeValidation?: string;
-  setImgVolSize: (v: number) => void;
   configureLoadBalancer: boolean;
-  configureLoadBalancerInitial: boolean;
-  setConfigureLoadBalancer: (v: boolean) => void;
+  addCiscoIntersightUrl: boolean;
+  ciscoIntersightURL: string;
 };
 
 export type CimConfigProgressAlertProps = {
@@ -26,12 +14,14 @@ export type CimConfigProgressAlertProps = {
   assistedServiceDeploymentUrl: string;
 };
 
-export type CimConfigurationFormProps = CimConfiguratioProps & {
-  onClose: () => void;
+export type CimConfigurationFormFieldsProps = {
   isEdit: boolean;
   isInProgressPeriod: boolean;
   docConfigUrl: string;
   docConfigAwsUrl: string;
+  platform: string;
+  configureLoadBalancerInitial: boolean;
+  setConfigureLoadBalancerInitial: (value: boolean) => void;
 };
 
 export type CimConfigurationModalProps = {
@@ -42,11 +32,6 @@ export type CimConfigurationModalProps = {
   docDisconnectedUrl: string;
   docConfigUrl: string;
   docConfigAwsUrl: string;
-
-  createResource: CreateResourceFuncType;
-  getResource: GetResourceFuncType;
-  listResources: ListResourcesFuncType;
-  patchResource: PatchResourceFuncType;
 };
 
 export type CimConfigMissingAlertProps = {

package/lib/cim/components/modals/EditBMHModal.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { Modal, ModalVariant } from '@patternfly/react-core';
+import { Modal, ModalVariant } from '@patternfly/react-core/deprecated';
 import BMCForm from '../Agent/BMCForm';
 import { SecretK8sResource } from '../../types';
 import { LoadingState } from '../../../common';

package/lib/cim/components/modals/EditNtpSourcesModal.tsx

@@ -1,16 +1,11 @@
 import * as React from 'react';
+import { Alert, Button, ButtonVariant, Form, Stack, StackItem } from '@patternfly/react-core';
 import {
-  Alert,
-  Button,
-  ButtonVariant,
-  Form,
   Modal,
   ModalBoxBody,
   ModalBoxFooter,
   ModalVariant,
-  Stack,
-  StackItem,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { Formik, FormikProps } from 'formik';
 import { InfraEnvK8sResource } from '../../types';
 import { RadioField, AdditionalNTPSourcesField } from '../../../common';

package/lib/cim/components/modals/EditProxyModal.tsx

@@ -1,17 +1,12 @@
 import * as React from 'react';
 import * as Yup from 'yup';
+import { Alert, Button, ButtonVariant, Form, Stack, StackItem } from '@patternfly/react-core';
 import {
-  Alert,
-  Button,
-  ButtonVariant,
-  Form,
   Modal,
   ModalBoxBody,
   ModalBoxFooter,
   ModalVariant,
-  Stack,
-  StackItem,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { EditProxyModalProps } from './types';
 import { useTranslation } from '../../../common/hooks/use-translation-wrapper';
 import { ProxyInputFields } from '../../../common/components/clusterConfiguration/ProxyFields';

package/lib/cim/components/modals/EditPullSecretModal.tsx

@@ -1,17 +1,13 @@
 import * as React from 'react';
 import * as Yup from 'yup';
 import { TFunction } from 'i18next';
+import { Alert, Button, ButtonVariant, Stack, StackItem } from '@patternfly/react-core';
 import {
-  Alert,
-  Button,
-  ButtonVariant,
   Modal,
   ModalBoxBody,
   ModalBoxFooter,
   ModalVariant,
-  Stack,
-  StackItem,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { Formik, useFormikContext } from 'formik';
 import { InfraEnvK8sResource, SecretK8sResource } from '../../types';
 import { LoadingState, PullSecretField, pullSecretValidationSchema } from '../../../common';

package/lib/cim/components/modals/EditSSHKeyModal.tsx

@@ -1,16 +1,12 @@
 import * as React from 'react';
 import * as Yup from 'yup';
+import { Alert, Button, ButtonVariant, Stack, StackItem } from '@patternfly/react-core';
 import {
-  Alert,
-  Button,
-  ButtonVariant,
   Modal,
   ModalBoxBody,
   ModalBoxFooter,
   ModalVariant,
-  Stack,
-  StackItem,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { Formik, FormikProps } from 'formik';
 
 import { InfraEnvK8sResource } from '../../types';

package/lib/cim/components/modals/MassApproveAgentModal.tsx

@@ -1,14 +1,6 @@
 import * as React from 'react';
-import {
-  Alert,
-  Button,
-  ButtonVariant,
-  Modal,
-  ModalBoxBody,
-  ModalBoxFooter,
-  Stack,
-  StackItem,
-} from '@patternfly/react-core';
+import { Alert, Button, ButtonVariant, Stack, StackItem } from '@patternfly/react-core';
+import { Modal, ModalBoxBody, ModalBoxFooter } from '@patternfly/react-core/deprecated';
 import { getAIHosts } from '../helpers';
 import { AgentK8sResource } from '../../types';
 import HostsTable from '../../../common/components/hosts/HostsTable';
@@ -27,7 +19,7 @@ type ApproveTableRowProps = {
 };
 
 const ApproveTableRow: React.FC<ApproveTableRowProps> = ({ agent, children }) => (
-  <div className={agent?.spec.approved ? 'pf-v5-u-color-200' : undefined}>{children}</div>
+  <div className={agent?.spec.approved ? 'pf-v6-u-color-200' : undefined}>{children}</div>
 );
 
 const hostnameColumn = (agents: AgentK8sResource[], t: TFunction): TableRow<Host> => {

package/lib/cim/components/modals/MassDeleteAgentModal.tsx

@@ -2,7 +2,7 @@ import * as React from 'react';
 import { Button, Flex, FlexItem, Popover } from '@patternfly/react-core';
 import { InfoCircleIcon } from '@patternfly/react-icons/dist/js/icons/info-circle-icon';
 import { Link } from 'react-router-dom-v5-compat';
-import { global_palette_blue_300 as blueInfoColor } from '@patternfly/react-tokens/dist/js/global_palette_blue_300';
+import { t_global_icon_color_status_info_default as blueInfoColor } from '@patternfly/react-tokens/dist/js/t_global_icon_color_status_info_default';
 import {
   getHostname,
   getInventory,
@@ -43,7 +43,7 @@ const hostnameColumn = (agents: AgentK8sResource[], t: TFunction): TableRow<Host
         ?.clusterDeploymentName?.name;
       return {
         title: (
-          <div className={assignedToCluster ? 'pf-v5-u-color-200' : undefined}>
+          <div className={assignedToCluster ? 'pf-v6-u-color-200' : undefined}>
             {computedHostname}
           </div>
         ),

package/lib/cim/components/modals/ScaleUpModal.tsx

@@ -1,9 +1,6 @@
 import React from 'react';
 import * as Yup from 'yup';
 import {
-  Modal,
-  ModalBoxBody,
-  ModalBoxFooter,
   Button,
   ButtonVariant,
   Alert,
@@ -12,6 +9,7 @@ import {
   Stack,
   StackItem,
 } from '@patternfly/react-core';
+import { Modal, ModalBoxBody, ModalBoxFooter } from '@patternfly/react-core/deprecated';
 import { Formik, FormikConfig, FormikProps } from 'formik';
 import { AgentK8sResource } from '../../types/k8s/agent';
 import { ClusterDeploymentK8sResource } from '../../types/k8s/cluster-deployment';

package/lib/cim/hooks/index.tsx

@@ -1 +1,2 @@
 export * from './useConfigMaps';
+export * from './useAgentServiceConfig';

package/lib/cim/hooks/useAgentServiceConfig.tsx

@@ -0,0 +1,15 @@
+import { useK8sWatchResource } from './useK8sWatchResource';
+import { AgentServiceConfigK8sResource } from '../types';
+import { K8sWatchHookProps } from './types';
+
+export const useAgentServiceConfig = (props: K8sWatchHookProps) =>
+  useK8sWatchResource<AgentServiceConfigK8sResource>(
+    {
+      groupVersionKind: {
+        group: 'agent-install.openshift.io',
+        kind: 'AgentServiceConfig',
+        version: 'v1beta1',
+      },
+    },
+    props,
+  );

package/lib/cim/types/index.ts

@@ -1,5 +1,4 @@
 export * from './fromOCP';
 export * from './k8s';
 export * from './metal3';
-export * from './resources';
 export * from './models';

package/lib/cim/types/models.tsx

@@ -1,12 +1,56 @@
 import { K8sModel } from '@openshift-console/dynamic-plugin-sdk/lib/api/common-types';
 
 export const AgentClusterInstallModel: K8sModel = {
+  kind: 'AgentClusterInstall',
   label: 'AgentClusterInstall',
-  apiVersion: 'v1beta1',
+  labelPlural: 'AgentClusterInstalls',
   plural: 'agentclusterinstalls',
+  apiVersion: 'v1beta1',
+  apiGroup: 'extensions.hive.openshift.io',
+  namespaced: true,
   abbr: 'ACI',
+};
+
+export const AgentServiceConfigModel: K8sModel = {
+  kind: 'AgentServiceConfig',
+  label: 'AgentServiceConfig',
+  labelPlural: 'AgentServiceConfigs',
+  plural: 'agentserviceconfigs',
+  apiVersion: 'v1beta1',
+  apiGroup: 'agent-install.openshift.io',
+  namespaced: false,
+  abbr: 'asc',
+};
+
+export const IngressControllerModel: K8sModel = {
+  kind: 'IngressController',
+  label: 'IngressController',
+  labelPlural: 'IngressControllers',
+  plural: 'ingresscontrollers',
+  apiVersion: 'v1',
+  apiGroup: 'operator.openshift.io',
   namespaced: true,
-  kind: 'AgentClusterInstall',
-  labelPlural: 'AgentClusterInstalls',
-  apiGroup: 'extensions.hive.openshift.io',
+  abbr: 'ic',
+};
+
+export const RouteModel: K8sModel = {
+  kind: 'Route',
+  label: 'Route',
+  labelPlural: 'Routes',
+  plural: 'routes',
+  apiVersion: 'v1',
+  apiGroup: 'route.openshift.io',
+  namespaced: true,
+  abbr: 'rt',
+};
+
+export const ProvisioningModel: K8sModel = {
+  kind: 'Provisioning',
+  label: 'Provisioning',
+  labelPlural: 'Provisionings',
+  plural: 'provisionings',
+  apiVersion: 'v1alpha1',
+  apiGroup: 'metal3.io',
+  namespaced: false,
+  abbr: 'p',
 };

package/lib/common/api/assisted-service/BundleAPI.ts

@@ -6,8 +6,21 @@ const BundleAPI = {
     return `/v2/operators/bundles`;
   },
 
-  list() {
-    return client.get<Bundle[]>(`${BundleAPI.makeBaseURI()}`);
+  list(
+    openshiftVersion?: string,
+    cpuArchitecture?: string,
+    platformType?: string,
+    featureIds?: string,
+  ) {
+    return client.get<Bundle[]>(`${BundleAPI.makeBaseURI()}`, {
+      params: {
+        openshift_version: openshiftVersion,
+        cpu_architecture: cpuArchitecture,
+        platform_type: platformType,
+        feature_ids: featureIds,
+        ...(platformType === 'external' ? { external_platform_name: 'oci' } : {}),
+      },
+    });
   },
 
   listOperatorsForBundle(bundleName: string) {