Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Move signing dir under eng/. Remove usages of ProjectDir. #8621

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

jkoritzinsky
Copy link
Member

Follow up on feedback to #8593

Copy link
Member

@dagood dagood left a comment

Choose a reason for hiding this comment

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

Looks reasonable. Have you ran a validation build? I can kick one off.

@jkoritzinsky
Copy link
Member Author

Can you kick one off?

@dagood
Copy link
Member

dagood commented Oct 21, 2019

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

Thanks :)

@ViktorHofer
Copy link
Member

Looks like the internal build failed.

@dagood
Copy link
Member

dagood commented Oct 21, 2019

The failure is the Microsoft.NET.HostModel test project being unable to access Microsoft.NET.HostModel.dll, I would guess because the signing process is trying to replace it.

Looking back at failed official builds, I see it happening at least once before.

I need to have the tests wait for signing to complete--this bug has been there for a while, apparently, it just hasn't happened enough for me to see it. Filed https://github.com/dotnet/core-setup/issues/8626.

Running a retry on that validation build to try to get past that flakiness and get on to the rest of the infra.

@dagood
Copy link
Member

dagood commented Oct 21, 2019

Retry worked. Thanks!

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.

3 participants