Skip to content

Disable a couple of pen tests on Windows on release/6.0 #60789

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
Merged
merged 1 commit into from
Nov 9, 2021
Merged

Disable a couple of pen tests on Windows on release/6.0 #60789

merged 1 commit into from
Nov 9, 2021

Conversation

safern
Copy link
Member

@safern safern commented Oct 22, 2021

I'm still investigating these failures which seem to have been caused by windows updates on the CI vms, but still confirming.

@ghost ghost added the area-System.Drawing label Oct 22, 2021
@ghost
Copy link

ghost commented Oct 22, 2021

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

Issue Details

I'm still investigating these failures which seem to have been caused by windows updates on the CI vms, but still confirming.

Author: safern
Assignees: -
Labels:

area-System.Drawing
Milestone: -

@safern safern mentioned this pull request Oct 22, 2021
Closed
@danmoseley
Copy link
Member

I think the branch is closed now -- please let @Anipik merge or check with him first.

@safern
Copy link
Member Author

safern commented Oct 22, 2021

Yup, I'll let him manage this PR and merge at his will 😄

@runfoapp runfoapp bot mentioned this pull request Oct 25, 2021
Closed
@Anipik
Copy link
Contributor

Anipik commented Oct 25, 2021

The branch is closed, we should wait for servicing.

@Anipik Anipik added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) area-Infrastructure-libraries labels Oct 25, 2021
@ghost
Copy link

ghost commented Oct 25, 2021

Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

I'm still investigating these failures which seem to have been caused by windows updates on the CI vms, but still confirming.

Author: safern
Assignees: -
Labels:

* NO MERGE *, area-Infrastructure-libraries, area-System.Drawing
Milestone: -

@Anipik Anipik removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Nov 9, 2021
@Anipik Anipik merged commit da3adc5 into dotnet:release/6.0 Nov 9, 2021
@safern safern deleted the DisablePenTests6.0 branch November 9, 2021 18:38
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@danmoseley danmoseley danmoseley approved these changes

@ericstj ericstj Awaiting requested review from ericstj

+1 more reviewer

@Anipik Anipik Anipik approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area-Infrastructure-libraries area-System.Drawing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@safern @danmoseley @Anipik