Skip to content

Update Datadog tracer version to v1.0.1#7701

Merged
mattklein123 merged 3 commits intoenvoyproxy:masterfrom
DataDog:cgilmour/update-dd-tracer
Jul 26, 2019
Merged

Update Datadog tracer version to v1.0.1#7701
mattklein123 merged 3 commits intoenvoyproxy:masterfrom
DataDog:cgilmour/update-dd-tracer

Conversation

@cgilmour
Copy link
Copy Markdown
Contributor

Signed-off-by: Caleb Gilmour caleb.gilmour@datadoghq.com

Description: Updates dd-opentracing-cpp from v0.4.2 to v1.0.0
Fairly minor changes/fixes/features under the hood.
Tiny update to the datadog tracer code in envoy, because it made too much noise with debug enabled.

Risk Level: Low.
Testing: No changes. Passed unit tests and manual E2E tests.
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Caleb Gilmour <caleb.gilmour@datadoghq.com>
@dio
Copy link
Copy Markdown
Member

dio commented Jul 24, 2019

/azp run envoy-macos

@cgilmour
Copy link
Copy Markdown
Contributor Author

I'll check into the CI errors.

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@dio
Copy link
Copy Markdown
Member

dio commented Jul 24, 2019

@cgilmour yeap. To be clear, macOS failure seems like to be real.

@cgilmour
Copy link
Copy Markdown
Contributor Author

@dio correct, they're both real and directly related.
Been working on fixes and making sure I can verify locally before pushing an update.

Note: if it passes, a new release of dd-opentracing-cpp will be
published. This is not intended for merging yet.

Signed-off-by: Caleb Gilmour <caleb.gilmour@datadoghq.com>
@cgilmour cgilmour force-pushed the cgilmour/update-dd-tracer branch from 10d4545 to c587e11 Compare July 25, 2019 23:36
@cgilmour
Copy link
Copy Markdown
Contributor Author

Note: this is not ready for merging yet.

I've made code changes to clear the original CI issues, and verified locally using ./ci/run_envoy_docker.sh. To double-check, I've made this PR point to a specific commit of dd-opentracing-cpp and make sure it got a green result.
I'll ship a patch release of dd-opentracing-cpp and change this PR to use the tagged version.

Signed-off-by: Caleb Gilmour <caleb.gilmour@datadoghq.com>
@cgilmour
Copy link
Copy Markdown
Contributor Author

I've made the updates described in the previous comment.
Everything's passing and working fine in E2E retest.

@cgilmour cgilmour changed the title Update Datadog tracer version to v1.0.0 Update Datadog tracer version to v1.0.1 Jul 26, 2019
Copy link
Copy Markdown
Member

@dio dio left a comment

Choose a reason for hiding this comment

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

Nice!

@dio dio requested review from mattklein123 and removed request for mattklein123 July 26, 2019 12:05
@mattklein123 mattklein123 merged commit cd47463 into envoyproxy:master Jul 26, 2019
@cgilmour cgilmour deleted the cgilmour/update-dd-tracer branch August 21, 2019 02:27
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.

4 participants