GC legacy dontcallupdatehooks metrics param.

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");