Skip to content

Commit

Permalink
kubeadm: skip calico / cilium tests for now on brightbox
Browse files Browse the repository at this point in the history
once we switch to butane, we can enable them.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
  • Loading branch information
tormath1 committed Nov 21, 2023
1 parent 1fd7edc commit bced13d
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions kola/tests/kubeadm/kubeadm.go
Original file line number Diff line number Diff line change
Expand Up @@ -224,6 +224,8 @@ func init() {
}
}

cni := CNI

register.Register(&register.Test{
Name: fmt.Sprintf("kubeadm.%s.%s%s.base", version, CNI, cgroupSuffix),
Distros: []string{"cl"},
Expand All @@ -236,10 +238,14 @@ func init() {
MinVersion: semver.Version{Major: major},
Flags: flags,
SkipFunc: func(version semver.Version, channel, arch, platform string) bool {
// LTS (3033) does not have the network-kargs service pulled in:
// * LTS (3033) does not have the network-kargs service pulled in:
// https://github.com/flatcar/coreos-overlay/pull/1848/commits/9e04bc12c3c7eb38da05173dc0ff7beaefa13446
// Let's skip this test for < 3034 on ESX.
return version.LessThan(semver.Version{Major: 3034}) && platform == "esx"
// Let's skip this test for < 3034 on ESX
// * For Cilium Calico/CNI on Brightbox:
// unprocessable_entity: User data is too long (maximum is 16384 characters)
// Should be reenabled once we switch to Butane provisioning because of internal compression.
return (version.LessThan(semver.Version{Major: 3034}) && platform == "esx") ||
(platform == "brightbox" && (cni == "cilium" || cni == "calico"))
},
})
}
Expand Down

0 comments on commit bced13d

Please sign in to comment.