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

Fix for k8s nfs related tests probably caused by ulimit changes #9799

Merged
merged 1 commit into from Feb 10, 2024

Conversation

dims
Copy link
Member

@dims dims commented Feb 10, 2024

on 2/7 the k8s upstream jobs picked up a new version of ubuntu kernel and some of the NFS related jobs started failing:

We had the same issue with kops based tests as well which we fixed by setting LimitNOFILE explicitly to a known previous value:
kubernetes/kops#16329

So we want to see if fixing this in the install scripts for the CI jobs works for us. Also note the pending PR here:
#9660

and the note back from 2019:
#3201 (comment)

@dims
Copy link
Member Author

dims commented Feb 10, 2024

cc @upodroid @henry118 @mikebrow

@dims
Copy link
Member Author

dims commented Feb 10, 2024

If this does not work, will file a revert for sure!

@dims
Copy link
Member Author

dims commented Feb 10, 2024

GOOD:

Feb 07 19:31:06.349093 ubuntu kernel: Linux version 6.2.0-1019-gcp (buildd@lcy02-amd64-032) (x86_64-linux-gnu-gcc-12 (Ubuntu 12.3.0-1ubuntu1~22.04) 12.3.0, GNU ld (GNU Binutils for Ubuntu) 2.38) #21~22.04.1-Ubuntu SMP Thu Nov 16 18:18:34 UTC 2023 (Ubuntu 6.2.0-1019.21~22.04.1-gcp 6.2.16)

BAD:

Feb 07 20:33:47.276593 ubuntu kernel: Linux version 6.5.0-1013-gcp (buildd@lcy02-amd64-064) (x86_64-linux-gnu-gcc-12 (Ubuntu 12.3.0-1ubuntu1~22.04) 12.3.0, GNU ld (GNU Binutils for Ubuntu) 2.38) #13~22.04.1-Ubuntu SMP Wed Jan 24 23:39:40 UTC 2024 (Ubuntu 6.5.0-1013.13~22.04.1-gcp 6.5.8)

Signed-off-by: Davanum Srinivas <davanum@gmail.com>
@dims dims force-pushed the fix-for-k8s-nfs-related-tests branch from 3324383 to 23ebfd0 Compare February 10, 2024 16:28
@dims dims added this pull request to the merge queue Feb 10, 2024
Merged via the queue into containerd:main with commit bb45ce1 Feb 10, 2024
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants