`-daemon` crash about docker

[AbcSxyZ]

It's interesting to notice, but I'm not sure if and how we should handle this case.

[patricklodder]

The only way to handle it is imho to disable -daemon. Problem is it can still be thrown into a config file. So maybe just document it?

[AbcSxyZ]

Tested on ruimarinho/bitcoin-core/, it's crashing too.

[patricklodder]

-daemon needs a root process e.g. init - so that won't work?

[AbcSxyZ]

Yeah we are having this kind of issue, without a running process as pid 1, it's stopping. Depend on the behavior of -daemon.

I'm not sure about the purpose of using a daemonized executable in a container, not really meaningful.

[xanimo]

yes my mistake. looks good to me.

…

On Fri, Dec 3, 2021 at 5:04 PM AbcSxyZ ***@***.***> wrote: Yeah we are having this kind of issue, without a running process as pid 1, it's stopping. I'm not sure about the exact behavior of -daemon. I'm not sure about the purpose of using a daemonized executable in a docker. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHTBONCV2V6QXGTASY7UXSLUPFSKJANCNFSM5JK5KU6A> .

[xanimo]

agree with @patricklodder on second thought. i don't think it's necessary since docker itself can run as daemon using -d. my mistake.

[AbcSxyZ]

I was thinking about it and how we should deal with this error. I had an incredible idea : doing nothing.

Regarding the other bitcoin repo, no one is raising issues related to this problem. We may assume users won't really try to use the -deamon option with images, advanced user who use more complex settings may understand the implication or find this PR, and at worst few people can raise an issue.

We need to hope people won't get stuck with an old dogecoin.conf containing deamon setting, it may definitively be confusing where the error is coming from.

Maybe we can leave it like it is, and if people are raising issues related to this point we can adapt the strategy and eventually add it to the doc ? This PR could be a sufficient documentation about this error.

[patricklodder]

Yes, I think we should just keep this issue open for now and keep it in the back of our heads that at some point we may need to do something about it. For now, it's easily negated.