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(deps): update module github.com/cilium/ebpf to v0.17.1 (main) #3206

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cilium-renovate[bot]
Copy link
Contributor

@cilium-renovate cilium-renovate bot commented Dec 9, 2024

This PR contains the following updates:

Package Type Update Change
github.com/cilium/ebpf require minor v0.16.1-0.20241205185900-f0eec7efba9d -> v0.17.1

Release Notes

cilium/ebpf (github.com/cilium/ebpf)

v0.17.1: Hotfix: don't retry endlessly on verifier errors on kernels <6.4

Compare Source

During the prior removal of ProgramOptions.LogSize in v0.16, the tests weren't updated to exercise the retry logic. With the last-minute addition of ProgramOptions.LogSizeStart, a bug was introduced that would cause program loads to retry indefinitely without ever growing the buffer in case of a verifier error on kernels before 6.4. This is now fixed, and the tests were updated to exercise the retry logic.


Configuration

📅 Schedule: Branch creation - "on monday and friday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@cilium-renovate cilium-renovate bot requested a review from a team as a code owner December 9, 2024 16:21
@cilium-renovate cilium-renovate bot added release-blocker This PR or issue is blocking the next release. release-note/dependency This PR updates one or multiple dependencies labels Dec 9, 2024
@cilium-renovate cilium-renovate bot requested a review from olsajiri December 9, 2024 16:21
Copy link
Member

@mtardy mtardy left a comment

Choose a reason for hiding this comment

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

I think we should wait for the next release, except if we need a special version to work with. Otherwise we'll have PRs every time they merge something into main.

@mtardy mtardy removed the request for review from olsajiri December 12, 2024 09:49
@kkourt
Copy link
Contributor

kkourt commented Dec 12, 2024

I think we should wait for the next release, except if we need a special version to work with. Otherwise we'll have PRs every time they merge something into main.

OK, removing the release-blocker tag then.

@kkourt kkourt removed the release-blocker This PR or issue is blocking the next release. label Dec 12, 2024
@cilium-renovate cilium-renovate bot changed the title fix(deps): update github.com/cilium/ebpf digest to 99e9ca6 (main) fix(deps): update github.com/cilium/ebpf digest to 98ede8a (main) Dec 12, 2024
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch from ce2d747 to 442749a Compare December 12, 2024 13:21
Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit f109cd3
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/6765364ab0339100081aeb9f
😎 Deploy Preview https://deploy-preview-3206--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch 2 times, most recently from 177359f to 66e06e7 Compare December 12, 2024 19:13
@cilium-renovate cilium-renovate bot changed the title fix(deps): update github.com/cilium/ebpf digest to 98ede8a (main) fix(deps): update all go dependencies main (main) Dec 12, 2024
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch from 66e06e7 to 6e7d251 Compare December 16, 2024 10:15
@cilium-renovate cilium-renovate bot changed the title fix(deps): update all go dependencies main (main) fix(deps): update github.com/cilium/ebpf digest to 0a77f25 (main) Dec 16, 2024
@cilium-renovate cilium-renovate bot changed the title fix(deps): update github.com/cilium/ebpf digest to 0a77f25 (main) fix(deps): update github.com/cilium/ebpf digest to 60405bb (main) Dec 16, 2024
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch 2 times, most recently from e193e0b to 5d36802 Compare December 16, 2024 18:16
@cilium-renovate cilium-renovate bot changed the title fix(deps): update github.com/cilium/ebpf digest to 60405bb (main) fix(deps): update all go dependencies main (main) Dec 16, 2024
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch 5 times, most recently from df9034e to 0ff8475 Compare December 18, 2024 16:22
@cilium-renovate cilium-renovate bot changed the title fix(deps): update all go dependencies main (main) fix(deps): update all go dependencies main (main) (minor) Dec 18, 2024
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch 2 times, most recently from b0a5f87 to f109cd3 Compare December 20, 2024 09:18
@cilium-renovate cilium-renovate bot changed the title fix(deps): update all go dependencies main (main) (minor) fix(deps): update module github.com/cilium/ebpf to v0.17.1 (main) Dec 20, 2024
Signed-off-by: cilium-renovate[bot] <134692979+cilium-renovate[bot]@users.noreply.github.com>
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch from f109cd3 to 9c44714 Compare December 20, 2024 11:19
@mtardy mtardy requested a review from olsajiri December 20, 2024 16:29
PR cilium/ebpf#1626 added the new pin package that simplifies a lot how
one can parse a bpf fs. I just love it.

Signed-off-by: Mahe Tardy <[email protected]>
@mtardy mtardy force-pushed the renovate/main-all-go-deps-main branch from 4c31e33 to ef53ab5 Compare December 20, 2024 16:30
@mtardy
Copy link
Member

mtardy commented Dec 20, 2024

Hey @olsajiri could you take a look at my commit and see if you need updates on your side? Since my part of this code breaks loudly with tests and your stuff recently broke quietly, you might need to test it manually and see if it needs to use this amazing pin.WalkDir stuff.

@cilium-renovate
Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@olsajiri
Copy link
Contributor

Hey @olsajiri could you take a look at my commit and see if you need updates on your side? Since my part of this code breaks loudly with tests and your stuff recently broke quietly, you might need to test it manually and see if it needs to use this amazing pin.WalkDir stuff.

added change to use it, was easy and seems to work, thanks

@olsajiri olsajiri force-pushed the renovate/main-all-go-deps-main branch 2 times, most recently from 3d03040 to 7528c11 Compare December 21, 2024 23:47
Using new pin.WalkDir interface to get all the tetragon
programs from bpffs directory.

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri force-pushed the renovate/main-all-go-deps-main branch from 7528c11 to 93db3c9 Compare December 21, 2024 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/dependency This PR updates one or multiple dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants