Skip to content

Remove alpine job and cut down on perftiger jobs #4886

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

Merged
merged 2 commits into from
Jul 15, 2025

Conversation

LoopedBard3
Copy link
Member

@LoopedBard3 LoopedBard3 commented Jul 14, 2025

Remove alpine job and cut down on perftiger jobs as the alpine machines have been decom'd and the perftiger machines have had a portion decom'd.

Should be merged with: dotnet/runtime#117628

…es have been decom'd and the perftiger machines have had a portion decom'd.
Copy link

@Copilot 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 removes deprecated Alpine and Perftiger pipeline jobs and trims unsupported platforms due to decommissioned machines.

  • Removed Alpine (linux_musl_x64) and typo’d Windows (windown_x86) configurations
  • Dropped Perftiger-specific Mono perf jobs in favor of the remaining PerfViper matrix
  • Pruned unsupported platforms from existing job definitions and updated inline comments
Comments suppressed due to low confidence (2)

eng/pipelines/runtime-perf-jobs.yml:189

  • [nitpick] The comment for this interpreter perf job has been generalized; consider specifying '- perfviper' to clearly communicate which machine this job targets.
  # run mono interpreter perf job

eng/pipelines/runtime-perf-jobs.yml:279

  • [nitpick] The inline comment provides context but could be more readable if moved above the logicalMachine setting or formatted as its own comment block.
          logicalMachine: 'perftiger_crossgen' # Update when re-enabling as PerfTiger is now fewer machines

Copy link
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

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

LGTM

@LoopedBard3 LoopedBard3 merged commit 0b09684 into dotnet:main Jul 15, 2025
72 of 80 checks passed
@LoopedBard3 LoopedBard3 deleted the CleanupPerfRuns branch July 15, 2025 18:30
LoopedBard3 added a commit that referenced this pull request Jul 16, 2025
Removes more runs that were missed during the tiger downsize here: #4886
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants