Show time alongside health check status

[afscrome]

Is there an existing issue for this?

• I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.

If the dashboard shows a health check failure, it's not entirely clear to me how up to date the health check is.

Describe the solution you'd like

It would be nice if the UI could display the time at which the health check last ran, to confirm that the app is still broken, and not caching a broken health check from a while ago.

I know aspire should be re-running the health checks and everythign shoudl be up to date, but seeing a time of last execution (and possibly also next execution) would help to quell any doubt I have as to whether the app's is really healthy and the status is just out of date.

Additional context

No response

[JamesNK]

Health checks, when a resource is running, should run up to every 30 seconds. Is that not happening in your experience, or are you unsure about how often they run?

There isn't much screen space to put a time so I'm hesistant to add it.

[afscrome]

Sorry, forgot to reply to this.

It's more to help the case where a health check is failing, but I don't believe it should be (or vice versa). Whilst academically I know the health check ought to be re-running, Having some kind of evidence in the UI that the health check has run would help settle that lingering doubt.

Some context for why I originally field this issue - my HTTP healthcheck was rendering perfectly fine when I hit it in my browser, but timed out from Aspire. Given that it was working fine in the browser, the one explanation I could not get out of my head was that the Aspire health check was not re-running - having some evidence that healthchecks were actually runnign would have helped me avoid going down some rabbit holes. It eventually turned out the problem was that the healthcheck endpoint was restricting it's port via RequireHost rather than HttpContext.Connection.LocalPort which explained why the health check worked fine in the browser, but not from Aspire (see https://learn.microsoft.com/en-us/aspnet/core/fundamentals/routing?view=aspnetcore-9.0#host-matching-in-routes-with-requirehost). But with aspire providing

[JamesNK]

What do you think about a tooltip on the state column, e.g. State: Unhealthy, last run: 1:50:34.652 pm

[afscrome]

That works for me.

[JamesNK]

I don't think we'll have time to do this right now, but PR is welcome. Note that the last run time isn't available in the dashboard so it will need to be added to the proto contract and snapshot types on the server.