Skip to content

Commit f77dc3f

Browse files
authored
chore(ci): base converged Agent image on -full (#1908)
> Stacked on #1906 (`jszwedko/renovate-agent-version-pins`). Review/merge that first; this PR will retarget to `main` automatically once it lands. ## Summary Switch to the `-full` image which contains both JMX and DDOT for correctness tests and the converged image we publish. This removes the need for the OTLP correctness tests to pin to published Agent `-full` images. It also enables deployment of the converged ADP image on internal clusters that also run DDOT. ## Test plan - [x] Verified `-full` includes JMX: both `-jmx` and `-full` carry the same `jmxfetch.jar` + Java 11 JRE; `-full` additionally has `otel-agent`. - [x] Rebuilt the converged `testing-release` image on `-full` and confirmed `otel-agent` (DDOT) is present. - [x] Ran locally on the `-full` converged image — all pass: `otlp-traces-ottl-filtering`, `otlp-traces-ottl-transform` (baseline now runs DDOT from the converged image), plus `otlp-traces`, `otlp-metrics`, and `dsd-plain` to confirm no regression from the base-image change. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: jesse.szwedko <jesse.szwedko@datadoghq.com>
1 parent f939603 commit f77dc3f

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.2"
59-
PUBLIC_DD_AGENT_VERSION: "7.80.2-jmx"
59+
PUBLIC_DD_AGENT_VERSION: "7.80.2-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.2-jmx
1+
ARG DD_AGENT_VERSION=7.80.2-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.2-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.2-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)