[8.x] [ECO][Inventory V2] Turn off entityCentricExperience feature flag by default (#204278)

x-pack/plugins/observability_solution/apm/public/hooks/use_entity_centric_experience_setting.tsx

@@ -13,7 +13,7 @@ export function useEntityCentricExperienceSetting() {
 
   const isEntityCentricExperienceEnabled = core.uiSettings.get<boolean>(
     entityCentricExperience,
-    true
+    false
   );
 
   return { isEntityCentricExperienceEnabled };

x-pack/plugins/observability_solution/infra/public/hooks/use_entity_centric_experience_setting.tsx

@@ -11,7 +11,7 @@ import { useKibanaContextForPlugin } from './use_kibana';
 export function useEntityCentricExperienceSetting() {
   const { uiSettings } = useKibanaContextForPlugin().services;
 
-  const isEntityCentricExperienceEnabled = uiSettings.get<boolean>(entityCentricExperience, true);
+  const isEntityCentricExperienceEnabled = uiSettings.get<boolean>(entityCentricExperience);
 
   return { isEntityCentricExperienceEnabled };
 }

x-pack/plugins/observability_solution/infra/server/routes/entities/get_data_stream_types.test.ts

@@ -35,13 +35,13 @@ describe('getDataStreamTypes', () => {
     jest.clearAllMocks();
   });
 
-  it('should return only metrics when entityCentriExperienceEnabled is false and hasMetricsData is true', async () => {
+  it('should return only metrics when entityCentricExperienceEnabled is false and hasMetricsData is true', async () => {
     (getHasMetricsData as jest.Mock).mockResolvedValue(true);
 
     const params = {
       entityId: 'entity123',
       entityType: 'host' as EntityType,
-      entityCentriExperienceEnabled: false,
+      entityCentricExperienceEnabled: false,
       infraMetricsClient,
       obsEsClient,
       entityManagerClient,
@@ -58,13 +58,13 @@ describe('getDataStreamTypes', () => {
     });
   });
 
-  it('should return an empty array when entityCentriExperienceEnabled is false and hasMetricsData is false', async () => {
+  it('should return an empty array when entityCentricExperienceEnabled is false and hasMetricsData is false', async () => {
     (getHasMetricsData as jest.Mock).mockResolvedValue(false);
 
     const params = {
       entityId: 'entity123',
       entityType: 'container' as EntityType,
-      entityCentriExperienceEnabled: false,
+      entityCentricExperienceEnabled: false,
       infraMetricsClient,
       obsEsClient,
       entityManagerClient,
@@ -75,7 +75,7 @@ describe('getDataStreamTypes', () => {
     expect(result).toEqual([]);
   });
 
-  it('should return metrics and entity source_data_stream types when entityCentriExperienceEnabled is true and has entity data', async () => {
+  it('should return metrics and entity source_data_stream types when entityCentricExperienceEnabled is true and has entity data', async () => {
     (getHasMetricsData as jest.Mock).mockResolvedValue(true);
     (getLatestEntity as jest.Mock).mockResolvedValue({
       sourceDataStreamType: ['logs', 'metrics'],
@@ -84,7 +84,7 @@ describe('getDataStreamTypes', () => {
     const params = {
       entityId: 'entity123',
       entityType: 'host' as EntityType,
-      entityCentriExperienceEnabled: true,
+      entityCentricExperienceEnabled: true,
       infraMetricsClient,
       obsEsClient,
       entityManagerClient,
@@ -104,14 +104,14 @@ describe('getDataStreamTypes', () => {
     });
   });
 
-  it('should return only metrics when entityCentriExperienceEnabled is true but entity data is undefined', async () => {
+  it('should return only metrics when entityCentricExperienceEnabled is true but entity data is undefined', async () => {
     (getHasMetricsData as jest.Mock).mockResolvedValue(true);
     (getLatestEntity as jest.Mock).mockResolvedValue(undefined);
 
     const params = {
       entityId: 'entity123',
       entityType: 'host' as EntityType,
-      entityCentriExperienceEnabled: true,
+      entityCentricExperienceEnabled: true,
       infraMetricsClient,
       obsEsClient,
       entityManagerClient,
@@ -131,7 +131,7 @@ describe('getDataStreamTypes', () => {
     const params = {
       entityId: 'entity123',
       entityType: 'host' as EntityType,
-      entityCentriExperienceEnabled: true,
+      entityCentricExperienceEnabled: true,
       infraMetricsClient,
       obsEsClient,
       entityManagerClient,

x-pack/plugins/observability_solution/infra/server/routes/entities/get_data_stream_types.ts

@@ -18,15 +18,15 @@ import { getLatestEntity } from './get_latest_entity';
 interface Params {
   entityId: string;
   entityType: 'host' | 'container';
-  entityCentriExperienceEnabled: boolean;
+  entityCentricExperienceEnabled: boolean;
   infraMetricsClient: InfraMetricsClient;
   obsEsClient: ObservabilityElasticsearchClient;
   entityManagerClient: EntityClient;
   logger: Logger;
 }
 
 export async function getDataStreamTypes({
-  entityCentriExperienceEnabled,
+  entityCentricExperienceEnabled,
   entityId,
   entityManagerClient,
   entityType,
@@ -42,7 +42,7 @@ export async function getDataStreamTypes({
 
   const sourceDataStreams = new Set(hasMetricsData ? [EntityDataStreamType.METRICS] : []);
 
-  if (!entityCentriExperienceEnabled) {
+  if (!entityCentricExperienceEnabled) {
     return Array.from(sourceDataStreams);
   }
 

x-pack/plugins/observability_solution/infra/server/routes/entities/index.ts

@@ -54,13 +54,13 @@ export const initEntitiesConfigurationRoutes = (libs: InfraBackendLibs) => {
         plugin: `@kbn/${METRICS_APP_ID}-plugin`,
       });
 
-      const entityCentriExperienceEnabled = await coreContext.uiSettings.client.get(
+      const entityCentricExperienceEnabled = await coreContext.uiSettings.client.get(
         entityCentricExperience
       );
 
       try {
         const sourceDataStreamTypes = await getDataStreamTypes({
-          entityCentriExperienceEnabled,
+          entityCentricExperienceEnabled,
           entityId,
           entityManagerClient,
           entityType,

x-pack/plugins/observability_solution/inventory/e2e/cypress/e2e/alert_count/alert_count.cy.ts

@@ -38,6 +38,10 @@ describe.skip('Alert count', () => {
   beforeEach(() => {
     cy.loginAsSuperUser();
 
+    cy.updateAdvancedSettings({
+      'observability:entityCentricExperience': true,
+    });
+
     cy.intercept('GET', '/internal/entities/managed/enablement', {
       fixture: 'eem_enabled.json',
     }).as('getEEMStatus');
@@ -50,6 +54,9 @@ describe.skip('Alert count', () => {
   });
 
   afterEach(() => {
+    cy.updateAdvancedSettings({
+      'observability:entityCentricExperience': false,
+    });
     entitiesSynthtrace.clean();
     cleanEntityAlerts();
   });

x-pack/plugins/observability_solution/inventory/e2e/cypress/e2e/home.cy.ts

@@ -15,6 +15,25 @@ const end = '2024-10-16T00:15:00.000Z';
 describe.skip('Home page', () => {
   beforeEach(() => {
     cy.loginAsSuperUser();
+    cy.updateAdvancedSettings({
+      'observability:entityCentricExperience': true,
+    });
+  });
+
+  afterEach(() => {
+    cy.updateAdvancedSettings({
+      'observability:entityCentricExperience': false,
+    });
+  });
+
+  describe('When the entityCentricExperience FF is disabled', () => {
+    it('Shows 404', () => {
+      cy.updateAdvancedSettings({
+        'observability:entityCentricExperience': false,
+      });
+      cy.visitKibana('/app/inventory');
+      cy.contains('Application Not Found');
+    });
   });
 
   describe('When EEM is disabled', () => {

x-pack/plugins/observability_solution/inventory/e2e/cypress/support/commands.ts

@@ -55,3 +55,17 @@ Cypress.Commands.add(
     );
   }
 );
+
+Cypress.Commands.add('updateAdvancedSettings', (settings: Record<string, unknown>) => {
+  const kibanaUrl = Cypress.env('KIBANA_URL');
+  cy.request({
+    log: false,
+    method: 'POST',
+    url: `${kibanaUrl}/internal/kibana/settings`,
+    body: { changes: settings },
+    headers: {
+      'kbn-xsrf': 'e2e_test',
+    },
+    auth: { user: 'elastic', pass: 'changeme' },
+  });
+});

x-pack/plugins/observability_solution/inventory/e2e/cypress/support/types.d.ts

@@ -13,6 +13,7 @@ declare namespace Cypress {
       password: string;
     }): Cypress.Chainable<Cypress.Response<any>>;
     getByTestSubj(selector: string): Chainable<JQuery<Element>>;
+    updateAdvancedSettings(settings: Record<string, unknown>): void;
     visitKibana(url: string): void;
   }
 }

x-pack/plugins/observability_solution/inventory/public/plugin.ts

@@ -57,7 +57,7 @@ export class InventoryPlugin
     const inventoryAPIClient = createCallInventoryAPI(coreSetup);
     const isEntityCentricExperienceSettingEnabled = coreSetup.uiSettings.get<boolean>(
       'observability:entityCentricExperience',
-      true
+      false
     );
 
     this.telemetry.setup({

x-pack/solutions/observability/plugins/observability/server/ui_settings.ts

@@ -352,7 +352,7 @@ export const uiSettings: Record<string, UiSettings> = {
       },
     }),
     schema: schema.boolean(),
-    value: true,
+    value: false,
     requiresPageReload: true,
     type: 'boolean',
   },