diff options
author | Geir Storli <geirst@verizonmedia.com> | 2021-01-26 13:18:37 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-26 13:18:37 +0100 |
commit | 8947cd201d5647cc0ebdccaa2235292ae24bac89 (patch) | |
tree | ec8d47f3df1506cc23d3863ec2e518df0b44206f | |
parent | fd0075f04835769d92263b5d852ff8e073634e42 (diff) | |
parent | a4ceb0eab70b53a40babc493450eb275be060447 (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.cpp | 8 | ||||
-rw-r--r-- | storage/src/vespa/storage/persistence/filestorage/service_layer_host_info_reporter.cpp | 2 |
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"); |