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

Fix Dockerfile for multiqc vs new python #23

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

marcodelapierre
Copy link
Member

No description provided.

@marcodelapierre marcodelapierre linked an issue Nov 6, 2023 that may be closed by this pull request
@pditommaso
Copy link
Member

Interesting, wonder if wave --conda-package multiqc=1.17 works of if it's broken as well

@emi80
Copy link

emi80 commented Nov 6, 2023

@pditommaso, indeed wave --conda-package multiqc=1.17 works as I have been using that version in my tests with Singularity 4

@marcodelapierre
Copy link
Member Author

interesting! below is what I needed for it to work:

  • python < 3.12
  • typing_extensions
  • importlib_metadata
FROM mambaorg/micromamba
MAINTAINER Paolo Di Tommaso <[email protected]>

RUN \
   micromamba install -y -n base -c defaults -c bioconda -c conda-forge \
      salmon=1.10.2 \
      fastqc=0.12.1 \
      multiqc=1.17 \
      python=3.11 \
      typing_extensions \
      importlib_metadata \
      procps-ng \
   && micromamba clean -a -y

ENV PATH="$MAMBA_ROOT_PREFIX/bin:$PATH"
USER root

@marcodelapierre
Copy link
Member Author

CI fails due to current image on quay.io:

In https://github.com/nextflow-io/rnaseq-nf/actions/runs/6770165713/workflow?pr=23:

    - name: Tests
        run: |
          curl -fsSL get.nextflow.io | bash
          ./nextflow run . -profile docker
        env:
          NXF_ANSI_LOG: false

In nextflow.config:

  docker {
    process.container = 'quay.io/nextflow/rnaseq-nf:v1.2.1'
    docker.enabled = true
  }

@pditommaso
Copy link
Member

Think I need to rebuild the container, let me try

@pditommaso
Copy link
Member

Should be ok

@pditommaso pditommaso merged commit 8253a58 into master Nov 6, 2023
1 check failed
@pditommaso pditommaso deleted the fix/dockerfile_multiqc_vs_py branch November 6, 2023 14:11
@marcodelapierre
Copy link
Member Author

yep

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.

Issue with multiqc installation with micromamba
3 participants