Repository navigation
Address ubuntu-26.04-related CI failures
#4267
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
base: main
Are you sure you want to change the base?
Changes from all commits
b63386c
b913bf6
ce7d3f7
dd68f13
eac0005
ab743b6
171607b
34b4917
63b184d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: we should remove this from the deflake workflow, since the list of deflaked workflows isn't autogenerated. |
This file was deleted.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,13 +6,13 @@ env: | |
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| - releases/v* | ||
| - main | ||
| - releases/v* | ||
| pull_request: | ||
| merge_group: | ||
| types: [checks_requested] | ||
| schedule: | ||
| - cron: '0 5 * * *' | ||
| - cron: "0 5 * * *" | ||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
|
|
@@ -30,10 +30,10 @@ jobs: | |
| fail-fast: false | ||
| matrix: | ||
| version: | ||
| - stable-v2.20.7 | ||
| - default | ||
| - linked | ||
| - nightly-latest | ||
| - stable-v2.20.7 | ||
| - default | ||
| - linked | ||
| - nightly-latest | ||
| name: Upload debug artifacts after failure in analyze | ||
| continue-on-error: true | ||
| env: | ||
|
|
@@ -43,7 +43,7 @@ jobs: | |
| # We currently need `security-events: read` to access feature flags. | ||
| security-events: read | ||
| timeout-minutes: 45 | ||
| runs-on: ubuntu-latest | ||
| runs-on: ${{ case(matrix.version == 'stable-v2.20.7', 'ubuntu-24.04', 'ubuntu-latest') }} | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You already added one in |
||
| steps: | ||
| - name: Dump GitHub event | ||
| run: cat "${GITHUB_EVENT_PATH}" | ||
|
|
@@ -60,7 +60,7 @@ jobs: | |
| - name: Install .NET | ||
| uses: actions/setup-dotnet@a98b56852c35b8e3190ac28c8c2271da59106c68 # v6.0.0 | ||
| with: | ||
| dotnet-version: '9.x' | ||
| dotnet-version: "9.x" | ||
| - name: Assert best-effort artifact scan completed | ||
| uses: ./../action/.github/actions/verify-debug-artifact-scan-completed | ||
| - uses: ./../action/init | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,13 +5,13 @@ env: | |
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| - releases/v* | ||
| - main | ||
| - releases/v* | ||
| pull_request: | ||
| merge_group: | ||
| types: [checks_requested] | ||
| schedule: | ||
| - cron: '0 5 * * *' | ||
| - cron: "0 5 * * *" | ||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
|
|
@@ -29,10 +29,10 @@ jobs: | |
| fail-fast: false | ||
| matrix: | ||
| version: | ||
| - stable-v2.20.7 | ||
| - default | ||
| - linked | ||
| - nightly-latest | ||
| - stable-v2.20.7 | ||
| - default | ||
| - linked | ||
| - nightly-latest | ||
| name: Upload debug artifacts | ||
| env: | ||
| CODEQL_ACTION_TEST_MODE: true | ||
|
|
@@ -41,7 +41,7 @@ jobs: | |
| contents: read | ||
| # We currently need `security-events: read` to access feature flags. | ||
| security-events: read | ||
| runs-on: ubuntu-latest | ||
| runs-on: ${{ case(matrix.version == 'stable-v2.20.7', 'ubuntu-24.04', 'ubuntu-latest') }} | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ditto |
||
| steps: | ||
| - name: Check out repository | ||
| uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 | ||
|
|
@@ -56,7 +56,7 @@ jobs: | |
| - name: Install .NET | ||
| uses: actions/setup-dotnet@a98b56852c35b8e3190ac28c8c2271da59106c68 # v6.0.0 | ||
| with: | ||
| dotnet-version: '9.x' | ||
| dotnet-version: "9.x" | ||
| - name: Assert best-effort artifact scan completed | ||
| uses: ./../action/.github/actions/verify-debug-artifact-scan-completed | ||
| - uses: ./../action/init | ||
|
|
||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -137,6 +137,14 @@ interface LanguageSetup { | |
| /** Describes partial mappings from built-in languages to their specific setup information. */ | ||
| type LanguageSetups = Partial<Record<BuiltInLanguage, LanguageSetup>>; | ||
|
|
||
| // A type representing version identifiers. | ||
| type Version = | ||
|
mbg marked this conversation as resolved.
|
||
| | "default" | ||
| | "linked" | ||
| | "nightly-latest" | ||
| // This is an approximation, but sufficient validation for our needs here. | ||
| | `stable-v${number}.${number}.${number}`; | ||
|
|
||
| // The default set of CodeQL Bundle versions to use for the PR checks. | ||
| const defaultTestVersions = [ | ||
| // The oldest supported CodeQL version. If bumping, update `CODEQL_MINIMUM_VERSION` in `codeql.ts` | ||
|
|
@@ -159,7 +167,10 @@ const defaultTestVersions = [ | |
| "linked", | ||
| // A nightly build directly from the our private repo, built in the last 24 hours. | ||
| "nightly-latest", | ||
| ]; | ||
| ] as const satisfies Version[]; | ||
|
|
||
| /** A type representing the default test versions. */ | ||
| type DefaultTestVersion = (typeof defaultTestVersions)[number]; | ||
|
|
||
| /** The default versions we use for languages / frameworks, if not specified as a workflow input. */ | ||
| const defaultLanguageVersions = { | ||
|
|
@@ -442,10 +453,23 @@ function generateJobMatrix( | |
| : [operatingSystemConfig["runner-image"]]; | ||
|
|
||
| for (const runnerImage of runnerImagesForOs) { | ||
| matrix.push({ | ||
| os: runnerImage, | ||
| version, | ||
| }); | ||
| // stable-v2.20.7 requires an older version of Java than the default on `ubuntu-latest`. | ||
| // We could install that older version on the newer runners, or use the older runner. | ||
| // We do the latter here. This can be removed once we stop supporting `v2.20.7`. | ||
| // Since this is typed as `DefaultTestVersion`, it will become an error once `stable-v2.20.7` | ||
| // is removed from `defaultTestVersions`. | ||
|
Comment on lines
+459
to
+460
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. (Though I believe we don't build the PR checks project as part of CI, so we'd only see this locally) |
||
| const requiresOlderRunnerImage: DefaultTestVersion = "stable-v2.20.7"; | ||
| if ( | ||
| runnerImage === "ubuntu-latest" && | ||
| version === requiresOlderRunnerImage | ||
| ) { | ||
| matrix.push({ os: "ubuntu-24.04", version }); | ||
| } else { | ||
|
Comment on lines
+456
to
+467
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Another valid approach here would be overriding the runner image for (I think just) the multi-language check, since I believe this issue only affects checks that analyze Java. |
||
| matrix.push({ | ||
| os: runnerImage, | ||
| version, | ||
| }); | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
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.
I presume this probably happened automatically, but we should consider adding a YAML formatter to CI to avoid this sort of noise.