Health check fails for AspNetCore.HealthChecks.Elasticsearch:9.0.0

[NikiforovAll]

Please, fill the following sections to help us fix the issue

What happened:

Health check doesn't work due to transitive dependency issue:

[16:28:09] fail: Microsoft.Extensions.Diagnostics.HealthChecks.DefaultHealthCheckService[104]
      Health check elasticsearch threw an unhandled exception after 2.8406ms
      System.TypeLoadException: Could not load type 'Elastic.Transport.TransportConfigurationBase`1' from assembly 'Elastic.Transport, Version=0.0.0.0, Culture=neutral, PublicKeyToken=069ca2728db333c1'.
         at HealthChecks.Elasticsearch.ElasticsearchHealthCheck.CheckHealthAsync(HealthCheckContext context, CancellationToken cancellationToken)
         at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
         at HealthChecks.Elasticsearch.ElasticsearchHealthCheck.CheckHealthAsync(HealthCheckContext context, CancellationToken cancellationToken)
         at Microsoft.Extensions.Diagnostics.HealthChecks.DefaultHealthCheckService.RunCheckAsync(HealthCheckRegistration registration, CancellationToken cancellationToken)

This happens because we have a dependency on Elastic.Clients.Elasticsearch:8.16.3 which resolves transitive dependency to Elastic.Transport:0.5.7

What you expected to happen:

Health check should work as expected.

How to reproduce it (as minimally and precisely as possible):

Add to a project that references Elastic.Clients.Elasticsearch:8.16.3 and AspNetCore.HealthChecks.Elasticsearch:9.0.0 at the same time.

[adamsitnik]

cc @Alirexaa @flobernd

[flobernd]

Our transport library Elastic.Transport is still in "pre-release" state (0.x.x version) which is bad, looking at this issue and other problems caused by the breaking changes that propagate to the Elastic.Clients.Elasticsearch package.

We are working on releasing a final version 1.0.0 for Elastic.Transport soon.

For now, there is not much we can do. It would probably be best to bump the Elasticsearch client package dependency in the next release of AspNetCore.HealthChecks.Elasticsearch.

[adamsitnik]

We are working on releasing a final version 1.0.0 for Elastic.Transport soon.

@flobernd Please ping us once it's available, we are then going to bump it and release a new version. Thanks!

[vik-torr]

should
? new ElasticsearchClientSettings(_options.CloudId!, new ApiKey(_options.ApiKey!))
be
? new ElasticsearchClientSettings(_options.CloudId!, new ApiKey(_options.CloudApiKey!))

[kaidl]

Is there some workaround in the meantime?

[sergey-tihon]

@kaidl downgrade to AspNetCore.HealthChecks.Elasticsearch 8.0.1