Removing all the internal API Cards from Overview page

A minimal change to remove all the internal API cards from 'Installed Operator' -> Overview page Signed-off-by: Arun Kumar Mohan <arun.iiird@gmail.com>
openshift · Jan 4, 2020 · d920f88 · d920f88
1 parent 9ef3ee3
commit d920f88
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/frontend/packages/operator-lifecycle-manager/src/components/clusterserviceversion.tsx b/frontend/packages/operator-lifecycle-manager/src/components/clusterserviceversion.tsx
@@ -81,6 +81,7 @@ import {
   SubscriptionKind,
   SubscriptionState,
 } from '../types';
+import { isInternal } from '../dev-catalog';
 import { ProvidedAPIsPage, ProvidedAPIPage } from './operand';
 import { createUninstallOperatorModal } from './modals/uninstall-operator-modal';
 import { operatorGroupFor, operatorNamespaceFor } from './operator-group';
@@ -570,6 +571,9 @@ export const CRDCard: React.SFC<CRDCardProps> = (props) => {
     `/k8s/ns/${csv.metadata.namespace}/${ClusterServiceVersionModel.plural}/${
       csv.metadata.name
     }/${reference}/~new`;
+  if (crd.displayName.startsWith("[Internal]")) {
+    return null
+  }
   return (
     <Card>
       <CardHeader>
@@ -616,7 +620,7 @@ export const CRDCardRow = connect(crdCardRowStateToProps)((props: CRDCardRowProp
     {_.isEmpty(props.crdDescs) ? (
       <span className="text-muted">No Kubernetes APIs are being provided by this Operator.</span>
     ) : (
-      props.crdDescs.map((desc) => (
+      props.crdDescs.filter((crd) => !isInternal(crd)).map((desc) => (
         <CRDCard
           key={referenceForProvidedAPI(desc)}
           crd={desc}

diff --git a/frontend/packages/operator-lifecycle-manager/src/dev-catalog.ts b/frontend/packages/operator-lifecycle-manager/src/dev-catalog.ts
@@ -4,7 +4,7 @@ import { ClusterServiceVersionKind } from './types';
 import { referenceForProvidedAPI, providedAPIsFor } from './components';
 import * as operatorLogo from './operator.svg';
 
-const isInternal = (crd: { name: string }): boolean => {
+export const isInternal = (crd: { name: string }): boolean => {
   const internalOpListString = _.get(
     crd,
     ['csv', 'metadata', 'annotations', 'operators.operatorframework.io/internal-objects'],
@@ -19,6 +19,7 @@ const isInternal = (crd: { name: string }): boolean => {
     return false;
   }
 };
+
 export const normalizeClusterServiceVersions = (
   clusterServiceVersions: ClusterServiceVersionKind[],
 ): K8sResourceKind[] => {