Skip to content

Commit 9c628cb

Browse files
committed
Simplify description of version variants
1 parent ad04b9f commit 9c628cb

11 files changed

Lines changed: 36 additions & 35 deletions

.claude/ci/appsec-gradle-integration.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ file.
1616
| `helper-rust integration coverage` | same | Integration coverage collection (not needed locally) |
1717

1818
Runner: `docker-in-docker:amd64`
19-
Matrix: PHP 7.0--8.5 × release/debug/zts/musl/ssi (varies by job group)
19+
Matrix: PHP 7.0+ × release/debug/zts/musl/ssi (varies by job group)
2020

2121
**Important:** `testX.Y-debug` are not gradle targets that are run on CI. They
2222
may, however, be useful for debugging.

.claude/ci/appsec-native-tests.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ file.
1616
| `appsec code coverage` | `datadog/dd-trace-ci:php-8.3_bookworm-6` | Coverage instrumented build (not needed locally) |
1717

1818
Runner: `arch:amd64` + `arch:arm64`
19-
Matrix: PHP 7.0--8.5 × {debug, debug-zts, debug-zts-asan (7.4+)}
19+
Matrix: PHP 7.0+ × {debug, debug-zts, debug-zts-asan (7.4+)}
2020

2121
The `{arch}` dimension only controls the GitLab runner tag. It has no
2222
effect on the Docker image or commands run. On macOS (Apple Silicon),
@@ -38,8 +38,7 @@ source directory. See [index.md](index.md) for how `--overlayfs` works.
3838

3939
### Full suite
4040

41-
All commands are run from the repo root. Replace `8.3` with the desired
42-
PHP version (`7.0`--`8.5`) and `debug` with the desired variant
41+
All commands are run from the repo root. Replace `8.3` with the desired PHP version and `debug` with the desired variant
4342
(`debug`, `debug-zts`, `debug-zts-asan`).
4443

4544
```bash

.claude/ci/compile-artifacts.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ If you need to run this step outside a build script and your host lacks `binutil
4848
| `Prepare code` | `php:8.2-cli` | Runs `composer update` + `make generate` to produce `src/bridge/_generated_*.php` |
4949

5050
Runner: `arch:{amd64,arm64}`
51-
Matrix (`compile extension: debug`): PHP 7.0--8.5 x {amd64, arm64}
52-
Matrix (`compile extension: debug-zts-asan`): PHP 7.4--8.5 x {amd64, arm64}
51+
Matrix (`compile extension: debug`): PHP 7.0+ x {amd64, arm64}
52+
Matrix (`compile extension: debug-zts-asan`): PHP 7.4+ x {amd64, arm64}
5353

5454
**Note on `Prepare code` vs `prepare code`:** These are two distinct jobs. The tracer
5555
pipeline `Prepare code` uses `php:8.2-cli` (which has no Composer), installs Composer
@@ -83,10 +83,10 @@ files via `classpreloader`: `_generated_api.php`, `_generated_tracer.php`, and
8383
| `pecl build` | `dd-trace-ci:php-7.4_bookworm-6` | Runs `tooling/bin/pecl-build` via `make build_pecl_package`; regenerates PHP bridge files via `composer -dtooling/generation`; mutates `package.xml` (version, date, file list) and `Cargo.toml` (strips profiling workspace member) in-place; produces `datadog_trace-*.tgz` via `pear package`; requires a clean tree to re-run |
8484

8585
Runner: `arch:{amd64,arm64}` (Linux jobs) or `windows-v2:2019` (Windows)
86-
Matrix (tracing/appsec extension): PHP 7.0--8.5 x 4 build platforms (x86_64-alpine-linux-musl, aarch64-alpine-linux-musl, x86_64-unknown-linux-gnu, aarch64-unknown-linux-gnu)
87-
Matrix (profiler extension): PHP 7.1--8.5 x same 4 platforms
88-
Matrix (ASAN tracing): PHP 7.4--8.5 x {x86_64-unknown-linux-gnu, aarch64-unknown-linux-gnu}
89-
Matrix (Windows): PHP 7.2--8.5
86+
Matrix (tracing/appsec extension): PHP 7.0+ x 4 build platforms (x86_64-alpine-linux-musl, aarch64-alpine-linux-musl, x86_64-unknown-linux-gnu, aarch64-unknown-linux-gnu)
87+
Matrix (profiler extension): PHP 7.1+ x same 4 platforms
88+
Matrix (ASAN tracing): PHP 7.4+ x {x86_64-unknown-linux-gnu, aarch64-unknown-linux-gnu}
89+
Matrix (Windows): PHP 7.2+
9090

