Skip to content

Conversation

@benwtrent
Copy link
Member

Wraps requests in the appropriate context so that the index can be written to in a secured cluster.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

Copy link

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

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

LGTM

@benwtrent
Copy link
Member Author

run the gradle build tests 1

1 similar comment
@benwtrent
Copy link
Member Author

run the gradle build tests 1

@benwtrent benwtrent merged commit 6b376a1 into elastic:master Jan 7, 2019
@benwtrent benwtrent deleted the bug/fix-annotations-on-secured-cluster branch January 7, 2019 21:18
benwtrent added a commit that referenced this pull request Jan 7, 2019
* changing executing context for writing annotation

* adjusting user

* removing unused import
benwtrent added a commit that referenced this pull request Jan 7, 2019
* changing executing context for writing annotation

* adjusting user

* removing unused import
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jan 7, 2019
…piration

* elastic/master:
  Removing unused methods in Numbers (elastic#37186)
  Fix setting by time unit (elastic#37192)
  [DOCS] Cleans up xpackml attributes
  ML: fix delayed data annotations on secured cluster (elastic#37193)
  Update version in SearchRequest and related test
  [DOCS] Adds overview and API ref for cluster voting configurations (elastic#36954)
  ML: changing JobResultsProvider.getForecastRequestStats to support > 1 index (elastic#37157)
  Separate out validation of groups of settings (elastic#34184)
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.

4 participants