make gen_compile_commands for tetragon

[0xMALVEE]

Fixes
#3697

Description

just like in cilium, make gen_compile_commands will generate compile_commands.json using bear.
It's used for some IDE to analyze codebase better and ensure no linting errors are shown by following includes paths

[ghost]

@0xMALVEE Thank you for the PR. For reference, here are the guidelines for commits in the Tetragon project: https://tetragon.io/docs/contribution-guide/submitting-a-pull-request/#commit-changes If you could write a more descriptive commit message that would be great, but IMO it's also fine to merge as is.

[0xMALVEE]

@lambdanis updated the commit according to commit guidelines. <3

[kkourt]

LGTM, thanks!

[kkourt]

@0xMALVEE 👋🏼

Could you please fix the checkpatch warnings?

[netlify]

✅ Deploy Preview for tetragon ready!

Name	Link
🔨 Latest commit	b11be28
🔍 Latest deploy log	https://app.netlify.com/projects/tetragon/deploys/68592935a160d10008c539be
😎 Deploy Preview	https://deploy-preview-3698--tetragon.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[mtardy]

I'm sorry this is still failing, I think you need to do that:

-Signed-off-by: M Alvee <m.alvee8141@gmail.com>
-Co-Authored-By: Kornilios Kourtis <kkourt@users.noreply.github.com>
+Co-authored-By: Kornilios Kourtis <kkourt@users.noreply.github.com>
+Signed-off-by: M Alvee <m.alvee8141@gmail.com>

remove A in authored, and maybe switch co-authored and signed-off.

Sorry I know this is a bit annoying

[kkourt]

Thanks!

[kkourt]

Warning: WARNING:BAD_SIGN_OFF: 'Co-authored-by:' is the preferred signature form

:(

Indeed very annoying. Feel free to remove it :)

[mtardy]

Warning: WARNING:BAD_SIGN_OFF: 'Co-authored-by:' is the preferred signature form

:(

Indeed very annoying. Feel free to remove it :)

oh my... I missed the uppercase B in my reco sorry

[0xMALVEE]

Warning: WARNING:BAD_SIGN_OFF: 'Co-authored-by:' is the preferred signature form

:(
Indeed very annoying. Feel free to remove it :)

oh my... I missed the uppercase B in my reco sorry

I have updated with "b"

[0xMALVEE]

I think the project should have pre commit hooks to check commits details when user comits, rather then in github actions. What you guys think

[mtardy]

I don't understand what's happening, I have been trying to repro locally but it doesn't make any sense. Let me try to push to your PR to rebase and see because even the error displayed doesn't make sense. 🤔

[mtardy]

I think the project should have pre commit hooks to check commits details when user comits, rather then in github actions. What you guys think

Yeah that could be an idea maybe just having a target for first contributors to trigger checkpatch and such static checks could be an idea as well (looking at the static-checks YAML GitHub actions could give an idea, in addition to checkpatch).

Unrelated to this, sorry the contribution was such a poor experience, I admit for this kind of patch, it should have been merged quickly and you shouldn't have to do all these edits! Sorry, I hope you still want to contribute to Tetragon :)

[mtardy]

Hopefully this passes CI and we can merge this :)