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

Documentation for security roles to be used for DLS parameter substit… #420

Merged
merged 1 commit into from
May 25, 2022

Conversation

mm-govau
Copy link
Contributor

@mm-govau mm-govau commented Feb 22, 2022

…ution. Added under opensearch-project/security/#1568

Signed-off-by: Matthew Moreland matthew.moreland1@defence.gov.au

Description

Minor update to add in the parameter substitution implemented under #1588.

Issues Resolved

Related to #1568.

Check List

  • Commits are signed per the DCO using --signoff

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

…ution. Added under opensearch-project/security/opensearch-project#1568

Signed-off-by: Matthew Moreland <matthew.moreland1@defence.gov.au>
Copy link
Contributor

@keithhc2 keithhc2 left a comment

Choose a reason for hiding this comment

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

Thanks @mm-govau! I'm guessing we can merge this PR when the mentioned security PR gets merged?

@mm-govau
Copy link
Contributor Author

Thanks @mm-govau! I'm guessing we can merge this PR when the mentioned security PR gets merged?

Sounds like a plan to me! Thanks @keithhc2 and @Naarcha-AWS for the speedy review/approval. I'll keep an eye on the PR and post back when we're in.

@keithhc2 keithhc2 added the upcoming release Don't merge until the version or feature is available label Feb 25, 2022
@Naarcha-AWS Naarcha-AWS merged commit 146a01c into opensearch-project:main May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upcoming release Don't merge until the version or feature is available
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants