search

zalando / spilo

Highly available elephant herd: HA PostgreSQL cluster using Docker
Apache License 2.0
1.51k stars 371 forks source link

fix(clone_with_wale): fix script to work with wal-g master #979

Open peterparser opened 4 months ago

peterparser commented 4 months ago

While working with a wal-g binary built from master branch and spilo, I found out that the newer version of wal-g uses backup_name instead of name in the output of the command, this change make the script not functional during a restore operation.

The change is pretty straightforward, it's also needed to replace the backup_name word to name to make sure that the dictionary is created accordingly to the rest of the script and to support also older versions of wal-g.

gabrielmocan commented 3 months ago

No need to work with master anymore as wal-g team just cutted a brand new release.

More on #986

peterparser commented 3 months ago

No need to work with master anymore as wal-g team just cutted a brand new release.

More on #986

Hi @gabrielmocan, the problem was affecting wal-g master and from what I see in #987 test logs is also affecting this new release.

gabrielmocan commented 3 months ago

No need to work with master anymore as wal-g team just cutted a brand new release. More on #986

Hi @gabrielmocan, the problem was affecting wal-g master and from what I see in #987 test logs is also affecting this new release.

There's more work to do in this PR, still a draft. Just wait...