Skip to content

remove request and upgrade artillery#426

Open
agnesgaroux wants to merge 1 commit intomainfrom
smoke-test-package-upgrades
Open

remove request and upgrade artillery#426
agnesgaroux wants to merge 1 commit intomainfrom
smoke-test-package-upgrades

Conversation

@agnesgaroux
Copy link
Copy Markdown
Contributor

@agnesgaroux agnesgaroux commented Mar 12, 2025

What does this change?

Critical vulnerabilties: dependabot alerts
request has been deprecated, use fetch instead
upgrading artillery
using aws-sdk v3 to get secret value

How to test

Run the tests in ci? 😬
running locally I get

Test run id: tyy46_4w7re3tjh5macgbtxexw6gc5mwd3n_4fbw
Phase started: unnamed (index: 0, duration: 1s) 11:06:31(+0000)
Phase completed: unnamed (index: 0, duration: 1s) 11:06:32(+0000)

but then it hangs there with Warning: multiple batches of metrics for period 1741777590000 2025-03-12T11:06:30.000Z

How can we measure success?

smoke tests run in ci, fewer vulnerability alerts

Have we considered potential risks?

Limited as only testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant