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.

Sign up for GitHub

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

Jump to bottom

[release/6.0] Disable CI job: Android x64 Release AllSubsets_Mono_RuntimeTests #96891

Merged
merged 1 commit into from
Jan 12, 2024
Merged

[release/6.0] Disable CI job: Android x64 Release AllSubsets_Mono_RuntimeTests #96891

merged 1 commit into from
Jan 12, 2024

Conversation

akoeplinger
Copy link
Member

It is causing a lot of flaky failures: #83422

.NET MAUI is already EOL on 6.0 so disabling the tests is fine.

Fixes #83422

Customer Impact

None, this is an infrastructure change.

Testing

CI testing.

Risk

Low.

@akoeplinger
[release/6.0] Disable CI job: Android x64 Release AllSubsets_Mono_Run…
142683c 
…timeTests

It is causing a lot of flaky failures: #83422

.NET MAUI is already EOL on 6.0 so disabling the tests is fine.

Fixes #83422
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 12, 2024
@ghost ghost assigned akoeplinger Jan 12, 2024
@akoeplinger
Copy link
Member Author

Infrastructure-only change, applying Servicing-approved.

@akoeplinger akoeplinger added Servicing-approved Approved for servicing release area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jan 12, 2024
@ghost
Copy link

ghost commented Jan 12, 2024

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

Issue Details

It is causing a lot of flaky failures: #83422

.NET MAUI is already EOL on 6.0 so disabling the tests is fine.

Fixes #83422

Customer Impact

None, this is an infrastructure change.

Testing

CI testing.

Risk

Low.

Author: akoeplinger
Assignees: akoeplinger
Labels:

Servicing-approved, area-Infrastructure
Milestone: -

@akoeplinger akoeplinger added this to the 6.0.x milestone Jan 12, 2024
@build-analysis build-analysis bot mentioned this pull request Jan 12, 2024
Open
@steveisok steveisok self-requested a review January 12, 2024 16:15
steveisok
steveisok approved these changes Jan 12, 2024
View reviewed changes
Copy link
Member

@steveisok steveisok left a comment

Choose a reason for hiding this comment

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

Probably a good idea to disable everything mobile in 6

@carlossanlop
Copy link
Member

Probably a good idea to disable everything mobile in 6

I support this. It can be done in a separate PR. This one is ready to merge.

@carlossanlop carlossanlop merged commit d8a2748 into release/6.0-staging Jan 12, 2024
169 of 171 checks passed
@carlossanlop carlossanlop deleted the akoeplinger-patch-1 branch January 12, 2024 22:19
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@steveisok steveisok steveisok approved these changes

@carlossanlop carlossanlop carlossanlop approved these changes

Assignees

@akoeplinger akoeplinger

Labels
area-Infrastructure Servicing-approved Approved for servicing release
Projects
None yet
Milestone
6.0.x
Development

Successfully merging this pull request may close these issues.
3 participants
@akoeplinger @carlossanlop @steveisok