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

Add reverse proxy for bioconductor (R) #126

Merged
merged 8 commits into from
Oct 17, 2024
Merged

Add reverse proxy for bioconductor (R) #126

merged 8 commits into from
Oct 17, 2024

Conversation

AaronJackson
Copy link
Member

No description provided.

@AaronJackson
Copy link
Member Author

I had been working on a test

  - name: bioconductor package installation
    shell:
      cmd: |
        #rm -rf /usr/local/lib/R/site-library/dplyr # remove if installed
        /usr/bin/R --no-save <<RPROG
        install.packages("BiocManager")
        options(BIOCONDUCTOR_CONFIG_FILE="file:///root/config.yaml")
        options(BioC_mirror="http://bioconductor.localhost")
        options(BIOCMANAGER_SITE_REPOSITORY="http://bioconductor.localhost")
        library(BiocManager)
        BiocManager::install("BiocVersion")
        q()
        RPROG
        #ls /usr/local/lib/R/site-library/dplyr
    failed_when: false
    changed_when: false
    register: bioc_install

but it doesn't work in this version of ubuntu (like the other R CRAN test). R is too old. I might look at changing ubuntu version next week.

I also need to figure out what to do about config.yaml. It would be nice if we could proxy that too. It comes from bioconductor.org/config.yaml regardless of the mirror, but can be overridden. I couldn't reverse proxy bioconductor.org directly

@AaronJackson
Copy link
Member Author

No idea if this will actually work from TRE workspaces, very confidence inspiring

Copy link
Member

@manics manics left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine to me!

reverse_proxy/defaults/main.yml Outdated Show resolved Hide resolved
reverse_proxy/molecule/default/verify.yml Outdated Show resolved Hide resolved
@AaronJackson AaronJackson merged commit ca91ae1 into main Oct 17, 2024
10 checks passed
@AaronJackson AaronJackson deleted the r-bioconductor branch October 17, 2024 12:38
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.

2 participants