From d3c6548214a21fbf105022512f621c4278493416 Mon Sep 17 00:00:00 2001 From: nexustar Date: Thu, 17 Nov 2022 19:31:55 +0800 Subject: [PATCH] cluster: mark kylin v10 as supported OS --- pkg/cluster/operation/check.go | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/pkg/cluster/operation/check.go b/pkg/cluster/operation/check.go index 44c81ea13f..1edd6b02d3 100644 --- a/pkg/cluster/operation/check.go +++ b/pkg/cluster/operation/check.go @@ -168,13 +168,10 @@ func checkOSInfo(opt *CheckOptions, osInfo *sysinfo.OS) *CheckResult { // check OS vendor switch osInfo.Vendor { case "kylin": - msg := "kylin support is not fully tested, be careful" - result.Err = fmt.Errorf("%s (%s)", result.Msg, msg) - result.Warn = true // VERSION_ID="V10" if ver, _ := strconv.ParseFloat(strings.Trim(osInfo.Version, "V"), 64); ver < 10 { - result.Err = fmt.Errorf("%s %s not supported, use version V10 or higher(%s)", - osInfo.Name, osInfo.Release, msg) + result.Err = fmt.Errorf("%s %s not supported, use version V10 or higher", + osInfo.Name, osInfo.Release) return result } case "amzn": @@ -187,8 +184,6 @@ func checkOSInfo(opt *CheckOptions, osInfo *sysinfo.OS) *CheckResult { } case "centos", "redhat", "rhel", "ol": // check version - // CentOS 8 is known to be not working, and we don't have plan to support it - // as of now, we may add support for RHEL 8 based systems in the future. if ver, _ := strconv.ParseFloat(osInfo.Version, 64); ver < 7 { result.Err = fmt.Errorf("%s %s not supported, use version 8 please", osInfo.Name, osInfo.Release)