github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-28 14:07:14 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #104077 from jsafrane/fix-skip-schedulable

Browse Source 
Fix zone calculation - consider only untainted nodes
This commit is contained in:
Kubernetes Prow Robot 2021-08-16 10:03:29 -07:00 committed by GitHub
commit ca31857c55
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
test/e2e/framework/node/resource.go
View File

@ -563,18 +563,15 @@ func GetClusterZones(c clientset.Interface) (sets.String, error) {
// GetSchedulableClusterZones returns the values of zone label collected from all nodes which are schedulable. // GetSchedulableClusterZones returns the values of zone label collected from all nodes which are schedulable.
func GetSchedulableClusterZones(c clientset.Interface) (sets.String, error) { func GetSchedulableClusterZones(c clientset.Interface) (sets.String, error) {
nodes, err := c.CoreV1().Nodes().List(context.TODO(), metav1.ListOptions{}) // GetReadySchedulableNodes already filters our tainted and unschedulable nodes.
nodes, err := GetReadySchedulableNodes(c)
if err != nil { if err != nil {
return nil, fmt.Errorf("Error getting nodes while attempting to list cluster zones: %v", err) return nil, fmt.Errorf("error getting nodes while attempting to list cluster zones: %v", err)
} }
// collect values of zone label from all nodes // collect values of zone label from all nodes
zones := sets.NewString() zones := sets.NewString()
for _, node := range nodes.Items { for _, node := range nodes.Items {
// We should have at least 1 node in the zone which is schedulable.
if !IsNodeSchedulable(&node) {
continue
}
if zone, found := node.Labels[v1.LabelFailureDomainBetaZone]; found { if zone, found := node.Labels[v1.LabelFailureDomainBetaZone]; found {
zones.Insert(zone) zones.Insert(zone)
} }