You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Bertrand Delacretaz (JIRA)" <ji...@apache.org> on 2017/06/09 14:56:18 UTC
[jira] [Comment Edited] (SLING-6855) Sticky Results Support
[ https://issues.apache.org/jira/browse/SLING-6855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16044469#comment-16044469 ]
Bertrand Delacretaz edited comment on SLING-6855 at 6/9/17 2:55 PM:
--------------------------------------------------------------------
Thanks [~henzlerg] - I added a sticky config to the async HC in the samples bundle, and initially that did not work as the HC results cache was not updated for async execution.
http://svn.apache.org/r1798222 fixes that for me, do you agree with this fix? Tests didn't detect that issue, if you have an idea for improving them for that that's great otherwise I can have a look _(update: I've added a test for that in http://svn.apache.org/r1798231 )_ .
With this fix, installing the {{org.apache.sling.hc.samples}} bundle and requesting http://localhost:8080/system/console/healthcheck?tags=async shows output like follows:
{code}
Sticky Asynchronous Health Check sample
Tags: [async, sticky] Finished: 2017-06-09 59:29 after 1ms
Result: HEALTH_CHECK_ERROR
INFO *** Current Result ***
INFO AsyncHealthCheckSample@3d846efa - counter value set to 788 at Fri Jun 09 15:59:29 CEST 2017
WARN *** Sticky Result CRITICAL from 15:59:24.002 ***
INFO AsyncHealthCheckSample@31a3fb2e - counter value set to 783 at Fri Jun 09 15:59:24 CEST 2017
CRITICAL Counter value (783) is not a multiple of 3 (critical) at Fri Jun 09 15:59:24 CEST 2017
WARN *** Sticky Result HEALTH_CHECK_ERROR from 15:58:46.002 ***
INFO AsyncHealthCheckSample@1b2b0a85 - counter value set to 745 at Fri Jun 09 15:58:46 CEST 2017
HEALTH_CHECK_ERROR Counter value (745) is not a multiple of 5 (healthCheckError) at Fri Jun 09 15:58:46 CEST 2017
{code}
The cache keeps one result of each type, by design, I think that's ok, it prevents the cache from growing indefinitely and gives useful information as to recent non-ok results. The results are not ordered by time which can be a bit surprising but is ok IMO.
If we agree on how this feature works we should document it at https://sling.apache.org/documentation/bundles/sling-health-check-tool.html
was (Author: bdelacretaz):
Thanks [~henzlerg] - I added a sticky config to the async HC in the samples bundle, and initially that did not work as the HC results cache was not updated for async execution.
http://svn.apache.org/r1798222 fixes that for me, do you agree with this fix? Tests didn't detect that issue, if you have an idea for improving them for that that's great otherwise I can have a look.
With this fix, installing the {{org.apache.sling.hc.samples}} bundle and requesting http://localhost:8080/system/console/healthcheck?tags=async shows output like follows:
{code}
Sticky Asynchronous Health Check sample
Tags: [async, sticky] Finished: 2017-06-09 59:29 after 1ms
Result: HEALTH_CHECK_ERROR
INFO *** Current Result ***
INFO AsyncHealthCheckSample@3d846efa - counter value set to 788 at Fri Jun 09 15:59:29 CEST 2017
WARN *** Sticky Result CRITICAL from 15:59:24.002 ***
INFO AsyncHealthCheckSample@31a3fb2e - counter value set to 783 at Fri Jun 09 15:59:24 CEST 2017
CRITICAL Counter value (783) is not a multiple of 3 (critical) at Fri Jun 09 15:59:24 CEST 2017
WARN *** Sticky Result HEALTH_CHECK_ERROR from 15:58:46.002 ***
INFO AsyncHealthCheckSample@1b2b0a85 - counter value set to 745 at Fri Jun 09 15:58:46 CEST 2017
HEALTH_CHECK_ERROR Counter value (745) is not a multiple of 5 (healthCheckError) at Fri Jun 09 15:58:46 CEST 2017
{code}
The cache keeps one result of each type, by design, I think that's ok, it prevents the cache from growing indefinitely and gives useful information as to recent non-ok results. The results are not ordered by time which can be a bit surprising but is ok IMO.
If we agree on how this feature works we should document it at https://sling.apache.org/documentation/bundles/sling-health-check-tool.html
> Sticky Results Support
> ----------------------
>
> Key: SLING-6855
> URL: https://issues.apache.org/jira/browse/SLING-6855
> Project: Sling
> Issue Type: New Feature
> Components: Health Check
> Reporter: Clinton H Goudie-Nice
> Assignee: Georg Henzler
> Fix For: Health Check Annotations 1.0.6, Health Check Core 1.2.10, Health Check API 1.0.2
>
>
> Introduce HC service property {{hc.warningsStickForMinutes}} to allow old WARN/CRITICAL/HEALTH_CHECK_ERROR results to be sticky (see also http://sling.markmail.org/thread/tawikgt7bqxvnlj5#query:+page:1+mid:57hhg55hekr7ib33+state:results)
> --- Original Request ----
> *Create ResultRegistry to provide health check behavior for executing code that does not want a HealthCheck*
> I want to provide a Registry service that can be leveraged to provide health check results.
> These results can be for a period of time through an expiration, until the JVM is restarted, or added and later removed.
> This can be useful when code observes a specific (possibly bad) state, and wants to alert through the health check API that this state has taken place.
> Some examples:
> An event pool has filled, and some events will be thrown away.
> This is a failure case that requires a restart of the instance.
> It would be appropriate to trigger a permanent failure.
>
> A quota has been tripped. This quota may immediately recover, but it is sensible to alert for 30 minutes that the quota has been tripped.
> If you expect the failure will clear itself within a certain window, setting the expiration to that window can be ideal.
> GHPR to follow
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)