aboutsummaryrefslogtreecommitdiffstats
path: root/config-model/src/test/java
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2024-06-11 12:05:15 +0200
committerHenning Baldersheim <balder@yahoo-inc.com>2024-06-11 12:05:15 +0200
commitb7bff19fc74ea04d4c63af49ffd548da5e582f14 (patch)
tree331b96a9bd1def1f0f485a669d69f7c3446529b8 /config-model/src/test/java
parent12150d28d136f5e3f2fae5acc551a5af0e7a9af3 (diff)
Disk size is given in units of 1*10^9(GB), not 1*1024^3(GiB)
Diffstat (limited to 'config-model/src/test/java')
-rw-r--r--config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java4
-rw-r--r--config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java2
-rw-r--r--config-model/src/test/java/com/yahoo/vespa/model/search/NodeResourcesTuningTest.java41
3 files changed, 24 insertions, 23 deletions
diff --git a/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java b/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java
index 2846925b9c3..737ac4c248c 100644
--- a/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java
+++ b/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java
@@ -60,7 +60,7 @@ import static com.yahoo.config.provision.NodeResources.DiskSpeed;
import static com.yahoo.config.provision.NodeResources.StorageType;
import static com.yahoo.vespa.defaults.Defaults.getDefaults;
import static com.yahoo.vespa.model.Host.memoryOverheadGb;
-import static com.yahoo.vespa.model.search.NodeResourcesTuning.GB;
+import static com.yahoo.vespa.model.search.NodeResourcesTuning.GiB;
import static com.yahoo.vespa.model.test.utils.ApplicationPackageUtils.generateSchemas;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
@@ -2580,7 +2580,7 @@ public class ModelProvisioningTest {
ProtonConfig cfg = getProtonConfig(model, cluster.getSearchNodes().get(0).getConfigId());
assertEquals(2000, cfg.flush().memory().maxtlssize()); // from config override
assertEquals(1000, cfg.flush().memory().maxmemory()); // from explicit tuning
- assertEquals((long) ((128 - memoryOverheadGb) * GB * 0.08), cfg.flush().memory().each().maxmemory()); // from default node flavor tuning
+ assertEquals((long) ((128 - memoryOverheadGb) * GiB * 0.08), cfg.flush().memory().each().maxmemory()); // from default node flavor tuning
}
private static ProtonConfig getProtonConfig(VespaModel model, String configId) {
diff --git a/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java b/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java
index adcfcda0713..a890f35caa8 100644
--- a/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java
+++ b/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java
@@ -1278,7 +1278,7 @@ public class ContentClusterTest extends ContentBaseTest {
void verify_max_tls_size() throws Exception {
var flavor = new Flavor(new FlavorsConfig.Flavor(new FlavorsConfig.Flavor.Builder().name("test").minDiskAvailableGb(100)));
assertEquals(21474836480L, resolveMaxTLSSize(Optional.empty()));
- assertEquals(2147483648L, resolveMaxTLSSize(Optional.of(flavor)));
+ assertEquals(2_000_000_000, resolveMaxTLSSize(Optional.of(flavor)));
}
void assertZookeeperServerImplementation(String expectedClassName,
diff --git a/config-model/src/test/java/com/yahoo/vespa/model/search/NodeResourcesTuningTest.java b/config-model/src/test/java/com/yahoo/vespa/model/search/NodeResourcesTuningTest.java
index fbc4f6768a2..998437b64f5 100644
--- a/config-model/src/test/java/com/yahoo/vespa/model/search/NodeResourcesTuningTest.java
+++ b/config-model/src/test/java/com/yahoo/vespa/model/search/NodeResourcesTuningTest.java
@@ -9,7 +9,8 @@ import org.junit.jupiter.api.Test;
import static com.yahoo.vespa.model.Host.memoryOverheadGb;
import static org.junit.jupiter.api.Assertions.assertEquals;
-import static com.yahoo.vespa.model.search.NodeResourcesTuning.MB;
+import static com.yahoo.vespa.model.search.NodeResourcesTuning.MiB;
+import static com.yahoo.vespa.model.search.NodeResourcesTuning.GiB;
import static com.yahoo.vespa.model.search.NodeResourcesTuning.GB;
/**
@@ -28,7 +29,7 @@ public class NodeResourcesTuningTest {
@Test
void require_that_hwinfo_memory_size_is_set() {
- assertEquals(24 * GB, configFromMemorySetting(24 + memoryOverheadGb, 0).hwinfo().memory().size());
+ assertEquals(24 * GiB, configFromMemorySetting(24 + memoryOverheadGb, 0).hwinfo().memory().size());
assertEquals(1.9585050869E10, configFromMemorySetting(24 + memoryOverheadGb, ApplicationContainerCluster.heapSizePercentageOfTotalAvailableMemoryWhenCombinedCluster * 0.01).hwinfo().memory().size(), 1000);
}
@@ -84,26 +85,26 @@ public class NodeResourcesTuningTest {
@Test
void require_that_document_store_maxfilesize_is_set_based_on_available_memory() {
- assertDocumentStoreMaxFileSize(256 * MB, 4);
- assertDocumentStoreMaxFileSize(256 * MB, 6);
- assertDocumentStoreMaxFileSize(256 * MB, 8);
- assertDocumentStoreMaxFileSize(256 * MB, 12);
- assertDocumentStoreMaxFileSize((long) (16 * GB * 0.02), 16);
- assertDocumentStoreMaxFileSize((long) (24 * GB * 0.02), 24);
- assertDocumentStoreMaxFileSize((long) (32 * GB * 0.02), 32);
- assertDocumentStoreMaxFileSize((long) (48 * GB * 0.02), 48);
- assertDocumentStoreMaxFileSize((long) (64 * GB * 0.02), 64);
- assertDocumentStoreMaxFileSize((long) (128 * GB * 0.02), 128);
- assertDocumentStoreMaxFileSize((long) (256 * GB * 0.02), 256);
- assertDocumentStoreMaxFileSize((long) (512 * GB * 0.02), 512);
+ assertDocumentStoreMaxFileSize(256 * MiB, 4);
+ assertDocumentStoreMaxFileSize(256 * MiB, 6);
+ assertDocumentStoreMaxFileSize(256 * MiB, 8);
+ assertDocumentStoreMaxFileSize(256 * MiB, 12);
+ assertDocumentStoreMaxFileSize((long) (16 * GiB * 0.02), 16);
+ assertDocumentStoreMaxFileSize((long) (24 * GiB * 0.02), 24);
+ assertDocumentStoreMaxFileSize((long) (32 * GiB * 0.02), 32);
+ assertDocumentStoreMaxFileSize((long) (48 * GiB * 0.02), 48);
+ assertDocumentStoreMaxFileSize((long) (64 * GiB * 0.02), 64);
+ assertDocumentStoreMaxFileSize((long) (128 * GiB * 0.02), 128);
+ assertDocumentStoreMaxFileSize((long) (256 * GiB * 0.02), 256);
+ assertDocumentStoreMaxFileSize((long) (512 * GiB * 0.02), 512);
}
@Test
void require_that_flush_strategy_memory_limits_are_set_based_on_available_memory() {
- assertFlushStrategyMemory((long) (4 * GB * DEFAULT_MEMORY_GAIN), 4);
- assertFlushStrategyMemory((long) (8 * GB * DEFAULT_MEMORY_GAIN), 8);
- assertFlushStrategyMemory((long) (24 * GB * DEFAULT_MEMORY_GAIN), 24);
- assertFlushStrategyMemory((long) (64 * GB * DEFAULT_MEMORY_GAIN), 64);
+ assertFlushStrategyMemory((long) (4 * GiB * DEFAULT_MEMORY_GAIN), 4);
+ assertFlushStrategyMemory((long) (8 * GiB * DEFAULT_MEMORY_GAIN), 8);
+ assertFlushStrategyMemory((long) (24 * GiB * DEFAULT_MEMORY_GAIN), 24);
+ assertFlushStrategyMemory((long) (64 * GiB * DEFAULT_MEMORY_GAIN), 64);
}
@Test
@@ -129,8 +130,8 @@ public class NodeResourcesTuningTest {
@Test
void require_that_summary_cache_max_bytes_is_set_based_on_memory() {
- assertEquals(1 * GB / 25, configFromMemorySetting(1 + memoryOverheadGb, 0).summary().cache().maxbytes());
- assertEquals(256 * GB / 25, configFromMemorySetting(256 + memoryOverheadGb, 0).summary().cache().maxbytes());
+ assertEquals(1 * GiB / 25, configFromMemorySetting(1 + memoryOverheadGb, 0).summary().cache().maxbytes());
+ assertEquals(256 * GiB / 25, configFromMemorySetting(256 + memoryOverheadGb, 0).summary().cache().maxbytes());
}
@Test