Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

✨ Load Test - Switch from Artillery to k6 #904

Merged
merged 3 commits into from
Jun 6, 2024
Merged

✨ Load Test - Switch from Artillery to k6 #904

merged 3 commits into from
Jun 6, 2024

Conversation

tombui99
Copy link
Contributor

@tombui99 tombui99 commented Jun 4, 2024 •
edited
Loading

#900

Back End - Docker, NodeJs

  • Added k6 load test tool installation in Dockerfile
  • Added function to run k6 script and output to a JSON file
  • Added function to read k6 output JSON file and upload to Azure Storage via Firebase API

image

Figure: New console box result with k6 load test

Back End - Firebase API

  • Created new table in Azure Blob Storage named "k6report"
  • Added API to upload k6 load test result to Blob Storage on Azure
  • Added k6 report summary data as new payload item to Scans table on Azure
  • Removed APIs that upload Artillery results

UI

  • Added new components to display k6 load test result
  • Added if else condition so that older scans with Artillery results still display as normal

image
Figure: k6 load test result

@tombui99 tombui99 changed the base branch from main to staging June 4, 2024 04:31
github-advanced-security[bot]
github-advanced-security bot found potential problems Jun 4, 2024
View reviewed changes
docker/utils.js Dismissed Show dismissed Hide dismissed
@tombui99 tombui99 changed the title Switch from Artillery to k6 load test ✨ Switch from Artillery to k6 load test Jun 4, 2024
@tombui99 tombui99 changed the title ✨ Switch from Artillery to k6 load test ✨ Load Test - Switch from Artillery to k6 Jun 4, 2024
zacharykeeping
zacharykeeping approved these changes Jun 4, 2024
View reviewed changes
Copy link
Member

@zacharykeeping zacharykeeping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tombui99 tombui99 merged commit fdd3a7a into staging Jun 6, 2024
4 checks passed
@tombui99 tombui99 deleted the switch-to-k6-load-test branch June 6, 2024 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zacharykeeping zacharykeeping zacharykeeping approved these changes

@william-liebenberg william-liebenberg Awaiting requested review from william-liebenberg

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tombui99 @zacharykeeping