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

update versions of miniconda and rstudio #38

Merged
merged 41 commits into from
Sep 4, 2023
Merged

update versions of miniconda and rstudio #38

merged 41 commits into from
Sep 4, 2023

Conversation

bgruening
Copy link
Collaborator

No description provided.

@bgruening
Copy link
Collaborator Author

This PR really just bumps the RStudio version and the Miniconda version.

I think the ie2 branch was the latest? Thats what I based it on and thats why the diff looks so odd.

@hexylena who do you want to proceed?

@bgruening
Copy link
Collaborator Author

The last commit adds some larger restructuring.

  • It uses miniforge
  • activate the mamba solver by default for conda
  • it removed some deps that are hopefully not needed anymore
  • and it removed nginx - it seems we have not used nginx in the last release at all

I'm unsure about the PROXY_PREFIX and if this is still needed for other deployments than EU, ORG ...

The container is available here https://quay.io/repository/galaxy/docker-rstudio-notebook

@bgruening
Copy link
Collaborator Author

bgruening commented Aug 17, 2023

Its deployed on EU, if you would like to test - verson 0.3

@hexylena
Copy link
Owner

hexylena commented Sep 4, 2023

The last commit adds some larger restructuring.

very cool!

@hexylena
Copy link
Owner

hexylena commented Sep 4, 2023

I guess there's still some ongoing work to align with the Bioconductor folks, but that will probably take some more time, so, we go with this now.

@hexylena
Copy link
Owner

hexylena commented Sep 4, 2023

I'm unsure about the PROXY_PREFIX and if this is still needed for other deployments than EU, ORG ...

if we want path based proxying to work, it probably needs to be supported somehow, but I guess that's a discussion with Sveinung

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.

None yet

4 participants