Skip to content

Conversation

@alnikola
Copy link
Contributor

Networking stress tests running on Linux are failing in master at building runtime inside container due to a missing _.pdb files (e.g. /repo/src/coreclr/src/.nuget/_.pdb) This PR makes Docker to include those files into containers.

Networking stress tests running on Linux are failing in master at building runtime inside container due to a missing `_.pdb` files (e.g. `/repo/src/coreclr/src/.nuget/_.pdb`) This PR makes Docker to include those files into containers.
@alnikola alnikola requested review from a team, ViktorHofer and safern June 17, 2020 13:22
@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. Please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Jun 17, 2020

Tagging subscribers to this area: @ViktorHofer
Notify danmosemsft if you want to be subscribed.

@alnikola
Copy link
Contributor Author

/azp run runtime-libraries stress-http

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@alnikola
Copy link
Contributor Author

/azp run runtime-libraries stress-ssl

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

**/*.obj
**/*.pch
**/*.pdb
!**/_.pdb
Copy link
Contributor

Choose a reason for hiding this comment

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

It worth a comment with explanation why one kind of pdb is ignored while other allowed.

@safern
Copy link
Member

safern commented Jun 17, 2020

I think I added this change to: #35011 -- but fine to split it in a separate PR.

@alnikola
Copy link
Contributor Author

Closing it as obsolete since #35011 was merged.

@alnikola alnikola closed this Jun 18, 2020
@alnikola alnikola deleted the alnikola/include-pdb-to-fix-stress-on-linux branch June 18, 2020 11:51
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
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.

5 participants