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.

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

3 changes regarding git running tests #205

Merged
merged 2 commits into from
Apr 22, 2021
Merged

3 changes regarding git running tests #205

merged 2 commits into from
Apr 22, 2021

Conversation

SudeshnaBora
Copy link
Contributor

3 changes -

2 in run_test_suite_on_cluster.sh
1 in _load_remote_config.sh

@denisalevi
Copy link
Member

I think you forgot the change in the mkdir -p command in run_test_suite_on_cluster.sh.

Instead of

ssh "$remote" "mkdir -p $test_suite_remote_logdir"

it should be

ssh "$remote" "mkdir -p $test_suite_remote_logdir $remote_b2c_dir"

If I remember correctly.

@denisalevi
Copy link
Member

I was fixing something else in the script and added the missing mkdir command already. Merging this.

@denisalevi denisalevi merged commit 81b51b1 into master Apr 22, 2021
@denisalevi
Copy link
Member

Just FYI, I changed the directory of the logfiles that are generated by the qsub command. Before, they were placed in the home directory on the cluster, which ended up in quite a mess. Now they are placed in the same directory on the cluster where the test suite logs and the synced brian2cuda repositories end up (by default ~/projects/brian2cuda/test-suite/ge-logs. So if there is an issue with running the test suite, look into the logs in that folder.

Also, you can delete oll those .o<job_id> and .e<job_id> files in your home directory on the cluster.

And finally, the naming for the submitted test suite jobs changed, theyr are now named b2c-<your-test-suite-name>__<timestemp>. This means we can delete them easier with qdel b2c-<your-test-sutie-name>* for example. Or to delete all running test suite jobs, run qdel b2c-*.

denisalevi added a commit that referenced this pull request Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants