search

opensearch-project / flow-framework

OpenSearch plugin that enables builders to innovate AI apps on OpenSearch
Apache License 2.0
32 stars 31 forks source link

[Backport-2.x] Added user level access control based on backend roles (#838) #847

Closed owaiskazi19 closed 1 month ago

owaiskazi19 commented 1 month ago

Backport of https://github.com/opensearch-project/flow-framework/pull/838

Signed-off-by: owaiskazi19 owaiskazi19@gmail.com Signed-off-by: Owais owaiskazi19@gmail.com (cherry picked from commit 60458a6275a2fe358a236583bce29db821d09ddc)

Description

[Describe what this change achieves]

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

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.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 71.95767% with 106 lines in your changes missing coverage. Please review.

Project coverage is 77.44%. Comparing base (f032867) to head (a20b569). Report is 1 commits behind head on 2.x.

Files Patch % Lines
.../org/opensearch/flowframework/util/ParseUtils.java 58.66% 26 Missing and 5 partials :warning:
...rk/transport/ProvisionWorkflowTransportAction.java 66.66% 20 Missing and 3 partials :warning:
.../transport/ReprovisionWorkflowTransportAction.java 79.68% 12 Missing and 1 partial :warning:
...ework/transport/CreateWorkflowTransportAction.java 70.00% 8 Missing and 1 partial :warning:
...ork/transport/GetWorkflowStateTransportAction.java 78.57% 5 Missing and 1 partial :warning:
...ework/transport/DeleteWorkflowTransportAction.java 75.00% 5 Missing :warning:
...ramework/transport/GetWorkflowTransportAction.java 80.76% 4 Missing and 1 partial :warning:
.../transport/DeprovisionWorkflowTransportAction.java 83.33% 4 Missing :warning:
...flowframework/transport/handler/SearchHandler.java 84.61% 3 Missing and 1 partial :warning:
.../transport/SearchWorkflowStateTransportAction.java 40.00% 3 Missing :warning:
... and 1 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## 2.x #847 +/- ## ============================================ - Coverage 77.54% 77.44% -0.10% - Complexity 928 963 +35 ============================================ Files 96 97 +1 Lines 4359 4536 +177 Branches 413 423 +10 ============================================ + Hits 3380 3513 +133 - Misses 806 841 +35 - Partials 173 182 +9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.