Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 19, 2025

Updates Fedora version references in CI/CD pipeline configuration files for Helix testing infrastructure.

Changes Made

  • Fedora Latest: Updated from version 42 to 43
  • Fedora Oldest: Updated from version 40 to 42

These changes update the Docker image references and platform definitions used by the Helix testing infrastructure to test against newer Fedora versions.

Files Modified

  • eng/pipelines/helix-platforms.yml - Updated both latest and oldest Fedora version definitions
  • eng/pipelines/libraries/helix-queues-setup.yml - Updated Fedora reference used in extra-platforms CoreCLR testing

Validation

  • ✅ YAML syntax validated successfully
  • ✅ No security vulnerabilities detected (CodeQL scan)
  • ✅ Changes follow established patterns in the configuration files
Original prompt

Update Fedora references in *.yml files. I believe there are only two relevant ones. They can be found by searching for "fedora-".

Where there is min and max, use 42 and 43, respectively. Where there is a single choice, use 43.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@richlander
Copy link
Member

@copilot I'm wondering why the build is being run before updating any files. Or am I missing the edits?

Copy link
Contributor Author

Copilot AI commented Nov 19, 2025

@copilot I'm wondering why the build is being run before updating any files. Or am I missing the edits?

You're right - I created the initial plan commit prematurely without making any file changes. I'll now make the actual updates to the Fedora references in the YML files and commit those changes properly.

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Nov 19, 2025
Co-authored-by: richlander <2608468+richlander@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Fedora references in YML files Update Fedora references in YML files from 42/40 to 43/41 Nov 19, 2025
Copilot finished work on behalf of richlander November 19, 2025 00:50
@richlander
Copy link
Member

@copilot Close. Oldest should be Fedora 42 and newest Fedora 43.

Co-authored-by: richlander <2608468+richlander@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 19, 2025

@copilot Close. Oldest should be Fedora 42 and newest Fedora 43.

Fixed in c3c4e7f. Updated oldest from 41 to 42.

Copilot AI changed the title Update Fedora references in YML files from 42/40 to 43/41 Update Fedora references in YML files from 42/40 to 43/42 Nov 19, 2025
Copilot finished work on behalf of richlander November 19, 2025 01:15
@richlander richlander marked this pull request as ready for review November 19, 2025 01:30
Copilot AI review requested due to automatic review settings November 19, 2025 01:30
@richlander
Copy link
Member

Copilot finished reviewing on behalf of richlander November 19, 2025 01:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates Fedora version references in Helix CI/CD pipeline configuration files to test against newer Fedora releases. The changes align with the infrastructure's practice of supporting the latest and oldest supported versions.

  • Updated Fedora "latest" version from 42 to 43
  • Updated Fedora "oldest" version from 40 to 42

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
eng/pipelines/helix-platforms.yml Updated Fedora version variable definitions for both latest (42→43) and oldest (40→42) supported versions, including comments and Docker image references
eng/pipelines/libraries/helix-queues-setup.yml Updated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander
Copy link
Member

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@richlander
Copy link
Member

PR inner-loop build is clean.

@richlander
Copy link
Member

richlander commented Nov 19, 2025

This looks clean for Fedora on PR and Extra Platforms. I don't see any linux-x64 failues.

Main place to look: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1207203&view=logs&jobId=803b4c9b-c8dc-5f8e-c2da-4890c86ff82f&j=803b4c9b-c8dc-5f8e-c2da-4890c86ff82f&t=b981e634-422d-532f-0103-2cf28078ad75

I saw a break for Ubuntu 26.04:
https://dev.azure.com/dnceng-public/public/_build/results?buildId=1207203&view=logs&jobId=f34c4883-3ffa-561e-aa56-47727d75db21&j=f34c4883-3ffa-561e-aa56-47727d75db21&t=94086b59-3b57-5d7f-6f81-edbeaf0f6cce

That break is unrelated to this PR. That same leg passed on #121730. I double checked just now. Just recording here if it turns out that root cause is the other PR. I looked at a later run in the same pipeline. I don't see the failure but do see the 26.04 queue. I'm going to ignore this problem until I hear more.

Ready for approval?

@richlander
Copy link
Member

/ba-g "Known issues and unrelated failures"

@richlander richlander enabled auto-merge (squash) November 19, 2025 19:40
@richlander richlander merged commit 3ac6077 into main Nov 19, 2025
206 of 221 checks passed
@richlander richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants