Skip to content

fix: Add missing otelcol alias to make OTel Engine work with OTel Collector helm chart#5473

Merged
thampiotr merged 3 commits into
mainfrom
thampiotr/otel-engine-helm-support
Feb 9, 2026
Merged

fix: Add missing otelcol alias to make OTel Engine work with OTel Collector helm chart#5473
thampiotr merged 3 commits into
mainfrom
thampiotr/otel-engine-helm-support

Conversation

@thampiotr
Copy link
Copy Markdown
Contributor

@thampiotr thampiotr commented Feb 6, 2026

Brief description of Pull Request

Turns out that we cannot change the OTel Collector Helm Chart's args to prepend an otel subcommand. To allow using Alloy's OTel Engine with OTel ecosystem more broadly, we add an otelcol alias to Alloy's Docker image.

Pull Request Details

Issue(s) fixed by this Pull Request

Notes to the Reviewer

This should be tested in the future integration test. For now I verified in local cluster:

image
root@otelcol-opentelemetry-collector-7786f6d646-fcklr:/# cat /bin/otelcol
#!/usr/bin/env sh
# Compatibility wrapper for OpenTelemetry Collector integrations.
# Delegate to Alloy's OTel engine and pass through all arguments.
exec /bin/alloy otel "$@"

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an otelcol compatibility alias to Alloy’s container images so OpenTelemetry ecosystem tooling (notably the OTel Collector Helm chart) can invoke Alloy’s OTel engine without needing to prepend an otel subcommand.

Changes:

  • Add a Linux /bin/otelcol wrapper script that delegates to alloy otel.
  • Add a Windows otelcol.cmd wrapper that delegates to alloy.exe otel.
  • Update Linux and Windows Dockerfiles to include these compatibility wrappers in the final images.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
packaging/docker/otelcol.sh Adds a small shell wrapper that maps otelcol ...alloy otel ....
packaging/docker/otelcol.cmd Adds a Windows CMD wrapper that maps otelcol ...alloy.exe otel ....
Dockerfile Installs /bin/otelcol into the Linux image and ensures it is executable.
Dockerfile.windows Installs otelcol.cmd into the Windows image for compatibility use cases.

@thampiotr thampiotr marked this pull request as ready for review February 6, 2026 17:13
@thampiotr thampiotr requested a review from a team as a code owner February 6, 2026 17:13
Copy link
Copy Markdown
Member

@blewis12 blewis12 left a comment

Choose a reason for hiding this comment

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

LGTM trying it out locally - perhaps we can add a follow up issue for integration testing

@thampiotr thampiotr merged commit 90478cd into main Feb 9, 2026
51 of 52 checks passed
@thampiotr thampiotr deleted the thampiotr/otel-engine-helm-support branch February 9, 2026 10:14
blewis12 pushed a commit that referenced this pull request Feb 10, 2026
…lector helm chart (#5473)

Turns out that we cannot change the OTel Collector Helm Chart's args to prepend an otel subcommand. To allow using Alloy's OTel Engine with OTel ecosystem more broadly, we add an otelcol alias to Alloy's Docker image.
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants