Skip to content

ci(codeql): fix go-analysis runner label (was queuing forever)#69

Merged
entlein merged 1 commit into
mainfrom
fix/codeql-go-runner-label
Jun 22, 2026
Merged

ci(codeql): fix go-analysis runner label (was queuing forever)#69
entlein merged 1 commit into
mainfrom
fix/codeql-go-runner-label

Conversation

@entlein

@entlein entlein commented Jun 22, 2026

Copy link
Copy Markdown

The CodeQL go job used oracle-16cpu-64gb-x86-64 (nonexistent label on this repo) → queued 12h+/16h+ forever, auto-cancelling at the 24h cap. python/javascript already use oracle-vm-16cpu-64gb-x86-64; this aligns go. Cancelled the two stuck runs (27917413327, 27911584113).

Context: this is what looked like 'AE aeprod20 hung 12h' — but the aeprod20 release itself succeeded (Build Release ✓ ~70min, image 0.14.19-aeprod20 + GH release published 22:26). The 12h hang was this codeql go-job, unrelated to the release.

…cpu)

The CodeQL 'go' job was on `oracle-16cpu-64gb-x86-64` — a label that doesn't
exist on this repo's runners — so it queued forever (12h+/16h+, older runs
auto-cancelled at the 24h cap). The python/javascript jobs already use the
correct `oracle-vm-16cpu-64gb-x86-64`. Align go to match. (Same upstream-label
trap that's bitten the vizier/bazel jobs.)
@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: c9db6fcf-419e-4976-83ac-591b9c189bff

📥 Commits

Reviewing files that changed from the base of the PR and between 855db81 and a84f28b.

📒 Files selected for processing (1)
  • .github/workflows/codeql.yaml

📝 Walkthrough

Walkthrough

The runs-on runner label for the analyze-go job in .github/workflows/codeql.yaml is updated from oracle-16cpu-64gb-x86-64 to oracle-vm-16cpu-64gb-x86-64. No other configuration changes are made.

Changes

CodeQL Runner Label Fix

Layer / File(s) Summary
Update analyze-go runner label
.github/workflows/codeql.yaml
runs-on value changed from oracle-16cpu-64gb-x86-64 to oracle-vm-16cpu-64gb-x86-64 for the analyze-go job.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • k8sstormcenter/pixie#64: Applies the same runner label rename (oracle-16cpu-64gb-x86-64oracle-vm-16cpu-64gb-x86-64) across release and mirror workflows, matching this fix in the CodeQL workflow.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a CodeQL runner label for the go-analysis job that was causing queuing issues.
Description check ✅ Passed The description is directly related to the changeset, providing context about the runner label fix and its impact on job queuing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/codeql-go-runner-label

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@entlein entlein merged commit 6ac471d into main Jun 22, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant