Skip to content

Commit 098da03

Browse files
jszwedkoclaude
andcommitted
chore(ci): base converged Agent image on -full; OTLP baselines use it
The two otlp-traces-ottl correctness tests pinned an external registry.datadoghq.com/agent:X-full image for their DDOT baseline, because the converged Agent+ADP image was built on -jmx (no OTel collector). Build the converged image on -full instead -- a superset of -jmx (same JMXFetch jar + Java 11 JRE, plus the DDOT otel-agent) -- and switch the two ottl baselines to the saluki-images/datadog-agent:testing-release placeholder like every other correctness test, so CI injects the bundled converged image for those baselines automatically. That removes the only external Agent image pins left in the test configs, so the renovate custom manager no longer needs to scan test/correctness: revert the ignorePaths override and the test fileMatch added alongside it (the manager now covers just the Makefile and Windows LTSC pins). Trade-off: this enlarges the publicly published converged image, since -full also bundles DDOT/system-probe. Opening for team feedback on whether that's acceptable versus keeping the test-config custom manager. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 32c00cb commit 098da03

5 files changed

Lines changed: 11 additions & 30 deletions

File tree

.gitlab-ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ variables:
5656
# Converged Datadog Agent-specific variables, which control how we build the converged Datadog Agent image that we
5757
# publicly publish.
5858
PUBLIC_DSD_VERSION: "7.80.1"
59-
PUBLIC_DD_AGENT_VERSION: "7.80.1-jmx"
59+
PUBLIC_DD_AGENT_VERSION: "7.80.1-full"
6060

6161
# Base images to copy Agent Data Plane into, depending on whether the image is meant for our internal environment or
6262
# public registries.

docker/Dockerfile.datadog-agent

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
ARG DD_AGENT_VERSION=7.80.1-jmx
1+
ARG DD_AGENT_VERSION=7.80.1-full
22
ARG DD_AGENT_IMAGE=registry.datadoghq.com/agent:${DD_AGENT_VERSION}
33
ARG ADP_IMAGE=saluki-images/agent-data-plane:testing-devel
44

renovate.json5

Lines changed: 7 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -50,25 +50,6 @@
5050
// weekly non-major group)
5151
"osvVulnerabilityAlerts": true,
5252

53-
// Paths Renovate skips entirely, before any manager (built-in or custom) extracts from them.
54-
//
55-
// This overrides the list inherited from `config:recommended` (via `:ignoreModulesAndTests`).
56-
// We reproduce its defaults but drop the broad `**/test/**` entry: that pattern would otherwise
57-
// hide test/correctness/cases/*/config.yaml from the custom Agent-pin manager below, so the
58-
// OTLP baseline image pins would silently keep drifting. The only dependency files under test/
59-
// that built-in managers recognize live in test/antithesis/deploy (a Dockerfile and a
60-
// docker-compose), which we keep ignoring here to preserve current behavior.
61-
"ignorePaths": [
62-
"**/node_modules/**",
63-
"**/bower_components/**",
64-
"**/vendor/**",
65-
"**/examples/**",
66-
"**/__tests__/**",
67-
"**/tests/**",
68-
"**/__fixtures__/**",
69-
"test/antithesis/**"
70-
],
71-
7253
// Package-manager-specific rules.
7354
"packageRules": [
7455
// Use Cargo's semver interpretation for all Cargo dependencies. Cargo
@@ -111,13 +92,14 @@
11192
// Custom managers for Datadog Agent version pins that Renovate's built-in managers don't see.
11293
//
11394
// The `dockerfile` manager already bumps docker/Dockerfile.datadog-agent (the
114-
// `registry.datadoghq.com/agent` FROM/ARG). But several other pins for the same Agent are
115-
// hand-maintained in files no default manager parses, so they were silently left behind on past
116-
// bumps (see the 7.80.1 -> 7.80.2 update). Manage them here, tied to the same depName
95+
// `registry.datadoghq.com/agent` FROM/ARG). But a couple of other pins for the same Agent are
96+
// hand-maintained in files no default manager parses -- the macOS test Agent version (Makefile)
97+
// and the Windows LTSC base image (.gitlab/windows.yml) -- so they were silently left behind on
98+
// past bumps (see the 7.80.1 -> 7.80.2 update). Manage them here, tied to the same depName
11799
// (`registry.datadoghq.com/agent`) so they bump in lockstep with -- and in the same PR as -- the
118100
// main Agent image.
119101
//
120-
// `currentValue` captures only the numeric version; any image-tag suffix (-full, -ltsc2022, -jmx)
102+
// `currentValue` captures only the numeric version; any image-tag suffix (-ltsc2022, -jmx, -full)
121103
// is matched but left out of the group, so Renovate rewrites just the number and preserves the
122104
// suffix in place. Because the suffixed tags are published together with the bare release, every
123105
// pin tracks the same single Agent version rather than drifting onto separate per-suffix streams.
@@ -126,13 +108,12 @@
126108
"customType": "regex",
127109
"fileMatch": [
128110
"(^|/)Makefile$",
129-
"^\\.gitlab/windows\\.yml$",
130-
"^test/correctness/cases/.+/config\\.yaml$"
111+
"^\\.gitlab/windows\\.yml$"
131112
],
132113
"matchStrings": [
133114
// Bare version, e.g. `MACOS_TEST_AGENT_VERSION ?= 7.80.2` (Makefile).
134115
"MACOS_TEST_AGENT_VERSION \\?= (?<currentValue>[\\d.]+)",
135-
// Image pins, e.g. `registry.datadoghq.com/agent:7.80.2-full` or `:7.80.2-ltsc2022`.
116+
// Image pin, e.g. `registry.datadoghq.com/agent:7.80.2-ltsc2022` (.gitlab/windows.yml).
136117
// The `-suffix` is matched but not captured, so only the number is rewritten.
137118
"registry\\.datadoghq\\.com/agent:(?<currentValue>[\\d.]+)(?:-[a-z0-9]+)?"
138119
],

test/correctness/cases/otlp-traces-ottl-filtering/config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ additional_span_ignore_fields:
1212
- metrics._top_level
1313
- metrics._dd.measured
1414
baseline:
15-
image: registry.datadoghq.com/agent:7.80.1-full
15+
image: saluki-images/datadog-agent:testing-release
1616
# Run DDOT (otel-agent) directly so the image entrypoint does not treat our args as a command to exec.
1717
#entrypoint:
1818
# - /opt/datadog-agent/embedded/bin/otel-agent

test/correctness/cases/otlp-traces-ottl-transform/config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ additional_span_ignore_fields:
1414
- metrics.http.status_code
1515
- meta.error.msg
1616
baseline:
17-
image: registry.datadoghq.com/agent:7.80.1-full
17+
image: saluki-images/datadog-agent:testing-release
1818
files:
1919
- collector.yaml:/etc/datadog-agent/otel-config.yaml
2020
- datadog_agent.yaml:/etc/datadog-agent/datadog.yaml

0 commit comments

Comments
 (0)