summaryrefslogtreecommitdiffstats
path: root/storage
diff options
context:
space:
mode:
Diffstat (limited to 'storage')
-rw-r--r--storage/src/vespa/storage/common/statusmetricconsumer.cpp16
1 files changed, 6 insertions, 10 deletions
diff --git a/storage/src/vespa/storage/common/statusmetricconsumer.cpp b/storage/src/vespa/storage/common/statusmetricconsumer.cpp
index 5998a2fe9a9..e23400063d4 100644
--- a/storage/src/vespa/storage/common/statusmetricconsumer.cpp
+++ b/storage/src/vespa/storage/common/statusmetricconsumer.cpp
@@ -51,16 +51,12 @@ bool
StatusMetricConsumer::reportStatus(std::ostream& out,
const framework::HttpUrlPath& path) const
{
- // Update metrics unless 'dontcallupdatehooks' is 1. Update
- // snapshot metrics too, if callsnapshothooks is set to 1.
- if (path.get("dontcallupdatehooks", 0) == 0) {
- bool updateSnapshotHooks = path.get("callsnapshothooks", 0) == 1;
- LOG(debug, "Updating metrics ahead of status page view%s",
- updateSnapshotHooks ? ", calling snapshot hooks too" : ".");
- _manager.updateMetrics(updateSnapshotHooks);
- } else {
- LOG(debug, "Not calling update hooks as dontcallupdatehooks option has been given");
- }
+ // snapshot metrics too, if callsnapshothooks is set to 1.
+ bool updateSnapshotHooks = path.get("callsnapshothooks", 0) == 1;
+ LOG(debug, "Updating metrics ahead of status page view%s",
+ updateSnapshotHooks ? ", calling snapshot hooks too" : ".");
+ _manager.updateMetrics(updateSnapshotHooks);
+
vespalib::system_time currentTime = _component.getClock().getSystemTime();
bool json = (path.getAttribute("format") == "json");