Cast bootstrap-expect (and node-id) to string when it's a number to prevent breaking containers created via the docker API about rqlite

[joostdecock]

Almost forgot: If you do want to make either change (docs or casting to string) I am happy to submit a pull request for either.

[joostdecock]

The exact same issue applies to -bootstrap-expect which is arguably more problematic as that one is a number. From the docs:

-bootstrap-expect should be set to the number of nodes that must be available before the bootstrapping process will commence, in this case 3.

I will update the issue accordingly.

[otoolep]

Thanks for the report. However I don't understand how changing rqlite source code can fix this. Is this error an error from the Docker API?

[otoolep]

Perhaps you could detect that node-id is a number and cast it to a string.

This is the part I don't understand. My mental model of what you are seeing is that all these issues occur before anything is passed to rqlited. Therefore I don't understand how I can address.

[joostdecock]

Please forgive me, I saw the Golang error and presumed it was coming from rqlite, when in fact, it comes from the Docker daemon.

Love rqlite by the way, can't thank you enough for your work on this 🙏

closing