Skip to content

Feat: improve disk space calculation for Incremental DB Restoration #2345

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

dlachaume
Copy link
Collaborator

@dlachaume dlachaume commented Mar 3, 2025

Content

This PR includes enhancements to the disk space computation for incremental Cardano DB snapshot restoration in the client CLI.
It leverages updated artifact metadata to compute the required disk space for both full and partial restorations, adding ancillary files sizes when necessary.
Additionally, for partial restorations, a 10% safety margin is applied to ensure that sufficient disk space is available before download, as the computation uses the average size of an immutable file.

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • CHANGELOG file is updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested
  • Documentation
    • Update README file (if relevant)
    • Update documentation website (if relevant)
    • Add dev blog post (if relevant)

Issue(s)

Closes #2292

@dlachaume dlachaume self-assigned this Mar 3, 2025
Copy link

github-actions bot commented Mar 3, 2025

Test Results

    3 files  ± 0     52 suites  ±0   10m 2s ⏱️ -8s
1 682 tests + 4  1 682 ✅ + 4  0 💤 ±0  0 ❌ ±0 
2 072 runs  +12  2 072 ✅ +12  0 💤 ±0  0 ❌ ±0 

Results for commit 3197a4a. ± Comparison against base commit b480fda.

♻️ This comment has been updated with latest results.

@dlachaume dlachaume temporarily deployed to testing-preview March 3, 2025 16:38 — with GitHub Actions Inactive
@dlachaume dlachaume marked this pull request as ready for review March 3, 2025 17:33
@dlachaume dlachaume marked this pull request as draft March 3, 2025 17:33
@dlachaume dlachaume requested review from jpraynaud and sfauvel March 3, 2025 17:33
Base automatically changed from sfa/2291/enhance_artifact_structure_for_incremental_cardano_db to main March 4, 2025 10:08
@dlachaume dlachaume force-pushed the dlachaume/2292/enhance-computation-required-disk-space-client-cli branch from 69afdfb to f05b4d6 Compare March 4, 2025 10:39
@dlachaume dlachaume marked this pull request as ready for review March 4, 2025 10:41
-- Enhance required disk space with margin computation
-- Enhance test stability to handle minor rounding differences
@dlachaume dlachaume temporarily deployed to testing-preview March 4, 2025 11:08 — with GitHub Actions Inactive
Copy link
Collaborator

@sfauvel sfauvel left a comment

Choose a reason for hiding this comment

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

LGTM

@dlachaume dlachaume temporarily deployed to testing-preview March 4, 2025 11:17 — with GitHub Actions Inactive
@dlachaume dlachaume requested a review from jpraynaud March 4, 2025 11:27
@dlachaume dlachaume temporarily deployed to testing-preview March 4, 2025 11:34 — with GitHub Actions Inactive
@dlachaume dlachaume temporarily deployed to testing-preview March 4, 2025 13:48 — with GitHub Actions Inactive
* mithril-client-cli from `0.11.6` to `0.11.7`
@dlachaume dlachaume force-pushed the dlachaume/2292/enhance-computation-required-disk-space-client-cli branch from 148e8aa to 3197a4a Compare March 4, 2025 14:13
@dlachaume dlachaume temporarily deployed to testing-preview March 4, 2025 14:21 — with GitHub Actions Inactive
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@dlachaume dlachaume merged commit d399cfe into main Mar 4, 2025
37 of 41 checks passed
@dlachaume dlachaume deleted the dlachaume/2292/enhance-computation-required-disk-space-client-cli branch March 4, 2025 14:40
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.

Enhance computation of required disk space for Incremental Cardano DB in client CLI
3 participants