Skip to content

Add logging to MVC tests #55592

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 11 commits into from
Jul 2, 2024
Merged

Add logging to MVC tests #55592

merged 11 commits into from
Jul 2, 2024

Conversation

@BrennanConroy
Copy link
Member

No description provided.

@BrennanConroy BrennanConroy requested review from a team, JamesNK, halter73 and mgravell as code owners May 7, 2024 23:51
@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label May 7, 2024
@captainsafia
Copy link
Member

captainsafia commented May 10, 2024 •
edited
Loading

Test failures are around the broken tests that need investigation. I'll merge this so we can start to get logs to investigate this.

Correction: these test failures are new and only surface in this PR.

@captainsafia captainsafia enabled auto-merge (squash) May 10, 2024 16:50
@captainsafia captainsafia disabled auto-merge May 10, 2024 16:50
@captainsafia captainsafia mentioned this pull request May 13, 2024
Open
This was referenced May 14, 2024
Merged
Merged
@BrennanConroy
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@BrennanConroy BrennanConroy added the * NO MERGE * Do not merge this PR as long as this label is present. label May 15, 2024
@dotnet-policy-service dotnet-policy-service bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label May 23, 2024
@build-analysis build-analysis bot mentioned this pull request Jun 15, 2024
Closed
3 tasks
@build-analysis build-analysis bot mentioned this pull request Jun 18, 2024
Open
3 tasks
@wtgodbe
Copy link
Member

wtgodbe commented Jun 26, 2024

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@BrennanConroy
Copy link
Member Author

/azp run

@dotnet-policy-service dotnet-policy-service bot removed the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Jun 27, 2024
@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@BrennanConroy
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@BrennanConroy
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@BrennanConroy
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@BrennanConroy BrennanConroy mentioned this pull request Jul 1, 2024
Open
@BrennanConroy BrennanConroy merged commit c790ed5 into main Jul 2, 2024
@BrennanConroy BrennanConroy deleted the brecon/mvclog branch July 2, 2024 03:34
@dotnet-policy-service dotnet-policy-service bot added this to the 9.0-preview7 milestone Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@captainsafia captainsafia captainsafia approved these changes

@halter73 halter73 Awaiting requested review from halter73 halter73 is a code owner

@JamesNK JamesNK Awaiting requested review from JamesNK JamesNK is a code owner

@mgravell mgravell Awaiting requested review from mgravell

Assignees

No one assigned

Labels

area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates * NO MERGE * Do not merge this PR as long as this label is present.

Projects

None yet

Milestone

9.0-preview7

Development

Successfully merging this pull request may close these issues.

4 participants

@BrennanConroy @captainsafia @wtgodbe