Commit f6b4ed33 authored by 蔡宇东's avatar 蔡宇东
Browse files

#1463 enable config version check 



Signed-off-by: default avataryudong.cai <yudong.cai@zilliz.com>
parent 600dac75

core/src/server/Config.cpp

+1 −1
Original line number Diff line number Diff line
@@ -617,7 +617,7 @@ Config::CheckConfigVersion(const std::string& value) { 
            std::string msg = "Invalid config version: " + value +

                              ". Expected config version: " + milvus_config_version_map.at(MILVUS_VERSION);

            SERVER_LOG_ERROR << msg;

            // return Status(SERVER_INVALID_ARGUMENT, msg);

            return Status(SERVER_INVALID_ARGUMENT, msg);

        }

    }

    return Status::OK();

core/unittest/server/test_config.cpp

+4 −4
Original line number Diff line number Diff line
@@ -650,10 +650,10 @@ TEST_F(ConfigTest, SERVER_CONFIG_VALID_FAIL_TEST) { 
    milvus::Status s = config.LoadConfigFile(config_path + VALID_CONFIG_FILE);

    ASSERT_TRUE(s.ok());



    // fiu_enable("check_config_version_fail", 1, NULL, 0);

    // s = config.ValidateConfig();

    // ASSERT_FALSE(s.ok());

    // fiu_disable("check_config_version_fail");

    fiu_enable("check_config_version_fail", 1, NULL, 0);

    s = config.ValidateConfig();

    ASSERT_FALSE(s.ok());

    fiu_disable("check_config_version_fail");



    /* server config */

    fiu_enable("check_config_address_fail", 1, NULL, 0);