Add path to the script for the securityadmin.sh troubleshooting topic #3645

cwillum · 2023-03-30T23:16:16Z

Description

The securityadmin.sh troubleshooting topic does not provide a path to the script so users can find it in their distribution.

Issues Resolved

Added the path to the securityadmin tool and a link to the topic that describes using the tool.

Fixes #3634.

Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: cwillum <cwmmoore@amazon.com>

_troubleshoot/security-admin.md

Signed-off-by: cwillum <cwmmoore@amazon.com>

cwillum · 2023-03-31T20:45:15Z

@hdhalter @Naarcha-AWS I'm not sure if explanation of the path where the script lives is necessary, especially for an admin who installed the instance of OpenSearch. But other members of the team have asked about this in the past. I imagine it could be helpful for some users. So I've added a couple sentences in the securityadmin.sh topic to draw some context.

Signed-off-by: cwillum <cwmmoore@amazon.com>

cwillum · 2023-04-03T18:24:05Z

@Naarcha-AWS I provided a little more explanation of where the tool is located in an OpenSearch installation. If this looks good, could you approve this? Thanks.

Naarcha-AWS

Small suggestion, otherwise LGTM

_troubleshoot/security-admin.md

Naarcha-AWS

Found two more instances of the same.

_security/configuration/security-admin.md

Signed-off-by: cwillum <cwmmoore@amazon.com>

…#3645) * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> --------- Signed-off-by: cwillum <cwmmoore@amazon.com> (cherry picked from commit 89b16a9) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3689) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3688) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3687) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3686) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3685) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3684) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3683) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) (#3682) * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path * fix missing securityadmin path --------- (cherry picked from commit 89b16a9) Signed-off-by: cwillum <cwmmoore@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

…#3645) * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> --------- Signed-off-by: cwillum <cwmmoore@amazon.com>

…ng topic (#3645)" This reverts commit ecd5563.

…#3645) * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> --------- Signed-off-by: cwillum <cwmmoore@amazon.com>

…ng topic (#3645)" This reverts commit e18b2b1.

…opensearch-project#3645) * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> * fix missing securityadmin path Signed-off-by: cwillum <cwmmoore@amazon.com> --------- Signed-off-by: cwillum <cwmmoore@amazon.com>

fix missing securityadmin path

cc32c5d

Signed-off-by: cwillum <cwmmoore@amazon.com>

cwillum self-assigned this Mar 30, 2023

cwillum requested a review from a team as a code owner March 30, 2023 23:16

cwillum mentioned this pull request Mar 30, 2023

[DOC] Feedback on /docs/latest/troubleshoot/security-admin/ page #3634

Closed

4 tasks

hdhalter reviewed Mar 30, 2023

View reviewed changes

_troubleshoot/security-admin.md Outdated Show resolved Hide resolved

hdhalter approved these changes Mar 30, 2023

View reviewed changes

cwillum added 2 commits March 31, 2023 13:29

fix missing securityadmin path

825f957

Signed-off-by: cwillum <cwmmoore@amazon.com>

fix missing securityadmin path

3c6a145

Signed-off-by: cwillum <cwmmoore@amazon.com>

fix missing securityadmin path

6442dd1

Signed-off-by: cwillum <cwmmoore@amazon.com>

cwillum requested a review from Naarcha-AWS April 3, 2023 20:53

Naarcha-AWS approved these changes Apr 5, 2023

View reviewed changes

_troubleshoot/security-admin.md Outdated Show resolved Hide resolved

Naarcha-AWS approved these changes Apr 5, 2023

View reviewed changes

_security/configuration/security-admin.md Outdated Show resolved Hide resolved

_security/configuration/security-admin.md Outdated Show resolved Hide resolved

fix missing securityadmin path

8effec9

Signed-off-by: cwillum <cwmmoore@amazon.com>

cwillum merged commit 89b16a9 into main Apr 5, 2023

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 1.3] Add path to the script for the securityadmin.sh troubleshooting topic #3682

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.0] Add path to the script for the securityadmin.sh troubleshooting topic #3683

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.1] Add path to the script for the securityadmin.sh troubleshooting topic #3684

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.2] Add path to the script for the securityadmin.sh troubleshooting topic #3685

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.3] Add path to the script for the securityadmin.sh troubleshooting topic #3686

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.4] Add path to the script for the securityadmin.sh troubleshooting topic #3687

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.5] Add path to the script for the securityadmin.sh troubleshooting topic #3688

Merged

opensearch-trigger-bot bot mentioned this pull request Apr 5, 2023

[Backport 2.6] Add path to the script for the securityadmin.sh troubleshooting topic #3689

Merged

cwillum removed the 4 - Doc review PR: Doc review in progress label Apr 5, 2023

cwillum deleted the fix#3634-securityadmin-location branch April 5, 2023 17:55

vagimeli added a commit that referenced this pull request Apr 25, 2023

Revert "Add path to the script for the securityadmin.sh troubleshooti…

619b3b3

…ng topic (#3645)" This reverts commit ecd5563.

vagimeli added a commit that referenced this pull request May 4, 2023

Revert "Add path to the script for the securityadmin.sh troubleshooti…

057db61

…ng topic (#3645)" This reverts commit e18b2b1.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add path to the script for the securityadmin.sh troubleshooting topic #3645

Add path to the script for the securityadmin.sh troubleshooting topic #3645

cwillum commented Mar 30, 2023 •

edited

Loading

cwillum commented Mar 31, 2023

cwillum commented Apr 3, 2023

Naarcha-AWS left a comment

Naarcha-AWS left a comment

Add path to the script for the securityadmin.sh troubleshooting topic #3645

Add path to the script for the securityadmin.sh troubleshooting topic #3645

Conversation

cwillum commented Mar 30, 2023 • edited Loading

Description

Issues Resolved

Checklist

cwillum commented Mar 31, 2023

cwillum commented Apr 3, 2023

Naarcha-AWS left a comment

Choose a reason for hiding this comment

Naarcha-AWS left a comment

Choose a reason for hiding this comment

cwillum commented Mar 30, 2023 •

edited

Loading