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

Remove support for CentOS 8 #12877

Merged
merged 1 commit into from Dec 3, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/releases/1.23-NOTES.md
Expand Up @@ -14,6 +14,8 @@ This is a document to gather the release notes prior to the release.

* Support for the Lyft CNI has been removed.

* Support for CentOS 8 has been removed.

* Cilium now has `disable-cnp-status-updates: true` by default. Set this to false if you rely on the CiliumNetworkPolicy status fields.

# Required actions
Expand All @@ -26,8 +28,6 @@ This is a document to gather the release notes prior to the release.

* Support for CentOS 7 is deprecated and will be removed in future versions of kOps.

* Support for CentOS 8 is deprecated and will be removed in future versions of kOps.

* Support for Debian 9 (Stretch) is deprecated and will be removed in future versions of kOps.

* Support for RHEL 7 is deprecated and will be removed in future versions of kOps.
Expand Down
2 changes: 1 addition & 1 deletion upup/pkg/fi/nodeup/nodetasks/package.go
Expand Up @@ -333,7 +333,7 @@ func (_ *Package) RenderLocal(t *local.LocalTarget, a, e, changes *Package) erro
args = []string{"apt-get", "install", "--yes", "--no-install-recommends"}
env = append(env, "DEBIAN_FRONTEND=noninteractive")
} else if d.IsRHELFamily() {
if d == distributions.DistributionCentos8 || d == distributions.DistributionRhel8 {
if d == distributions.DistributionRhel8 {
args = []string{"/usr/bin/dnf", "install", "-y", "--setopt=install_weak_deps=False"}
} else {
args = []string{"/usr/bin/yum", "install", "-y"}
Expand Down
1 change: 0 additions & 1 deletion util/pkg/distributions/distributions.go
Expand Up @@ -51,7 +51,6 @@ var (
DistributionRhel7 = Distribution{packageFormat: "rpm", project: "rhel", id: "rhel7", version: 7}
DistributionCentos7 = Distribution{packageFormat: "rpm", project: "centos", id: "centos7", version: 7}
DistributionRhel8 = Distribution{packageFormat: "rpm", project: "rhel", id: "rhel8", version: 8}
DistributionCentos8 = Distribution{packageFormat: "rpm", project: "centos", id: "centos8", version: 8}
DistributionFlatcar = Distribution{packageFormat: "", project: "flatcar", id: "flatcar", version: 0}
DistributionContainerOS = Distribution{packageFormat: "", project: "containeros", id: "containeros", version: 0}
)
Expand Down
2 changes: 0 additions & 2 deletions util/pkg/distributions/identify.go
Expand Up @@ -52,8 +52,6 @@ func FindDistribution(rootfs string) (Distribution, error) {
return DistributionAmazonLinux2, nil
case "centos-7":
return DistributionCentos7, nil
case "centos-8":
return DistributionCentos8, nil
case "debian-9":
return DistributionDebian9, nil
case "debian-10":
Expand Down
4 changes: 2 additions & 2 deletions util/pkg/distributions/identify_test.go
Expand Up @@ -41,8 +41,8 @@ func TestFindDistribution(t *testing.T) {
},
{
rootfs: "centos8",
err: nil,
expected: DistributionCentos8,
err: fmt.Errorf("unsupported distro: centos-8"),
expected: Distribution{},
},
{
rootfs: "coreos",
Expand Down