-
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
[ML] Data Frame Analytics audit messages api test #125325
[ML] Data Frame Analytics audit messages api test #125325
Conversation
Pinging @elastic/ml-ui (:ml) |
85ca4b9
to
2cd2f89
Compare
); | ||
}); | ||
|
||
it('should not allow to retrieve a job stats for the user without required permissions', async () => { |
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.
Is this a copy and paste from one of the job stats tests?
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.
Yep - accidentally left in - fixed in 49371743f19221b4c20d62d87e9227545366a280
This has been updated and is ready for a final look cc @pheyos, @peteharverson |
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
4937174
to
174ce1a
Compare
174ce1a
to
c0c1a13
Compare
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
* wip: add beginning of messages api test * adds dfa audit messages api test * use retry instead of running job (cherry picked from commit eafa4d9)
Summary
Related meta issue: #117501
Checklist
Delete any items that are not applicable to this PR.