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

cmd/roachtest: skip clearrange/zfs/* tests #72044

Merged
merged 1 commit into from
Oct 27, 2021

Conversation

nicktrav
Copy link
Collaborator

The clearrange tests consistently fail when run with a ZFS filesystem.
Given that wider ZFS support is minimal, disable these tests until time
can be devoted to investigating the cause of the failures.

Release note: None.

@nicktrav nicktrav requested review from bananabrick and a team October 27, 2021 17:16
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@bananabrick bananabrick left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @bananabrick and @nicktrav)


-- commits, line 4 at r1:
Does it fail everytime? Afaik, there's failure once every couple of runs.

The clearrange tests occasionally fail when run with a ZFS filesystem.
Given that wider ZFS support is minimal, disable these tests until time
can be devoted to investigating the cause of the failures.

Release note: None.
@nicktrav nicktrav force-pushed the nickt.disable-clearrange branch from a54372f to 00f125f Compare October 27, 2021 18:23
Copy link
Collaborator Author

@nicktrav nicktrav left a comment

Choose a reason for hiding this comment

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

Dismissed @bananabrick from a discussion.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @bananabrick)


-- commits, line 4 at r1:

Previously, bananabrick (Arjun Nair) wrote…

Does it fail everytime? Afaik, there's failure once every couple of runs.

Perhaps "consistently" isn't the right word. I'll amend.

@nicktrav
Copy link
Collaborator Author

bors r+

@craig
Copy link
Contributor

craig bot commented Oct 27, 2021

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Oct 27, 2021

Build succeeded:

@craig craig bot merged commit b5aedea into cockroachdb:master Oct 27, 2021
@nicktrav nicktrav deleted the nickt.disable-clearrange branch October 27, 2021 21:58
@nicktrav
Copy link
Collaborator Author

TFTR!

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.

3 participants