[bug]: better docs for the db.skip-native-sql-migration option

[ZZiigguurraatt]

We need a bit more explanation of the new db.skip-native-sql-migration config option at

lnd/sample-lnd.conf

Lines 1484 to 1489 in 6a3845b

	; If set to true, the KV to native SQL migration will be skipped. Note that
	; this option is intended for users who experience non-resolvable migration
	; errors.
	; db.skip-native-sql-migration=false

and

lnd/lncfg/db.go

Lines 89 to 91 in 6a3845b

	SkipNativeSQLMigration bool `long:"skip-native-sql-migration" description:"Do not run the KV to native SQL migration. It should only be used if errors are encountered normally."`

.

What the user is not told is that if we have db.use-native-sql=true and db.skip-native-sql-migration=true that basically means you're starting with an empty invoice DB and forgetting their existing invoice database.

[ZZiigguurraatt]

Also, the option db.skip-native-sql-migration=true seems to exist so that if you have some error when trying to do DB migration, you can stop LND, enable this option, then restart LND using the old KV DB and the partially migrated DB is not used.

[Lymah123]

Hi @ZZiigguurraatt , can I work on the issue?