Skip to content

[Backport 2.x] System index permissions (#2887)#3325

Merged
peternied merged 1 commit intoopensearch-project:2.xfrom
DarshitChanpura:backport/backport-2887-to-2.x
Sep 7, 2023
Merged

[Backport 2.x] System index permissions (#2887)#3325
peternied merged 1 commit intoopensearch-project:2.xfrom
DarshitChanpura:backport/backport-2887-to-2.x

Conversation

@DarshitChanpura
Copy link
Copy Markdown
Member

@DarshitChanpura DarshitChanpura commented Sep 7, 2023

Backports #2887 to 2.x
(cherry picked from commit 1379234)

Manual backport was required since SystemIndicesTests.java was deleted in main in lieu of the new testing for system indices. Also had to change http5 imports in tests to http4

Testing

automated tests

Check List

  • New functionality includes testing
  • New functionality has been documented
  • Commits are signed per the DCO using --signoff

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.

System index permissions

Signed-off-by: Sam <samuel.costa@eliatra.com>
Signed-off-by: Sam <128482925+samuelcostae@users.noreply.github.com>
Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Darshit Chanpura <dchanp@amazon.com>
Co-authored-by: Peter Nied <peternied@hotmail.com>
(cherry picked from commit 1379234)
@DarshitChanpura DarshitChanpura force-pushed the backport/backport-2887-to-2.x branch from 99778a2 to a4bd99d Compare September 7, 2023 15:13
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 7, 2023

Codecov Report

Merging #3325 (a4bd99d) into 2.x (552de1b) will increase coverage by 0.98%.
The diff coverage is 78.89%.

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #3325      +/-   ##
============================================
+ Coverage     63.10%   64.08%   +0.98%     
- Complexity     3343     3366      +23     
============================================
  Files           256      256              
  Lines         19456    19515      +59     
  Branches       3286     3297      +11     
============================================
+ Hits          12277    12507     +230     
+ Misses         5584     5364     -220     
- Partials       1595     1644      +49     
Files Changed Coverage Δ
...pensearch/security/securityconf/ConfigModelV6.java 31.71% <52.63%> (+31.71%) ⬆️
...pensearch/security/securityconf/ConfigModelV7.java 67.66% <66.66%> (-0.44%) ⬇️
...urity/privileges/SecurityIndexAccessEvaluator.java 79.80% <87.87%> (+4.48%) ⬆️
.../opensearch/security/OpenSearchSecurityPlugin.java 84.46% <100.00%> (+0.07%) ⬆️
...earch/security/privileges/PrivilegesEvaluator.java 73.29% <100.00%> (+0.08%) ⬆️
...g/opensearch/security/support/ConfigConstants.java 95.00% <100.00%> (+0.26%) ⬆️

... and 8 files with indirect coverage changes

@DarshitChanpura
Copy link
Copy Markdown
Member Author

code-ql is failing because of the new test framework is not present in 2.x: #3230

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v2.10.0 Issues targeting release v2.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants