Skip to content

Replace Ubuntu 22.04 with 24.04 in CI#26787

Closed
fmeum wants to merge 2 commits into
bazelbuild:masterfrom
fmeum:ubuntu-24.04
Closed

Replace Ubuntu 22.04 with 24.04 in CI#26787
fmeum wants to merge 2 commits into
bazelbuild:masterfrom
fmeum:ubuntu-24.04

Conversation

@fmeum
Copy link
Copy Markdown
Collaborator

@fmeum fmeum commented Aug 18, 2025

The order of fields in gcov's JSON output changed, which requires switching to JSON-aware matching via jq rather than a simple grep.

Work towards #22553

@fmeum fmeum requested a review from meteorcloudy August 18, 2025 17:14
@fmeum fmeum marked this pull request as ready for review August 18, 2025 17:15
@fmeum fmeum requested a review from Wyverald August 18, 2025 17:15
@github-actions github-actions Bot added the awaiting-review PR is awaiting review from an assigned reviewer label Aug 18, 2025
@iancha1992 iancha1992 added the team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website label Aug 18, 2025
Copy link
Copy Markdown
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally we should drop ubuntu2004, but looks like we don't have ubuntu2204_arm64.

FYI @fweikert

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Aug 19, 2025
@github-actions github-actions Bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Aug 19, 2025
@fmeum
Copy link
Copy Markdown
Collaborator Author

fmeum commented Aug 19, 2025

FYI @PikachuHyA

Wyverald pushed a commit that referenced this pull request May 21, 2026
The order of fields in gcov's JSON output changed, which requires switching to JSON-aware matching via jq rather than a simple grep.

Work towards #22553

Closes #26787.

PiperOrigin-RevId: 796985639
Change-Id: I2b3feb5b5747143751127b6d69a1c295cc621773
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants