-
Notifications
You must be signed in to change notification settings - Fork 297
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.
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
Development
: Add documentation for benchmarking tool
#8047
Conversation
WalkthroughThe documentation now features the Artemis Benchmarking Tool, designed for testing Artemis under heavy load with a focus on exam scenarios. This tool assesses performance and scalability by simulating user interactions, offering optimization insights. Additionally, a reference to the tool has been included in the documentation index for easy access. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Documentation
Add documentation for Benchmarking ToolDevelopment
: Add documentation for Benchmarking Tool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well-written! lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documentation explains the benchmarking process well. I ran the text through a language tool and checked it with Grammarly, and there were some minor mistakes. You can decide which ones you want to correct.
This is the corrected text:
The Artemis Benchmarking Tool is a web application designed for Artemis administrators to test the performance of Artemis under heavy load. It is available on GitHub.
The Benchmarking Tool for Artemis is designed to simulate realistic user interactions with the Artemis platform, particularly focusing on exam conduction where the system’s performance and scalability are most critical.
This tool enables system administrators and developers to assess and analyze Artemis’s behavior under simulated load conditions, providing valuable insights into response times, system workload, and overall stability. These insights can be used to identify the limits of the hardware infrastructure as well as the software architecture, thereby allowing for targeted optimizations and improvements. The main goal is to ensure that the respective Artemis setup can handle the expected load during exams without any performance issues.
During an exam simulation, the tool participates in the exam on behalf of a configurable number of students. It performs various actions, such as logging in, starting the exam, submitting answers, and performing git operations. For each action, the tool measures the time it takes to complete the action and records the result. Additionally, the tool can be connected to Prometheus to collect and visualize workload metrics.
For more information on how to set up and use the Benchmarking Tool, please refer to the README.
50644be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammar wise it is well written and for me it's understandable. LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re approve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation lgtm
Development
: Add documentation for Benchmarking ToolDevelopment
: Add documentation for benchmarking tool
Checklist
General
Motivation and Context
The main documentation of the Benchmarking Tool is available in its README. To make sure that Artemis admins find the tool, I added a short section about it to the Artemis docs. The goal of this section is simply to give a short overview about the tool and then link to its Readme for more information.
Description
I added a section called "Benchmarking Tool" to the Administration Guide. It gives a short summary about the tool's core functionality and then links to the repo.
Steps for Testing
Take a look at the new section in the docs: https://artemis-platform--8047.org.readthedocs.build/en/8047/admin/benchmarking-tool.html
Testserver States
Note
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Review Progress
Code Review
Manual Tests
Summary by CodeRabbit
admin/benchmarking-tool
to the list of exercises in the documentation.