Skip to content

GH-55112: [Packaging][Debian] Add missing opentelemetry-cpp dependency - #55249

Merged
raulcd merged 1 commit into
apache:mainfrom
kou:deb-opentelemetry
Oct 26, 2026
Merged

raulcd merged 1 commit into
apache:mainfrom
kou:deb-opentelemetry

Conversation

@kou

@kou kou commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Rationale for this change

debuild doesn't add OpenTelemetry dependencies to libarrow.so.XXX automatically because OpenTelemetry package on Debian doesn't have SO version.

What changes are included in this PR?

Add opentelemetry-cpp dependency explicitly.

Are these changes tested?

Yes.

Are there any user-facing changes?

Yes.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

Copilot AI balanced review requested due to automatic review settings October 25, 2026 11:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@kou kou added the CI: Extra: Package: Linux Run extra Linux Packages CI label Oct 25, 2026
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #55112 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Oct 25, 2026
@raulcd
raulcd merged commit f10fadc into apache:main Oct 26, 2026
80 checks passed
@raulcd raulcd removed the awaiting committer review Awaiting committer review label Oct 26, 2026
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Oct 26, 2026
@kou
kou deleted the deb-opentelemetry branch October 26, 2026 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge CI: Extra: Package: Linux Run extra Linux Packages CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants