Kubernetes performance test#84
Merged
AlexanderWells-diamond merged 19 commits intomainfrom Jul 11, 2023
Merged
Conversation
Sets up a Coniql server, IOC, and a variable number of Python client programs. Collects various data about the system and prints it to terminal.
This is very convenient for various Kubernetes deployment files
This one was recommended to us by the Cloud team as being little used and so therefore the most likely to be most stable
This allows easy usage of parameters to configure the tests
This helps ensure we retrieve the right data from Prometheus
When we run the test repeatedly, its possible for the previous Coniql server to not have finished terminating when the new one is starting
This means we won't accidentally start running the Jobs before Coniql is ready to receive data. Previously we would occasionally see TimeoutError() coming from websockets.connect(). Also reduced CPU usage as the clients really don't need much.
This allows us to see progress messages in Kubernetes pods
Most stdout output is now dumped into /dev/null as we just don't need it
Codecov Report
@@ Coverage Diff @@
## main #84 +/- ##
=======================================
Coverage 93.30% 93.30%
=======================================
Files 10 10
Lines 807 807
=======================================
Hits 753 753
Misses 54 54 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
They don't belong in this repository
This appeared a few times while running the Kubernetes tests.
Some of this was remnants from when the kubernetes test was in this repo
AlexanderWells-diamond
force-pushed
the
kubernetes_performance_test
branch
from
4dfdf64 to
86054d9
Compare
MJGaughran
approved these changes
Jul 11, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains changes necessary to build the Performance Test script into a pair of Containers, one for the IOC and one for the Python test client. There are also many tweaks to the scripts to make them work in Kubernetes.
See the PR the deployment repo for further details.