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.

Sign up for GitHub

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

Jump to bottom

ZTS: Add LUKS sanity test #16681

Merged
merged 1 commit into from
Oct 25, 2024
Merged

ZTS: Add LUKS sanity test #16681

merged 1 commit into from
Oct 25, 2024

Conversation

tonyhutter
Copy link
Contributor

Motivation and Context

Add a LUKS sanity test to trigger: #16631

Description

Add a basic LUKS test that will verify the fix from #16678.

How Has This Been Tested?

Tested locally

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@tonyhutter tonyhutter mentioned this pull request Oct 23, 2024
Closed
13 tasks
@tonyhutter tonyhutter force-pushed the zts-luks branch 2 times, most recently from 7751f9d to 12d3ab2 Compare October 23, 2024 17:23
@behlendorf behlendorf added the Status: Accepted Ready to integrate (reviewed, tested) label Oct 24, 2024
@tonyhutter
ZTS: Add LUKS sanity test
efaaeb9 
Add a LUKS sanity test to trigger: openzfs#16631

Signed-off-by: Tony Hutter <hutter2@llnl.gov>
@tonyhutter
Copy link
Contributor Author

@mcmilk I just rebased this to pull in 2e4e092

@behlendorf behlendorf merged commit e5d1f68 into openzfs:master Oct 25, 2024
19 of 20 checks passed
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Nov 1, 2024
@tonyhutter @behlendorf
ZTS: Add LUKS sanity test
f7b4bca 
Add a LUKS sanity test to trigger: openzfs#16631

Reviewed-by: Tino Reichardt <milky-zfs@mcmilk.de>
Reviewed-by: Rob Norris <robn@despairlabs.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes openzfs#16681
robn pushed a commit to robn/zfs that referenced this pull request Nov 5, 2024
@tonyhutter
ZTS: Add LUKS sanity test
6d64dbd 
Add a LUKS sanity test to trigger: openzfs#16631

Reviewed-by: Tino Reichardt <milky-zfs@mcmilk.de>
Reviewed-by: Rob Norris <robn@despairlabs.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes openzfs#16681
ptr1337 pushed a commit to CachyOS/zfs that referenced this pull request Nov 14, 2024
@tonyhutter
ZTS: Add LUKS sanity test
f263314 
Add a LUKS sanity test to trigger: openzfs#16631

Reviewed-by: Tino Reichardt <milky-zfs@mcmilk.de>
Reviewed-by: Rob Norris <robn@despairlabs.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Closes openzfs#16681
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amotin amotin amotin left review comments

@mcmilk mcmilk mcmilk approved these changes

@robn robn robn left review comments

@behlendorf behlendorf behlendorf approved these changes

Assignees
No one assigned
Labels
Status: Accepted Ready to integrate (reviewed, tested)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tonyhutter @robn @behlendorf @mcmilk @amotin