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

Adding security installation information #20231130 #6605

Conversation

AntonEliatra
Copy link
Contributor

Description

Adding security Installation page in case users disabled the security plugin or installed Min Version of OpenSearch

Issues Resolved

Relates to user comment in #4314 #20231130

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.

@AntonEliatra AntonEliatra mentioned this pull request Mar 6, 2024
20 tasks
fixed a dead link

Signed-off-by: AnthonyEliatra <[email protected]>
@Naarcha-AWS Naarcha-AWS self-assigned this Mar 6, 2024
@AntonEliatra AntonEliatra marked this pull request as draft March 6, 2024 16:41
@AntonEliatra AntonEliatra marked this pull request as ready for review March 11, 2024 19:03
@hdhalter hdhalter added the 4 - Doc review PR: Doc review in progress label Mar 11, 2024
Copy link
Contributor

@hdhalter hdhalter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @AntonEliatra ! Looks good! Just a few suggestions for the title and some style things.

_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
_security/configuration/security-installation.md Outdated Show resolved Hide resolved
AntonEliatra and others added 10 commits March 12, 2024 09:23
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
@AntonEliatra
Copy link
Contributor Author

Excellent suggestions @hdhalter, I've now updated accordingly

@Naarcha-AWS Naarcha-AWS added the backport 2.12 PR: Backport label for 2.12 label Mar 27, 2024
Copy link
Contributor

@hdhalter hdhalter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, a few tweaks and a question.

_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
AntonEliatra and others added 5 commits March 29, 2024 12:52
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
@AntonEliatra AntonEliatra requested a review from epugh as a code owner April 3, 2024 15:12
@Naarcha-AWS Naarcha-AWS dismissed hdhalter’s stale review April 3, 2024 15:12

Review feedback already implemented.

Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AntonEliatra @Naarcha-AWS Please see my comments and changes and let me know if you have any questions. Thanks!

_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
_security/configuration/disable-enable-security.md Outdated Show resolved Hide resolved
AntonEliatra and others added 2 commits April 4, 2024 15:45
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
@Naarcha-AWS Naarcha-AWS merged commit 1350c5a into opensearch-project:main Apr 4, 2024
4 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 4, 2024
* additing security installation page #20231130

Signed-off-by: Anthony7774 <[email protected]>

* Update security-installation.md

Signed-off-by: Anthony7774 <[email protected]>

* Update security-installation.md

Signed-off-by: Anthony7774 <[email protected]>

* Update security-installation.md

fixed a dead link

Signed-off-by: AnthonyEliatra <[email protected]>

* Update security-installation.md

Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* renaming to enabling security

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* Update enabling-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update enabling-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* moving security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* moving security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* fixes on security disable-enable page

Signed-off-by: AntonEliatra <[email protected]>

* Made some updates to the structure

Signed-off-by: Heather Halter <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* fixes on security disable-enable page

Signed-off-by: AntonEliatra <[email protected]>

* adding link for installation method on security installation page

Signed-off-by: AntonEliatra <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Heather Halter <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update disable-enable-security.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Update disable-enable-security.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Anthony7774 <[email protected]>
Signed-off-by: AnthonyEliatra <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
(cherry picked from commit 1350c5a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Naarcha-AWS Naarcha-AWS added the backport 2.13 PR: Backport label for 2.13 label Apr 4, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 4, 2024
* additing security installation page #20231130

Signed-off-by: Anthony7774 <[email protected]>

* Update security-installation.md

Signed-off-by: Anthony7774 <[email protected]>

* Update security-installation.md

Signed-off-by: Anthony7774 <[email protected]>

* Update security-installation.md

fixed a dead link

Signed-off-by: AnthonyEliatra <[email protected]>

* Update security-installation.md

Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/security-installation.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* renaming to enabling security

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* additing security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* Update enabling-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update enabling-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* moving security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* moving security installation page #20231130

Signed-off-by: AntonEliatra <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

* fixes on security disable-enable page

Signed-off-by: AntonEliatra <[email protected]>

* Made some updates to the structure

Signed-off-by: Heather Halter <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* fixes on security disable-enable page

Signed-off-by: AntonEliatra <[email protected]>

* adding link for installation method on security installation page

Signed-off-by: AntonEliatra <[email protected]>

* Update disable-enable-security.md

Signed-off-by: Heather Halter <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update _security/configuration/disable-enable-security.md

Co-authored-by: Heather Halter <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update disable-enable-security.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Update disable-enable-security.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Anthony7774 <[email protected]>
Signed-off-by: AnthonyEliatra <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
(cherry picked from commit 1350c5a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Naarcha-AWS added a commit that referenced this pull request Apr 4, 2024
* additing security installation page #20231130



* Update security-installation.md



* Update security-installation.md



* Update security-installation.md

fixed a dead link



* Update security-installation.md



* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* Update _security/configuration/security-installation.md




* renaming to enabling security



* additing security installation page #20231130



* additing security installation page #20231130



* additing security installation page #20231130



* additing security installation page #20231130



* Update enabling-security.md



* Update enabling-security.md



* moving security installation page #20231130



* moving security installation page #20231130



* Update disable-enable-security.md



* Update disable-enable-security.md



* Update disable-enable-security.md



* Apply suggestions from code review



* Apply suggestions from code review




* fixes on security disable-enable page



* Made some updates to the structure



* Apply suggestions from code review



* Apply suggestions from code review



* Apply suggestions from code review



* fixes on security disable-enable page



* adding link for installation method on security installation page



* Update disable-enable-security.md



* Update _security/configuration/disable-enable-security.md




* Update _security/configuration/disable-enable-security.md




* Update _security/configuration/disable-enable-security.md




* Update _security/configuration/disable-enable-security.md




* Update disable-enable-security.md



* Apply suggestions from code review



* Update disable-enable-security.md



* Apply suggestions from code review




* Apply suggestions from code review



---------









(cherry picked from commit 1350c5a)

Signed-off-by: Anthony7774 <[email protected]>
Signed-off-by: AnthonyEliatra <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Signed-off-by: Heather Halter <[email protected]>
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>
Co-authored-by: Heather Halter <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Editorial review PR: Editorial review in progress backport 2.12 PR: Backport label for 2.12 backport 2.13 PR: Backport label for 2.13 security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants