Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

Update Fedora references in YML files from 42/40 to 43/42 - #121767

Merged
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml
Nov 19, 2025
Merged

Update Fedora references in YML files from 42/40 to 43/42#121767
richlander merged 3 commits into
mainfrom
copilot/update-fedora-references-yml

Conversation

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

@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-actionsgithub-actionsBot 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>
CopilotAI changed the title [WIP] Update Fedora references in YML filesUpdate Fedora references in YML files from 42/40 to 43/41Nov 19, 2025
@richlander

Copy link
Copy Markdown
Member

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

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

CopilotAI commented Nov 19, 2025

Copy link
Copy Markdown
ContributorAuthor

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

Fixed in c3c4e7f. Updated oldest from 41 to 42.

CopilotAI changed the title Update Fedora references in YML files from 42/40 to 43/41Update Fedora references in YML files from 42/40 to 43/42Nov 19, 2025
@richlander
richlander marked this pull request as ready for review November 19, 2025 01:30
CopilotAI review requested due to automatic review settings November 19, 2025 01:30
@richlander

Copy link
Copy Markdown
Member

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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.

FileDescription
eng/pipelines/helix-platforms.ymlUpdated 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.ymlUpdated Fedora Docker image reference in extra-platforms CoreCLR testing from version 42 to 43

@richlander

Copy link
Copy Markdown
Member

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@richlander

Copy link
Copy Markdown
Member

PR inner-loop build is clean.

@richlander

richlander commented Nov 19, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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 mainNov 19, 2025
206 of 221 checks passed
@richlander
richlander deleted the copilot/update-fedora-references-yml branch November 19, 2025 19:40
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Dec 20, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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.

4 participants

@richlander@agocke