-
Notifications
You must be signed in to change notification settings - Fork 8k
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
[FTR](data views) update common serverless api tests to use api keys #183386
[FTR](data views) update common serverless api tests to use api keys #183386
Conversation
3dc2712
to
84279fd
Compare
84279fd
to
60e1f91
Compare
Flaky Test Runner Stats🟠 Some tests failed. - kibana-flaky-test-suite-runner#6015[❌] x-pack/test_serverless/api_integration/test_suites/observability/common_configs/config.group1.ts: 0/2 tests passed. |
Flaky Test Runner Stats🟠 Some tests failed. - kibana-flaky-test-suite-runner#6016[❌] x-pack/test_serverless/api_integration/test_suites/observability/common_configs/config.group1.ts: 0/2 tests passed. |
bb4cdef
to
f93db86
Compare
Flaky Test Runner Stats🟠 Some tests failed. - kibana-flaky-test-suite-runner#6023[❌] x-pack/test_serverless/api_integration/test_suites/observability/common_configs/config.group1.ts: 0/2 tests passed. |
/ci |
1 similar comment
/ci |
@elasticmachine merge upstream |
1 similar comment
@elasticmachine merge upstream |
/ci |
24e3456
to
033310e
Compare
033310e
to
3369c2c
Compare
/ci |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6072[✅] x-pack/test_serverless/api_integration/test_suites/search/common_configs/config.group1.ts: 5/5 tests passed. |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes look great and the tests pass! 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the improvements
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…lastic#183386) Use api keys for api calls, to act as the user, within: `x-pack/test_serverless/api_integration/test_suites/common/data_views/` Contributes to: elastic#180834 --------- Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Use api keys for api calls, to act as the user, within:
x-pack/test_serverless/api_integration/test_suites/common/data_views/
Contributes to: #180834