9191
## What It Builds
9292

.claude/ci/github-actions-profiler.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515
| `Profiling correctness / prof-correctness ({ver}, zts)` | `ubuntu-24.04` | Same + `exceptions_zts` (requires `parallel` PECL extension) |
1616
| `Profiling ASAN Tests / prof-asan ({ver}, {arch})` | `arm-8core-linux` / `ubuntu-8-core-latest` | Builds profiler with ASAN + runs `.phpt` profiling tests |
1717

18-
Correctness matrix: PHP 8.0--8.5 × {nts, zts}.
19-
ASAN matrix: PHP {8.3, 8.4, 8.5} × {arm64, amd64}.
18+
Correctness matrix: PHP 8.0+ × {nts, zts}.
19+
ASAN matrix: PHP 8.3+ × {arm64, amd64}.
2020

2121
## What It Tests
2222

.claude/ci/meta-job-group-doc.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ with columns `CI Job`, `Image`, `What it does`:
3838
| `appsec code coverage` | `dd-trace-ci:php-8.3_bookworm-6` | (not needed locally) |
3939
4040
Runner: `arch:amd64` + `arch:arm64`
41-
Matrix: PHP 7.0--8.5 × {debug, debug-zts, debug-zts-asan (7.4+)}
41+
Matrix: PHP 7.0+ × {debug, debug-zts, debug-zts-asan (7.4+)}
4242
```
4343

4444
If there is only one source file, the single-line form is fine:

.claude/ci/package-native-verification.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ in the `verify` stage of the **package-trigger** child pipeline and use native r
3636

3737
**verify alpine:**
3838
Runner: `arch:amd64`
39-
Matrix: (Alpine 3.8--3.20 + latest with `php7`/`php` packages) + (`php:{ver}-fpm-alpine` for PHP 7.0--8.5)
39+
Matrix: (Alpine 3.8+ including latest, with `php7`/`php` packages) + (`php:{ver}-fpm-alpine` for PHP 7.0+)
4040
Install types: `php_installer` (uses `datadog-setup.php`) and `native_package` (uses `.apk`)
4141

4242
**verify centos:**
@@ -45,7 +45,7 @@ Matrix: PHP 7.0--8.3 x {php_installer, native_package}
4545

4646
**verify debian:**
4747
Runner: `arch:amd64`
48-
Matrix: PHP 7.0--8.5 x {php_installer, native_package} x {bullseye-slim, bookworm-slim}
48+
Matrix: PHP 7.0+ x {php_installer, native_package} x {bullseye-slim, bookworm-slim}
4949

5050
**verify .tar.gz:**
5151
Runner: `arch:{amd64,arm64}` (amd64 tests PHP 7.0 package, arm64 tests PHP 8.1 package)
@@ -57,8 +57,8 @@ Runner: `arch:amd64`
5757

5858
**Loader test on {arch} libc:**
5959
Runner: `arch:{amd64,arm64}`
60-
Matrix (amd64): PHP 5.6 (buster) + 7.0--7.3 (nts) + 7.4--8.5 (nts + zts, with valgrind)
61-
Matrix (arm64): PHP 7.0--7.3 (nts) + 7.4--8.5 (nts + zts)
60+
Matrix (amd64): PHP 5.6 (buster) + 7.0--7.3 (nts) + 7.4+ (nts + zts, with valgrind)
61+
Matrix (arm64): PHP 7.0--7.3 (nts) + 7.4+ (nts + zts)
6262

6363
**Loader test on {arch} alpine:**
6464
Runner: `arch:{amd64,arm64}`
@@ -68,14 +68,14 @@ Runner: `arch:amd64`
6868

6969
**pecl tests:**
7070
Runner: `arch:amd64`
71-
Matrix: PHP 7.0--8.5
71+
Matrix: PHP 7.0+
7272

7373
**test early PHP 8.1:**
7474
Runner: `arch:amd64`
7575

7676
**x-profiling phpt tests on Alpine:**
7777
Runner: `arch:amd64`
78-
Matrix: PHP 7.1--8.5
78+
Matrix: PHP 7.1+
7979

8080
## What It Tests
8181

.claude/ci/shared-zai-tea-tests.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@ child pipeline; all job definitions and matrices are inline.
1818
Runner: `arch:amd64` (all jobs in this pipeline are amd64-only)
1919

2020
Matrix:
21-
- **Build & Test Tea**: PHP 7.0--8.5 x {debug, debug-zts-asan (7.4+),
21+
- **Build & Test Tea**: PHP 7.0+ x {debug, debug-zts-asan (7.4+),
2222
nts, zts}. Pre-7.4 versions skip `debug-zts-asan` and use
2323
`debug-zts` instead.
2424
- **ZAI Tests**: same matrix as TEA, plus UBSAN toolchain for `debug`
2525
variant on PHP 7.4+.
26-
- **Extension Tea Tests**: PHP 7.0--8.5 x {debug, debug-zts-asan
26+
- **Extension Tea Tests**: PHP 7.0+ x {debug, debug-zts-asan
2727
(7.4+), nts, zts}. Pre-7.4 skips `debug-zts-asan`.
2828
- **ZAI Shared Tests**: PHP 7.4, 8.0 only, `nts` variant only.
2929
- **C components ASAN**: three images (centos-7, alpine, bookworm-6);

.claude/ci/tracer-integration-tests.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,9 @@
77
pipeline; integration test jobs are generated from `TEST_INTEGRATIONS_{XY}`
88
lists in the `Makefile`, expanded by the PHP loop starting at the
99
`foreach ($jobs as $type => $type_jobs)` block
10-
- `Makefile` -- defines `TEST_INTEGRATIONS_70` through `TEST_INTEGRATIONS_85`
11-
target lists and the individual `test_integrations_*`, `test_opentelemetry_*`,
10+
- `Makefile` -- defines per-version `TEST_INTEGRATIONS_{XY}` (where XY is the
11+
PHP major+minor digits, e.g. `70` for PHP 7.0) target lists and the
12+
individual `test_integrations_*`, `test_opentelemetry_*`,
1213
`test_opentracing_*` make targets
1314
- `.gitlab/generate-common.php` -- shared service definitions (test-agent,
1415
request-replayer, httpbin, mysql, redis, memcache, amqp, mongodb, kafka,
@@ -45,7 +46,7 @@
4546
| `test_opentracing_10: [{php}]` | same | OpenTracing 1.0 bridge tests |
4647

4748
Runner: `arch:amd64`
48-
Matrix: PHP 7.0--8.5 (varies per integration; see `TEST_INTEGRATIONS_{XY}` in Makefile)
49+
Matrix: PHP 7.0+ (varies per integration; see `TEST_INTEGRATIONS_{XY}` in Makefile)
4950

5051
Stage: `integrations test`
5152

@@ -74,7 +75,7 @@ substring.
7475
| Service | Image | Alias | Port | Purpose |
7576
|---------|-------|-------|------|---------|
7677
| test-agent | `ddapm-test-agent:v1.22.1` | `test-agent` | 9126 | Receives traces; validates snapshots |
77-
| request-replayer | `dd-trace-ci:php-request-replayer-2.0` | `request-replayer` | 80 | HTTP request replay |
78+
| request-replayer | `dd-trace-ci:php-request-replayer-*` | `request-replayer` | 80 | HTTP request replay |
7879
| httpbin | `kong/httpbin:0.2.2` | `httpbin-integration` | 8080 | HTTP echo service |
7980

8081
### Additional services by target substring

.claude/ci/tracer-unit-tests.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,18 @@ Runner: `arch:amd64` for all test jobs. Compile jobs also run on
3636
`arch:arm64`.
3737

3838
Matrix:
39-
- **Non-ASAN jobs**: PHP 7.0--8.5, amd64 only.
40-
- **ASAN jobs**: PHP 7.4--8.5 x {amd64, arm64} for `ASAN test_c`;
41-
PHP 7.4--8.5 amd64-only for other ASAN jobs. `ASAN test_c with
39+
- **Non-ASAN jobs**: PHP 7.0+, amd64 only.
40+
- **ASAN jobs**: PHP 7.4+ x {amd64, arm64} for `ASAN test_c`;
41+
PHP 7.4+ amd64-only for other ASAN jobs. `ASAN test_c with
4242
multiple observers` is PHP 8.0+ only.
4343
- **xDebug tests**: specific (PHP, xdebug) version pairs; see the
4444
`$xdebug_test_matrix` array in `generate-tracer.php`. Xdebug is
4545
not yet supported on PHP 8.5.
46-
- **`test_distributed_tracing`**: PHP 7.0--8.5 x {cli-server,
46+
- **`test_distributed_tracing`**: PHP 7.0+ x {cli-server,
4747
cgi-fcgi}. The cgi-fcgi variant sets
4848
`DD_DISTRIBUTED_TRACING=false`.
4949
- **`test_auto_instrumentation`, `test_composer`, `test_integration`**:
50-
PHP 7.0--8.5, amd64.
50+
PHP 7.0+, amd64.
5151

5252
## Quick start: build once, run many
5353

.claude/ci/tracer-web-tests.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
pipeline; the web test jobs are generated from `TEST_WEB_{XY}` lists in
88
the `Makefile`, expanded by the PHP loop starting at the
99
`foreach ($jobs as $type => $type_jobs)` block
10-
- `Makefile` -- defines `TEST_WEB_70` through `TEST_WEB_85` target lists
10+
- `Makefile` -- defines per-version `TEST_WEB_{XY}` target lists (where XY is
11+
the PHP major+minor digits, e.g. `70` for PHP 7.0)
1112
and the individual `test_web_*` / `test_metrics` make targets
1213
- `.gitlab/generate-common.php` -- shared service definitions (test-agent,
1314
request-replayer, httpbin-integration, mysql)
@@ -31,7 +32,7 @@
3132
| `test_metrics: [{php}, {sapi}]` | same | Metrics integration tests |
3233

3334
Runner: `arch:amd64`
34-
Matrix: PHP 7.0--8.5 (varies per framework) x SAPI {cli-server, cgi-fcgi, apache2handler}
35+
Matrix: PHP 7.0+ (varies per framework) x SAPI {cli-server, cgi-fcgi, apache2handler}
3536
(PHP >= 7.2 gets all three SAPIs; PHP 7.0--7.1 gets only a bare run without SAPI dimension.
3637
`test_web_custom` additionally gets `fpm-fcgi`.)
3738

@@ -61,7 +62,7 @@ All web test jobs use four GitLab service containers:
6162
| Service | Image | Alias | Port | Purpose |
6263
|---------|-------|-------|------|---------|
6364
| test-agent | `ddapm-test-agent:v1.22.1` | `test-agent` | 9126 | Receives traces; validates snapshots |
64-
| request-replayer | `dd-trace-ci:php-request-replayer-2.0` | `request-replayer` | 80 | Replays HTTP requests for trace forwarding |
65+
| request-replayer | `dd-trace-ci:php-request-replayer-*` | `request-replayer` | 80 | Replays HTTP requests for trace forwarding |
6566
| httpbin | `kong/httpbin:0.2.2` | `httpbin-integration` | 80 | HTTP echo service for curl/guzzle tests |
6667
| mysql | `dd-trace-ci:php-mysql-dev-5.6` | `mysql-integration` | 3306 | MySQL for WordPress, Drupal, Magento, etc. |
6768

0 commit comments

Comments
 (0)