Amulety modules bump versions#11645
Conversation
|
@mashehu thanks for the review, I'll address your points! I also saw that the tests failed due to no space left in the runner, even though the new container seemed to be smaller than the previous one so that's a bit puzzling. Can something be done like increasing the runners size for this specific module test? |
mashehu
left a comment
There was a problem hiding this comment.
please do the same for the gpu test
|
@mashehu thanks for the review, I've implemented the suggested changes. I needed |
|
hmm, can you bump it up to get one passing run and then bump it down to the default. just don't want to waste too much resources for all the different tests we run. I can force merge the PR once you have one commit with ✅ |
|
@mashehu, thank you all tests pass now, so I'll bump it down to 40gb which was the previous default |
PR checklist
Closes #XXX
topic: versions- See version_topicslabelnf-core modules test <MODULE> --profile dockernf-core modules test <MODULE> --profile singularitynf-core modules test <MODULE> --profile condanf-core subworkflows test <SUBWORKFLOW> --profile dockernf-core subworkflows test <SUBWORKFLOW> --profile singularitynf-core subworkflows test <SUBWORKFLOW> --profile conda