search

opendistro-for-elasticsearch / opendistro-build

🧰 Open Distro Build Scripts
https://opendistro.github.io/
Apache License 2.0
344 stars 175 forks source link

Indentation bug fix in values.yaml : Security Configuration #688

Closed mdiver closed 3 years ago

mdiver commented 3 years ago

Issue #, if available:

687

Description of changes: Lines 175 to 181 are separated by 3 spaces, so change to 2 spaces. 스크린샷 2021-03-15 23 09 02

Test Results: By configuring Security Configuration normally, it is successful to install helm by configuring oidc backend.

Note: If this PR is related to Helm, please also update the README for related documentation changes. Thanks. https://github.com/opendistro-for-elasticsearch/opendistro-build/blob/master/helm/README.md

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 signing off your commits, please check here.

sreekarjami commented 3 years ago

Hi @mdiver, Thanks for informing about the bug and opening PR for the fix. Can you please send the test results after the change.

Thanks for your contribution.

peterzhuamazon commented 3 years ago

Thanks @mdiver we have approved and merged this PR.