-
Notifications
You must be signed in to change notification settings - Fork 169
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
add set_es_document_id parameter in doc and fix es_datastream_name #3914
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
This pull request does not have a backport label. Could you fix it @kaiyan-sheng? 🙏
|
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.
Thanks! I pushed a small change.
I'm monitoring the ms-108 release. It looks like we'll be able to merge this either later today or on Monday.
ms-108 is live |
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.
🐦
…3914) * add set_es_document_id parameter in doc and fix es_datastream_name * Update monitor-aws-firehose.asciidoc --------- Co-authored-by: Brandon Morelli <[email protected]> (cherry picked from commit dc68564)
…3914) (#3967) * add set_es_document_id parameter in doc and fix es_datastream_name * Update monitor-aws-firehose.asciidoc --------- Co-authored-by: Brandon Morelli <[email protected]> (cherry picked from commit dc68564) Co-authored-by: kaiyan-sheng <[email protected]>
This PR is to add documentation for the new config parameter added in this PR: https://github.com/elastic/cloud/pull/126020
This PR should be merged after ms-108 public release.
Also I see
es_datastream_name
documentation is outdated after this PR got merged.