-
Notifications
You must be signed in to change notification settings - Fork 595
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
Kuberun: add plugins to config #3766
Conversation
Signed-off-by: Lehmann_Fabian <fabian.lehmann@informatik.hu-berlin.de>
I feel a test is missing 😄 |
Signed-off-by: Lehmann_Fabian <fabian.lehmann@informatik.hu-berlin.de>
Had the same feeling... |
The problem seems not to be related to me, but to the changes in this file: https://github.com/nextflow-io/hello/blob/master/nextflow.config |
Not sure, it looks the test is failing is this
|
This test reads the config from here: https://github.com/nextflow-io/hello if I see correctly. |
You are right. I've reverted it |
Are you able to trigger tests again? https://github.com/Lehmann-Fabian/nextflow/actions/runs/4437679938 |
Signed-off-by: Lehmann_Fabian <fabian.lehmann@informatik.hu-berlin.de>
Thank you, seems to fix it :) |
Signed-off-by: Lehmann_Fabian <fabian.lehmann@informatik.hu-berlin.de>
Using
kuberun
with-plugins xy
does not yet hand this over to the head-pod. This is implemented in this PR. It would be nice to have this, especially for CWS.