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

Temporarily disable the reguid test #997

Closed
wants to merge 1 commit into from

Conversation

dechamps
Copy link
Contributor

Currently, ztest fails with the following error:

error: Pool 'ztest' has encountered an uncorrectable I/O failure and the failure mode property for this pool is set to panic.

We know how to fix it (see issue #939), but it may take some time before we get around to merging the fix, which has some heavy dependencies.

In the mean time, it is not ideal to be unable to use ztest just because of a small isolated issue, so this patch works around the problem by disabling the reguid test. This is just a temporary hack to keep ztest usable.

The reguid test will be enabled again when the proper fix is merged.

Currently, ztest fails with the following error:

    error: Pool 'ztest' has encountered an uncorrectable I/O failure and the failure mode property for this pool is set to panic.

We know how to fix it (see issue openzfs#939), but it may take some time
before we get around to merging the fix, which has some heavy
dependencies.

In the mean time, it is not ideal to be unable to use ztest just
because of a small isolated issue, so this patch works around the
problem by disabling the reguid test. This is just a temporary hack to
keep ztest usable.

The reguid test will be enabled again when the proper fix is merged.
@dechamps
Copy link
Contributor Author

@behlendorf: with #991, #992 and this one, you should be able to add ztest to your automated testing chain without any issues.

@dechamps dechamps mentioned this pull request Oct 1, 2012
@behlendorf behlendorf closed this in d135245 Oct 3, 2012
behlendorf added a commit that referenced this pull request Jan 8, 2013
This reverts commit d135245.
Since feature flags have now been merged we can apply the real
upstream fix from Illumos.

Signed-off-by: Brian Behlendorf <[email protected]>
Issue #997
unya pushed a commit to unya/zfs that referenced this pull request Dec 13, 2013
This reverts commit d135245.
Since feature flags have now been merged we can apply the real
upstream fix from Illumos.

Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#997
pcd1193182 pushed a commit to pcd1193182/zfs that referenced this pull request Sep 26, 2023
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.

2 participants