ManagedIndex Explain Action - Index permissions check fix - #783
Open
petardz wants to merge 13 commits into
Open
Conversation
petardz
requested review from
Hailong-am,
bowenlan-amzn,
gaobinlong,
getsaurabh02,
lezzago,
praveensameneni and
xluo-aws
as code owners
May 16, 2023 23:02
Codecov Report
@@ Coverage Diff @@
## main #783 +/- ##
============================================
+ Coverage 75.53% 75.63% +0.10%
- Complexity 2831 2833 +2
============================================
Files 364 364
Lines 16176 16177 +1
Branches 2324 2324
============================================
+ Hits 12219 12236 +17
+ Misses 2619 2603 -16
Partials 1338 1338
|
| } | ||
|
|
||
| private fun searchForMetadata(searchRequest: SearchRequest) { | ||
| client.threadPool().threadContext.stashContext().use { threadContext -> |
Member
There was a problem hiding this comment.
Stash context so that plugin can search the ISM system index, otherwise the search won't return anything
Contributor
Author
There was a problem hiding this comment.
this is called from coroutine so TLS should be clean at this point
petardz
force-pushed
the
explain-index-check-fix
branch
from
May 19, 2023 00:30
d80fcd1 to
7f73a2c
Compare
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
…urity tests into security package Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
…roject#722) * notification crud api Signed-off-by: zhichao-aws <zhichaog@amazon.com> * changes based on comments; add tests Signed-off-by: zhichao-aws <zhichaog@amazon.com> * changes for comment Signed-off-by: zhichao-aws <zhichaog@amazon.com> * change behavior for get action index not exists, fix test cases Signed-off-by: zhichao-aws <zhichaog@amazon.com> * fix weak password for changes in security plugin Signed-off-by: zhichao-aws <zhichaog@amazon.com> --------- Signed-off-by: zhichao-aws <zhichaog@amazon.com>
…urity tests into security package Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
petardz
force-pushed
the
explain-index-check-fix
branch
2 times, most recently
from
May 31, 2023 23:37
4b7d6a2 to
95d4c8c
Compare
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
Signed-off-by: Petar Dzepina <petar.dzepina@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
Fixed verifying index permissions when calling Explain API on managed index
CheckList:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.