-
Notifications
You must be signed in to change notification settings - Fork 151
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
[DevOps] adding readme for how to deploy elasticstack using helm charts #565
[DevOps] adding readme for how to deploy elasticstack using helm charts #565
Conversation
For APM an issue has been opened to track #566 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. This is enough of a readme to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's more appropriate if we file this information under docs/
. Maybe docs/helm
or something like that.
Er... make that |
Moving to Docs makes sense. I was hoping that this README would live next to the customized values yaml files. BUT maybe this does make more sense in the docs directory and a more terse readme can live here, and reference the docs directory. |
@fxdgear We have generally been keeping documentation in |
…to nick/elastic-stack-helm
@lottspot PTAL. I've updated the locations of files as well as updated the README to reference the new locations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Link to issue
Changelog entry
Additional info
This has been deployed already for testing into the
test-es
namespace on preprod. This namespace can be deleted at anytime.I'd suggest a review of the available overrides for each helmchart to ensure that the right settings are getting set that meet the needs for this project.
The default settings have been applied to each service in the Elastic Stack so far and that means:
Pull Request Checklist
For Reviewer