Repository navigation
chore(deps): bump the codeql-action group with 3 updates - #6209
Conversation
Bumps the codeql-action group with 3 updates: [github/codeql-action/init](https://github.com/github/codeql-action), [github/codeql-action/analyze](https://github.com/github/codeql-action) and [github/codeql-action/upload-sarif](https://github.com/github/codeql-action). Updates `github/codeql-action/init` from 4.38.2 to 4.38.3 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@2892aa5...24c5418) Updates `github/codeql-action/analyze` from 4.38.2 to 4.38.3 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@2892aa5...24c5418) Updates `github/codeql-action/upload-sarif` from 4.38.2 to 4.38.3 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@2892aa5...24c5418) --- updated-dependencies: - dependency-name: github/codeql-action/init dependency-version: 4.38.3 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: codeql-action - dependency-name: github/codeql-action/analyze dependency-version: 4.38.3 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: codeql-action - dependency-name: github/codeql-action/upload-sarif dependency-version: 4.38.3 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: codeql-action ... Signed-off-by: dependabot[bot] <support@github.com>
|
Hi @dependabot[bot]. Thanks for your PR. I'm waiting for a fluid-cloudnative member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
There was a problem hiding this comment.
🟢 Approval recommended
All three pins match the official v4.38.3 release, with no blocking compatibility issues identified.
0 open findings
What changed in this PR
Updates Fluid’s security-scanning workflows to CodeQL Action v4.38.3.
Changes:
- Updates the
init,analyze, andupload-sarifcommit pins from v4.38.2 to v4.38.3. - Preserves existing workflow configuration.
| File | Description |
|---|---|
.github/workflows/sonarcloud.yaml |
Updates the SARIF upload action pin. |
.github/workflows/codeql-analysis.yml |
Updates CodeQL initialization and analysis action pins. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
cheyang
left a comment
There was a problem hiding this comment.
Verified this Dependabot bump end to end: all three github/codeql-action pins move together from the exact v4.38.2 release-tag commit (2892aa5e) to the exact v4.38.3 release-tag commit (24c54180), checked against the upstream repository rather than the PR body, and the diff touches nothing but those three lines. The repo's own CodeQL workflow already ran the new init/analyze pins green on this PR's head, and nothing in 4.38.3 affects this repo's usage (no custom codeql config, no tools: pin, GitHub-hosted runners). CI is fully green on the head.
Two pre-existing, non-blocking notes worth a maintainer's attention:
- The trailing version comments (
# v3.29.5on init/analyze,# v2.22.11on upload-sarif) haven't matched the pinned version for a long while — since PR #5200 (Aug 2025) and the bump after PR #3646 (Dec 2023) respectively. Once a comment falls out of sync, Dependabot stops maintaining it. Correcting all three to# v4.38.3here would also re-enable that; skipping is fine too. sonarcloud.yaml'supload-sarifpin isn't exercised before merge — that workflow only runs on push/schedule, so the merge itself is its first run. Low risk (the same commit's init/analyze already passed, and no 4.38.3 change touches upload-sarif), but a glance at that first post-merge run is cheap insurance.
Next round
[MUST] (none — no PR-introduced blockers; the MUST list is empty and every gating acceptance criterion passes)
[SHOULD] Correct the three stale version comments to # v4.38.3 (pre-existing, optional — see appendix F1), and glance at the first post-merge sonarcloud.yaml run that exercises the new upload-sarif pin (appendix F2).
[Filed] (none)
Reviewer C (Copilot) raised no findings. The two debaters agreed on every finding and disposition; the only disagreement was nit-vs-minor on the comment hygiene, with no material effect on the verdict.
Non-blocking observations (2) — not part of this round's ask; fix now, follow up, or file an issue as you see fit
- .github/workflows/codeql-analysis.yml:49 — stale pin version comments (pre-existing, consensus) [minor] — pre-existing, not introduced by this PR
- .github/workflows/sonarcloud.yaml:54 — upload-sarif pin first exercised only after merge (pre-existing, consensus) [minor] — pre-existing, not introduced by this PR
Verified review delta
- F1-stale-pin-comments: open (preexisting)
- F2-upload-sarif-post-merge-validation: open (preexisting)
Code: pending. CI: success; attribution: baseline. Required CI checks still apply.
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cheyang The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |



Bumps the codeql-action group with 3 updates: github/codeql-action/init, github/codeql-action/analyze and github/codeql-action/upload-sarif.
Updates
github/codeql-action/initfrom 4.38.2 to 4.38.3Release notes
Sourced from github/codeql-action/init's releases.
Changelog
Sourced from github/codeql-action/init's changelog.
... (truncated)
Commits
24c5418Merge pull request #4214 from github/update-v4.38.3-e0b52dbdc69e87c5Add changelog note for #418423b6828Update changelog for v4.38.3e0b52dbMerge pull request #4210 from github/henrymercer/pin-python-for-older-clis-in-ci6b99698Pin Python 3.13.15 for older CLI versions in multi-language check63d3d63Merge pull request #4203 from github/update-bundle/codeql-bundle-v2.27.27377a10Add changelog notec1a30cdUpdate default bundle to codeql-bundle-v2.27.2d208ec7Merge pull request #4200 from github/mbg/esm/migrate-pr-checksa71b0c6Replace__dirnameinsync.tsUpdates
github/codeql-action/analyzefrom 4.38.2 to 4.38.3Release notes
Sourced from github/codeql-action/analyze's releases.
Changelog
Sourced from github/codeql-action/analyze's changelog.
... (truncated)
Commits
24c5418Merge pull request #4214 from github/update-v4.38.3-e0b52dbdc69e87c5Add changelog note for #418423b6828Update changelog for v4.38.3e0b52dbMerge pull request #4210 from github/henrymercer/pin-python-for-older-clis-in-ci6b99698Pin Python 3.13.15 for older CLI versions in multi-language check63d3d63Merge pull request #4203 from github/update-bundle/codeql-bundle-v2.27.27377a10Add changelog notec1a30cdUpdate default bundle to codeql-bundle-v2.27.2d208ec7Merge pull request #4200 from github/mbg/esm/migrate-pr-checksa71b0c6Replace__dirnameinsync.tsUpdates
github/codeql-action/upload-sariffrom 4.38.2 to 4.38.3Release notes
Sourced from github/codeql-action/upload-sarif's releases.
Changelog
Sourced from github/codeql-action/upload-sarif's changelog.
... (truncated)
Commits
24c5418Merge pull request #4214 from github/update-v4.38.3-e0b52dbdc69e87c5Add changelog note for #418423b6828Update changelog for v4.38.3e0b52dbMerge pull request #4210 from github/henrymercer/pin-python-for-older-clis-in-ci6b99698Pin Python 3.13.15 for older CLI versions in multi-language check63d3d63Merge pull request #4203 from github/update-bundle/codeql-bundle-v2.27.27377a10Add changelog notec1a30cdUpdate default bundle to codeql-bundle-v2.27.2d208ec7Merge pull request #4200 from github/mbg/esm/migrate-pr-checksa71b0c6Replace__dirnameinsync.tsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions