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

[Security Solution] Introducing PLI testing on AET Cypress tests for serverless #165254

Merged
merged 5 commits into from
Aug 31, 2023

Conversation

MadameSheema
Copy link
Member

@MadameSheema MadameSheema commented Aug 30, 2023

Summary

In this PR we are introducing PLI testing on AET cypress tests for serverless.

  • We have set the Security complete tier with all the add-ons by default.
  • We have updated the README file to explain how PLIs work on our Cypress tests.
  • We have added a test that can be used as example.

@MadameSheema MadameSheema changed the title adding security,endpoint and cloud complete as default PLI [Security Solution] Introducing PLI testing on AET Cypress tests for serverless Aug 30, 2023
@MadameSheema MadameSheema marked this pull request as ready for review August 30, 2023 17:21
@MadameSheema MadameSheema requested review from a team as code owners August 30, 2023 17:21
@MadameSheema MadameSheema self-assigned this Aug 30, 2023
@MadameSheema MadameSheema added v8.11.0 release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. labels Aug 30, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

Copy link
Contributor

@charlie-pichette charlie-pichette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woohoo! 👍🏼

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @MadameSheema

Copy link
Member

@machadoum machadoum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is awesome! 👏 👏

Thank you, Glo!

I left a nit comment.


Per the way we set the environment during the execution process on CI, the above configuration is going to be valid when the test is executed on headless mode.

For test developing or test debugging purposes, you need to modify the configuration but without the changes in `x-pack/test/security_solution_cypress/serverless_config.ts`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... you need to modify the configuration but without the changes in x-pack/test/security_solution_cypress/serverless_config.ts

This part is confusing to me. Should I update serverless_config.ts or not? If not, which file do I need to update?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the comment @machadoum!! I wanted to say:

For test developing or test debugging purposes, you need to modify the configuration but without committing and pushing the changes in `x-pack/test/security_solution_cypress/serverless_config.ts`

Seems that I was tired and I didn't write the sentence properly. Can you please let me know if is clear now? Thanks!! :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks great. Thank you!

@MadameSheema MadameSheema enabled auto-merge (squash) August 31, 2023 08:46
@MadameSheema MadameSheema merged commit b55c949 into elastic:main Aug 31, 2023
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Aug 31, 2023
@MadameSheema MadameSheema deleted the cypress/introducing-plis branch August 31, 2023 09:20
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants