Skip to content

fix(serverless,tests): switch Security Cypress tests to use UIAM authentication#261873

Merged
azasypkin merged 1 commit intoelastic:mainfrom
azasypkin:issue-xxx-fix-cypress
Apr 9, 2026
Merged

fix(serverless,tests): switch Security Cypress tests to use UIAM authentication#261873
azasypkin merged 1 commit intoelastic:mainfrom
azasypkin:issue-xxx-fix-cypress

Conversation

@azasypkin
Copy link
Copy Markdown
Contributor

@azasypkin azasypkin commented Apr 8, 2026

Summary

Switch Security Cypress tests to use UIAM authentication.

Related: #260546

@azasypkin azasypkin added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting ci:all-cypress-suites labels Apr 8, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #165 / discover/esql_4 discover esql controls when unlinking a ES|QL panel with controls and explorting it in discover should retain the controls and their state

Metrics [docs]

✅ unchanged

@@ -17,6 +17,7 @@ import { resolve } from 'path';
import axios from 'axios';
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

From what I can tell this file is only used in serverless tests, and hence it should use UIAM mode (see #260546)

cloudUsersFilePath,
serverless: {
uiam: true,
projectType: 'security',
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I see security is already hard-coded in rolesPath, so I'm doing the same here.

@azasypkin azasypkin marked this pull request as ready for review April 8, 2026 10:09
@azasypkin azasypkin requested review from a team as code owners April 8, 2026 10:09
Copy link
Copy Markdown
Contributor

@alainnahalliday alainnahalliday left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@rylnd rylnd left a comment

Choose a reason for hiding this comment

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

I don't know how to review this other than observing that the tests are still passing after these changes. If that's sufficient: LGTM

@azasypkin
Copy link
Copy Markdown
Contributor Author

I don't know how to review this other than observing that the tests are still passing after these changes. If that's sufficient: LGTM

Yeah, that should be enough (I've added ci:all-cypress-suites to run all Cypress tests to be sure it works), thanks!

@azasypkin azasypkin merged commit 58459c2 into elastic:main Apr 9, 2026
30 checks passed
@azasypkin azasypkin deleted the issue-xxx-fix-cypress branch April 9, 2026 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:all-cypress-suites release_note:skip Skip the PR/issue when compiling release notes v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants