aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Francois Tur <ftur@infoblox.com> 2018-05-09 10:09:06 -0400
committerGravatar Miek Gieben <miek@miek.nl> 2018-05-09 15:09:06 +0100
commit30309861c56157e38eb06c20e3bc62c1e5a818bc (patch)
treecf44cec0c7d0a28a0fc7d2e34fdaf22c6780611a
parentb9f0d55fc94aafa4060cbf2d4f91d34a4a87c043 (diff)
downloadcoredns-30309861c56157e38eb06c20e3bc62c1e5a818bc.tar.gz
coredns-30309861c56157e38eb06c20e3bc62c1e5a818bc.tar.zst
coredns-30309861c56157e38eb06c20e3bc62c1e5a818bc.zip
- review BUG related doc for Health and Premotheus after change of behavior to be compatible with reload feature. (#1790)
-rw-r--r--plugin/health/README.md7
-rw-r--r--plugin/metrics/README.md6
2 files changed, 11 insertions, 2 deletions
diff --git a/plugin/health/README.md b/plugin/health/README.md
index 5cb6c7beb..b7d4892b3 100644
--- a/plugin/health/README.md
+++ b/plugin/health/README.md
@@ -93,3 +93,10 @@ Set a lameduck duration of 1 second:
}
}
~~~
+
+## Bugs
+
+When reloading, the Health handler is stopped before the new server instance is started.
+If that new server fails to start, then the initial server instance is still available and DNS queries still served,
+but Health handler stays down.
+Health will not reply HTTP request until a successful reload or a complete restart of CoreDNS.
diff --git a/plugin/metrics/README.md b/plugin/metrics/README.md
index c962068a4..2a914fe44 100644
--- a/plugin/metrics/README.md
+++ b/plugin/metrics/README.md
@@ -71,5 +71,7 @@ then:
## Bugs
-When reloading, we keep the handler running, meaning that any changes to the handler's address
-aren't picked up. You'll need to restart CoreDNS for that to happen.
+When reloading, the Prometheus handler is stopped before the new server instance is started.
+If that new server fails to start, then the initial server instance is still available and DNS queries still served,
+but Prometheus handler stays down.
+Prometheus will not reply HTTP request until a successful reload or a complete restart of CoreDNS.