${CLI_ARGS} are not working

[mikey0000]

---

Expected Behavior

${CLI_ARGS} should be interpreted correctly

Current Behavior

minetestserver fails and prints options list

Steps to Reproduce

1. Create new docker image using docker-compose
2. Run docker logs

Environment

OS: debian
CPU architecture: arm64
How docker service was installed:
followed official docker-compose instructions, however I used portainer.io to create the stack

[github-actions]

Thanks for opening your first issue here! Be sure to follow the bug or feature issue templates!

[Code-Slave]

did you fix this. same issue

[mikey0000]

Yeah docker exec - it /bin/sh and edit the start _script_ 7/12/2022 5:47:41 am CodeS1ave ***@***.***>:

…

did you fix this. same issue — Reply to this email directly, view it on GitHub[#47 (comment)], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AALDOPPEJTOGUXBPBMM7GDLWL5U2BANCNFSM6AAAAAASFUZTUY]. You are receiving this because you authored the thread.[Tracking image][https://github.com/notifications/beacon/AALDOPOUKA4CBR6U4FV5RJ3WL5U2BA5CNFSM6AAAAAASFUZTU2WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSP3GOP6.gif]

[mclang]

Hello.
Tried to spin up minetest server for the first time but stumbled upon this same problem. Running e.g minetestserver --gameid mineclone2 --info seems to work though.

Could you @mikey0000 maybe elaborate a bit which file I need to edit and what modifications to make 🙏

Why this is tagged as invalid @aptalca ?

[mikey0000]

docker exec {docker_id} -it /bin/bash
vi /etc/services.d/minetest/run

exec s6-setuidgid abc minetestserver --port 30000 --config /config/.minetest/main-config/minetest.conf

[mclang]

Okay found out the problem.

The example docker-compose.yml given in README has still double-quotes around CLI_ARGS parameter. Problem was reported ages ago in #30 but sadly the documentation hasn't been fixed yet 😞

Right way to pass client arguments is without quotes, e.g

- CLI_ARGS=--gameid minetest --port 30123 --logfile /config/.minetest/minetest-server.log --info

I was about to make PR for the change, but Contributing to minetest discouraged me...

[aptalca]

It was marked invalid because the issue template was not filled out properly

I now see that the readme-vars incorrectly has quotes for the value. But if we removed the quotes, then the cli version would be incorrectly generated because our templating engine does not provide a solution for this yet.

This needs to be fixed in our jenkins-builder, which generates the templates. The fix is in the works.

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions]

This issue is locked due to inactivity