added create-snapshot/restore-snapshot on the sidecar CLI + readme

[BrendanCoughlan5]

Loom to show the working create and restore:
https://www.loom.com/share/0022a4cac88343999505ae452992768b?sid=a2e1bd4e-6054-4cb1-92d2-8507d48f28f9

[seanmcgary]

Start with these comments, most will apply to the restore command as well.

[seanmcgary]

By more composable I meant:

GetTestPostgresDatabase() <-- get the database, but dont run the migrator
GetTestPostgresDatabaseWithMigrations() <-- should call GetTestPostgresDatabase and subsequently run the migrations.

But to do this without breaking things, youd first need to rename GetTestPostgresDatabase across the project to GetTestPostgresDatabaseWithMigrations then re-introduce GetTestPostgresDatabase

[seanmcgary]

Issue found with providing a output_file path of ~/snapshot-test

go run main.go create-snapshot \
    --ethereum.rpc-url="..." \
    --chain="preprod" \
    --database.host="localhost" \
    --database.port="5432" \
    --database.user="seanmcgary" \
    --database.password="" \
    --database.db_name="sidecar_testnet_holesky" \
    --ethereum.use_native_batch_call=false \
    --output_file=~/snapshot-test

{"level":"error","ts":"2025-01-10T10:44:02.338-0600","caller":"snapshot/snapshot.go:50","msg":"Failed to create database snapshot","error":"exit status 1","output":"pg_dump: error: could not open output file \"~/snapshot-test\": No such file or directory\n","stacktrace":"github.com/Layr-Labs/sidecar/pkg/snapshot.(*SnapshotService).CreateSnapshot\n\t/Users/seanmcgary/Code/sidecar/pkg/snapshot/snapshot.go:50\ngithub.com/Layr-Labs/sidecar/cmd.init.func1\n\t/Users/seanmcgary/Code/sidecar/cmd/createSnapshot.go:37\ngithub.com/spf13/cobra.(*Command).execute\n\t/Users/seanmcgary/Code/gocode/pkg/mod/github.com/spf13/[email protected]/command.go:985\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/Users/seanmcgary/Code/gocode/pkg/mod/github.com/spf13/[email protected]/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\t/Users/seanmcgary/Code/gocode/pkg/mod/github.com/spf13/[email protected]/command.go:1041\ngithub.com/Layr-Labs/sidecar/cmd.Execute\n\t/Users/seanmcgary/Code/sidecar/cmd/root.go:19\nmain.main\n\t/Users/seanmcgary/Code/sidecar/main.go:8\nruntime.main\n\t/opt/homebrew/opt/go/libexec/src/runtime/proc.go:272"}
Error: failed to create snapshot: exit status 1

[seanmcgary]

Restoring a snapshot that was dumped using a schema (e.g. public) to a different schema, doesnt work but gives me a success message. New schema I specified ends up as completely empty

# implicitly uses the public schema
go run main.go create-snapshot \
    --ethereum.rpc-url="..." \
    --chain="preprod" \
    --database.host="localhost" \
    --database.port="5432" \
    --database.user="seanmcgary" \
    --database.password="" \
    --database.db_name="sidecar_testnet_holesky" \
    --ethereum.use_native_batch_call=false \
    --input_file=/Users/seanmcgary/snapshot-test

# specifies a schema that is not public
go run main.go restore-snapshot \
    --ethereum.rpc-url="..." \
    --chain="preprod" \
    --database.host="localhost" \
    --database.port="5432" \
    --database.user="seanmcgary" \
    --database.password="" \
    --database.db_name="sidecar_testnet_holesky" \
    --ethereum.use_native_batch_call=false \
    --input_file=/Users/seanmcgary/snapshot-test \
    --database.schema_name=test

[BrendanCoughlan5]

Restoring a snapshot that was dumped using a schema (e.g. public) to a different schema, doesnt work but gives me a success message. New schema I specified ends up as completely empty

# implicitly uses the public schema
go run main.go create-snapshot \
    --ethereum.rpc-url="..." \
    --chain="preprod" \
    --database.host="localhost" \
    --database.port="5432" \
    --database.user="seanmcgary" \
    --database.password="" \
    --database.db_name="sidecar_testnet_holesky" \
    --ethereum.use_native_batch_call=false \
    --input_file=/Users/seanmcgary/snapshot-test

# specifies a schema that is not public
go run main.go restore-snapshot \
    --ethereum.rpc-url="..." \
    --chain="preprod" \
    --database.host="localhost" \
    --database.port="5432" \
    --database.user="seanmcgary" \
    --database.password="" \
    --database.db_name="sidecar_testnet_holesky" \
    --ethereum.use_native_batch_call=false \
    --input_file=/Users/seanmcgary/snapshot-test \
    --database.schema_name=test

More detail here:
https://www.notion.so/TDD-SideCar-Snapshots-14d13c11c3e0809c960cf1307d91d39e?d=17a13c11c3e0803c9ed6001cd067759d&pvs=4#17a13c11c3e0808e92a5e3800d1685bb

I thought about this case a fair bit,
I updated the CLI and docs accordingly and put in the docs how to convert the dump to a new schema.

[seanmcgary]

If we're adding this, why is this not part of the cli directly as a flag that the user can pass?