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

Fix quoted "otel-cli exec ..." example in README.md #357

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shastah
Copy link

@shastah shastah commented Aug 12, 2024

This example from README:

# otel-cli propagates context via envvars so you can chain it to create child spans
otel-cli exec --kind producer "otel-cli exec --kind consumer sleep 1"

doesn't work, because we're running into #356.
It worked until /bin/sh -c was removed in #202.

This example from README:

    # otel-cli propagates context via envvars so you can chain it to create child spans
    otel-cli exec --kind producer "otel-cli exec --kind consumer sleep 1"

doesn't work, because we're running into equinix-labs#356.
It worked until `/bin/sh -c` was removed in equinix-labs#202.
@shastah shastah marked this pull request as ready for review August 12, 2024 23:27
Copy link
Collaborator

@tobert tobert left a comment

Choose a reason for hiding this comment

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

Good catch, thank you!

@shastah
Copy link
Author

shastah commented Sep 3, 2024

@tobert feel free to merge this, I don't have the permissions to :)

@hertznsk
Copy link

Thanks for this PR! I just want to remind @tobert that this is still waiting to be merged.

@tobert
Copy link
Collaborator

tobert commented Nov 14, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants