Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check system info throws an error “os vendor rhel not supported” #1332

Closed
7yyo opened this issue Apr 27, 2021 · 2 comments · Fixed by #1336
Closed

Check system info throws an error “os vendor rhel not supported” #1332

7yyo opened this issue Apr 27, 2021 · 2 comments · Fixed by #1336
Assignees
Labels
component/cluster Issues about the tiup-cluster component type/bug Categorizes issue as related to a bug.

Comments

@7yyo
Copy link

7yyo commented Apr 27, 2021

Bug Report

Please answer these questions before submitting your issue. Thanks!

  1. What did you do?

run tiup check

  1. What did you expect to see?

run success.

  1. What did you see instead?

os vendor rhel not supported

  1. What version of TiUP are you using (tiup --version)?

I think it should be rhel ?

case "centos", "redhat":

switch si.OS.Vendor {
	case "debian":
		si.OS.Release = slurpFile("/etc/debian_version")
	case "ubuntu":
		if m := reUbuntu.FindStringSubmatch(si.OS.Name); m != nil {
			si.OS.Release = m[1]
		}
	case "centos":
		if release := slurpFile("/etc/centos-release"); release != "" {
			if m := reCentOS.FindStringSubmatch(release); m != nil {
				si.OS.Release = m[2]
			}
		}
	case "rhel":
		if release := slurpFile("/etc/redhat-release"); release != "" {
			if m := reRHEL.FindStringSubmatch(release); m != nil {
				si.OS.Release = m[1] + " " + m[2]
			}
		}
	}
@7yyo 7yyo added the type/bug Categorizes issue as related to a bug. label Apr 27, 2021
@AstroProfundis AstroProfundis added the component/cluster Issues about the tiup-cluster component label Apr 27, 2021
@AstroProfundis AstroProfundis self-assigned this Apr 27, 2021
@AstroProfundis
Copy link
Contributor

AstroProfundis commented Apr 27, 2021

Thanks for the report, could you also share with us the content of your /etc/os-release and /etc/redhat-release?

@7yyo
Copy link
Author

7yyo commented Apr 28, 2021

Thanks for the report, could you also share with us the content of your /etc/os-release and /etc/redhat-release?

Red Hat Enterprise Linux Server release 7.9 (Maipo)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/cluster Issues about the tiup-cluster component type/bug Categorizes issue as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants