Skip to content

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 27, 2025

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.

  • Stateful (docs/en/observability/*)
  • Serverless (docs/en/serverless/*)
  • Integrations Developer Guide (docs/en/integrations/*)
  • [] None of the above

Related issue

Related : elastic/integrations#12150
Related : #4784

Checklist

  • Product/Engineering Review
  • Writer Review

Follow-up tasks

Select one.

  • This PR does not need to be ported to another doc set because:
    • The concepts in this PR only apply to one doc set (serverless or stateful)
    • The PR contains edits to both doc sets (serverless and stateful)
  • This PR needs to be ported to another doc set:
    • Port to stateful docs: <link to PR or tracking issue>
    • Port to serverless docs: <link to PR or tracking issue>

This is an automatic backport of pull request #4818 done by [Mergify](https://mergify.com).

* update firehose endpoint and data stream details

Signed-off-by: Kavindu Dodanduwa <[email protected]>

* improve api key and retry duration section

Signed-off-by: Kavindu Dodanduwa <[email protected]>

* fix wording of storage location

Signed-off-by: Kavindu Dodanduwa <[email protected]>

* improve further by modifying all related documents

Signed-off-by: Kavindu Dodanduwa <[email protected]>

* Update docs/en/observability/cloud-monitoring/aws/ingest-aws-firehose.asciidoc

Co-authored-by: Arianna Laudazzi <[email protected]>

---------

Signed-off-by: Kavindu Dodanduwa <[email protected]>
Co-authored-by: Arianna Laudazzi <[email protected]>
(cherry picked from commit 9813ea7)
@mergify mergify bot requested a review from a team as a code owner February 27, 2025 20:59
@mergify mergify bot added the backport label Feb 27, 2025
Copy link
Contributor

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

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.

Copy link
Contributor

@alaudazzi alaudazzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alaudazzi alaudazzi merged commit 1b22c58 into 8.17 Feb 28, 2025
9 checks passed
@alaudazzi alaudazzi deleted the mergify/bp/8.17/pr-4818 branch February 28, 2025 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants