cgroupv1: do not run tests for version >= 4179

[ader1990]

Flatcar with systemd >= 256 does not support cgroupv1 anymore.

See: flatcar/scripts#2145

[tormath1]

Thanks, this can be done on the kubeadm test too:

mantle/kola/tests/kubeadm/kubeadm.go

Lines 148 to 166 in 28a2bfa

	cgroupSuffix := ""
	var major int64 = 0
	if testParams["cgroupv1"].(bool) {
	cgroupSuffix = ".cgroupv1"
	major = 3140
	}
	if CNI == "flannel" {
	flags = append(flags, register.NoEnableSelinux)
	}
	if mmvi, ok := testParams["MinMajorVersion"]; ok {
	mmv := (int64)(mmvi.(int))
	// Careful, so we don't lower
	// the min version too much.
	if mmv > major {
	major = mmv
	}
	}

[ader1990]

Thanks, this can be done on the kubeadm test too:

mantle/kola/tests/kubeadm/kubeadm.go

Lines 148 to 166 in 28a2bfa

	cgroupSuffix := ""
	var major int64 = 0
	if testParams["cgroupv1"].(bool) {
	cgroupSuffix = ".cgroupv1"
	major = 3140
	}
	if CNI == "flannel" {
	flags = append(flags, register.NoEnableSelinux)
	}
	if mmvi, ok := testParams["MinMajorVersion"]; ok {
	mmv := (int64)(mmvi.(int))
	// Careful, so we don't lower
	// the min version too much.
	if mmv > major {
	major = mmv
	}
	}

Yes, that needs to be addressed, but do not know where is the best place to compare with a max version aka EndVersion. I think it should be done in a unified way, only defined once and used accordingly.

[ader1990]

Hello @flatcar/flatcar-maintainers, I would like to have this PR reviewed, to unblock flatcar/scripts#2145

[tormath1]

I think the next alpha will be greater than 4152. So we might need to bump this?

[ader1990]

depends on when we merge the systemd PR. now (aka today), or after the next alpha gets released.

[ader1990]

I saw that the next alpha release will be 4179, let's get this merged after this next alpha release to have time for better functional testing.

[ader1990]

updated to 4179 for ample of time.