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.17] Unskips api key functional tests in MKI (#196572) #201394

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.17:

Questions ?

Please refer to the Backport tool documentation

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)
@kibanamachine kibanamachine force-pushed the backport/8.17/pr-196572 branch from 5d8585b to d6a7f84 Compare November 22, 2024 14:20
@kibanamachine kibanamachine merged commit 3e09ffc into elastic:8.17 Nov 22, 2024
21 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 542.6KB 542.6KB +13.0B

cc @jeramysoucy

SiddharthMantri added a commit that referenced this pull request Nov 25, 2024
# Backport

This will backport the following commits from `main` to `8.15`:
- [Unskips api key functional tests in MKI
(#196572)](#196572)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jeramy
Soucy","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-21T12:33:34Z","message":"Unskips
api key functional tests in MKI (#196572)\n\nCloses #186619\r\n\r\n##
Summary\r\n\r\nTested against a new QA project, the platform security
api key\r\nfunctional test now passes as expected.\r\n\r\nUpdate: the
serverless functional UI API key test has also been modified\r\nto
account for a non-empty start state, and to be non-destructive.
If\r\nany API keys exist prior to running this test, they will not
interfere\r\nwith the test and they will remain after the test
completes.\r\n\r\nFlaky test
runners:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\r\n-
Security
re-run\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195","sha":"492a9a4643b37d2bf393a3d549c50dcfba16907f","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Security","release_note:skip","v9.0.0","backport:prev-major","v8.17.0","v8.18.0","v8.16.2"],"number":196572,"url":"https://github.com/elastic/kibana/pull/196572","mergeCommit":{"message":"Unskips
api key functional tests in MKI (#196572)\n\nCloses #186619\r\n\r\n##
Summary\r\n\r\nTested against a new QA project, the platform security
api key\r\nfunctional test now passes as expected.\r\n\r\nUpdate: the
serverless functional UI API key test has also been modified\r\nto
account for a non-empty start state, and to be non-destructive.
If\r\nany API keys exist prior to running this test, they will not
interfere\r\nwith the test and they will remain after the test
completes.\r\n\r\nFlaky test
runners:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\r\n-
Security
re-run\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195","sha":"492a9a4643b37d2bf393a3d549c50dcfba16907f"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196572","number":196572,"mergeCommit":{"message":"Unskips
api key functional tests in MKI (#196572)\n\nCloses #186619\r\n\r\n##
Summary\r\n\r\nTested against a new QA project, the platform security
api key\r\nfunctional test now passes as expected.\r\n\r\nUpdate: the
serverless functional UI API key test has also been modified\r\nto
account for a non-empty start state, and to be non-destructive.
If\r\nany API keys exist prior to running this test, they will not
interfere\r\nwith the test and they will remain after the test
completes.\r\n\r\nFlaky test
runners:\r\n-\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\r\n-
Security
re-run\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195","sha":"492a9a4643b37d2bf393a3d549c50dcfba16907f"}},{"branch":"8.x","label":"v8.17.0","labelRegex":"^v8.17.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/201395","number":201395,"state":"MERGED","mergeCommit":{"sha":"ec39ce4966c14468b7684719324881e419791aa9","message":"[8.x]
Unskips api key functional tests in MKI (#196572) (#201395)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.x`:\n- [Unskips api key functional tests in
MKI\n(#196572)](https://github.com/elastic/kibana/pull/196572)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Jeramy\nSoucy\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-10-21T12:33:34Z\",\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"Team:Security\",\"release_note:skip\",\"v9.0.0\",\"backport:prev-major\"],\"title\":\"Unskips\napi
key functional tests
in\nMKI\",\"number\":196572,\"url\":\"https://github.com/elastic/kibana/pull/196572\",\"mergeCommit\":{\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v9.0.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/196572\",\"number\":196572,\"mergeCommit\":{\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Jeramy Soucy
<[email protected]>"}},{"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/elastic/kibana/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/elastic/kibana/pull/196572)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Jeramy\nSoucy\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-10-21T12:33:34Z\",\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"Team:Security\",\"release_note:skip\",\"v9.0.0\",\"backport:prev-major\"],\"title\":\"Unskips\napi
key functional tests
in\nMKI\",\"number\":196572,\"url\":\"https://github.com/elastic/kibana/pull/196572\",\"mergeCommit\":{\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v9.0.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/196572\",\"number\":196572,\"mergeCommit\":{\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Jeramy Soucy
<[email protected]>"}},{"url":"https://github.com/elastic/kibana/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/elastic/kibana/pull/196572)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Jeramy\nSoucy\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-10-21T12:33:34Z\",\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"Team:Security\",\"release_note:skip\",\"v9.0.0\",\"backport:prev-major\"],\"title\":\"Unskips\napi
key functional tests
in\nMKI\",\"number\":196572,\"url\":\"https://github.com/elastic/kibana/pull/196572\",\"mergeCommit\":{\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v9.0.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/196572\",\"number\":196572,\"mergeCommit\":{\"message\":\"Unskips\napi
key functional tests in MKI (#196572)\\n\\nCloses
#186619\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTested against a new QA
project, the platform security\napi key\\r\\nfunctional test now passes
as expected.\\r\\n\\r\\nUpdate: the\nserverless functional UI API key
test has also been modified\\r\\nto\naccount for a non-empty start
state, and to be non-destructive.\nIf\\r\\nany API keys exist prior to
running this test, they will not\ninterfere\\r\\nwith the test and they
will remain after the test\ncompletes.\\r\\n\\r\\nFlaky
test\nrunners:\\r\\n-\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7184\\r\\n-\nSecurity\nre-run\\r\\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/7195\",\"sha\":\"492a9a4643b37d2bf393a3d549c50dcfba16907f\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Jeramy Soucy <[email protected]>"}}]}] BACKPORT-->

---------

Co-authored-by: Jeramy Soucy <[email protected]>
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.

3 participants