-
Notifications
You must be signed in to change notification settings - Fork 179
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
[8.x] Custom roles & privileges: Revise, reorder, add serverless requirements pages (backport #5779) #5850
Conversation
…ts pages (#5779) * First draft: add Defend feature privs page * Test if bad link within conditional will break * Alas, CI doesn't catch bad links in conditionals (rolling back the test) * Temporarily comment out conditionals We have to do this to create a PR preview * Li'l edit for active voice * Reorg serverless req pages to match ESS * Align page titles * Move serverless source files for req pages * Remove old draft files (duplicates) * Revise "Elastic Security requirements" * Minor edits * Another edit * Revise "Network map data" page * Minor revision to "ML job and rule reqs" * Change terminology: predefined user role * Revise "Entity risk scoring" page * Revise "Cases requirements" Lil edit in AsciiDoc too * Remove image Seems unnecessary, and would require custom image each for ESS and serverless * Remove image: Defend feature privs Seems unnecessary and is highly subject to change. May also require custom image each for ESS and serverless * Revise "Detections requirements" Add big ol' table back in * Temporarily comment out conditionals in Detections * Revise "Advanced settings" * Revise "Endpoint response actions" * Un-comment conditionals Previously commented out to generate PR preview * Indent conditional contents (for readability) * Fix ML requirements * Apply feedback from Janeen's review (cherry picked from commit 9b556a7) # Conflicts: # docs/serverless/advanced-entity-analytics/ml-requirements.mdx # docs/serverless/endpoint-response-actions/response-actions.mdx # docs/serverless/explore/conf-map-ui.mdx # docs/serverless/serverless-security.docnav.json # docs/serverless/settings/advanced-settings.mdx
Cherry-pick of 9b556a7 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
run docs-build |
Contributes to https://github.com/elastic/security-docs-internal/issues/11.
Also incorporates changes originally drafted in #5763 and #5772, but consolidated into this branch.
Previews
Preview serverless
Generating a preview is tricky, because a lot of content currently is hidden behind the
serverlessCustomRoles
docs feature flag until the feature is available in Production (ETA TBA). I temporarily commented out the conditionals to show the content for review purposes, and this is the latest preview build without the conditionals:https://elastic-dot-co-docs-production-bc2c75hpv-elastic-dev.vercel.app/current/serverless/security/what-is-security-serverless
I then un-commented the conditionals to re-hide the new content behind the flag. (This is more complicated than preferred, but I'd rather get this content merged soon — even with the added complexity — so we can continue to build around it, versus the content only existing in a PR branch and getting stale).
Preview ESS
These pages have minor edits for consistency with serverless counterparts:
Summary of changes
Adding content
For serverless MVP, we omitted a lot of content because custom roles and feature privileges weren't available yet. We can add that content back in now. As much as possible, I've tried to make the serverless content match ESS exactly, but there are spots where we also need to mention serverless-specific predefined roles (since serverless customers can use either predefined or custom roles now).
In some areas, I also tried not to introduce granular differences like changing
Kibana
toElastic Security
, because our assumptions have changed about that (the term still appears all over the serverless UI). Unifying content across serverless and ESS is a priority now.Reordering pages
We no longer need a separate section of requirements pages in serverless, so this PR moves those pages to locations that match ESS docs: each page is within a feature section.
Aligning titles
Our requirements pages currently go by lots of names: some are "prerequisites," some are "requirements," and one is even "prerequisites and requirements"! This PR uses a single unified name for all pages (for example, "Detections requirements"), in both ESS and serverless.
Housekeeping
This PR also moves around the serverless source files so they'll be easier to find in their respective feature folders. Doesn't affect output but improves authoring experience.
This is an automatic backport of pull request #5779 done by Mergify.