-
Notifications
You must be signed in to change notification settings - Fork 719
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
Test fails when using GCP due to missing tools in the basic biocontainer #764
Comments
Thanks @hnawar ! I need to prep a release soon so will try to look into this. Will need to find a way to reproduce on GCP. Quite a few modules will be using the Biocontainers BusyBox base image which seems to work fine on AWS and Azure. Any ideas why this error is raised?
|
@drpatelh
Here is the
|
I will do some more tests with just the Life Sciences API and the BusyBox container image |
can you add these to your profile and see if it completes?
|
I was able to replicate this issue on GCP with the google-lifesciences executor on nf-core/rnaseq 3.6. Similar to @hnawar's solution, I was able to resolve this by switching the container for any process that used the Here's the configuration I used:
Any ideas if there's a more graceful method of changing the container for processes that use the Also I expect that this will impact any workflow using nf-core/modules, should I open an issue there? |
I wonder what's the rationale to use a fancy |
Thanks again for looking into this @andrewfrank @hnawar @leipzig. I will replace this container in the next release to one that is compatible with GCP too. The main advantage of using a Biocontainer here is that it comes with both Docker and Singularity images we can use off the shelf. Depending on the execution environment either of these can be pulled directly. As it stands, the |
Does not nextflow convert it automatically to singularity on-demand? |
It does but that has had inherent issues in the past where for example user's home directories are filled up with temp files due to the conversion from Docker -> Singularity. There is also a time overhead in the conversion which doesn't always work seamlessly either. If you can directly download a container and avoid all of that then why not?! Especially if it means pipelines break less and users are on Cloud 9. |
With some digging it turns out that the Galaxy project is also hosting a Singularity image for |
@drpatelh if you wanted to maintain consistency with biocontainers, I confirmed that |
Thanks @andrewfrank. Let's let the size of the container be the deciding factor here:
I may rope you into testing when I have made the changes if that's ok? |
@drpatelh 👍 happy to help with testing |
@andrewfrank this should now be fixed on Be great if you can give the pipeline a go with Will leave this issue open until we have managed to get the pipeline running out of the box on GCP. Thanks! |
@drpatelh Success! Config from
|
Awesome! Thanks for testing so quickly and for all of the infra set-up info. I want to do a little more testing and then we can get a release in the next week or two. |
Thanks @drpatelh and @andrewfrank I have also tested it and it worked on the same setup it was failing before. I will test a few more nf-core pipelines that had the same issue and open other issues when needed |
I was experiencing the same issues and can confirm using the dev branch resolved the errors. I can also confirm changing biocontainers/biocontainers:v1.2.0_cv1 to ubuntu:20.04 in nf-core/cutandrun fixed similar errors that were occurring in that pipeline. |
This issue also appears in nf-core/mag, tagging @skrakau on this. |
Can someone help me understand how using ubuntu:20.04 addresses the problem? What was the problem with biocontainers/biocontainers:v1.2.0_cv1? I ask because I may be running into the same problem. I am using a custom image though, so ubuntu:20.04 does not work for me. I would like to update my image to get it to work, but I am not sure where to begin. |
Description of the bug
When trying to run the pipeline with profiles test,google the pipeline fails in the prep steps
After initial debugging, it seems the container suggested in the untar and the gunzip which is the base biocontainer is the issue.
When replacing it in my fork with another container image I built that had those tools the pipeline ran further and failed in a later step.
I updated the containers here
https://github.com/nf-core/rnaseq/blob/master/modules/nf-core/modules/gunzip/main.nf#L8
and here
https://github.com/nf-core/rnaseq/blob/master/modules/nf-core/modules/untar/main.nf#L8
The same issue seems to affect a number of pipelines that uses these modules at least on Google Cloud and potentially in other cases as well.
Command used and terminal output
Relevant files
No response
System information
Nextflow version: 21.10.6
Hardware: Cloud
Executor: google (And potentially others)
Container engine: Docker
OS Linux
Version of nf-core/rnaseq 3.5
The text was updated successfully, but these errors were encountered: