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] Read Only Tenant Mode #5436

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport abc97ea from #4498.

* merge conflict resolved
* Restore config/opensearch_dashboards.yml
* Fix capabilities tsc
* docs: read only tenant mode
* feat: introduce security service in core and readonly service
* fix: adds securityServiceMock
* feat: adds tests and default default readonly service
* docs: fill up docs for read only tenant mode

---------

Signed-off-by: jakubp-eliatra <jakub.przybylski@theworkingroup.net>
Signed-off-by: Kajetan Nobel <k.nobel@routegroup.pl>
Signed-off-by: Kajetan Nobel <kajetan.nobel@eliatra.com>
Co-authored-by: jakubp-eliatra <jakub.przybylski@theworkingroup.net>
Co-authored-by: Kajetan Nobel <k.nobel@routegroup.pl>
Co-authored-by: Peter Nied <peternied@hotmail.com>
Co-authored-by: Ashwin P Chandran <ashwinpc@amazon.com>
Co-authored-by: Kajetan Nobel <kajetan.nobel@eliatra.com>
Co-authored-by: Anan Zhuang <ananzh@amazon.com>
(cherry picked from commit abc97ea)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #5436 (e2fbfc4) into 2.x (603ecff) will decrease coverage by 0.01%.
The diff coverage is 82.60%.

@@            Coverage Diff             @@
##              2.x    #5436      +/-   ##
==========================================
- Coverage   66.81%   66.81%   -0.01%     
==========================================
  Files        3284     3287       +3     
  Lines       63149    63172      +23     
  Branches    10048    10049       +1     
==========================================
+ Hits        42192    42206      +14     
- Misses      18491    18498       +7     
- Partials     2466     2468       +2     
Flag Coverage Δ
Linux_1 35.26% <0.00%> (-0.01%) ⬇️
Linux_2 55.29% <85.71%> (+0.02%) ⬆️
Linux_3 43.82% <19.04%> (-0.03%) ⬇️
Linux_4 ?
Windows_1 35.28% <0.00%> (-0.01%) ⬇️
Windows_2 55.26% <85.71%> (+0.02%) ⬆️
Windows_3 43.83% <19.04%> (-0.02%) ⬇️
Windows_4 35.48% <21.73%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...rc/core/public/application/application_service.tsx 83.44% <ø> (ø)
src/core/server/index.ts 100.00% <ø> (ø)
src/core/server/legacy/legacy_service.ts 74.24% <ø> (ø)
src/core/server/mocks.ts 100.00% <ø> (ø)
src/core/server/plugins/plugin_context.ts 45.45% <ø> (ø)
src/core/server/security/security_service.mock.ts 100.00% <100.00%> (ø)
src/core/server/security/security_service.ts 100.00% <100.00%> (ø)
src/core/server/server.ts 98.92% <100.00%> (+0.04%) ⬆️
src/core/server/security/readonly_service.ts 50.00% <50.00%> (ø)
.../plugins/saved_objects_management/server/plugin.ts 58.33% <50.00%> (-1.67%) ⬇️
... and 1 more

... and 2 files with indirect coverage changes

@joshuarrrr joshuarrrr merged commit b635c4b into 2.x Nov 17, 2023
83 of 84 checks passed
@joshuarrrr joshuarrrr deleted the backport/backport-4498-to-2.x branch November 17, 2023 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports first-time-contributor v2.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants