Skip to content

Revert "[mono] Update runtimeFlavor property for Mono AOT perf jobs" - #90535

Merged
LoopedBard3 merged 1 commit into
mainfrom
revert-90215-bugfix/perf-jobs-params
Aug 14, 2023
Merged

Revert "[mono] Update runtimeFlavor property for Mono AOT perf jobs"#90535
LoopedBard3 merged 1 commit into
mainfrom
revert-90215-bugfix/perf-jobs-params

Conversation

@kotlarmilos

@kotlarmilos kotlarmilos commented Aug 14, 2023

Copy link
Copy Markdown
Member

This PR fixes failing Mono AOT perf jobs. If runtime flavor is set to mono, artifacts can't be found.

https://dev.azure.com/dnceng/internal/_build/results?buildId=2242319&view=logs&j=3ad9afb5-5506-517e-402a-ab4e139d0d70&t=8949bdb7-a752-5bf3-8dd8-c76bc9ac64db

Reverts #90215

@ghost

ghost commented Aug 14, 2023

Copy link
Copy Markdown

Tagging subscribers to this area: @directhex
See info in area-owners.md if you want to be subscribed.

Issue Details

This PR fixes failing Mono AOT perf jobs. If runtime flavor set to mono, artifacts can't be found.

https://dev.azure.com/dnceng/internal/_build/results?buildId=2242319&view=logs&j=3ad9afb5-5506-517e-402a-ab4e139d0d70&t=8949bdb7-a752-5bf3-8dd8-c76bc9ac64db

Reverts #90215

Author: kotlarmilos
Assignees: kotlarmilos
Labels:

area-Infrastructure-mono

Milestone: 8.0.0

@SamMonoRT

Copy link
Copy Markdown
Member

@kotlarmilos @LoopedBard3 - I don't believe it is necessary to wait for CI to finish to merge this PR. The change is trivial, non risky, and should get in prior to the RC1 snap. I'm ok merging now.

@LoopedBard3
LoopedBard3 merged commit 7583f4b into main Aug 14, 2023
@LoopedBard3
LoopedBard3 deleted the revert-90215-bugfix/perf-jobs-params branch August 14, 2023 19:36
@LoopedBard3

Copy link
Copy Markdown
Member

Merged per ^ and this doesn't make any changes outside the perf-infra.

@ghost ghost locked as resolved and limited conversation to collaborators Sep 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants