-
Notifications
You must be signed in to change notification settings - Fork 30
No server-id.json file created in the config directory #3
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
Comments
I just had this same issue when trying to deploy. Maybe its due to changes in the most recent update? Is this a newly generated file? |
Would you mind placing in your values file? I am wondering if it's related to 1.1 that was released, I haven't created a new server on that version yet. |
|
Only new thing I added was |
Just for clarification, are you saying you are building the 1.1 docker container locally? If so, can you place your docker file also? I apologize for the questions, I'm just trying to gather the information to debug it :) |
I am just running |
The It seems like this |
Neat, this didn't come up in my testing. I'll work on this over the weekend. Since it is the first time slot I will have available. I believe you are correct it's how I'm mounting my configs into that directory. I believe I can specify the subpath in the mountvolume section, this should allow the main folder to have the proper permissions to write the file it needs to. |
Yeah I think it would only have come up if you had set the game to public. |
Hey guys, just wanted to post a follow up here, I have been looking into this issue:
I am looking at trying to sidecar a container to properly move the files into the correct location so we can have a directory that allows us to write to the proper location. However, I can also see there is a pull request we can put into the factoriotools repository. If we can get them to expose the locations of the configs as ENVs instead of just copying them to a location it would also solve this problem. Specifically, if we can just get the --server-id location as an ENV we would be fixed here. This did also remind me that we are not listing the adminlist.json, whitelist.json, or the banlist.json in the configmap. So I imagine this would also cause some issues when we go to create these files IE ban someone from the server. So we would also run into this issue from this perspective. The below code snippit is from the docker-entrypoint.sh that is run on startup so you can easily see what I am referencing
I might see if I can get ahold of the maintainers of the docker project to see if they would be open to changing that script specifically as I continue to work on the sidecar solution. In the meantime @samcat116 , do you want to see if you can modify the docker-entrypoint.sh to point to a different location? |
Hey guys sorry for the delay, had some real life distractions come up, I am in the middle of switching jobs. I put in a pull request for new ENV vars that would address this issue. |
Is there anything I can do to help with this? Public server listing without having to modify the upstream container would be kinda handy for a thing I'm working on. |
It looks like I'm still waiting for the other project to approve my pull request. In the meantime you can build the container with the changes I made. That worked for me when I was testing. |
Solved with #13 |
I am trying to deploy this chart but am getting the following error
Unhandled exception: Error when opening /factorio/config/server-id.json for writing: No such file or directory
This seems to be an autogenerated file, but since all the config files are mounted in read only as a ConfigMap, it cannot create it.
The text was updated successfully, but these errors were encountered: