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

[8.15] Unskips api key functional tests in MKI (#196572) #201617

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

SiddharthMantri
Copy link
Contributor

Backport

This will backport the following commits from main to 8.15:

Questions ?

Please refer to the Backport tool documentation

\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sqren/backport)\n\n\n\nCo-authored-by: Jeramy Soucy "}},{"branch":"8.18","label":"v8.18.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.2","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com//pull/201393","number":201393,"state":"MERGED","mergeCommit":{"sha":"b938b43a5fdc2d091c02a4c54ad0485cb119c680","message":"[8.16] Unskips api key functional tests in MKI (#196572) (#201393)\n\n# Backport\n\nThis will backport the following commits from `main` to `8.16`:\n- [Unskips api key functional tests in MKI\n(#196572)](https://github.com//pull/196572)\n\n\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sqren/backport)\n\n\n\nCo-authored-by: Jeramy Soucy "}},{"url":"https://github.com//pull/201394","number":201394,"branch":"8.17","state":"MERGED","mergeCommit":{"sha":"3e09ffc239215999f28bed0d3f7a7cebf4a92657","message":"[8.17] Unskips api key functional tests in MKI (#196572) (#201394)\n\n# Backport\n\nThis will backport the following commits from `main` to `8.17`:\n- [Unskips api key functional tests in MKI\n(#196572)](https://github.com//pull/196572)\n\n\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sqren/backport)\n\n\n\nCo-authored-by: Jeramy Soucy "}}]}] BACKPORT-->

Closes elastic#186619

## Summary

Tested against a new QA project, the platform security api key
functional test now passes as expected.

Update: the serverless functional UI API key test has also been modified
to account for a non-empty start state, and to be non-destructive. If
any API keys exist prior to running this test, they will not interfere
with the test and they will remain after the test completes.

Flaky test runners:
-
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184
- Security re-run
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195

(cherry picked from commit 492a9a4)

# Conflicts:
#	x-pack/test_serverless/functional/test_suites/common/platform_security/api_keys.ts
@SiddharthMantri
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

There are no new commits on the base branch.

@SiddharthMantri
Copy link
Contributor Author

/ci

@SiddharthMantri SiddharthMantri merged commit 12771dd into elastic:8.15 Nov 25, 2024
22 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
security 587.5KB 587.5KB +13.0B

History

cc @SiddharthMantri

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants