-
Notifications
You must be signed in to change notification settings - Fork 632
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 Documentation on Deploy without Docker #10623
Comments
Hi Jeremy - do we have ETA for this? Is it only documentation or also the files (pom, jsp etc)? |
Working on the update right now. @barykada90 Would you be available to test those steps? |
Sure thing, happy to test! |
@barykada90 Can you test these out when you have a chance: Step 1 - Software Requirements |
@JREastonMarks - testing on both Win11 and Ubuntu: Step 2 instructs copying application.properties to root but then maven is unable to find it when compiling as it's looking at src/main/resources path (because of the pom.xml entry) Win11 - error and fail when trying to start the application (no authentication): I'll test Ubuntu next |
Hi @barykada90, Win11 Error - That looks almost like it couldn't find the application.properties file. When you get a chance please take a look again. |
Thank you @JREastonMarks - amazing
I can confirm the session service is no longer needed to be spin out separately, however, the connection string must be still there, i.e.: session.service.url=http://localhost:8091/session_service/api/sessions/public_portal/ otherwise there's an error Step 1 - Java 21 is now a must, I've tried running on 19 and hit a brick wall FYI, I didn't test STEP 4 as I'm running my own set of data STEP 5 - typo command Thank you! |
Hi @barykada90 In application.properties.example session.service.url has a default value |
Thanks @JREastonMarks - i just scanned the documents and apart from the typo not fixed on step 5, all good to go, thank you! Re session.service.url - yes, but I don't understand what heroku is capturing therefore I'm not happy for it to stay there = override Thank you for making that happen! |
Thank you again for reviewing this @barykada90 . I'm not sure what typo on step 5 you are talking about. Can you expand? |
My absolute pleasure @JREastonMarks I'm facing 2 problems with that build - CORS for API (platform is unable to fetch studies if accessed via URL (not localhost)) and no files to import studies. |
Hey - just whilst making more progress, I'm trying to upload some data and the error is thrown So I read multiple docs and found on STEP 4 reference in core of the below which doesn't exist - is that still correct? Essentially I'm unable to load any data :) |
Hi @JREastonMarks - it appears that when using SAML authentication, session service is still required (context: https://groups.google.com/g/cbioportal/c/lPk-_NxaggU) |
With the rollout of 6.0.0 we have a different methodology for deploying cBioPortal without Docker. The documentation for this is currently incomplete.
Some things that are know to be needed to address
The text was updated successfully, but these errors were encountered: