-
-
Notifications
You must be signed in to change notification settings - Fork 50
docs: Clarify Complete Example and Inputs #30
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
docs: Clarify Complete Example and Inputs #30
Conversation
Signed-off-by: younsl <[email protected]>
* To help users understand, an example setting for the multi_az_with_standby_enabled value is explicitly declared. Signed-off-by: younsl <[email protected]>
* Included multi-AZ standby and dedicated master node settings for clarity. * Provided a link to AWS documentation for additional context on off-peak windows. * Sort cluster_config values
* Using pre-commit run -a before submit PR
* Apply linting in example written by markdown.
|
why? |
|
@bryantbiggs To make it easier for Terraform users to work with the module. |
|
How so? |
Based on my user experience (using this module Additionally, the fact that |
|
thank you but unfortunately, this is what the documented variables are for. examples just demonstrate various configurations but they are not exhaustive. The documented inputs are there to show users what values the module accepts as inputs, not the examples https://github.com/terraform-aws-modules/terraform-aws-opensearch/blob/master/README.md#inputs |
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
No code changes, only update docs and
completeexamples.Motivation and Context
engine_versionandoff_peak_window_optionsinput in docsBreaking Changes
How Has This Been Tested?
examples/*to demonstrate and validate my change(s)examples/*projectspre-commit run -aon my pull request