Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Detection Engine] Add basic serverless rbac tests for detection engine #192952

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

yctercero
Copy link
Contributor

@yctercero yctercero commented Sep 14, 2024

Summary

Adding FTR RBAC tests for serverless prebuilt roles.

Following Serverless Security Roles spreadsheet.

Needs clarification

The following tests returned unexpected results and need confirmation of what the intended behavior should be. @approksiu @caitlinbetz

Actions

Our serverless spreadsheet denotes the following should be READ on actions, but:

  • endpoint_operations_analyst requires feature_actions.all to manage endpoint actions and can't differentiate between endpoint/rule actions, so will have all. Also requires feature_siem.all to manage rules, so will also inherit being able to update rule actions.
  • endpoint_policy_manager requires feature_actions.all to manage endpoint actions and can't differentiate between endpoint/rule actions, so will have all. Also requires feature_siem.all to manage rules, so will also inherit being able to update rule actions.
  • rule_author requires feature_actions.all to manage endpoint actions and can't differentiate between endpoint/rule actions, so will have all. Also requires feature_siem.all to manage rules, so will also inherit being able to update rule actions.
  • threat_intelligence_analyst requires feature_actions.all to manage endpoint actions and can't differentiate between endpoint/rule actions, so will have all. Also requires feature_siem.all to manage rules, so will also inherit being able to update rule actions.
  • t3_analyst requires feature_siem.all to manage rules, so will also inherit being able to update rule actions.

Rule exceptions

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine

This comment was marked as outdated.

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7260

[❌] x-pack/test/security_solution_api_integration/test_suites/detections_response/detection_engine/authentication/alerts/trial_license_complete_tier/configs/serverless.config.ts: 0/25 tests passed.
[❌] x-pack/test/security_solution_api_integration/test_suites/detections_response/detection_engine/authentication/exceptions/trial_license_complete_tier/configs/serverless.config.ts: 24/25 tests passed.

see run history

@yctercero
Copy link
Contributor Author

/ci

@yctercero yctercero marked this pull request as ready for review October 28, 2024 21:41
@yctercero yctercero requested review from a team as code owners October 28, 2024 21:41
@yctercero yctercero requested a review from rylnd October 28, 2024 21:41
@yctercero yctercero added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Detection Engine Security Solution Detection Engine Area backport:version Backport to applied version labels v8.17.0 labels Oct 29, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detection-engine (Team:Detection Engine)

@yctercero yctercero self-assigned this Oct 29, 2024
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @yctercero

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Detection Engine Security Solution Detection Engine Area v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants