-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Serverless] Disable UI of Users, Roles, and Role Mappings #158186
[Serverless] Disable UI of Users, Roles, and Role Mappings #158186
Conversation
Pinging @elastic/platform-deployment-management (Team:Deployment Management) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ElenaStoeva thanks for putting this PR together for us, much appreciated! I left a couple of comments below for your review.
x-pack/plugins/security/public/management/management_service.test.ts
Outdated
Show resolved
Hide resolved
Thanks a lot for the review @legrego! I made the suggested changes and left a couple of follow-up comments. Let me know your thoughts on these. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look great, thanks! One last request around unit tests below, then I think we're good to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM on green CI!
💚 Build Succeeded
Metrics [docs]Page load bundle
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @ElenaStoeva |
…58186) Partially addresses elastic#157756 ## Summary This PR makes the UI of the following Security apps disable-able for serverless: - Users - Roles - Role Mappings **How to test:** 1. Start Elasticsearch with `yarn es snapshot` and Kibana with yarn `serverless-{mode}` where `{mode}` can be `es`, `security`, or `oblt`. 2. Verify that the Users app is not accessible and its path (`management/security/users`) leads to the Stack Management landing page. 3. Verify that the Roles app is not accessible and its path (`management/security/roles`) leads to the Stack Management landing page. 4. Verify that the Role Mappings app is not accessible and its path (`management/security/role_mappings`) leads to the Stack Management landing page. Test the Security apps in regular (non-serverless) mode: 1. Start Elasticsearch with `yarn es snapshot` and Kibana with `yarn start`. 2. Verify that Users, Roles, and Role Mappings apps work as expected.
…#162187) This is a follow-up to #160671, where the Management plugins were disabled using `contextRef`. The configs for disabling the UI of the security management plugins were added in #158186. In this PR, they are changed so that they follow the same convention for disabling the Management plugins - setting the default values of the configs to `true` and explicitly setting them to `false` in the `serverless.yml` file. This way, we have a clear view in `serverless.yml` of all plugins/functionalities that have been disabled.
…elastic#162187) This is a follow-up to elastic#160671, where the Management plugins were disabled using `contextRef`. The configs for disabling the UI of the security management plugins were added in elastic#158186. In this PR, they are changed so that they follow the same convention for disabling the Management plugins - setting the default values of the configs to `true` and explicitly setting them to `false` in the `serverless.yml` file. This way, we have a clear view in `serverless.yml` of all plugins/functionalities that have been disabled.
Partially addresses #157756
Summary
This PR makes the UI of the following Security apps disable-able for serverless:
How to test:
yarn es snapshot
and Kibana with yarnserverless-{mode}
where{mode}
can bees
,security
, oroblt
.management/security/users
) leads to the Stack Management landing page.management/security/roles
) leads to the Stack Management landing page.management/security/role_mappings
) leads to the Stack Management landing page.Test the Security apps in regular (non-serverless) mode:
yarn es snapshot
and Kibana withyarn start
.