-
Notifications
You must be signed in to change notification settings - Fork 49
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
[Feature Request] Documentation to run this locally #402
Comments
There's actually no need to go through the build process and create your own docker image. We push docker images of the latest builds out to the AWS ECR Public Gallery. But you are correct, we need to update the docs with this info. Executing the following should get you started:
Then browse to |
@vemonet Thank you for the very detailed issue. It's always helpful when we have more details. And, thank you @triggan for your fantastic response to this! I agree the documentation can be improved for setup in both local dev mode and local docker runtime. I recently made some minor improvements to the dev mode setup documentation, but I can see now that we need more. @vemonet the issue for the port has an easy solution for you. You can change the port used for the local proxy server here: graph-explorer/packages/graph-explorer-proxy-server/node-server.js Lines 515 to 517 in 4d4ae7e
Don't forget to also change the proxy server URL in the Graph Explorer connection settings to include whatever port you end up using. Long term we will need a better solution. We are currently investigating options. |
Thanks @triggan and @kmcginnes for the responses! I think just adding the I authorized all unprivileged ports so now I can also run the dev deployment but it is not ideal, ideally there could be a way to easily deploy locally on port 443 and 80 (maybe some env variables?) |
I wholeheartedly agree. I have created a development project to cover these changes. |
Description
I have tried to simply deploy this project locally to test it quickly on publicly available SPARQL endpoints, but the current documentation only explain in details how to deploy this on remote AWS machines in the cloud. It would be nice to be able to deploy this on a local machine easily.
I tried to use docker:
I am getting this output:
But nothing is available on http://localhost:8000/explorer
I also tried to start it in development mode with
pnpm start
(on Ubuntu 22.04) but I am rightfully getting permission denied to deploy the proxy server on port 80The proxy is trying to deploy on port 80 locally which is blocked on many operating systems for security reason. What would be the best solution to bypass this issue and tell the proxy to deploy on another port than 80 when running in development?
Preferred Solution
README.md
there should be a command to easily deploy the graph-explorer with docker on a local machine, and simply making it available on a non-privileged port (e.g. 8000 or 8080 or 3000, etc)This command could use the published graph-explorer image available on ECR (https://gallery.ecr.aws/neptune/graph-explorer) to avoid having to build from source.
Ideally there would be a complete command that works out of the box with one of the many publicly available SPARQL endpoint available (e.g. https://www.bgee.org/sparql, https://sparql.uniprot.org/sparql/, https://bio2rdf.org/sparql/), that would be a really nice way to make this stack easy to test and discover for newcomers (and potential new contributors)
pnpm start
orpnpm dev
development deployment should use a non-privileged port (e.g. 8000 or 8080 or 3000, etc)Additional Context
I can look into it myself if necessary, but I expect some contributors already know how to do this well, so that could save us some time, and it would probably be better done by someone with already good knowledge of the stack! I can also help with documenting it for future use
Related Issues
Tasks
Important
If you are interested in working on this issue or have submitted
a pull request, please leave a comment.
Tip
Please use a 👍 reaction to provide a +1/vote.
This helps the community and maintainers prioritize this request.
The text was updated successfully, but these errors were encountered: