Remove Okta monitor beta flag (#7360)

Author: jpople

changelog/7360-remove-okta-beta-flag.yaml

@@ -0,0 +1,7 @@
+# Copy this file and rename it (e.g., pr-number.yaml or feature-name.yaml)
+# Fill in the required fields and delete this comment block
+
+type: Changed # One of: Added, Changed, Developer Experience, Deprecated, Docs, Fixed, Removed, Security
+description: Removed beta flag for Okta monitor features
+pr: 7360 # PR number
+labels: [] # Optional: ["high-risk", "db-migration"]

clients/admin-ui/cypress/e2e/config-wizard.cy.ts

@@ -99,7 +99,6 @@ describe("Config Wizard", () => {
     beforeEach(() => {
       stubSystemCrud();
       stubTaxonomyEntities();
-      cy.overrideFeatureFlag("oktaMonitor", false);
 
       cy.visit(ADD_SYSTEMS_ROUTE);
       // Select Okta to move to form step.

clients/admin-ui/src/features/data-discovery-and-detection/action-center/MonitorList.tsx

@@ -26,15 +26,15 @@ import { SearchFormQueryState } from "./MonitorList.const";
 const MonitorList = () => {
   const toast = useToast();
   const {
-    flags: { webMonitor: webMonitorEnabled, oktaMonitor: oktaMonitorEnabled },
+    flags: { webMonitor: webMonitorEnabled },
   } = useFeatures();
   const { paginationProps, pageIndex, pageSize, resetPagination } =
     useAntPagination();
 
   const availableMonitorTypes = [
     ...(webMonitorEnabled ? [MONITOR_TYPES.WEBSITE] : []),
     MONITOR_TYPES.DATASTORE,
-    ...(oktaMonitorEnabled ? [MONITOR_TYPES.INFRASTRUCTURE] : []),
+    MONITOR_TYPES.INFRASTRUCTURE,
   ] as const;
 
   const currentUser = useAppSelector(selectUser);

clients/admin-ui/src/features/integrations/add-integration/SelectIntegrationType.tsx

@@ -32,7 +32,7 @@ const SelectIntegrationType = ({
   const [isFiltering, setIsFiltering] = useState(false);
 
   const {
-    flags: { oktaMonitor, newIntegrationManagement },
+    flags: { newIntegrationManagement },
   } = useFlags();
 
   // Fetch connection types for SAAS integration generation
@@ -86,10 +86,7 @@ const SelectIntegrationType = ({
   const availableCategories = useMemo(() => {
     const allCategories: IntegrationCategoryFilter[] = [
       "ALL",
-      ...Object.values(ConnectionCategory).filter(
-        (category) =>
-          category !== ConnectionCategory.IDENTITY_PROVIDER || oktaMonitor,
-      ),
+      ...Object.values(ConnectionCategory),
     ];
 
     // If new integration management is disabled, filter out categories that have no integrations
@@ -108,7 +105,7 @@ const SelectIntegrationType = ({
     }
 
     return allCategories;
-  }, [oktaMonitor, newIntegrationManagement, allIntegrationTypes]);
+  }, [newIntegrationManagement, allIntegrationTypes]);
 
   // Filter integrations based on search and category
   const filteredTypes = useMemo(() => {
@@ -127,21 +124,13 @@ const SelectIntegrationType = ({
       );
     }
 
-    // Apply flag-based filtering
-    filtered = filtered.filter((i) => {
-      if (!oktaMonitor && i.placeholder.connection_type === "okta") {
-        return false;
-      }
-      return true;
-    });
-
     // Sort integrations alphabetically by display name
     return filtered.sort((a, b) => {
       const nameA = a.placeholder.name || "";
       const nameB = b.placeholder.name || "";
       return nameA.localeCompare(nameB);
     });
-  }, [searchTerm, selectedCategory, oktaMonitor, allIntegrationTypes]);
+  }, [searchTerm, selectedCategory, allIntegrationTypes]);
 
   const handleCategoryChange = (value: IntegrationCategoryFilter) => {
     setIsFiltering(true);

clients/admin-ui/src/features/integrations/useIntegrationFilterTabs.tsx

@@ -1,6 +1,5 @@
 import { useState } from "react";
 
-import { useFlags } from "~/features/common/features/features.slice";
 import { IntegrationTypeInfo } from "~/features/integrations/add-integration/allIntegrationTypes";
 
 export enum IntegrationFilterTabs {
@@ -15,11 +14,7 @@ export enum IntegrationFilterTabs {
 }
 
 const useIntegrationFilterTabs = (integrationTypes?: IntegrationTypeInfo[]) => {
-  const { flags } = useFlags();
-  const tabs = Object.values(IntegrationFilterTabs).filter(
-    (tab) =>
-      tab !== IntegrationFilterTabs.IDENTITY_PROVIDER || flags.oktaMonitor,
-  );
+  const tabs = Object.values(IntegrationFilterTabs);
 
   const [tabIndex, setTabIndex] = useState(0);
   const currentTab = tabs[tabIndex];

clients/admin-ui/src/flags.json

@@ -35,13 +35,6 @@
     "test": true,
     "production": false
   },
-  "oktaMonitor": {
-    "label": "Okta monitor",
-    "description": "Monitor Okta for activity using OAuth2 authentication and Identity Provider Monitor endpoints",
-    "development": true,
-    "test": true,
-    "production": false
-  },
   "downloadAccessRequestResults": {
     "label": "Download access request results",
     "description": "Download access data from completed requests",