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

feat: bump bpftrace version to v0.19.1 #205

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cakemanny
Copy link

@cakemanny cakemanny commented May 4, 2024

This resolves #200

It bumps the bptrace version to v0.19.1 .

I chose 0.19.1 as it's the most recent quay.io/iovisor/bpftrace tag. (It seems they are working on switching to using a Nix based build for later versions).
I wasn't able to successfully able to bump bcc.

The other changes are related to

  • getting the Github Actions to run in the forked repo - i.e. $GIT_ORG is expanded in integration tests
  • getting the integration tests to pass with more recent versions of Minikube and Kind.
    • I ended up bumping the version of go in order to be able to debug locally, so I hope it's ok that that's included also.

By all means let me know if it needs additional touches or splitting out into separate PRs.

@cakemanny cakemanny marked this pull request as draft May 4, 2024 20:54
@cakemanny
Copy link
Author

cakemanny commented May 4, 2024

It seems the github actions are actually failing still, I'll have to come back to this. I've put in draft mode for now.
I also tried to follow the instructions for signing the commits but for some reason they're showing as unverified so I'll have to revisit that too.

Now fixed. I'd simply gone a bit too far with the go builder base image; switching from bookworm back to bullseye sorted that. I also added something to output pod log contents when a test case fails.

@cakemanny cakemanny marked this pull request as ready for review May 4, 2024 23:36
@cakemanny
Copy link
Author

@fntlnz, would you be the one to look at this?

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.

update bpftrace to 0.18.0
1 participant