From d57bc9a5d7d64f86b550eff7997605a3090aee9a Mon Sep 17 00:00:00 2001 From: Georgii Gorbachev Date: Fri, 11 Oct 2024 18:00:58 +0200 Subject: [PATCH] [Security Solution] Skip recently added tests for updating restricted fields in prebuilt rules (#195926) ## Summary The new tests added yesterday in https://github.com/elastic/kibana/pull/195318 have failed in the periodic pipeline ([build](https://buildkite.com/elastic/kibana-serverless-security-solution-quality-gate-rule-management/builds/1408#019279c2-233d-405e-8758-7864b84d0524)). Skipping for now in MKI pipelines (periodic pipeline and the 2nd quality gate), otherwise it will block the next Serverless release. Ticket for unskipping: https://github.com/elastic/kibana/issues/195921 --- .../rule_patch/basic_license_essentials_tier/patch_rules.ts | 3 ++- .../basic_license_essentials_tier/patch_rules_bulk.ts | 3 ++- .../rule_update/basic_license_essentials_tier/update_rules.ts | 3 ++- .../basic_license_essentials_tier/update_rules_bulk.ts | 3 ++- 4 files changed, 8 insertions(+), 4 deletions(-) diff --git a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules.ts b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules.ts index 41f207c90f319..be8e76f058b14 100644 --- a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules.ts +++ b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules.ts @@ -241,7 +241,8 @@ export default ({ getService }: FtrProviderContext) => { }); }); - it('throws an error if rule has external rule source and non-customizable fields are changed', async () => { + // Unskip: https://github.com/elastic/kibana/issues/195921 + it('@skipInServerlessMKI throws an error if rule has external rule source and non-customizable fields are changed', async () => { // Install base prebuilt detection rule await createHistoricalPrebuiltRuleAssetSavedObjects(es, [ createRuleAssetSavedObject({ rule_id: 'rule-1', author: ['elastic'] }), diff --git a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules_bulk.ts b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules_bulk.ts index 7929b912768ff..f02ab60eaabf7 100644 --- a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules_bulk.ts +++ b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules_bulk.ts @@ -351,7 +351,8 @@ export default ({ getService }: FtrProviderContext) => { ]); }); - it('throws an error if rule has external rule source and non-customizable fields are changed', async () => { + // Unskip: https://github.com/elastic/kibana/issues/195921 + it('@skipInServerlessMKI throws an error if rule has external rule source and non-customizable fields are changed', async () => { // Install base prebuilt detection rule await createHistoricalPrebuiltRuleAssetSavedObjects(es, [ createRuleAssetSavedObject({ rule_id: 'rule-1', author: ['elastic'] }), diff --git a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules.ts b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules.ts index c84236a14eb37..4aa264238ffad 100644 --- a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules.ts +++ b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules.ts @@ -313,7 +313,8 @@ export default ({ getService }: FtrProviderContext) => { }); }); - it('throws an error if rule has external rule source and non-customizable fields are changed', async () => { + // Unskip: https://github.com/elastic/kibana/issues/195921 + it('@skipInServerlessMKI throws an error if rule has external rule source and non-customizable fields are changed', async () => { // Install base prebuilt detection rule await createHistoricalPrebuiltRuleAssetSavedObjects(es, [ createRuleAssetSavedObject({ rule_id: 'rule-1', license: 'elastic' }), diff --git a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules_bulk.ts b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules_bulk.ts index cdca9e3ca6e1a..409b07f301c0d 100644 --- a/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules_bulk.ts +++ b/x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_update/basic_license_essentials_tier/update_rules_bulk.ts @@ -374,7 +374,8 @@ export default ({ getService }: FtrProviderContext) => { ]); }); - it('throws an error if rule has external rule source and non-customizable fields are changed', async () => { + // Unskip: https://github.com/elastic/kibana/issues/195921 + it('@skipInServerlessMKI throws an error if rule has external rule source and non-customizable fields are changed', async () => { // Install base prebuilt detection rule await createHistoricalPrebuiltRuleAssetSavedObjects(es, [ createRuleAssetSavedObject({ rule_id: 'rule-1', author: ['elastic'] }),