Skip to content

Update version of cloudcache lib we reference#55016

Merged
CyrusNajmabadi merged 5 commits intodotnet:mainfrom
CyrusNajmabadi:updateVersion2
Jul 24, 2021
Merged

Update version of cloudcache lib we reference#55016
CyrusNajmabadi merged 5 commits intodotnet:mainfrom
CyrusNajmabadi:updateVersion2

Conversation

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

Fixes #54869

@CyrusNajmabadi CyrusNajmabadi requested review from a team as code owners July 21, 2021 18:15
@ghost ghost added the Area-IDE label Jul 21, 2021
ExtraLarge,
}

private const int Iterations = 100;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

note: even running all tests 100 times each, we still run all sqlite and cloudcache tests in under 30s total. so this seems worth it for the coverage.

<MicrosoftServiceHubFrameworkVersion>2.8.10</MicrosoftServiceHubFrameworkVersion>
<MicrosoftVisualBasicVersion>10.1.0</MicrosoftVisualBasicVersion>
<MicrosoftVisualStudioCacheVersion>16.10.40-alpha</MicrosoftVisualStudioCacheVersion>
<MicrosoftVisualStudioCacheVersion>17.0.13-alpha</MicrosoftVisualStudioCacheVersion>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

might need to go into main-vs-deps unless we're shipping this in VS, I don't think we've merged dev17 back into main yet (though hopefully soon)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

main-vs-deps already has this. this is to unblock main through from teh test failures we're seeing there. do you know when we're going to be on dev17 for main? @jinujoseph ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I believe we're waiting on preview2 images? @allisonchou was working on it actively if I remember right

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@dibarbet can this be merged in now?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yep I think so!

@CyrusNajmabadi CyrusNajmabadi requested a review from dibarbet July 23, 2021 07:05
@CyrusNajmabadi CyrusNajmabadi merged commit b60fe7f into dotnet:main Jul 24, 2021
@ghost ghost added this to the Next milestone Jul 24, 2021
@CyrusNajmabadi CyrusNajmabadi deleted the updateVersion2 branch July 24, 2021 00:26
@allisonchou allisonchou modified the milestones: Next, 17.0.P3 Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CloudCachePersistentStorageTests.TestReadChecksumProducesLastWrittenChecksum1 failed in CI: Pipelines - Run 20210715.30 (azure.com)

3 participants