Skip to content
This repository has been archived by the owner on Sep 30, 2021. It is now read-only.

Change deployment to catalina ROOT #42

Open
pieterlukasse opened this issue Nov 22, 2018 · 6 comments
Open

Change deployment to catalina ROOT #42

pieterlukasse opened this issue Nov 22, 2018 · 6 comments

Comments

@pieterlukasse
Copy link
Contributor

Change deployment to catalina ROOT so that the final URL reads https://servername.com instead of https://servername.com/cbioportal.

@pieterlukasse
Copy link
Contributor Author

pieterlukasse commented Nov 22, 2018

In the Dockerfile it is as easy as (before the maven step)

RUN rm -rf $CATALINA_HOME/webapps/ROOT

and change

	&& unzip portal/target/cbioportal-*.war -d $CATALINA_HOME/webapps/cbioportal \

to

	&& unzip portal/target/cbioportal-*.war -d $CATALINA_HOME/webapps/ROOT \

⚠️ keep in mind related documentation should also be updated in cbioportal/cbioportal (e.g. like the SAML/Keycloak documentation containing URLs ending in /cbioportal).

@fedde-s
Copy link
Member

fedde-s commented Nov 22, 2018

I'll update the Docker deployment instructions once it gets documented in the upstream cBioPortal deployment manual.

@pieterlukasse
Copy link
Contributor Author

Yes, ideally this can be done together, with 1 PR to cbioportal/cbioportal (for the documentation) and 1 PR here (for the Docker file).

@fedde-s
Copy link
Member

fedde-s commented Apr 24, 2019

@pieterlukasse, has your feature proposal been reported upstream? This docker setup will follow once it's a documented way to deploy cBioPortal; or maybe as soon as it's in progress there if it immediately greatly benefits our deployments, like we did with cBioPortal/cbioportal#4832.

@pieterlukasse
Copy link
Contributor Author

No, there is no upstream PR yet.

@pieterlukasse
Copy link
Contributor Author

Is this file already solving this issue...and introducing a new one (namely a documentation issue)?
https://github.com/cBioPortal/cbioportal/blob/master/docker/web/Dockerfile
CC @inodb

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants