-
Notifications
You must be signed in to change notification settings - Fork 75
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]: Add Docker support for ease of setup #59
Comments
Hi @DanielDKnudsen that's great. What are these best practices? Do you have a fork of the repo I can take a look at? Or if you are happy with your setup and have it running you can always do a pull request and I can merge it in after I test it. Would be good to offer people more options of running the repo. |
As for the best practices there are a couple I think I am not adhering to: 2. Migrate "Install.sh" script to Dockerfile: 3. Volume mount refactoring: 4. Automate Docker Image build: These are the major points as I see it, but I probably have overlooked quite a few as well. Not sure - would love some feedback in the pull request. |
Points 2 and 3 are now handled in the PR. I won't have time to look into 1 and 4 in the foreseeable future. But they could be looked into at a later stage. |
Hello, at first thanks for your work! I tried to start the server via docker but i always get this issue:
testet it on a windows machine and linux machine with docker. Edit: mkdir -p /home/${user}/.steam/sdk64/ then i replaced the old script with and now it works! |
I encounter the same error. And i found the link code has been writtened in line 133-141 of the docker_install.sh script. By uncommenting line 133-141 in the docker_install.sh script, the problem has been solved! |
@DanielDKnudsen any thoughts on this? Why did you comment out the symlink to |
My process for fixing this problem was:
Now my process for updating is to git pull, docker build, and then docker compose up and using git stash if ever needed, should any of the mentioned files be updated from upstream This allows me to make modifications to my config files and the likes and seems to be a fairly pain-free process I agree with @DanielDKnudsen on his opinions on better adhering to some of the best practices on building docker images An important design decision before refactoring the docker process is to decide how files are "personalized", e.g. I map the entire repo, whilst the current method seems to use specific mounts for config files and binaries only - which is neater in some ways, but means fixing the bug in this issue is more complicated as seen. My recommendation is to avoid having scripts move files around too much, and instead placing them in their actual location as how they are used in the docker container, although there may be some undesired consequence of this, that I am not aware of, @DanielDKnudsen might know more. I suggest a PR is made to resolve the current issues for using docker images, and then afterwards looking into the best practices and design decisions :-) |
Hey so i managed to get the server running, buydoing the uncomment in the docker_install.sh realted above But none of the ! commands work, looking in the mounted files it seems the files in the /csgo folder int he repo just get replaced? any idea what i might be messing up |
Ah figured it out. needed to change the
|
Information
I've been playing around with creating a Docker image using this repo.
This would make the server very simple to setup and run in different environments.
I've made a working version - BUT it could use some changes to follow best practices etc.
Linked Discussion: #25
The text was updated successfully, but these errors were encountered: