Skip to content
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

[Backport 2.x] Integrate k-NN functionality with security plugin #2455

Merged
merged 3 commits into from
Feb 21, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e7a120c from #2274

Adds k-NN read only and full access roles to the default roles file.
Adds k-NN model index to demo for system index.

Signed-off-by: John Mazanec <jmazane@amazon.com>
(cherry picked from commit e7a120c)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team February 21, 2023 20:10
@codecov-commenter
Copy link

codecov-commenter commented Feb 21, 2023

Codecov Report

Merging #2455 (a333212) into 2.x (823739a) will increase coverage by 0.02%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #2455      +/-   ##
============================================
+ Coverage     61.07%   61.09%   +0.02%     
- Complexity     3252     3253       +1     
============================================
  Files           258      258              
  Lines         18130    18130              
  Branches       3231     3231              
============================================
+ Hits          11073    11077       +4     
+ Misses         5486     5483       -3     
+ Partials       1571     1570       -1     
Impacted Files Coverage Δ
...ecurity/configuration/ConfigurationRepository.java 72.13% <0.00%> (-2.19%) ⬇️
...security/auditlog/sink/ExternalOpenSearchSink.java 61.72% <0.00%> (+2.46%) ⬆️
...nsearch/security/dlic/rest/api/AuditApiAction.java 68.08% <0.00%> (+4.25%) ⬆️
...ch/security/auditlog/routing/AsyncStoragePool.java 55.55% <0.00%> (+5.55%) ⬆️
...ecurity/configuration/StaticResourceException.java 25.00% <0.00%> (+25.00%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@cwperks cwperks merged commit 6ded82d into 2.x Feb 21, 2023
@cwperks cwperks deleted the backport/backport-2274-to-2.x branch February 21, 2023 23:23
jmazanec15 added a commit to jmazanec15/security that referenced this pull request Feb 24, 2023
peternied added a commit that referenced this pull request Feb 24, 2023
peternied added a commit that referenced this pull request Feb 24, 2023
)"

This reverts commit 6ded82d.

Resolves #2478

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
…#2274) (opensearch-project#2455)

Adds k-NN read only and full access roles to the default roles file.
Adds k-NN model index to demo for system index.

Signed-off-by: John Mazanec <jmazane@amazon.com>
(cherry picked from commit e7a120c)

Co-authored-by: John Mazanec <jmazane@amazon.com>
Co-authored-by: Craig Perkins <cwperx@amazon.com>
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
…-project#2274) (opensearch-project#2455)"

This reverts commit 6ded82d.

Resolves opensearch-project#2478

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants