Skip to content

Backup stellar-misc.db#370

Merged
marta-lokhova merged 2 commits intostellar:mainfrom
sisuresh:misc-backup
Apr 14, 2026
Merged

Backup stellar-misc.db#370
marta-lokhova merged 2 commits intostellar:mainfrom
sisuresh:misc-backup

Conversation

@sisuresh
Copy link
Copy Markdown
Contributor

Fixes a DatabasInplaceUpgrade failure we're now running into.

Copilot AI review requested due to automatic review settings April 14, 2026 15:25
@sisuresh sisuresh requested a review from marta-lokhova April 14, 2026 15:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses failures in DatabasInplaceUpgrade by adding backup/restore support for the stellar-misc.db SQLite database, while keeping compatibility with older peers that predate the main/misc DB split.

Changes:

  • Add stellar-misc.db backup creation during peer data dumps (optional when the misc DB does not exist).
  • Add optional restore of the misc DB when fetching a DB backup from another peer.
  • Introduce new config constants/URLs for misc DB paths and transfer locations.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/FSLibrary/StellarKubeSpecs.fs Adds an optional misc DB restore step during DB fetch from a peer.
src/FSLibrary/StellarDataDump.fs Adds misc DB backup generation, guarded by existence checks.
src/FSLibrary/StellarCoreCfg.fs Adds misc DB path/backup path/URL constants used by backup/restore flows.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/FSLibrary/StellarDataDump.fs Outdated
Comment thread src/FSLibrary/StellarCoreCfg.fs Outdated
Comment thread src/FSLibrary/StellarKubeSpecs.fs Outdated
@marta-lokhova marta-lokhova merged commit e22f281 into stellar:main Apr 14, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants