-
Notifications
You must be signed in to change notification settings - Fork 9
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
need to validate the nuget publication before 1.0 ships #155
Comments
@Arkatufus guess we'll do this via 1.0.0-beta1 and doing some integration testing into other projects / packages |
I managed to implement 1.0.0-beta1 in my project fyi For the Akka project I added:
Called AkkaHealthCheck.For(actorSystem); And used hocon:
For ASP.NET integration I added: using Akka.HealthCheck.Hosting.Web;
.WithAkkaLivenessProbe()
.WithAkkaReadinessProbe()
.WithAkkaPersistenceLivenessProbe()
endpoints
.MapAkkaHealthCheckRoutes(
HealthChecksWriter.HealthCheckEndpoint,
(_, options) =>
{
options.AllowCachingResponses = false;
options.ResponseWriter = HealthChecksWriter.WriteHealthCheckResponse;
}); And everything worked for my use case :) |
@CumpsD You shouldn't need to set the HOCON settings when you use |
I like to be explicit with the packages :) As for the HOCON, it's |
Got it :) |
Related: akkadotnet/Akka.Hosting#188 |
Originally posted by @Aaronontheweb in #148 (review)
The text was updated successfully, but these errors were encountered: