-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
CodeQL: Fix Go 1.21 build fix and update branch list #31892
Conversation
b9241f6
to
86681fc
Compare
@@ -17,7 +17,7 @@ jobs: | |||
strategy: | |||
fail-fast: false | |||
matrix: | |||
branch: [ 'master', 'branch/v11', 'branch/v12', 'branch/v13' ] | |||
branch: [ 'master', 'branch/v12', 'branch/v13', 'branch/v14' ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we keep branch/v11 for now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For CodeQL I think it is safe to drop v11.
We're getting little negative signal out of these the recurring checks AFAIK. The hassle of remembering to remove v11 is a greater cost to the org than the chance we miss something important on v11 in the next week.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wadells is echo'ing my thoughts as well. It's a small time window and any changes in v11 should go through other branches first anyways. So the window of risk is small and since this is a workflow update this PR needs significant approval: https://github.com/gravitational/teleport/blob/master/.github/CODEOWNERS#L3
Fix CodeQL Build and update branch list to include the v14 branch.
This PR fixes https://github.com/gravitational/SecOps/issues/414
Test build and analysis can be seen in this run: https://github.com/gravitational/teleport/actions/runs/6188331568/job/16800106312?pr=31892