Skip to content

chore(azure_logs_ingestion sink): migrate azure_logs_ingestion sink to the native Hyper 1 client - #26757

Merged
pront merged 3 commits into
masterfrom
http-1-azure-logs-ingestion
Oct 21, 2026
Merged

pront merged 3 commits into
masterfrom
http-1-azure-logs-ingestion

Conversation

@thomasqueirozb

Copy link
Copy Markdown
Member

Summary

Migrates the azure_logs_ingestion sink, its healthcheck, and its tower service to the native Hyper 1 client. The response stores http_1::StatusCode and relies on #26756 for retry classification.

References

Vector configuration

NA

How did you test this PR?

  • sinks::azure_logs_ingestion unit tests
  • cargo vdev int test azure

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Oct 20, 2026
@github-actions github-actions Bot added the domain: sinks Anything related to Vector's sinks label Oct 20, 2026
@thomasqueirozb
thomasqueirozb marked this pull request as ready for review October 20, 2026 20:20
@thomasqueirozb
thomasqueirozb requested a review from a team as a code owner October 20, 2026 20:20
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-20T21:45:59.474437Z 35dea91 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@datadoghq-integration datadoghq-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bits Code Review: PASS

More details

The native client uses the matching transport-error retry classifier, and the static review found no actionable regression in request construction or healthcheck behavior.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Bits Code Review · Commit ac763bb

Base automatically changed from http-1-status-code to master October 20, 2026 20:47

@datadoghq-integration datadoghq-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bits Code Review: PASS

More details

Azure’s new client uses its matching transport-error classifier, preserving retry handling across the migration. Runtime validation was blocked by the unavailable offline ntapi Git dependency.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Bits Code Review · Commit ac763bb

@pront
pront added this pull request to the merge queue Oct 21, 2026
Merged via the queue into master with commit a794c21 Oct 21, 2026
85 checks passed
@pront
pront deleted the http-1-azure-logs-ingestion branch October 21, 2026 01:10
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

domain: sinks Anything related to Vector's sinks no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants