Skip to content

Commit 0341e89

Browse files
authored
chore: update test-all-versions configs to test fewer package versions (open-telemetry#2464)
The TAV tests in CI are taking 2.5h - 3h, for a total usage of over 13h compute time. E.g.: https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/11233652488/usage This change reduces the number of tested versions from 839 to 317 (for a TAV run with Node.js v20). For a few of the .tav.yml files the tested version range was actually increased to capture some versions that the instrumentation supported, but were not being tested by TAV.
1 parent ad560df commit 0341e89

File tree

33 files changed

+278
-186
lines changed

33 files changed

+278
-186
lines changed

package-lock.json

Lines changed: 123 additions & 25 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
'amqplib':
2-
versions: ">=0.5.5"
1+
amqplib:
2+
versions:
3+
include: ">=0.5.5"
4+
mode: latest-minors
35
commands:
4-
- yarn test
6+
- npm test
Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,11 @@
11
'@cucumber/cucumber':
2-
- versions: '^8.0.0 || ^9.0.0'
2+
- versions:
3+
include: '^8.0.0 || ^9.0.0'
4+
mode: latest-minors
35
node: '>=14'
46
commands: npm test
5-
- versions: '^10.0.0'
7+
- versions:
8+
include: '^10.0.0'
9+
mode: latest-minors
610
node: '>=18'
711
commands: npm test
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
dataloader:
22
# Testing ^2.0.0 covers about 90% of the downloaded versions
3-
versions: "^2.0.0"
3+
versions:
4+
include: "^2.0.0"
5+
mode: latest-minors
46
commands: npm run test
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
'lru-memoizer':
2-
versions: ">=1.3 <3"
2+
versions:
3+
include: ">=1.3 <3"
4+
mode: latest-minors
35
commands:
46
- npm test
Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
'socket.io':
2-
- versions: "2.1.1 || 2.2.0 || 2.3.0 || 2.4.0 || ^2.5.0"
2+
- versions:
3+
include: "^2.1.1"
4+
mode: latest-minors
35
peerDependencies: socket.io-client@^2
46
commands:
57
- npm run test
@@ -9,7 +11,9 @@
911
commands:
1012
- npm run test
1113

12-
- versions: "4.5.3 || 4.6.2 || ^4.7.1"
14+
- versions:
15+
include: "^4.5.3"
16+
mode: latest-minors
1317
peerDependencies: socket.io-client@^4
1418
commands:
1519
- npm run test
Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,28 @@
11
"aws-sdk":
22
# A small subset of releases in the range [2.308.0, 3) to reduce testing time.
3-
versions: "2.308.0 || 2.556.0 || 2.801.0 || 2.1049.0 || 2.1297.0 || 2.1546.0 || >=2.1548.0"
3+
versions:
4+
include: "^2.308.0"
5+
mode: max-7
46
commands:
57
- npm run test
68

9+
# Versions [3.363.0, 3.377.0] of all @aws-sdk/client-* were bad releases. See:
10+
# - https://github.com/open-telemetry/opentelemetry-js-contrib/pull/2464#issuecomment-2403652552
11+
# - https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1828#issuecomment-1834276719
12+
713
"@aws-sdk/client-s3":
8-
# A small subset of releases in the range [3.6.1, 4) to reduce testing time.
9-
# - 3.377.0 was a bad release (see issue #1828).
1014
# - 3.529.0 was missing the fast-xml-parser dependency (https://github.com/aws/aws-sdk-js-v3/releases/tag/v3.529.1)
11-
versions: "3.6.1 || 3.55.0 || 3.180.0 || 3.289.0 || 3.385.0 || 3.498.0 || >=3.529.1"
15+
versions:
16+
include: "^3.6.1"
17+
exclude: "3.529.0 || >=3.363.0 <=3.377.0"
18+
mode: "max-7"
1219
commands:
1320
- npm run test
1421

1522
"@aws-sdk/client-sqs":
16-
# A small subset of releases in the range [3.24.0, 4) to reduce testing time.
17-
versions: "3.24.0 || 3.94.0 || 3.202.0 || 3.296.0 || 3.388.0 || 3.496.0 || >=3.503.1"
23+
versions:
24+
include: "^3.24.0"
25+
exclude: ">=3.363.0 <=3.377.0"
26+
mode: "max-7"
1827
commands:
1928
- npm run test
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
bunyan:
22
# Testing ^1.8.12 covers at least 95% of the downloaded versions
3-
versions: "^1.8.12"
3+
versions:
4+
include: "^1.8.12"
5+
mode: latest-minors
46
commands: npm run test
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
express:
2-
- versions: "^4.16.2"
2+
- versions:
3+
include: "^4.16.2"
4+
mode: latest-minors
35
commands: npm run test

plugins/node/opentelemetry-instrumentation-fastify/.tav.yml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
"fastify":
2-
# Sanity check the first 4.x release, instead of all releases, plus recent
3-
# releases.
4-
- versions: "4.0.0 || >=4.24.3 <5"
2+
- versions:
3+
include: "^4.0.0"
4+
mode: max-7
55
commands: npm run test
6-
- versions: ">=5 <6"
6+
- versions:
7+
include: ">=5 <6"
8+
mode: max-7
79
commands: npm run test
810
peerDependencies: "@fastify/[email protected]"
911
node: '>=20'

0 commit comments

Comments
 (0)