kubectl-trace icon indicating copy to clipboard operation
kubectl-trace copied to clipboard

feat: bump bpftrace version to v0.19.1

Open cakemanny opened this issue 9 months ago • 4 comments

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.

PS: also fixes #174

cakemanny avatar May 04 '24 20:05 cakemanny

~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 avatar May 04 '24 21:05 cakemanny

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

cakemanny avatar May 15 '24 18:05 cakemanny

Thanks @cakemanny - This was long due. I'll take a look

fntlnz avatar May 20 '24 12:05 fntlnz

I'm still looking into this @cakemanny - I did not forget! Thanks again !

fntlnz avatar Jun 25 '24 00:06 fntlnz

Hey @fntlnz, I guess you've still not had time? Or is there something I can do to help, such as break parts out or explain better anything that looks a bit mysterious?

cakemanny avatar Jul 07 '24 09:07 cakemanny

@cakemanny yes it's just my schedule the PR looks good to review, please hang on and thanks again for the great work.

fntlnz avatar Jul 08 '24 16:07 fntlnz