Commit Graph

774 Commits

Author SHA1 Message Date
Ryan Nowak bb3a30a2fc
Merge branch 'master' into merge/release/2.2-to-master 2018-10-09 13:31:31 -07:00
Ryan Nowak 1f31e0556d
Add IHealthCheckPublisher for push-based checks (#498)
IHealthCheckPublisher allows you to configure and run health checks
regularly inside an application, and push the notifications elsewhere.

All publishers are part of a single queue with a configurable period and
timeout.
2018-10-08 17:58:45 -07:00
Ryan Nowak 3eb50aebff
Merge pull request #497 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-10-06 16:57:30 -07:00
Ryan Nowak 9722d89572 Adjust log levels 2018-10-06 16:18:55 -07:00
Ryan Nowak f0623fff60 Merge branch 'release/2.2' 2018-10-01 13:38:48 -07:00
Ryan Nowak 014e7eb963 Add EFCore DbContext check 2018-10-01 13:33:00 -07:00
ASP.NET CI 6a8494f938 Update dependencies.props
[auto-updated: dependencies]
2018-09-30 12:10:59 -07:00
ASP.NET CI f1058f0d5d Update dependencies.props
[auto-updated: dependencies]
2018-09-30 12:10:26 -07:00
Nate McMaster f1bb7acd5a
automated: merge branch release/2.2 2018-09-28 17:27:28 -07:00
Nate McMaster 4fc5c7c11b
automated: bulk infrastructure updates. Update bootstrapper scripts and remove unnecessary signing properties 2018-09-28 17:10:32 -07:00
Unai Zorrilla fa961b003f Added execution time duration (HealthReportEntry TotalDuration) (#493)
* Added execution time duration into HealthReportEntry and TotalDuration on HealthReport

* review PR feedback from @rynowak.

* added the same duration into HealthReportEntry and log when the health check throw
2018-09-28 09:14:58 -07:00
Ryan Nowak 258a62f10f
Merge pull request #492 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-09-27 23:04:47 -07:00
Eilon Lipton c29b0e6131
Update LICENSE.txt 2018-09-27 15:30:51 -07:00
Ryan Nowak 51847c9f85
Merge pull request #488 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-09-24 11:21:46 -07:00
Ryan Nowak 071a49e4da
Merge branch 'master' into merge/release/2.2-to-master 2018-09-24 11:12:00 -07:00
Ryan Nowak 204ff0a785 Set cache headers in health check middleware 2018-09-24 10:43:57 -07:00
Ryan Nowak 18145880fa Improved logging for health checks
- Add logging of description/data
- Add logging for aggregate begin/end
2018-09-24 10:29:23 -07:00
ASP.NET CI ea980c55f6 Update dependencies.props
[auto-updated: dependencies]
2018-09-23 19:10:46 +00:00
ASP.NET CI dd46e7e46b Update dependencies.props
[auto-updated: dependencies]
2018-09-23 12:09:41 -07:00
Ryan Nowak db826f13ed
Merge pull request #484 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-09-20 12:18:26 -07:00
Ryan Nowak 8fb6c2a50a Allow cancellation to propagate 2018-09-20 11:04:35 -07:00
Ryan Nowak 4cecbc668a
Merge pull request #481 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-09-19 19:12:35 -07:00
Ryan Nowak 787283ec2f
Merge branch 'master' into merge/release/2.2-to-master 2018-09-19 18:23:12 -07:00
Ryan Nowak 4259b65c16
Use options for registering health checks (#479)
* Use options for registering health checks

This change pivots to use options for registering health checks. We get
a few pretty nice things out of this, and it unblocks some of our
requirements.

Now all registration methods support the application developer
configuring the name, failure-status, and tags for each health check.
This is a requirment, that we weren't really satisfying - which is what
led to this redesign. In support of this health checks now return pass/fail,
and the service is responsible for assigning the status.

----

Health check authors that need configuration data (connection string as
an example) now have three ways to do this depending on their
requirements.
1. Create an instance and register that (easiest)
2. Use Type Activation and pass parameters (middle)
3. Use named options (richest)

We expect most health checks to need/want some kind of configuration -
which 1) works pretty well to solve. However many other health checks
will need DI + configuration. It was also a gap that we didn't have a
good way to use named options, when it's such a good fit for our
scenarios.

Added new registration methods for type activation that allow you to
pass parameters for 2).

Added a context type that allows the running health check access to its
registration for 3).

----

