-
Notifications
You must be signed in to change notification settings - Fork 449
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 Dockerfile to use Python base image and add input args #7579
Conversation
f515555
to
acc5e7b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've reviewed the Python portion of the PR and it seems to be a nice improvement. Thank you!
I've left a few minor comments.
Codacy issues should be addressed.
@xoriole, could you please review the Docker-related part of this PR? |
9dd6a4d
to
3baef2c
Compare
Running the built docker image, I'm getting the following error. This might not be the result of this PR but would need to be investigated further
Next, running the image with docker-compose, it fails with error
|
@xoriole the issue with the program being unable to open database is probably due to broken permissions. I can remove the network mode host or the ports section. I pushed a change where ports has been removed. |
I have removed myself from the list of reviewers, as it gives the impression that the PR is awaiting my review, which I have already provided. From my perspective, the Python changes are acceptable. We are now awaiting a response from @xoriole. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jutoft Could you update the run commands in the ReadMe file?
https://github.com/Tribler/tribler/blob/main/README.rst
@xoriole I was wondering why you had the file permission issues and i found it to be "" around the state dir path. Would you like me to add a section about the command line arguments in general, currently there is'nt really any docs about it. |
7f8b0c5
to
1b4bbba
Compare
…irectory Fixes Tribler#7563, Tribler#7560 Rename downloaddir to download_dir. add --no-install-recommends as requested by codacy. Add direct support for CORE_API_PORT in run_tribler_headless.py fix version info checkfor win32 set runtime user to be user fix incorrect environment config and refactor setting http or https enabled. Remove ports & rely on hosts mode for the docker-compose.yml example file. Fix " in env property Add http(s) host as env args on Dockerfile
Update docker compose to include port mapping and http host env var Add default /state and /downloads for simplified run Update ReadMe to run docker with volume mounts
1b4bbba
to
eae4a3b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jutoft Thank you for your contribution 🎉
Uses Python base image & add more input args to allow changing the mount directory.
input to the handling of input arguments is much appreciated
Fixes #7563, #7560