-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
Comprehensive tests #105
Comments
You may be interested to borrow from the CI set-up (based on |
Oh that is rather interesting! I see you are running it on travis. |
I managed to set up Travis testing on SlurmTools.jl using the docker images built by PySlurm: |
I was also able to setup testing infrastructure on travis for SlurmClusterManager using docker-compose to create a small cluster. It seems to work pretty well. |
It would be nice to unite these efforts here. We all need cluster managers and yet the current state of affairs is quite fragmented for a reliable experience with testing, etc. |
Start on #105, alternative to #125 Uses container from https://github.com/giovtorres/docker-centos7-slurm
Start on #105, alternative to #125 Uses container from https://github.com/giovtorres/docker-centos7-slurm
* Add Slurm Docker tests Start on #105, alternative to #125 Uses container from https://github.com/giovtorres/docker-centos7-slurm * test more combinations * sbatch output
Status of this? Is there still only slurm testing? Seems like PBS and SGE have been broken for a while: #179 |
WIP PR for this: #193 |
One of the issues (and one that became even more apparent during the 1.0 transistion) is that it is really hard to test this package.
Without CI development is slow since we are likely to break use-cases that we can't test.
Ideally we could use docker environments to instantiate a "minimal" cluster environment in which we then can run tests.
As an example see:
The text was updated successfully, but these errors were encountered: