Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Aug 28, 2024

Backport of #5476 to release/8.2

/cc @eerhardt

Customer Impact

Customers are unable to use the new MongoDB.Driver version because it is now strong name signed. For example, if the user is using the 8.1 version of MongoDB.EntityFrameworkCore and Aspire.MongoDB.Driver, their build is broken today. This allows our customers to upgrade to the latest MongoDB.Driver dependency.

Testing

Automated tests in the repo.

Risk

Only affects MongoDB. If something else in the users application uses a the 2.27 version of MongoDB.Driver, updating to the new version of Aspire.MongoDB.Driver will break them.

Regression?

No. But scenarios no longer work if the users updates MongoDB.EntityFrameworkCore from 8.0.0 to 8.0.1.

There was a breaking change between 2.27 and 2.28 - MongoDB.Driver added an assembly strong name.

We can now remove our NoWarn workarounds for MongoDB. Except Aspire.MongoDB.Driver still needs the NoWarn for 2 the health check and DiagnosticSource packages.
@ghost ghost added the area-integrations Issues pertaining to Aspire Integrations packages label Aug 28, 2024
@joperezr joperezr added the Servicing-approved Approved for servicing release label Aug 28, 2024
@joperezr
Copy link
Member

Fixing an already broken scenario (people using Mongo higher than 2.27), customer reported, low (isolated) risk. Approved.

@eerhardt eerhardt enabled auto-merge (squash) August 28, 2024 21:46
@joperezr
Copy link
Member

Failure is unrelated and seems to be an issue with our elasticsearch image in the container registry. I'm merging this in favor of getting an 8.2 build out.

@joperezr joperezr disabled auto-merge August 29, 2024 00:06
@joperezr joperezr merged commit 75fdcff into release/8.2 Aug 29, 2024
@joperezr joperezr deleted the backport/pr-5476-to-release/8.2 branch August 29, 2024 00:07
@github-actions github-actions bot locked and limited conversation to collaborators Sep 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-integrations Issues pertaining to Aspire Integrations packages Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants