diff --git a/src/config.c b/src/config.c index b26704283..5cb5b6a64 100644 --- a/src/config.c +++ b/src/config.c @@ -634,8 +634,7 @@ void loadServerConfigFromString(char *config) { loaderr: if (argv) sdsfreesplitres(argv,argc); - fprintf(stderr, "\n*** FATAL CONFIG FILE ERROR (Redis %s) ***\n", - REDIS_VERSION); + fprintf(stderr, "\n*** FATAL CONFIG FILE ERROR (Version %s) ***\n", SERVER_VERSION); if (i < totlines) { fprintf(stderr, "Reading the configuration file, at line %d\n", linenum); fprintf(stderr, ">>> '%s'\n", lines[i]); diff --git a/src/debug.c b/src/debug.c index 684f692af..29ec87a40 100644 --- a/src/debug.c +++ b/src/debug.c @@ -2122,7 +2122,7 @@ void sigsegvHandler(int sig, siginfo_t *info, void *secret) { bugReportStart(); serverLog(LL_WARNING, - "Redis %s crashed by signal: %d, si_code: %d", REDIS_VERSION, sig, info->si_code); + SERVER_NAME " %s crashed by signal: %d, si_code: %d", SERVER_VERSION, sig, info->si_code); if (sig == SIGSEGV || sig == SIGBUS) { serverLog(LL_WARNING, "Accessing address: %p", (void*)info->si_addr); diff --git a/src/lolwut.c b/src/lolwut.c index c014840e9..08c1f5899 100644 --- a/src/lolwut.c +++ b/src/lolwut.c @@ -44,7 +44,7 @@ void lolwut6Command(client *c); * This is what unstable versions of Redis will display. */ void lolwutUnstableCommand(client *c) { sds rendered = sdsnew("Redis ver. "); - rendered = sdscat(rendered,REDIS_VERSION); + rendered = sdscat(rendered,SERVER_VERSION); rendered = sdscatlen(rendered,"\n",1); addReplyVerbatim(c,rendered,sdslen(rendered),"txt"); sdsfree(rendered); @@ -52,7 +52,7 @@ void lolwutUnstableCommand(client *c) { /* LOLWUT [VERSION ] [... version specific arguments ...] */ void lolwutCommand(client *c) { - char *v = REDIS_VERSION; + char *v = SERVER_VERSION; char verstr[64]; if (c->argc >= 3 && !strcasecmp(c->argv[1]->ptr,"version")) { diff --git a/src/lolwut5.c b/src/lolwut5.c index 1240168d0..dcc7e7ad9 100644 --- a/src/lolwut5.c +++ b/src/lolwut5.c @@ -169,7 +169,7 @@ void lolwut5Command(client *c) { sds rendered = renderCanvas(canvas); rendered = sdscat(rendered, "\nGeorg Nees - schotter, plotter on paper, 1968. Redis ver. "); - rendered = sdscat(rendered,REDIS_VERSION); + rendered = sdscat(rendered,SERVER_VERSION); rendered = sdscatlen(rendered,"\n",1); addReplyVerbatim(c,rendered,sdslen(rendered),"txt"); sdsfree(rendered); diff --git a/src/lolwut6.c b/src/lolwut6.c index 1ba111c2d..38733505a 100644 --- a/src/lolwut6.c +++ b/src/lolwut6.c @@ -193,7 +193,7 @@ void lolwut6Command(client *c) { rendered = sdscat(rendered, "\nDedicated to the 8 bit game developers of past and present.\n" "Original 8 bit image from Plaguemon by hikikomori. Redis ver. "); - rendered = sdscat(rendered,REDIS_VERSION); + rendered = sdscat(rendered,SERVER_VERSION); rendered = sdscatlen(rendered,"\n",1); addReplyVerbatim(c,rendered,sdslen(rendered),"txt"); sdsfree(rendered); diff --git a/src/mkreleasehdr.sh b/src/mkreleasehdr.sh index 117b9e86f..26f7d505d 100755 --- a/src/mkreleasehdr.sh +++ b/src/mkreleasehdr.sh @@ -12,5 +12,5 @@ echo "#define REDIS_GIT_SHA1 \"$GIT_SHA1\"" > release.h echo "#define REDIS_GIT_DIRTY \"$GIT_DIRTY\"" >> release.h echo "#define REDIS_BUILD_ID \"$BUILD_ID\"" >> release.h echo "#include \"version.h\"" >> release.h -echo "#define REDIS_BUILD_ID_RAW REDIS_VERSION REDIS_BUILD_ID REDIS_GIT_DIRTY REDIS_GIT_SHA1" >> release.h +echo "#define REDIS_BUILD_ID_RAW SERVER_NAME SERVER_VERSION REDIS_BUILD_ID REDIS_GIT_DIRTY REDIS_GIT_SHA1" >> release.h touch release.c # Force recompile of release.c diff --git a/src/module.c b/src/module.c index ac6cbbb78..705f54326 100644 --- a/src/module.c +++ b/src/module.c @@ -13153,7 +13153,7 @@ int RM_GetKeyspaceNotificationFlagsAll(void) { * Example for 6.0.7 the return value will be 0x00060007. */ int RM_GetServerVersion(void) { - return REDIS_VERSION_NUM; + return SERVER_VERSION_NUM; } /** diff --git a/src/networking.c b/src/networking.c index 7696e8c28..0d2d7e951 100644 --- a/src/networking.c +++ b/src/networking.c @@ -3603,7 +3603,7 @@ void helloCommand(client *c) { addReplyBulkCString(c,"redis"); addReplyBulkCString(c,"version"); - addReplyBulkCString(c,REDIS_VERSION); + addReplyBulkCString(c,SERVER_VERSION); addReplyBulkCString(c,"proto"); addReplyLongLong(c,c->resp); diff --git a/src/rdb.c b/src/rdb.c index ed30b6523..f9b542fec 100644 --- a/src/rdb.c +++ b/src/rdb.c @@ -1195,6 +1195,8 @@ int rdbSaveInfoAuxFields(rio *rdb, int rdbflags, rdbSaveInfo *rsi) { int aof_base = (rdbflags & RDBFLAGS_AOF_PREAMBLE) != 0; /* Add a few fields about the state when the RDB was created. */ + if (rdbSaveAuxFieldStrStr(rdb,"server-ver",SERVER_VERSION) == -1) return -1; + /* Print Redis version for backwards compatibility with redis. */ if (rdbSaveAuxFieldStrStr(rdb,"redis-ver",REDIS_VERSION) == -1) return -1; if (rdbSaveAuxFieldStrInt(rdb,"redis-bits",redis_bits) == -1) return -1; if (rdbSaveAuxFieldStrInt(rdb,"ctime",time(NULL)) == -1) return -1; @@ -3138,7 +3140,10 @@ int rdbLoadRioWithLoadingCtx(rio *rdb, int rdbflags, rdbSaveInfo *rsi, rdbLoadin } else if (!strcasecmp(auxkey->ptr,"lua")) { /* Won't load the script back in memory anymore. */ } else if (!strcasecmp(auxkey->ptr,"redis-ver")) { - serverLog(LL_NOTICE,"Loading RDB produced by version %s", + serverLog(LL_NOTICE,"Loading RDB produced by Redis version %s", + (char*)auxval->ptr); + } else if (!strcasecmp(auxkey->ptr,"server-ver")) { + serverLog(LL_NOTICE,"Loading RDB produced by server version %s", (char*)auxval->ptr); } else if (!strcasecmp(auxkey->ptr,"ctime")) { time_t age = time(NULL)-strtol(auxval->ptr,NULL,10); diff --git a/src/script_lua.c b/src/script_lua.c index 8cdd80523..c6a353241 100644 --- a/src/script_lua.c +++ b/src/script_lua.c @@ -1373,6 +1373,7 @@ void luaSetTableProtectionRecursively(lua_State *lua) { } void luaRegisterVersion(lua_State* lua) { + /* For legacy compatibility reasons include Redis versions. */ lua_pushstring(lua,"REDIS_VERSION_NUM"); lua_pushnumber(lua,REDIS_VERSION_NUM); lua_settable(lua,-3); @@ -1380,6 +1381,19 @@ void luaRegisterVersion(lua_State* lua) { lua_pushstring(lua,"REDIS_VERSION"); lua_pushstring(lua,REDIS_VERSION); lua_settable(lua,-3); + + /* Now push the Valkey version information. */ + lua_pushstring(lua,"SERVER_VERSION_NUM"); + lua_pushnumber(lua,SERVER_VERSION_NUM); + lua_settable(lua,-3); + + lua_pushstring(lua,"SERVER_VERSION"); + lua_pushstring(lua,SERVER_VERSION); + lua_settable(lua,-3); + + lua_pushstring(lua,"SERVER_NAME"); + lua_pushstring(lua,SERVER_NAME); + lua_settable(lua,-3); } void luaRegisterLogFunction(lua_State* lua) { diff --git a/src/sentinel.c b/src/sentinel.c index 238be905f..56b5e2896 100644 --- a/src/sentinel.c +++ b/src/sentinel.c @@ -1846,8 +1846,7 @@ void loadSentinelConfigFromQueue(void) { return; loaderr: - fprintf(stderr, "\n*** FATAL CONFIG FILE ERROR (Redis %s) ***\n", - REDIS_VERSION); + fprintf(stderr, "\n*** FATAL CONFIG FILE ERROR (Version %s) ***\n", SERVER_VERSION); fprintf(stderr, "Reading the configuration file, at line %d\n", linenum); fprintf(stderr, ">>> '%s'\n", line); fprintf(stderr, "%s\n", err); diff --git a/src/server.c b/src/server.c index 10f10976b..8a636b6bb 100644 --- a/src/server.c +++ b/src/server.c @@ -5489,6 +5489,8 @@ sds genRedisInfoString(dict *section_dict, int all_sections, int everything) { info = sdscatfmt(info, "# Server\r\n" "redis_version:%s\r\n" + "server_name:%s\r\n", + "server_version:%s\r\n", "redis_git_sha1:%s\r\n" "redis_git_dirty:%i\r\n" "redis_build_id:%s\r\n" @@ -5513,6 +5515,8 @@ sds genRedisInfoString(dict *section_dict, int all_sections, int everything) { "config_file:%s\r\n" "io_threads_active:%i\r\n", REDIS_VERSION, + SERVER_NAME, + SERVER_VERSION, redisGitSHA1(), strtol(redisGitDirty(),NULL,10) > 0, redisBuildIdString(), @@ -6376,8 +6380,8 @@ void daemonize(void) { } void version(void) { - printf("Redis server v=%s sha=%s:%d malloc=%s bits=%d build=%llx\n", - REDIS_VERSION, + printf("Server v=%s sha=%s:%d malloc=%s bits=%d build=%llx\n", + SERVER_VERSION, redisGitSHA1(), atoi(redisGitDirty()) > 0, ZMALLOC_LIB, @@ -6431,7 +6435,7 @@ void redisAsciiArt(void) { ); } else { snprintf(buf,1024*16,ascii_logo, - REDIS_VERSION, + SERVER_VERSION, redisGitSHA1(), strtol(redisGitDirty(),NULL,10) > 0, (sizeof(long) == 8) ? "64" : "32", @@ -7281,9 +7285,8 @@ int main(int argc, char **argv) { if (background) daemonize(); serverLog(LL_NOTICE, "oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo"); - serverLog(LL_NOTICE, - "Redis version=%s, bits=%d, commit=%s, modified=%d, pid=%d, just started", - REDIS_VERSION, + serverLog(LL_NOTICE, SERVER_NAME " version=%s, bits=%d, commit=%s, modified=%d, pid=%d, just started", + SERVER_VERSION, (sizeof(long) == 8) ? 64 : 32, redisGitSHA1(), strtol(redisGitDirty(),NULL,10) > 0, diff --git a/src/version.h b/src/version.h index 7c6eea6a4..eaaaac2ec 100644 --- a/src/version.h +++ b/src/version.h @@ -1,2 +1,8 @@ +#define SERVER_NAME "valkey" +#define SERVER_VERSION "7.2.4" +#define SERVER_VERSION_NUM 0x00070204 + +/* Redis compatibility version, should never + * exceed 7.2.x. */ #define REDIS_VERSION "7.2.4" #define REDIS_VERSION_NUM 0x00070204 diff --git a/tests/unit/moduleapi/misc.tcl b/tests/unit/moduleapi/misc.tcl index cf205462e..2b76ee34e 100644 --- a/tests/unit/moduleapi/misc.tcl +++ b/tests/unit/moduleapi/misc.tcl @@ -21,8 +21,7 @@ start_server {overrides {save {900 1}} tags {"modules"}} { } test {test redis version} { - set version [s redis_version] - assert_equal $version [r test.redisversion] + assert_equal [s server_version] [r test.redisversion] } test {test long double conversions} {