Skip to content

Add authorization object to ML datafeed, job, transform specifications#1780

Merged
sethmlarson merged 7 commits intoelastic:mainfrom
lcawl:ml-auth
Jul 5, 2022
Merged

Add authorization object to ML datafeed, job, transform specifications#1780
sethmlarson merged 7 commits intoelastic:mainfrom
lcawl:ml-auth

Conversation

@lcawl
Copy link
Copy Markdown
Member

@lcawl lcawl commented Jul 4, 2022

@lcawl lcawl changed the title Add authorization object to ML datafeed specification Add authorization object to ML datafeed and job specifications Jul 4, 2022
@lcawl lcawl changed the title Add authorization object to ML datafeed and job specifications Add authorization object to ML datafeed, job, transform specifications Jul 4, 2022
@lcawl lcawl requested a review from sethmlarson July 5, 2022 14:50
@sethmlarson
Copy link
Copy Markdown
Contributor

@lcawl The linter is unhappy, can you run make contrib and commit the results? Also next time you contribute you can run from a branch in the elastic/elasticsearch-specification rather than a fork so our API validator can run.

Copy link
Copy Markdown
Contributor

@sethmlarson sethmlarson left a comment

Choose a reason for hiding this comment

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

LGTM

@sethmlarson sethmlarson merged commit 1b56d7e into elastic:main Jul 5, 2022
@lcawl lcawl deleted the ml-auth branch July 5, 2022 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants