Skip to content

Commit f6f08ba

Browse files
vjfridgeCodex GPT-6
andcommitted
Merge main into Lambda telemetry flush branch
Incorporate upstream Lambda v130 and instrumentation updates for FFL-3412. Environment: Datadog workspace Co-Authored-By: Codex GPT-6 <noreply@localhost>
2 parents 22bc92c + 5b86bdd commit f6f08ba

6 files changed

Lines changed: 26 additions & 11 deletions

File tree

‎Dockerfile‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,12 @@ RUN rm -rf \
4242

4343
# CI Visibility paths/integrations
4444
RUN rm -rf \
45-
./python/lib/$runtime/site-packages/ddtrace/contrib/coverage/ \
46-
./python/lib/$runtime/site-packages/ddtrace/contrib/pytest/ \
47-
./python/lib/$runtime/site-packages/ddtrace/contrib/pytest_bdd/ \
48-
./python/lib/$runtime/site-packages/ddtrace/contrib/pytest_benchmark/ \
49-
./python/lib/$runtime/site-packages/ddtrace/contrib/selenium/ \
50-
./python/lib/$runtime/site-packages/ddtrace/contrib/unittest/ \
45+
./python/lib/$runtime/site-packages/ddtrace/contrib/internal/coverage/ \
46+
./python/lib/$runtime/site-packages/ddtrace/contrib/internal/pytest/ \
47+
./python/lib/$runtime/site-packages/ddtrace/contrib/internal/pytest_bdd/ \
48+
./python/lib/$runtime/site-packages/ddtrace/contrib/internal/pytest_benchmark/ \
49+
./python/lib/$runtime/site-packages/ddtrace/contrib/internal/selenium/ \
50+
./python/lib/$runtime/site-packages/ddtrace/contrib/internal/unittest/ \
5151
./python/lib/$runtime/site-packages/ddtrace/ext/ci_visibility \
5252
./python/lib/$runtime/site-packages/ddtrace/ext/test_visibility \
5353
./python/lib/$runtime/site-packages/ddtrace/internal/ci_visibility \

‎ci/input_files/build.yaml.tpl‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ build-layer ({{ $runtime.name }}-{{ $runtime.arch }}):
4040
tags: ["arch:amd64"]
4141
image: registry.ddbuild.io/images/docker:20.10
4242
artifacts:
43-
expire_in: 1 hr # Unsigned zips expire in 1 hour
43+
expire_in: 1 day # Unsigned zips expire in 1 day
4444
paths:
4545
- .layers/datadog_lambda_py-{{ $runtime.arch }}-{{ $runtime.python_version }}.zip
4646
variables:
@@ -227,7 +227,7 @@ layer bundle:
227227
- build-layer ({{ .name }}-{{ .arch }})
228228
{{ end }}
229229
artifacts:
230-
expire_in: 1 hr
230+
expire_in: 1 day
231231
paths:
232232
- datadog_lambda_py-bundle-${CI_JOB_ID}/
233233
name: datadog_lambda_py-bundle-${CI_JOB_ID}

‎datadog_lambda/patch.py‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,10 @@ def patch_all():
3030
_patch_for_integration_tests()
3131

3232
if config.trace_enabled:
33-
patch_all_dd()
33+
# unittest and selenium are CI Visibility integrations. The layer strips
34+
# their code and ddtrace.internal.ci_visibility, so leaving them enabled
35+
# logs an import error whenever the user's code imports those modules.
36+
patch_all_dd(unittest=False, selenium=False)
3437
else:
3538
_patch_http()
3639
_ensure_patch_requests()

‎datadog_lambda/version.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
__version__ = "8.130.0.dev0"
1+
__version__ = "8.130.0"

‎pyproject.toml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[tool.poetry]
22
name = "datadog_lambda"
3-
version = "8.130.0.dev0"
3+
version = "8.130.0"
44
description = "The Datadog AWS Lambda Library"
55
authors = ["Datadog, Inc. <dev@datadoghq.com>"]
66
license = "Apache-2.0"

‎tests/test_patch.py‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,3 +94,15 @@ def test_patch_requests_with_headers_none(self):
9494
self.assertEqual(r.request.headers[TraceHeader.TRACE_ID], "123")
9595
self.assertEqual(r.request.headers[TraceHeader.PARENT_ID], "321")
9696
self.assertEqual(r.request.headers[TraceHeader.SAMPLING_PRIORITY], "2")
97+
98+
99+
@patch("datadog_lambda.patch.patch_all_dd")
100+
@patch("datadog_lambda.patch.config")
101+
def test_patch_all_disables_ci_visibility_integrations(mock_config, mock_patch_all_dd):
102+
from datadog_lambda.patch import patch_all
103+
104+
mock_config.trace_enabled = True
105+
mock_config.integration_test = False
106+
patch_all()
107+
108+
mock_patch_all_dd.assert_called_once_with(unittest=False, selenium=False)

0 commit comments

Comments
 (0)