public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Michael Kubacki" <mikuback@linux.microsoft.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>
Subject: [edk2-devel] CodeQL Failures in PR Checks
Date: Fri, 26 Jul 2024 10:07:42 -0400	[thread overview]
Message-ID: <afa7ce2f-c524-4503-b090-c6420ff9cfa8@linux.microsoft.com> (raw)

TLDR: If CodeQL failed in a PR. Rebase your PR branch on master and push.

---

edk2 uses an application called the CodeQL CLI to run CodeQL in CI builds.

https://github.com/github/codeql-cli-binaries/

A fixed version of the application is used to allow controlled updates 
where the version is tracked in YAML files like this:

https://github.com/tianocore/edk2/blob/master/BaseTools/Plugin/CodeQL/codeqlcli_ext_dep.yaml

Yesterday, a new CodeQL CLI version was released v2.18.1:

https://github.com/github/codeql-cli-binaries/releases/tag/v2.18.1

CodeQL uses queries to find issues that target certain CodeQL CLI 
versions. v2.18.1 published a new set of queries incompatible with the 
CodeQL CLI version fixed in edk2.

Last month, this PR intended to fix the queries as well to prevent the 
CLI from using an incompatible (latest) query set, but it wasn't merged 
due to some mergify issues. It eventually fell off the radar.

https://github.com/tianocore/edk2/pull/5720

---

Actions Taken:

To solve the immediate issue, the CodeQL CLI issue is updated in edk2 so 
it is compatible with the latest queries moving the project to the 
latest release and fixing the PR status checks. This was done in 
https://github.com/tianocore/edk2/commit/6589843cc619b3a5e2d2c0e5b12451b11a3f2288.

I will recreate 5720 to account for this new CodeQL version in a new PR 
to prevent this from happening in the future.

Thanks,
Michael


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120049): https://edk2.groups.io/g/devel/message/120049
Mute This Topic: https://groups.io/mt/107561854/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



                 reply	other threads:[~2024-07-26 14:07 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=afa7ce2f-c524-4503-b090-c6420ff9cfa8@linux.microsoft.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox