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: Remove zfs_allow_010_pos expection for FreeBSD #15382

Merged
merged 1 commit into from
Oct 10, 2023
Merged

ZTS: Remove zfs_allow_010_pos expection for FreeBSD #15382

merged 1 commit into from
Oct 10, 2023

Conversation

behlendorf
Copy link
Contributor

Motivation and Context

Update zts-report.py to accurately reflect non-passing tests.

Description

This issue should now be address by PR #15376 and the exception for this test case be removed.

How Has This Been Tested?

Will be verified by the CI.

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:

@behlendorf
ZTS: Remove zfs_allow_010_pos expection for FreeBSD
d99fbdf 
This issue should now be address by PR openzfs#15376 and the exception
for this test case be removed.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
@behlendorf behlendorf added the Status: Code Review Needed Ready for review and testing label Oct 10, 2023
@behlendorf
Copy link
Contributor Author

@usaleem-ix after merging I realized we should have also removed this from zts-report.py since we believe it to be fixed. Please take a look.

@behlendorf behlendorf added Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels Oct 10, 2023
@behlendorf behlendorf merged commit 483ccf0 into openzfs:master Oct 10, 2023
22 of 26 checks passed
behlendorf added a commit that referenced this pull request Oct 11, 2023
@behlendorf
ZTS: Remove zfs_allow_010_pos expection for FreeBSD
75a7740 
This issue should now be address by PR #15376 and the exception
for this test case be removed.

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by:  Umer Saleem <usaleem@ixsystems.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #15382
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Dec 12, 2023
@behlendorf @lundman
ZTS: Remove zfs_allow_010_pos expection for FreeBSD
e9b284d 
This issue should now be address by PR openzfs#15376 and the exception
for this test case be removed.

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by:  Umer Saleem <usaleem@ixsystems.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#15382
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amotin amotin amotin approved these changes

@usaleem-ix usaleem-ix usaleem-ix 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.
3 participants
@behlendorf @amotin @usaleem-ix