Skip to content

Release v1.16.1#366

Merged
xlamorlette-datadog merged 6 commits into
masterfrom
xlamorlette/release-with-updated-dd-trace-cpp
May 6, 2026
Merged

Release v1.16.1#366
xlamorlette-datadog merged 6 commits into
masterfrom
xlamorlette/release-with-updated-dd-trace-cpp

Conversation

@xlamorlette-datadog

@xlamorlette-datadog xlamorlette-datadog commented May 6, 2026

Copy link
Copy Markdown
Contributor

Prepare release v1.16.1, with the following changes:

  • Update dd-trace-cpp to v2.1.0.
  • Remove support of Nginx 1.24.0 (which reached end of life more than one year ago, on May 1st 2025).
  • Upgrade Amazon Linux to latest available version (we used a 2 years old that embedded Nginx 1.24.0).

@xlamorlette-datadog
xlamorlette-datadog marked this pull request as ready for review May 6, 2026 16:37
@xlamorlette-datadog
xlamorlette-datadog requested a review from a team as a code owner May 6, 2026 16:37
@xlamorlette-datadog xlamorlette-datadog changed the title Update dd-trace-cpp to v2.1.0, to get the baggage limits fix Release v1.16.1 May 6, 2026
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented May 6, 2026

Copy link
Copy Markdown

Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 67.62% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: d83a752 | Docs | Datadog PR Page | Give us feedback!

@zacharycmontoya

Copy link
Copy Markdown
Contributor

Under investigation: Upgrading Nginx from 1.24.0 to 1.30.0 for the tests with amazonlinux fails…

@xlamorlette-datadog if this is leading to failures, we can also consider separating this into a separate PR

@xlamorlette-datadog

Copy link
Copy Markdown
Contributor Author

@zacharycmontoya

if this is leading to failures, we can also consider separating this into a separate PR

This comes with the removal of the support of Nginx 1.24.0. I think I could get a fix within a reasonable timeframe by upgrading the used amazonlinux image (by the way, is this test really needed?…). Else, I will revert this change…

@pawelchcki
pawelchcki self-requested a review May 6, 2026 17:05
@zacharycmontoya

Copy link
Copy Markdown
Contributor

@zacharycmontoya

if this is leading to failures, we can also consider separating this into a separate PR

This comes with the removal of the support of Nginx 1.24.0. I think I could get a fix within a reasonable timeframe by upgrading the used amazonlinux image (by the way, is this test really needed?…). Else, I will revert this change…

@xlamorlette-datadog yep I understand that this came with the removal of Nginx 1.24.0. If it takes more than an hour to resolve, I suggest reverting the change and we can tackle separately

@xlamorlette-datadog
xlamorlette-datadog force-pushed the xlamorlette/release-with-updated-dd-trace-cpp branch from e54f9a8 to d83a752 Compare May 6, 2026 18:07
@xlamorlette-datadog
xlamorlette-datadog merged commit 40ddfa2 into master May 6, 2026
162 checks passed
@xlamorlette-datadog
xlamorlette-datadog deleted the xlamorlette/release-with-updated-dd-trace-cpp branch May 6, 2026 20:56
pawelchcki added a commit to DataDog/httpd-datadog that referenced this pull request May 8, 2026
Relative submodule URLs (`../foo.git`) get resolved against the parent
repo's URL — which on DDbuild is the gitretriever URL the runner uses
to fetch the main repo. The existing
`url.https://[email protected]/DataDog/.insteadOf` rule in
.gitlab-ci.yml only rewrites `https://github.com/DataDog/`, so the
gitretriever-resolved submodule URLs slipped past it and hit
`fatal: could not read Username` in get_sources (pipeline #420).

Switching the submodule URLs to absolute github.com URLs makes the
existing insteadOf rule actually match, so submodule clones go
through the authenticated gitlab.ddbuild.io mirror. Same fix landed
in DataDog/nginx-datadog#366 (commit fc330c9).
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