Skip to content

Commit 99340a1

Browse files
[Security Solution] remove riskScoringPersistence feature flag (elastic#240538)
## Summary This PR removes the `riskScoringPersistence` feature flag. This feature flag was enabled back in 2023 . **_Feel free to close this PR if our intention is to keep the feature flag around!_** > [!IMPORTANT] > As the feature flag has been eanbled for a while,, no UI or behavior changes should be introduced by this PR! ### Checklist - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels.
1 parent f257a65 commit 99340a1

File tree

3 files changed

+15
-23
lines changed

3 files changed

+15
-23
lines changed

x-pack/solutions/security/plugins/security_solution/common/experimental_features.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,6 @@ export const allowedExperimentalValues = Object.freeze({
6060
*/
6161
newUserDetailsFlyoutManagedUser: false,
6262

63-
/**
64-
* Enable risk engine client and initialisation of datastream, component templates and mappings
65-
*/
66-
riskScoringPersistence: true,
67-
6863
/**
6964
* Enables experimental Entity Analytics HTTP endpoints
7065
*/

x-pack/solutions/security/plugins/security_solution/server/plugin.ts

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
import type { Observable } from 'rxjs';
99
import { QUERY_RULE_TYPE_ID, SAVED_QUERY_RULE_TYPE_ID } from '@kbn/securitysolution-rules';
10-
import type { LogMeta, Logger } from '@kbn/core/server';
10+
import type { Logger, LogMeta } from '@kbn/core/server';
1111
import { SavedObjectsClient } from '@kbn/core/server';
1212
import type { UsageCounter } from '@kbn/usage-collection-plugin/server';
1313
import { ECS_COMPONENT_TEMPLATE_NAME } from '@kbn/alerting-plugin/server';
@@ -70,12 +70,12 @@ import { initUsageCollectors } from './usage';
7070
import type { SecuritySolutionRequestHandlerContext } from './types';
7171
import { securitySolutionSearchStrategyProvider } from './search_strategy/security_solution';
7272
import type { ITelemetryEventsSender } from './lib/telemetry/sender';
73-
import { type IAsyncTelemetryEventsSender } from './lib/telemetry/async_sender.types';
7473
import { TelemetryEventsSender } from './lib/telemetry/sender';
74+
import { type IAsyncTelemetryEventsSender } from './lib/telemetry/async_sender.types';
7575
import {
76+
AsyncTelemetryEventsSender,
7677
DEFAULT_QUEUE_CONFIG,
7778
DEFAULT_RETRY_CONFIG,
78-
AsyncTelemetryEventsSender,
7979
} from './lib/telemetry/async_sender';
8080
import type { ITelemetryReceiver } from './lib/telemetry/receiver';
8181
import { TelemetryReceiver } from './lib/telemetry/receiver';
@@ -108,7 +108,7 @@ import type {
108108
} from './plugin_contract';
109109
import { featureUsageService } from './endpoint/services/feature_usage';
110110
import { setIsElasticCloudDeployment } from './lib/telemetry/helpers';
111-
import { type CdnConfig, artifactService } from './lib/telemetry/artifact';
111+
import { artifactService, type CdnConfig } from './lib/telemetry/artifact';
112112
import { events } from './lib/telemetry/event_based/events';
113113
import { endpointFieldsProvider } from './search_strategy/endpoint_fields';
114114
import {
@@ -126,8 +126,8 @@ import {
126126
} from './lib/entity_analytics/entity_store/tasks';
127127
import { registerProtectionUpdatesNoteRoutes } from './endpoint/routes/protection_updates_note';
128128
import {
129-
latestRiskScoreIndexPattern,
130129
allRiskScoreIndexPattern,
130+
latestRiskScoreIndexPattern,
131131
} from '../common/entity_analytics/risk_engine';
132132
import { isEndpointPackageV2 } from '../common/endpoint/utils/package_v2';
133133
import { assistantTools } from './assistant/tools';
@@ -251,18 +251,16 @@ export class Plugin implements ISecuritySolutionPlugin {
251251

252252
registerDeprecations({ core, config: this.config, logger: this.logger });
253253

254-
if (experimentalFeatures.riskScoringPersistence) {
255-
registerRiskScoringTask({
256-
getStartServices: core.getStartServices,
257-
kibanaVersion: pluginContext.env.packageInfo.version,
258-
logger: this.logger,
259-
auditLogger: plugins.security?.audit.withoutRequest,
260-
taskManager: plugins.taskManager,
261-
telemetry: core.analytics,
262-
entityAnalyticsConfig: config.entityAnalytics,
263-
experimentalFeatures,
264-
});
265-
}
254+
registerRiskScoringTask({
255+
getStartServices: core.getStartServices,
256+
kibanaVersion: pluginContext.env.packageInfo.version,
257+
logger: this.logger,
258+
auditLogger: plugins.security?.audit.withoutRequest,
259+
taskManager: plugins.taskManager,
260+
telemetry: core.analytics,
261+
entityAnalyticsConfig: config.entityAnalytics,
262+
experimentalFeatures,
263+
});
266264

267265
scheduleEntityAnalyticsMigration({
268266
getStartServices: core.getStartServices,

x-pack/solutions/security/test/security_solution_api_integration/config/ess/config.base.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ export function createTestConfig(options: CreateTestConfigOptions, testFiles?: s
108108
'--xpack.ruleRegistry.unsafe.legacyMultiTenancy.enabled=true',
109109
`--xpack.securitySolution.enableExperimental=${JSON.stringify([
110110
'previewTelemetryUrlEnabled',
111-
'riskScoringPersistence',
112111
'riskScoringRoutesEnabled',
113112
'bulkEditAlertSuppressionEnabled',
114113
'doesNotMatchForIndicatorMatchRuleEnabled',

0 commit comments

Comments
 (0)