diff options
author | Harald Musum <musum@yahooinc.com> | 2023-06-13 09:17:31 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2023-06-13 09:17:31 +0200 |
commit | ccc9f2829e918846a444daea4434cab01e93241d (patch) | |
tree | 7fa35ef18fe0562d2a807765c137fe9058263172 /configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java | |
parent | 481892cdb069e0379fb36490ec53fda9588c1a29 (diff) |
Log or fail config server startup when upgrading between 2 versions too far apart
Fail on hosted, log on self serve Vespa
Diffstat (limited to 'configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java')
-rw-r--r-- | configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java index 1f49d496f70..092243bac6d 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/version/VersionStateTest.java @@ -33,14 +33,14 @@ public class VersionStateTest { VersionState state = createVersionState(); assertEquals(unknownVersion, state.storedVersion()); assertTrue(state.isUpgraded()); - state.saveNewVersion(); + state.storeCurrentVersion(); assertFalse(state.isUpgraded()); - state.saveNewVersion("badversion"); + state.storeVersion("badversion"); assertEquals(unknownVersion, state.storedVersion()); assertTrue(state.isUpgraded()); - state.saveNewVersion("5.0.0"); + state.storeVersion("5.0.0"); assertEquals(new Version(5, 0, 0), state.storedVersion()); assertTrue(state.isUpgraded()); @@ -50,12 +50,12 @@ public class VersionStateTest { assertTrue(state.isUpgraded()); // Save new version, remove version in file, should find version in ZooKeeper - state.saveNewVersion("6.0.0"); + state.storeVersion("6.0.0"); Files.delete(state.versionFile().toPath()); assertEquals(new Version(6, 0, 0), state.storedVersion()); assertTrue(state.isUpgraded()); - state.saveNewVersion(); + state.storeCurrentVersion(); assertEquals(state.currentVersion(), state.storedVersion()); assertFalse(state.isUpgraded()); } @@ -64,7 +64,7 @@ public class VersionStateTest { public void serverdbfile() throws IOException { File dbDir = tempDir.newFolder(); VersionState state = new VersionState(new ConfigserverConfig.Builder().configServerDBDir(dbDir.getAbsolutePath()).build(), curator); - state.saveNewVersion(); + state.storeCurrentVersion(); File versionFile = new File(dbDir, "vespa_version"); assertTrue(versionFile.exists()); Version stored = Version.fromString(IOUtils.readFile(versionFile)); @@ -72,7 +72,7 @@ public class VersionStateTest { } private VersionState createVersionState() throws IOException { - return new VersionState(tempDir.newFile(), curator); + return new VersionState(tempDir.newFile(), curator, false); } } |