-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Search profiler functional test -- using "test_user" with limited role. #69841
Search profiler functional test -- using "test_user" with limited role. #69841
Conversation
💚 CLA has been signed |
b03ac4f
to
6f29cce
Compare
6f29cce
to
3577e73
Compare
💚 Build SucceededBuild metricspage load asset sizebeta
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.
LGTM. Ran it in my local.
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 - Code review and checked Jenkins output. Found problem where these tests aren't being reported in test results, but they are in the console output of x-pack ciGroup10 (I'll file a separate issue to resolve that).
(ignore the console errors in the output below. They didn't impact the test)
21:27:04 └-: Search Profiler Editor
21:27:04 └-> "before all" hook
21:27:04 └-> "before all" hook
21:27:10 │ERROR browser[SEVERE] http://localhost:61111/api/saved_objects/_bulk_get - Failed to load resource: the server responded with a status of 403 (Forbidden)
21:27:10 │ERROR browser[SEVERE] http://localhost:61111/34012/bundles/core/core.entry.js 19:16140
21:27:13 └-> correctly parses triple quotes in JSON
21:27:13 └-> "before each" hook: global before each
21:27:39 └- ✓ pass (22.7s) "console Search Profiler Editor correctly parses triple quotes in JSON"
21:27:39 └-> "after all" hook
…e. (elastic#69841) * using test_user with limited read permission to search profiler test * gitcheck * search profiler test using test_user
Ref meta issue: #60815
Summary
This PR replaces
elastic
superuser withtest_user
with read access only to access Search Profiler .global_devtools_read
Checklist
Delete any items that are not applicable to this PR.