Skip to content

Integrity support#3790

Draft
jbaublitz wants to merge 2 commits intostratis-storage:masterfrom
jbaublitz:issue-stratisd-3787
Draft

Integrity support#3790
jbaublitz wants to merge 2 commits intostratis-storage:masterfrom
jbaublitz:issue-stratisd-3787

Conversation

@jbaublitz
Copy link
Copy Markdown
Member

@jbaublitz jbaublitz commented Mar 10, 2025

Closes #3787

@jbaublitz jbaublitz self-assigned this Mar 10, 2025
@jbaublitz jbaublitz moved this to In Progress in 2025March Mar 10, 2025
@jbaublitz jbaublitz marked this pull request as draft March 10, 2025 14:26
@jbaublitz jbaublitz force-pushed the issue-stratisd-3787 branch from da8919e to 7a0d7ff Compare March 10, 2025 14:27
@packit-as-a-service
Copy link
Copy Markdown

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/stratis-storage-stratisd-3790
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@jbaublitz jbaublitz force-pushed the issue-stratisd-3787 branch from 7a0d7ff to 7754907 Compare March 10, 2025 18:55
#[allow(dead_code)]
fn generate_segments_from_metadata(
data_tier: DataTier<StratBlockDev>,
) -> StratisResult<(
Copy link
Copy Markdown

@siteshwar siteshwar Mar 10, 2025

Choose a reason for hiding this comment

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

OpenScanHub reports a warning:

#     |
#  28 |   ) -> StratisResult<(
#     |  ______^
#  29 | |     HashMap<Device, Vec<(Sectors, Sectors)>>,
#  30 | |     HashMap<Device, Vec<(Sectors, Sectors)>>,
#  31 | | )> {
#     | |__^
#     |
#     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity
#     = note: `-D clippy::type-complexity` implied by `-D clippy::all`
#     = help: to override `-D clippy::all` add `#[allow(clippy::type_complexity)]`

Please fix it before merging. Thanks!

@jbaublitz jbaublitz force-pushed the issue-stratisd-3787 branch 2 times, most recently from e95d924 to 78b9fa8 Compare March 10, 2025 20:42
@jbaublitz jbaublitz force-pushed the issue-stratisd-3787 branch from 78b9fa8 to fcf4114 Compare March 14, 2025 18:22
@mulkieran mulkieran removed this from 2025March Mar 31, 2025
@mulkieran mulkieran moved this to In Progress in 2025April Mar 31, 2025
@mulkieran mulkieran removed this from 2025April May 5, 2025
@mulkieran mulkieran added this to 2025May May 5, 2025
@mulkieran mulkieran moved this to In Progress (long term) in 2025May May 5, 2025
@mulkieran mulkieran removed this from 2025May Jun 9, 2025
@mulkieran mulkieran moved this to In Progress (long term) in 2025June Jun 9, 2025
@mulkieran mulkieran removed this from 2025June Jul 8, 2025
@mulkieran mulkieran moved this to In Progress (long term) in 2025July Jul 8, 2025
@mulkieran mulkieran removed this from 2025July Aug 11, 2025
@mulkieran mulkieran moved this to In Progress (long term) in 2025August Aug 11, 2025
@mulkieran mulkieran removed this from 2025August Sep 2, 2025
@mulkieran mulkieran moved this to In Progress (long term) in 2025September Sep 2, 2025
@mulkieran mulkieran removed this from 2025October Oct 11, 2025
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.

Add dm-integrity support for stratisd

3 participants