Skip to content

[ML] Include authorization info in responses when creating jobs#87950

Merged
droberts195 merged 4 commits intoelastic:masterfrom
droberts195:auth_info_for_put_update
Jun 23, 2022
Merged

[ML] Include authorization info in responses when creating jobs#87950
droberts195 merged 4 commits intoelastic:masterfrom
droberts195:auth_info_for_put_update

Conversation

@droberts195
Copy link
Copy Markdown

#87884 added
authorization information to the datafeed and data frame
analytics job configs returned by listing them, but not to
the ones returned from creating or updating them. For
consistency it's best that the same fields are present in
both places.

elastic#87884 added
authorization information to the datafeed and data frame
analytics job configs returned by listing them, but not to
the ones returned from creating or updating them. For
consistency it's best that the same fields are present in
both places.
@droberts195 droberts195 added >non-issue :ml Machine learning Team:Clients Meta label for clients team v8.4.0 labels Jun 23, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/clients-team (Team:Clients)

@elasticmachine elasticmachine added the Team:ML Meta label for the ML team label Jun 23, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/ml-core (Team:ML)

@droberts195
Copy link
Copy Markdown
Author

Note: this is a >non-issue because it's logically part of #87884. The doc changes are being tracked in #87905.

Copy link
Copy Markdown
Contributor

@dimitris-athanasiou dimitris-athanasiou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@droberts195 droberts195 merged commit ced9e5c into elastic:master Jun 23, 2022
@droberts195 droberts195 deleted the auth_info_for_put_update branch June 23, 2022 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml Machine learning >non-issue Team:Clients Meta label for clients team Team:ML Meta label for the ML team v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants