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

.github: dump buddyinfo and pagetypeinfo when ci-e2e fails #26600

Merged
merged 1 commit into from Jul 4, 2023
Merged

.github: dump buddyinfo and pagetypeinfo when ci-e2e fails #26600

merged 1 commit into from Jul 4, 2023

Conversation

lmb
Copy link
Contributor

@lmb lmb commented Jul 3, 2023

There is a persistent CI flake where the kernel returns ENOMEM from BPF map
updates. Grab buddyinfo and pagetypeinfo from /proc to see whether the
allocation failures are related to memory fragmentation.

Updates #26062

@lmb lmb added area/CI Continuous Integration testing issue or flake ci/flake This is a known failure that occurs in the tree. Please investigate me! labels Jul 3, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jul 3, 2023
@lmb
Copy link
Contributor Author

lmb commented Jul 3, 2023

/test

@lmb
.github: dump buddyinfo and pagetypeinfo when ci-e2e fails
4ebbb33 
There is a persistent CI flake where the kernel returns ENOMEM from
BPF map updates. Grab buddyinfo and pagetypeinfo from /proc to see
whether the allocation failures are related to memory fragmentation.

Signed-off-by: Lorenz Bauer <lmb@isovalent.com>
@lmb lmb force-pushed the pr/lmb/ci-e2e-fragmentation branch from 524b8b6 to ba0279f Compare July 3, 2023 14:44
@lmb
Copy link
Contributor Author

lmb commented Jul 3, 2023

/test-ci-e2e

@lmb lmb added the release-note/minor This PR changes functionality that users may find relevant to operating Cilium. label Jul 3, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 3, 2023
@lmb
Copy link
Contributor Author

lmb commented Jul 3, 2023

Works in the sense that we get the expected output: https://github.com/cilium/cilium/actions/runs/5445805364/jobs/9906896104#step:11:225

@lmb lmb force-pushed the pr/lmb/ci-e2e-fragmentation branch from ba0279f to 4ebbb33 Compare July 3, 2023 16:14
@lmb lmb marked this pull request as ready for review July 3, 2023 16:15
@lmb lmb requested review from a team as code owners July 3, 2023 16:15
@lmb lmb requested a review from viktor-kurchenko July 3, 2023 16:15
@lmb
Copy link
Contributor Author

lmb commented Jul 4, 2023

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 4, 2023
@borkmann borkmann merged commit 55a09e5 into main Jul 4, 2023
65 checks passed
@borkmann borkmann deleted the pr/lmb/ci-e2e-fragmentation branch July 4, 2023 13:36
@joestringer joestringer added release-note/ci This PR makes changes to the CI. and removed release-note/minor This PR changes functionality that users may find relevant to operating Cilium. labels Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@viktor-kurchenko viktor-kurchenko viktor-kurchenko approved these changes

Assignees
No one assigned
Labels
area/CI Continuous Integration testing issue or flake ci/flake This is a known failure that occurs in the tree. Please investigate me! ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@lmb @viktor-kurchenko @borkmann @joestringer