Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Mongo.Driver to 3.0.0 #2312

Closed
wants to merge 1 commit into from
Closed

Conversation

rohitvipin
Copy link

@rohitvipin rohitvipin commented Nov 1, 2024

What this PR does / why we need it:

  • Doesn’t currently work when used with 3.0.0

Which issue(s) this PR fixes:

Please reference the issue this PR will close: #[issue number]

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Unit tests passing
  • End-to-end tests passing
  • Extended the documentation
  • Provided sample for the feature

@rohitvipin
Copy link
Author

@sungam3r - can you take a look, this helps to update Mongo 3.0.0 as it currently fails healthcheck

@Kiechlus
Copy link

Could a maintainer approve this? This is blocking us currently.

@VSUT-se
Copy link

VSUT-se commented Nov 14, 2024

Same issue on my side ...

@GabrielZalisz
Copy link

Waiting for this too...

@peterkiss1
Copy link

@ytsiupiak-ecw @rchaidez-cricut please check on the failure here, the target framework for the package must be changed in order to be compatible with the v3 MongoDB driver.

@eerhardt
Copy link
Collaborator

I put up an alternate PR using the same approach we are using in .NET Aspire (see RabbitMQ.Client v7 breaks Aspire.RabbitMQ.Client component (dotnet/aspire#3956) for the high-level description of the strategy being employed).

Add support for MongoDB.Driver version 3 (Xabaril/AspNetCore.Diagnostics.HealthChecks#2324)

The idea is to support MongoDB.Driver version 2 and version 3 for a while, until users are off of version 2 (or it is out of support).

@rohitvipin
Copy link
Author

Closing this as there is an alternative one open to support both.
#2324

@rohitvipin rohitvipin closed this Dec 3, 2024
@rohitvipin rohitvipin deleted the patch-1 branch December 3, 2024 06:07
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.

8 participants