aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGeir Storli <geirst@verizonmedia.com>2021-01-26 13:18:37 +0100
committerGitHub <noreply@github.com>2021-01-26 13:18:37 +0100
commit8947cd201d5647cc0ebdccaa2235292ae24bac89 (patch)
treeec8d47f3df1506cc23d3863ec2e518df0b44206f
parentfd0075f04835769d92263b5d852ff8e073634e42 (diff)
parenta4ceb0eab70b53a40babc493450eb275be060447 (diff)
Merge pull request #16231 from vespa-engine/toregge/reduce-diff-slack
Reduce diff slack to 0.001
-rw-r--r--storage/src/tests/persistence/filestorage/service_layer_host_info_reporter_test.cpp8
-rw-r--r--storage/src/vespa/storage/persistence/filestorage/service_layer_host_info_reporter.cpp2
2 files changed, 5 insertions, 5 deletions
diff --git a/storage/src/tests/persistence/filestorage/service_layer_host_info_reporter_test.cpp b/storage/src/tests/persistence/filestorage/service_layer_host_info_reporter_test.cpp
index c4489b42978..86595fb44e8 100644
--- a/storage/src/tests/persistence/filestorage/service_layer_host_info_reporter_test.cpp
+++ b/storage/src/tests/persistence/filestorage/service_layer_host_info_reporter_test.cpp
@@ -78,10 +78,10 @@ TEST_F(ServiceLayerHostInfoReporterTest, request_almost_immediate_node_state_as_
EXPECT_EQ(1, requested_almost_immediate_replies());
EXPECT_EQ(ResourceUsage(0.5, 0.4), get_old_usage());
EXPECT_EQ(ResourceUsage(0.5, 0.4), get_usage());
- notify(0.501, 0.401);
+ notify(0.5001, 0.4001);
EXPECT_EQ(1, requested_almost_immediate_replies());
EXPECT_EQ(ResourceUsage(0.5, 0.4), get_old_usage());
- EXPECT_EQ(ResourceUsage(0.501, 0.401), get_usage());
+ EXPECT_EQ(ResourceUsage(0.5001, 0.4001), get_usage());
notify(0.8, 0.4);
EXPECT_EQ(2, requested_almost_immediate_replies());
EXPECT_EQ(ResourceUsage(0.8, 0.4), get_old_usage());
@@ -90,10 +90,10 @@ TEST_F(ServiceLayerHostInfoReporterTest, request_almost_immediate_node_state_as_
EXPECT_EQ(3, requested_almost_immediate_replies());
EXPECT_EQ(ResourceUsage(0.8, 0.7), get_old_usage());
EXPECT_EQ(ResourceUsage(0.8, 0.7), get_usage());
- notify(0.799, 0.699);
+ notify(0.7999, 0.6999);
EXPECT_EQ(3, requested_almost_immediate_replies());
EXPECT_EQ(ResourceUsage(0.8, 0.7), get_old_usage());
- EXPECT_EQ(ResourceUsage(0.799, 0.699), get_usage());
+ EXPECT_EQ(ResourceUsage(0.7999, 0.6999), get_usage());
notify(0.8, 0.7, {0.1, attr_es_name}, {});
EXPECT_EQ(ResourceUsage(0.8, 0.7, {0.1, attr_es_name}, {}), get_old_usage());
EXPECT_EQ(ResourceUsage(0.8, 0.7, {0.1, attr_es_name}, {}), get_usage());
diff --git a/storage/src/vespa/storage/persistence/filestorage/service_layer_host_info_reporter.cpp b/storage/src/vespa/storage/persistence/filestorage/service_layer_host_info_reporter.cpp
index 01a923d7d90..c78b24bee78 100644
--- a/storage/src/vespa/storage/persistence/filestorage/service_layer_host_info_reporter.cpp
+++ b/storage/src/vespa/storage/persistence/filestorage/service_layer_host_info_reporter.cpp
@@ -11,7 +11,7 @@ using End = vespalib::JsonStream::End;
namespace {
-constexpr double diff_slack = 0.01;
+constexpr double diff_slack = 0.001;
const vespalib::string memory_label("memory");
const vespalib::string disk_label("disk");