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

Make the TRACE_ID unique across repos and re-runs #20

Merged
merged 2 commits into from
Dec 15, 2020

Conversation

DavidS
Copy link
Contributor

@DavidS DavidS commented Dec 14, 2020

This change adds the repo name and a random suffix to TRACE_ID. Otherwise
this ID would not be unique across repositories and re-runs (which preserve
the RUN_ID, see https://docs.github.com/en/free-pro-team@latest/actions/reference/environment-variables#default-environment-variables).

This fixes #16.

This change adds the repo name and a random suffix to TRACE_ID. Otherwise
this ID would not be unique across repositories and re-runs (which preserve
the RUN_ID, see https://docs.github.com/en/free-pro-team@latest/actions/reference/environment-variables#default-environment-variables).
@yvrhdn
Copy link

yvrhdn commented Dec 14, 2020

Hi @DavidS, thanks for all the contributions! The changes look fine but I haven't had time yet to check them out in detail. I should be able to do that tomorrow or Wednesday at the latest.

@DavidS
Copy link
Contributor Author

DavidS commented Dec 15, 2020

Hi @DavidS, thanks for all the contributions! The changes look fine but I haven't had time yet to check them out in detail. I should be able to do that tomorrow or Wednesday at the latest.

no worries - the entire team over here is about to head into vacation for two weeks. This ain't a hurry.

Merry Christmas and I hope you get some good down time over the holidays!

Thanks for all the time and work you put into this, really made a difference for us.

Copy link

@yvrhdn yvrhdn left a comment

Choose a reason for hiding this comment

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

This looks good to me, I just added a little comment. Thanks!

@yvrhdn yvrhdn merged commit 2041175 into honeycombio:main Dec 15, 2020
@DavidS DavidS deleted the unique-trace-id branch December 16, 2020 11:04
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.

run_number is not unique enough for trace_id
2 participants