Skip to content
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

[azure-eventhub] Update sdk/messaging/azeventhubs to v1.3.0 #42794

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zmoog
Copy link
Contributor

@zmoog zmoog commented Feb 19, 2025

Proposed commit message

Upgrade the sdk/messaging/azeventhubs package to the latest stable version v1.3.0.

This package is the core library of the modern Event Hub SDK we use in the azure-eventhub input v2. From v1.2.1 to v1.3.0 we get several bug fixes and a new TCP proxies with AMQP feature.

Author's Checklist

  • Run a input v2 smoke test ingesting activity logs

How to test this PR locally

# x-pack/filebeat/modules.d/azure.yml
- module: azure
  # All logs
  activitylogs:
    enabled: true
    var:
      # eventhub name containing the activity logs, overwrite he default value if the logs are exported in a different eventhub
      eventhub: "logs"
      # consumer group name that has access to the event hub, we advise creating a dedicated consumer group for the azure module
      consumer_group: "$Default"
      # the connection string required to communicate with Event Hubs, steps to generate one here https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-get-connection-string
      connection_string: '${EVENTHUB_CONNECTION_STRING:""}'
      # the name of the storage account the state/offsets will be stored and updated
      storage_account: '${EVENTHUB_STORAGE_ACCOUNT:""}'
      # the name of the storage account container you would like to store the offset information in.
      # storage_account_container: ""
      # the storage account key, this key will be used to authorize access to data in your storage account
      storage_account_key: '${EVENTHUB_STORAGE_ACCOUNT_KEY:""}'
      # the connection string required to communicate with the storage account, steps to generate one here https://docs.microsoft.com/en-us/azure/storage/common/storage-account-keys-manage?tabs=azure-portal
      storage_account_connection_string: '${EVENTHUB_STORAGE_ACCOUNT_CONNECTION_STRING:""}'
      # the processor version to use, v1 is the legacy processor and v2 is the new processor
      processor_version: v2

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 19, 2025
@mergify mergify bot assigned zmoog Feb 19, 2025
Copy link
Contributor

mergify bot commented Feb 19, 2025

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @zmoog? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@zmoog zmoog added Team:obs-ds-hosted-services Label for the Observability Hosted Services team input:azure-eventhub labels Feb 19, 2025
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Feb 19, 2025
@zmoog zmoog added backport-8.x Automated backport to the 8.x branch with mergify backport-8.16 Automated backport with mergify backport-8.17 Automated backport with mergify labels Feb 19, 2025
@zmoog zmoog changed the title [azure-eventhub] Bump messaging/azeventhubs v1.3.0 [azure-eventhub] Update sdk/messaging/azeventhubs to v1.3.0 Feb 19, 2025
@zmoog zmoog marked this pull request as ready for review February 20, 2025 12:30
@zmoog zmoog requested a review from a team as a code owner February 20, 2025 12:30
@elasticmachine
Copy link
Collaborator

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)

@zmoog
Copy link
Contributor Author

zmoog commented Feb 20, 2025

Hey @elastic/beats-tech-leads, the PR is ready for your review! 🙇

@pierrehilbert pierrehilbert removed the backport-8.16 Automated backport with mergify label Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.x Automated backport to the 8.x branch with mergify backport-8.17 Automated backport with mergify input:azure-eventhub Team:obs-ds-hosted-services Label for the Observability Hosted Services team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants