beats icon indicating copy to clipboard operation
beats copied to clipboard

[azure-eventhub] Add health status reporting to processor v2

Open zmoog opened this issue 5 months ago • 2 comments

Proposed commit message

WIP

Checklist

  • [ ] My code follows the style guidelines of this project
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have made corresponding change to the default configuration files
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

zmoog avatar Jun 16 '25 20:06 zmoog

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.

mergify[bot] avatar Jun 16 '25 20:06 mergify[bot]

:robot: GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

github-actions[bot] avatar Jun 16 '25 20:06 github-actions[bot]

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

elasticmachine avatar Aug 25 '25 21:08 elasticmachine

This pull request is now in conflicts. Could you fix it? 🙏 To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b zmoog/feat/azure-eventhub-v2-status-reporting upstream/zmoog/feat/azure-eventhub-v2-status-reporting
git merge upstream/main
git push upstream zmoog/feat/azure-eventhub-v2-status-reporting

mergify[bot] avatar Sep 05 '25 05:09 mergify[bot]

@joecompute @bhapas, I don't consider adding the link a blocker. When in doubt, we should merge the PR to include this change in the upcoming releases.

zmoog avatar Sep 08 '25 14:09 zmoog

Removed backport tag active-all so that we don't backport to 8.18 or 9.0. We need the statusreporterhelper in 8.19/9.1 at minimum for this, as it was introduced in https://github.com/elastic/beats/pull/45679 and improved upon in https://github.com/elastic/beats/pull/45748 , both which only backported to 8.19 and 9.1.

joecompute avatar Sep 08 '25 22:09 joecompute