Redesigned and renamed how status gets reported. Health checks return
pass/fail result, but the overall HealthReport includes entries of a
different type. This seems fine because there isn't really a way to
consume a HealthCheckResult directly - the service is the only consumer.

----

Added support for tags. This was easy to add now that we have a separate
registration type, and it's quite handy for building filters (see
sample).

* HARDER BETTER STRONGER FASTER
2018-09-19 14:48:34 -07:00
ASP.NET CI af2abc420c Update dependencies.props
[auto-updated: dependencies]
2018-09-16 19:09:58 +00:00
ASP.NET CI 3ac6439dcf Update dependencies.props
[auto-updated: dependencies]
2018-09-16 12:09:03 -07:00
ASP.NET CI bd87f9bb5b Update dependencies.props
[auto-updated: dependencies]
2018-09-09 19:11:15 +00:00
ASP.NET CI f94ad0f202 Update dependencies.props
[auto-updated: dependencies]
2018-09-09 12:10:09 -07:00
ASP.NET CI c486ff0b96 Update dependencies.props
[auto-updated: dependencies]
2018-09-05 23:42:22 +00:00
ASP.NET CI b3e163e44f
Update branding to 2.2.0-preview3 2018-09-05 16:34:00 -07:00
ASP.NET CI e681b344a6 Update dependencies.props
[auto-updated: dependencies]
2018-09-02 19:10:08 +00:00
ASP.NET CI dd07e6743c Update dependencies.props
[auto-updated: dependencies]
2018-09-02 12:09:26 -07:00
Ryan Nowak bf31749e9f
Merge pull request #475 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-08-30 17:06:50 -07:00
Ryan Nowak 525fbf495b Fix #468 - all UseHealthChecks without a path 2018-08-30 15:33:49 -07:00
Ryan Nowak 0bb4e948f7
Merge pull request #473 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-08-30 12:47:29 -07:00
Ryan Nowak 0001581ffc
Merge branch 'master' into merge/release/2.2-to-master 2018-08-30 11:25:57 -07:00
Ryan Nowak 3e4a3d0b90
Allow health checks to use any DI lifetime (#466)
* Allow health checks to use any DI lifetime

This change allows registered IHealthCheck implementations to use any DI
lifetime. This is necessary for scenarios like using EF which requires a
scope.

The works by having the health check service create a scope for each
time it queries health checks. This scope does not overlap or share
state with other scopes (the request scope) so there is no crosstalk
between processing going on per-request in ASP.NET Core and the health
check operation.

* PR feedback and some logging cleanup
2018-08-30 10:51:48 -07:00
Pranav K db40c80cea
Merge remote-tracking branch 'origin/release/2.2'
# Conflicts:
#	version.props
2018-08-29 10:13:33 -07:00
Zhiliang 6cd4cf1a7a Fix typos in samples 2018-08-29 10:12:10 -07:00
martincostello fc7fa4f0f0 Fix sample typo
Fix typo in sample comment.
2018-08-28 15:41:18 -07:00
ASP.NET CI 241208dee0 Update dependencies.props
[auto-updated: dependencies]
2018-08-26 19:10:15 +00:00
ASP.NET CI 5c8b549882 Update package branding for 2.2.0-preview2 2018-08-21 13:33:49 -07:00
ASP.NET CI 4762c651d5 Update dependencies.props
[auto-updated: dependencies]
2018-08-19 19:10:55 +00:00
Ryan Nowak 4f908628bf
Merge pull request #462 from dotnet-maestro-bot/merge/release/2.2-to-master
[automated] Merge branch 'release/2.2' => 'master'
2018-08-13 13:07:46 -07:00
Ryan Nowak 3d208181b9
Merge branch 'master' into merge/release/2.2-to-master 2018-08-13 11:13:54 -07:00
ASP.NET CI 3c5e8fb78e Update dependencies.props
[auto-updated: dependencies]
2018-08-12 19:56:32 +00:00
ASP.NET CI 983d7836ab Update dependencies.props
[auto-updated: dependencies]
2018-08-12 19:10:01 +00:00
Arthur Vickers 6b09fa4e86
Merge pull request #431 from andrewjsaid/dev
Show correct missing key name in NoContextType error message
2018-08-08 10:28:15 -07:00
Ryan Nowak 8ec69456ca
Update CustomWriterStartup.cs
Fix a misleading comment in sample
2018-08-06 16:44:54 -07:00
ASP.NET CI 602faf5baa Update dependencies.props
[auto-updated: dependencies]
2018-08-06 20:33:14 +00:00