Skip to content

Removed nightly to workaround failing Kestrel init. - #41934

Merged
ManickaP merged 1 commit into
dotnet:masterfrom
ManickaP:mapichov/temp_fix_stress
Sep 7, 2020
Merged

ManickaP merged 1 commit into
dotnet:masterfrom
ManickaP:mapichov/temp_fix_stress

Conversation

@ManickaP

@ManickaP ManickaP commented Sep 7, 2020 •

Copy link
Copy Markdown
Member

The http-stress pipeline doesn't work now since ASP.NET Core depended on having IO.Pipelines in-box, which has changed with Connection abstraction removal.
To get the pipeline working, referenced non-nightly docker image, that still works.

Eventually this either can be kept or reverted, depending on how fresh the images we want.

@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added the area-Infrastructure-coreclr Only use for closed issues label Sep 7, 2020
@ManickaP ManickaP added area-System.Net.Http and removed area-Infrastructure-coreclr Only use for closed issues labels Sep 7, 2020
@ghost

ghost commented Sep 7, 2020

Copy link
Copy Markdown

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

@ManickaP

ManickaP commented Sep 7, 2020

Copy link
Copy Markdown
Member Author

/azp run runtime-libraries stress-http

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@ManickaP
ManickaP marked this pull request as ready for review September 7, 2020 11:10
@ManickaP
ManickaP requested a review from a team September 7, 2020 11:10

@alnikola alnikola left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ManickaP
ManickaP merged commit a84720c into dotnet:master Sep 7, 2020
@ManickaP
ManickaP deleted the mapichov/temp_fix_stress branch September 7, 2020 11:25
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
@karelz karelz added this to the 6.0.0 milestone Jan 26, 2021
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