Repository navigation
Remove disconnected Citrine benchmark profiles - #2218
Merged
LoopedBard3 merged 1 commit intoOct 19, 2026
Merged
LoopedBard3 merged 1 commit into
LoopedBard3 merged 1 commit into
Conversation
Remove retired Citrine and dependent ARM/Siryn profile definitions, update examples to active Gold profiles, and reuse shared profiles in legacy scenarios. Correct the Gold Windows relay database address while preserving controller queues and active machine configurations. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 947a7758-5660-4f23-b447-e7006ace68fd
This was referenced Oct 18, 2026
There was a problem hiding this comment.
🟢 Approval recommended
The scoped configuration cleanup has no identified blocking issues, and reported validation covers profile composition and scheduler preservation.
No open issues 🎉
What changed in this PR
Completes the retired-machine profile cleanup following #2169 while preserving active controller queues and generated scheduling configuration.
Changes:
- Remove disconnected Citrine and dependent ARM/Siryn profiles.
- Update examples, reuse shared profiles, and replace the obsolete JSON Wrk import.
- Correct the Gold Windows relay database address to
10.0.0.125.
| File | Description |
|---|---|
| src/BenchmarksApps/Samples/Template/sample.yml | Update example comment to Gold Linux. |
| src/BenchmarksApps/BuildPerformance/buildperformance.yml | Replace inline Citrine profile with shared profiles. |
| src/Benchmarks/json.benchmarks.yml | Import shared profiles and current Wrk job; refresh example. |
| scenarios/README.md | Update profile documentation and GC example. |
| scenarios/platform.benchmarks.yml | Update example comment to Gold Linux. |
| scenarios/mono.benchmarks.yml | Update example comment to Gold Linux. |
| scenarios/aspnet.profiles.yml | Remove retired profiles and correct Gold database address. |
| scenarios/aspnet.profiles.standard.yml | Remove retired ARM and Siryn profiles. |
| build/ci.profile.yml | Remove retired CI machine profiles. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Context
Authenticated Azure inventory confirmed citrinelin, citrinedb, and citrineload have zero connected listeners; citrinearmlin, citrinearmwin, citrinearm, and sirynlin no longer exist. This completes the remaining profile cleanup after #2169. A separate PR migrates the PR-benchmark bot's machine selections to Gold.
Validation