Skip to content

Commit

Permalink
test(aws-sdk): set v3 versions based on Node.js support (#2628)
Browse files Browse the repository at this point in the history
  • Loading branch information
trivikr authored Jan 8, 2025
1 parent d4d3c4f commit 12ee8db
Showing 1 changed file with 42 additions and 12 deletions.
54 changes: 42 additions & 12 deletions plugins/node/opentelemetry-instrumentation-aws-sdk/.tav.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,19 +10,49 @@
# - https://github.com/open-telemetry/opentelemetry-js-contrib/pull/2464#issuecomment-2403652552
# - https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1828#issuecomment-1834276719

# node version support in JS SDK v3:
# - 14.x dropped in v3.567.0 https://github.com/aws/aws-sdk-js-v3/pull/6034
# - 16.x dropped in v3.723.0 https://github.com/aws/aws-sdk-js-v3/pull/6775

"@aws-sdk/client-s3":
# - 3.529.0 was missing the fast-xml-parser dependency (https://github.com/aws/aws-sdk-js-v3/releases/tag/v3.529.1)
versions:
include: "^3.6.1"
exclude: "3.529.0 || >=3.363.0 <=3.377.0"
mode: "max-7"
commands:
- npm run test
jobs:
- node: ">=18"
versions:
include: "^3.6.1"
exclude: "3.529.0 || >=3.363.0 <=3.377.0"
mode: "max-7"
commands: npm run test
- node: "16"
versions:
include: "^3.6.1 && <3.723.0"
exclude: "3.529.0 || >=3.363.0 <=3.377.0"
mode: "max-7"
commands: npm run test
- node: "14"
versions:
include: "^3.6.1 && <3.567.0"
exclude: "3.529.0 || >=3.363.0 <=3.377.0"
mode: "max-7"
commands: npm run test

"@aws-sdk/client-sqs":
versions:
include: "^3.24.0"
exclude: ">=3.363.0 <=3.377.0"
mode: "max-7"
commands:
- npm run test
jobs:
- node: ">=18"
versions:
include: "^3.24.0"
exclude: ">=3.363.0 <=3.377.0"
mode: "max-7"
commands: npm run test
- node: "16"
versions:
include: "^3.24.0 && <3.723.0"
exclude: ">=3.363.0 <=3.377.0"
mode: "max-7"
commands: npm run test
- node: "14"
versions:
include: "^3.24.0 && <3.567.0"
exclude: ">=3.363.0 <=3.377.0"
mode: "max-7"
commands: npm run test

0 comments on commit 12ee8db

Please sign in to comment.