-
Notifications
You must be signed in to change notification settings - Fork 242
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
ci: Add CodeQL analysis #475
Conversation
Signed-off-by: Tom Payne <tom@isovalent.com>
Thanks @twpayne for the PR! Is the "analysis not found" expected? https://github.com/cilium/hubble/pull/475/checks?check_run_id=1807367595 |
Nope, definitely not expected. With these CI changes, you only get to debug them once you push a branch :) |
So what would be the way to make forward progress on this? |
I need to do some debugging :) |
I think this is just because this is the first CodeQL analysis for this repo, so there's no other analysis to compare with to find what's been fixed or newly introduced. Relevant doc. |
That make sense. What we could do is merge this PR, and then you make another (dummy or cosmetic) PR to see if everything work as expected. How does that sound? |
Good idea. I've rebased #474 on top of this, which should serve the purpose. |
This adds CodeQL analysis to Hubble, similar to cilium/cilium#14514.