-
Notifications
You must be signed in to change notification settings - Fork 5
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
Updates to Ansible scripts to include an (optional) benchmark test #583
Conversation
…st to run benchmark)
I've included a commit / change 0d91081 to address a recent issue I found which is documented here: The commit changes the pip installation script to install a few libraries which seem to be required by a library which was failing the installation process in my recent deploys. |
This is a code change needed to run Spark 3, not to install and run the test suite ?
|
Looking good. |
I am tied up with ESCAPE/IVOA work today/tomorrow, but I'll try to put aside some time to test this branch on Monday. |
I've made the changes you suggested so that we now run the benchmark from the Ansible client rather than the monitor, and we run it against a specific version of the benchmark tool. I've also run a test and included the notes. I also merged with the latest from master, let me know when you get a chance to test it out if you have any other comments. |
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.
Cluster deployed and tests run successfully 👍
Description:
Contains updates to the Ansible scripts to include an optional "test" deploymode. When run as a test, the deploy runs a set of known notebooks, and compares the duration against a known value (https://github.com/wfau/aglais-testing/blob/main/config/notebooks/notebooks.json)
The results display one of:
Issue addressed:
#394