Fix the watchtower pre-check random wait

filecoin-saturn / L1-node

Filecoin Saturn L1 Node • The edge cache layer of Filecoin's decentralized CDN 🪐

Other

140 stars 50 forks source link

Fix the watchtower pre-check random wait #331

Closed joaosa closed 1 year ago

joaosa commented 1 year ago

I noticed this was broken when revisiting this for allowing faster updates on testnet. To fix this:

I added running the l1 container with --init so signals can be properly caught as we're running multiple processes;
I also added a new env var to configure the order of magnitude of the random wait;
I used a script for the random wait as that is the supported way to do this with watchtower.

guanzo commented 1 year ago

huh, so this is why deployments were rolling out so quickly

joaosa commented 1 year ago

@guanzo @DiegoRBaquero We cannot merge this or it'll break auto-update. This is because docker-compose.yml changes make it to the mainnet as soon as they're merged. ~I'm going to try to find a way to make this a bit saner~.

Here's the fix: https://github.com/filecoin-saturn/L1-node/pull/335

DiegoRBaquero commented 1 year ago

Let's merge this tomorrow or monday