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

Disable tests that are failing because of resize2fs kernel bug #14146

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

markylaing
Copy link
Contributor

This kernel bug is causing CI failures for Ceph and LVM. This PR disables those tests for now. We need to re-enable these tests when the kernel bug is fixed.

@markylaing markylaing self-assigned this Sep 23, 2024
@markylaing markylaing marked this pull request as draft September 23, 2024 13:41
@markylaing
Copy link
Contributor Author

Marked as draft to see the result of the workflow first, if many other tests need to be disabled, we'll turn those off for ceph and LVM only.

@simondeziel
Copy link
Member

Marked as draft to see the result of the workflow first, if many other tests need to be disabled, we'll turn those off for ceph and LVM only.

I think that anything that involves a block device and ext4 resizing risks triggering the issue. This would mean that creating any instance on top of LVM and Ceph is at risk, unless we can avoid the resizing part but that seems to big for a quick workaround.

@markylaing markylaing force-pushed the disable-failing-tests branch 2 times, most recently from d6992c9 to ea421ca Compare September 23, 2024 15:00
@markylaing markylaing force-pushed the disable-failing-tests branch 4 times, most recently from f7893c1 to 550f052 Compare September 23, 2024 16:45
@markylaing markylaing marked this pull request as ready for review September 23, 2024 17:05
@markylaing markylaing marked this pull request as draft September 23, 2024 17:05
@markylaing
Copy link
Contributor Author

@tomponline @simondeziel this works as is so I can change pull_request to push OR I can go back to excluding the event type in the matrix, which will stop all of the additional steps (e.g. remove_docker) from running. Let me know what you'd prefer.

Signed-off-by: Mark Laing <mark.laing@canonical.com>
@markylaing markylaing marked this pull request as ready for review September 23, 2024 17:29
@markylaing
Copy link
Contributor Author

@tomponline @simondeziel I've opted with the matrix exclusions as I think it's easier to read and also means we're not using unnecessary compute.

@tomponline
Copy link
Member

@tomponline @simondeziel I've opted with the matrix exclusions as I think it's easier to read and also means we're not using unnecessary compute.

Ok but we need to exclude "schedule" not just "push" for the tiobe daily checks

Copy link
Member

@tomponline tomponline left a comment

Choose a reason for hiding this comment

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

Merging this for now

@tomponline tomponline merged commit aa06597 into canonical:main Sep 23, 2024
28 of 30 checks passed
@markylaing
Copy link
Contributor Author

@tomponline @simondeziel I've opted with the matrix exclusions as I think it's easier to read and also means we're not using unnecessary compute.

Ok but we need to exclude "schedule" not just "push" for the tiobe daily checks

Yep this excludes schedule as well 👍

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