-
Notifications
You must be signed in to change notification settings - Fork 18
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
dockerd.json not updated #3
Comments
Thanks for catching this issue, it confirms my suspicion for issue #1. The log driver should be "json-file" instead of "db", as the db driver is specific to Synology's Docker package. It looks like "execute_update_log()" in the script does not work as expected, I'll look into it. |
Line 394:
Is the second half right? If it found a match, it would return
|
You're right, the second half is incorrect. I updated the function as follows:
A new branch 'logdriver' is available, care to try it? |
Sure :) I'm just restoring everything and then will attempt the update again :) |
/Edit: Ignore Me :p |
OK, I did a rerun on my NAS too, and it looks like $SYNO_DOCKER_JSON_CONFIG is not parsed correctly yet... I'll update the variable and will push it to the branch |
So, I couldn't restore the backup... I got this:
So I extracted it myself and restored the files manually. Just noticed the "Step 2 from 5" path... guessing the two variables shouldn't be put together? |
Ahh yes, I just noticed that :) At least it is a step in the right direction :) |
ok, $SYNO_DOCKER_JSON_CONFIG is working on my NAS now. I had to manually delete |
|
Weird... the restore process worked perfectly this time round. I ran the same command as the one I mentioned here: |
That's weird indeed. I checked the code and noticed an issue here:
Print_status incorrectly prepends |
Hey guys, |
Hi Max,
To better understand potential causes, can you share the following?
- Your Dockerfile and/or Docker command for running influxdb
- The syno_docker_update.sh command and console output
- Your Docker path (should be /var/packages/Docker)
Thanks,
Mark.
… On 24 Aug 2020, at 12:35, Max Beckenbauer ***@***.***> wrote:
Hey guys,
i still get the error on my influxdb docker file when i try to start it after the update:
Create container InfluxDB failed: {"message":"logger: no log driver named 'db' is registered"}.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Hi @markdumay ,
|
Ok, so the syno_docker_update command didn’t show any messages at all? Are you sure the script output is directed to your terminal then?
… On 25 Aug 2020, at 08:36, Max Beckenbauer ***@***.***> wrote:
Hi @markdumay ,
Dockerfile command:
$ docker run -p 8086:8086 \ -v $PWD:/var/lib/influxdb \ influxdb
Syno_docker_update command:
sudo ./syno_docker_update.sh update (Console output was uneventful)
Docker is in /var/packages/Docker
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
There were a lot of messages. But no fail messages. And i didn't save them, sorry :/ |
Ok, then it’s going to be difficult to understand to what point the script actually ran... if you’re up for it, would you mind rerunning the script and sharing the exact console output?
… On 27 Aug 2020, at 17:39, Max Beckenbauer ***@***.***> wrote:
There were a lot of messages. But no fail messages. And i didn't save them, sorry :/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Hey there
then I try to run my portainer container with: Anything else I should look for ? Rui |
and maybe this helps as well:
|
As a note, after clearing my issue in #48, I also ran into this problem. Here's my post-update dockerd.json.
|
Hello, same problem as @rui-nar, i have ths error message:
i have this dockerd.json file: someone have solve this problem? The hard drive on Docker is working is an ext4 |
|
It seems |
I'll probably be trying it now. Allegedly it's the same thing (with log rotation support and stuff) as |
It seems that |
Just did the full update process and found that my containers wouldn't start due to the "db" log driver not being found.
After a little digging, I noticed that the:
/var/packages/Docker/etc/dockerd.json
Still contains:
The last time this file was changed was a month ago... guess that is when I installed a DSM update.
The text was updated successfully, but these errors were encountered: