Skip to content

AWS Firehose for 9.0.0 enablement#12687

Merged
gizas merged 1 commit intomainfrom
enablefirehose9
Feb 12, 2025
Merged

AWS Firehose for 9.0.0 enablement#12687
gizas merged 1 commit intomainfrom
enablefirehose9

Conversation

@gizas
Copy link
Copy Markdown
Contributor

@gizas gizas commented Feb 10, 2025

  • Enhancement

Proposed commit message

WHAT: Enabling support for AWS Firehose integrations for 9.0 version
WHY: Is needed in order to enable above integrations in version 9.0.0

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  1. Clone Pr
  2. elastic-package build with v0.109.1
  3. elastic-package stack up -d -v --version=9.0.0-SNAPSHOT to install a local ES

Related issues

Signed-off-by: Andreas Gkizas <andreas.gkizas@elastic.co>
@gizas gizas requested a review from a team as a code owner February 10, 2025 14:11
@gizas gizas changed the title merging changes for 9.0.0 enablement AWS Firehose for 9.0.0 enablement Feb 10, 2025
@gizas gizas added Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] Integration:awsfirehose Amazon Data Firehose labels Feb 10, 2025
@gizas
Copy link
Copy Markdown
Contributor Author

gizas commented Feb 10, 2025

/test stack 9.0.0-SNAPSHOT

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elastic-sonarqube
Copy link
Copy Markdown

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

@gizas
Copy link
Copy Markdown
Contributor Author

gizas commented Feb 11, 2025

/test stack 9.0.0-SNAPSHOT

1 similar comment
@gizas
Copy link
Copy Markdown
Contributor Author

gizas commented Feb 11, 2025

/test stack 9.0.0-SNAPSHOT

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

@@ -1,7 +1,7 @@
format_version: "3.1.0"
format_version: "3.3.0"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ques: where is this taken from?

Copy link
Copy Markdown
Contributor Author

@gizas gizas Feb 12, 2025

Choose a reason for hiding this comment

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

The minimum format_version should be 3.0 for 9.0.0 migration. See https://github.com/elastic/ingest-dev/issues/4484#issuecomment-2551112710

Copy link
Copy Markdown
Contributor

@Kavindu-Dodan Kavindu-Dodan left a comment

Choose a reason for hiding this comment

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

@gizas gizas merged commit b807d35 into main Feb 12, 2025
@gizas gizas deleted the enablefirehose9 branch February 12, 2025 06:36
flexitrev pushed a commit that referenced this pull request Mar 20, 2025
Signed-off-by: Andreas Gkizas <andreas.gkizas@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:awsfirehose Amazon Data Firehose Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants