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

Update SAML tutorial with new default Admin settings #6279

Merged
merged 4 commits into from
Feb 1, 2024

Conversation

Naarcha-AWS
Copy link
Collaborator

This PR updates the SAML tutorial based off of PR #5887

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: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Copy link
Member

@DarshitChanpura DarshitChanpura left a comment

Choose a reason for hiding this comment

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

Ty @Naarcha-AWS for taking care of this!!

@@ -34,20 +34,20 @@ We provide a fully functional example that can help you understand how to use SA

```yml
opensearch-saml-node1:
image: opensearchproject/opensearch:2.8.0
image: opensearchproject/opensearch:2.12.0
Copy link
Member

Choose a reason for hiding this comment

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

we should change version on line 33 as well since it points to 2.6

Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
@Naarcha-AWS
Copy link
Collaborator Author

@DarshitChanpura: I went ahead and used our site variable here so it'll update to whatever version the user is on currently.

Copy link
Member

@DarshitChanpura DarshitChanpura left a comment

Choose a reason for hiding this comment

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

Nicely done @Naarcha-AWS !!

Copy link
Collaborator

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

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

LGTM

_security/authentication-backends/saml.md Outdated Show resolved Hide resolved
@hdhalter hdhalter added the release-notes PR: Include this PR in the automated release notes label Feb 1, 2024
Co-authored-by: Melissa Vagi <vagimeli@amazon.com>
Signed-off-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
@Naarcha-AWS Naarcha-AWS added 6 - Done but waiting to merge PR: The work is done and ready to merge and removed 4 - Doc review PR: Doc review in progress labels Feb 1, 2024
@hdhalter hdhalter merged commit 0801f38 into main Feb 1, 2024
5 checks passed
@hdhalter hdhalter added 3 - Done Issue is done/complete and removed 6 - Done but waiting to merge PR: The work is done and ready to merge labels Feb 1, 2024
@Naarcha-AWS Naarcha-AWS deleted the Naarcha-AWS-saml-tutorial branch March 28, 2024 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete release-notes PR: Include this PR in the automated release notes v2.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants