Skip to content
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

runtime-staging pipeline running on PRs #87958

Closed
am11 opened this issue Jun 23, 2023 · 5 comments · Fixed by #88036
Closed

runtime-staging pipeline running on PRs #87958

am11 opened this issue Jun 23, 2023 · 5 comments · Fixed by #88036
Assignees

Comments

@am11
Copy link
Member

am11 commented Jun 23, 2023

runtime-staging pipeline is running with PRs. It has only one leg iossimulator-x64 Release AllSubsets_Mono_RuntimeTests minijit which gets skipped with PR runs and only run against main branch (where it is failing for months..)

Should we remove it? At least from PR where it is a no-op and cluttering the build analysis?

@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jun 23, 2023
@ghost
Copy link

ghost commented Jun 23, 2023

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

runtime-staging pipeline is running with PRs. It has only one leg iossimulator-x64 Release AllSubsets_Mono_RuntimeTests minijit which gets skipped with PR runs and only run against main branch (where it is failing for months..)

Should we remove it? At least from PR where it is a no-op and cluttering the build analysis?

Author: am11
Assignees: -
Labels:

area-Infrastructure, blocking-clean-ci-staging

Milestone: -

@steveisok
Copy link
Member

@SamMonoRT should this be shut down?

@SamMonoRT
Copy link
Member

Are people using the runtime-staging for new lanes being created or should we add them to extra-platforms or ios-like* lanes?

(@kotlarmilos is trying to enable the aot-llvm ios device lane - and at some point over the next couple months will investigate/triage all ios* runtime test failures)

@kotlarmilos kotlarmilos self-assigned this Jun 23, 2023
@steveisok
Copy link
Member

Are people using the runtime-staging for new lanes being created or should we add them to extra-platforms or ios-like* lanes?

I think we can bring it up in ios-like lanes. runtime-staging doesn't have much relevance any longer.

@SamMonoRT
Copy link
Member

In that case, it seems ok to turn it off running via the runtime-staging and enabling as part of ios-lanes.

@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jun 26, 2023
@ghost ghost removed in-pr There is an active PR which will close this issue when it is merged untriaged New issue has not been triaged by the area owner labels Jun 26, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jul 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants