Skip to content
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

feat: extend Snapshotter to archive only specific files and directories #2189

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dlachaume
Copy link
Collaborator

@dlachaume dlachaume commented Dec 20, 2024

Content

This PR includes an update to the Snapshotter that allows it to create archives from specific directories and files.
It’s a preparatory step for the creation of archives from ancillary files.

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)

Relates to #2151

Copy link

github-actions bot commented Dec 20, 2024

Test Results

    4 files  ±0     52 suites  ±0   10m 26s ⏱️ +3s
1 465 tests +5  1 465 ✅ +5  0 💤 ±0  0 ❌ ±0 
1 721 runs  +5  1 721 ✅ +5  0 💤 ±0  0 ❌ ±0 

Results for commit 58133cb. ± Comparison against base commit d3e21fb.

♻️ This comment has been updated with latest results.

@dlachaume dlachaume temporarily deployed to testing-sanchonet December 20, 2024 11:20 — with GitHub Actions Inactive
@dlachaume dlachaume force-pushed the ensemble/2151/extend-snapshotter-for-specific-files-directories branch from 2f79c05 to 4331d07 Compare December 20, 2024 13:35
@dlachaume dlachaume temporarily deployed to testing-sanchonet December 20, 2024 13:53 — with GitHub Actions Inactive
@dlachaume dlachaume marked this pull request as ready for review December 20, 2024 13:53
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 👍

mithril-aggregator/src/snapshotter.rs Show resolved Hide resolved
Copy link
Collaborator

@Alenar Alenar left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

I had a sightly different idea regarding the snapshotter, but your design may work as well and is flexible enough to be adapted if needed.

What I had in mind was to let the snapshotter compute itself the subset of files that must be included in a ancillary archive, the only parameter to pass would be the immutable to include.

dlachaume and others added 2 commits December 20, 2024 17:16
* mithril-aggregator from `0.6.5` to `0.6.6`
@dlachaume dlachaume force-pushed the ensemble/2151/extend-snapshotter-for-specific-files-directories branch from 4331d07 to 58133cb Compare December 20, 2024 16:16
@dlachaume dlachaume temporarily deployed to testing-sanchonet December 20, 2024 16:25 — with GitHub Actions Inactive
@dlachaume dlachaume mentioned this pull request Dec 23, 2024
12 tasks
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.

4 participants