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

[exporter/datadog] The command should be /otelcol-contrib in the example #9425

Merged
merged 4 commits into from
Apr 25, 2022

Conversation

keisku
Copy link
Contributor

@keisku keisku commented Apr 25, 2022

Description:

The command in exporter/datadogexporter/example/example_k8s_manifest.yaml should be /otelcol-contrib according to the Dockerfile.

Facing the error when applying the existing example.

failed to create containerd task: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "/otelcontribcol": stat /otelcontribcol: no such file or directory: stat --config=/conf/otel-agent-config.yaml: no such file or directory: unknown: RunContainerError

Link to tracking Issue:

Testing:

Documentation:

@keisku keisku requested a review from a team April 25, 2022 12:29
@keisku keisku requested a review from mx-psi as a code owner April 25, 2022 12:29
@keisku keisku changed the title [exporter/datadog] The command should be /otelcol-contrib [exporter/datadog] The command should be /otelcol-contrib in the example Apr 25, 2022
Copy link
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

👋 thanks for fixing this, this changed on v0.40.0, I think with the move of release operations to opentelemetry-collector-releases.

Since this is technically code that end-users run, would you mind adding a note on the CHANGELOG.md file documenting the fix? Thanks!

@keisku keisku requested a review from mx-psi April 25, 2022 13:55
CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
@mx-psi mx-psi added the ready to merge Code review completed; ready to merge by maintainers label Apr 25, 2022
@jpkrohling jpkrohling merged commit 622fdb6 into open-telemetry:main Apr 25, 2022
djaglowski pushed a commit to djaglowski/opentelemetry-collector-contrib that referenced this pull request May 10, 2022
…ample (open-telemetry#9425)

* [exporter/datadog] Fix command in the example

* pin version

* update changelog

* Update CHANGELOG.md

Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>

Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
@keisku keisku deleted the patch-1 branch July 11, 2022 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants