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

Add note about unprivileged mode on 'install' tabs (take2) #1144

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kilfoyle
Copy link
Contributor

This is a second attempt since the first attempt fails CI checks.


This updates the Install standalone Elastic Agents docs page, so the install tabs for macOS, Linux, DEB and RPM would have the following note:

Screenshot 2024-06-27 at 12 53 58 PM

@ycombinator I know that in this comment you mentioned only RPM and DEB, but I think this note should apply to all four operating systems, right, since they all show sudo in the install command? I tried the phrase the text so that it applies to all four.

Rel: #4125

@kilfoyle kilfoyle requested a review from ycombinator June 27, 2024 16:56
@kilfoyle kilfoyle requested a review from a team as a code owner June 27, 2024 16:56
Copy link

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

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.

chriscressman
chriscressman previously approved these changes Jun 27, 2024
@ycombinator
Copy link
Contributor

ycombinator commented Jun 27, 2024

Let's wait to merge this PR until we have a resolution to elastic/elastic-agent#4125 (comment), in case it has any docs impact.

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

Successfully merging this pull request may close these issues.

3 participants