Skip to content

Commit

Permalink
Fix Red Hat os detection
Browse files Browse the repository at this point in the history
  • Loading branch information
Ecsy committed Oct 15, 2019
1 parent 736f857 commit 59cd61d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
11 changes: 8 additions & 3 deletions cmd/pke/app/util/linux/autodetect.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,16 @@ import (
)

func KubernetesPackagesImpl(out io.Writer) (KubernetesPackages, error) {
if ver, err := CentOSVersion(out); err == nil {
if ver == "7" {
ver, err := CentOSVersion(out)
if err != nil {
ver, err = RedHatVersion(out)
}
if err == nil {
v, _ := semver.NewVersion(ver)
c, _ := semver.NewConstraint("7.x-0 || 8.x-0")
if c.Check(v) {
return NewYumInstaller(), nil
}
return nil, constants.ErrUnsupportedOS
}

if distro, err := LSBReleaseDistributorID(out); err == nil {
Expand Down
2 changes: 2 additions & 0 deletions cmd/pke/app/util/linux/kernel.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@ func KernelVersionConstraint(out io.Writer, constraint string) error {
if err != nil {
return err
}
// Red Hat Linux uses underscore: 3.10.0-327.el7.x86_64
version = bytes.ReplaceAll(version, []byte("_"), []byte(""))
v := string(bytes.TrimSpace(version))
ver, err := semver.NewVersion(v)
if err != nil {
Expand Down

0 comments on commit 59cd61d

Please sign in to comment.