-
Notifications
You must be signed in to change notification settings - Fork 175
[8.16](backport #4818) [AWS Firehose] Improve Firehose documentation #4828
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
Conversation
Cherry-pick of 9813ea7 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
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. |
d52e832
to
9eecfed
Compare
* update firehose endpoint and data stream details Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co> * improve api key and retry duration section Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co> * fix wording of storage location Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co> * improve further by modifying all related documents Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co> * Update docs/en/observability/cloud-monitoring/aws/ingest-aws-firehose.asciidoc Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> --------- Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co> Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com> (cherry picked from commit 9813ea7) # Conflicts: # docs/en/observability/cloud-monitoring/aws/monitor-aws-cloudtrail-firehose.asciidoc # docs/en/observability/cloud-monitoring/aws/monitor-aws-cloudwatch-firehose.asciidoc # docs/en/observability/cloud-monitoring/aws/monitor-aws-firewall-firehose.asciidoc # docs/en/observability/cloud-monitoring/aws/monitor-aws-waf-firehose.asciidoc
9eecfed
to
087f8a4
Compare
This pull request has not been merged yet. Could you please review and merge it @Kavindu-Dodan? 🙏 |
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.
LGTM
Description
Improves AWS Firehose documentation, aligning with the update done on AWS Firehose Integration documentation at elastic/integrations#12732
Further, updates all Firehose related documentations to have the same or similar recommended configurations.
Documentation sets edited in this PR
Check all that apply.
docs/en/observability/*
)docs/en/serverless/*
)docs/en/integrations/*
)Related issue
Related : elastic/integrations#12150
Related : #4784
Checklist
Follow-up tasks
Select one.
This is an automatic backport of pull request #4818 done by [Mergify](https://mergify.com).