Skip to content

Update Drop.App version#5239

Merged
JohnTortugo merged 1 commit intodotnet:release/3.xfrom
rainersigwald:new-drop-app
May 8, 2020
Merged

Update Drop.App version#5239
JohnTortugo merged 1 commit intodotnet:release/3.xfrom
rainersigwald:new-drop-app

Conversation

@rainersigwald
Copy link
Member

@rainersigwald rainersigwald commented Apr 10, 2020

Description

Old drop.exe has an issue with a recent Windows security update, which
causes failures like

ContentStore.Exceptions.CacheException: Could not download into the
cache: ErrorMessage=[FileNotFoundException:
System.IO.FileNotFoundException: Could not find file
'C:\_cache\contents\Shared\VSO0\ED8\ED878A5B6B0E382A858F9F34A562E42BFE8A70032E7E900F13CF4EE3621D8E4900.blob'.

When downloading drop data. The Azure Artifacts team recommends updating
to 18.164.29918 or newer.

Customer Impact

Builds that acquire OptProf data fail with mysterious error (on some OSes).

Regression

Yes, but not in Arcade: Windows and/or Drop.exe is the problem here.

Risk

Low. Slight chance that the latest Drop.App package has some functional regression.

Workarounds

Override the package version at a repo level.

@tmat
Copy link
Member

tmat commented Apr 10, 2020

Will this flow to master?

@markwilkie
Copy link
Member

I presume it should flow to master yes?

Also, could you fill out the template please @rainersigwald ?

@rainersigwald
Copy link
Member Author

This change should go to master. But it looks like there's no automatic RI in this repo? https://github.com/dotnet/arcade/tree/release/3.x says This branch is 22 commits ahead, 625 commits behind master.

Do y'all do dual checkins?

@riarenas
Copy link
Contributor

Do y'all do dual checkins?

Yep, we do. The prefered flow in Arcade is to make the change in Master first, then port your change to release/3.x and fill out the template:

https://github.com/dotnet/arcade/blob/master/Documentation/Policy/ArcadeServicing.md#how-do-i-get-my-servicing-fix-into-master

Template for the PR to release branch: https://github.com/dotnet/arcade/blob/master/Documentation/Policy/AskModeTellModeTemplate.md

@riarenas
Copy link
Contributor

Ah, I was missing a refresh to see that you were already using the template.

Old `drop.exe` has an issue with a recent Windows security update, which
causes failures like

```
ContentStore.Exceptions.CacheException: Could not download into the
cache: ErrorMessage=[FileNotFoundException:
System.IO.FileNotFoundException: Could not find file
'C:\_cache\contents\Shared\VSO0\ED8\ED878A5B6B0E382A858F9F34A562E42BFE8A70032E7E900F13CF4EE3621D8E4900.blob'.
```

When downloading drop data. The Azure Artifacts team recommends updating
to 18.164.29918 or newer.
@rainersigwald
Copy link
Member Author

Ok. I rebased this back to the merge-base, so it can go in whichever branch y'all want to land first. I can then open another PR for the other one.

@JohnTortugo
Copy link
Contributor

Ping. Should we merge this?

@rainersigwald
Copy link
Member Author

@JohnTortugo who are you pinging, exactly? I'm waiting for someone with permissions to merge :)

@JohnTortugo JohnTortugo merged commit 6eba692 into dotnet:release/3.x May 8, 2020
rainersigwald added a commit to rainersigwald/msbuild that referenced this pull request Jun 4, 2020
…5258)"

This reverts commit f516095.

This workaround is no longer needed after dotnet/arcade#5239.
rainersigwald added a commit to dotnet/msbuild that referenced this pull request Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants