Merge pull request #19449 from hongchaodeng/fitport

PodFitsHostPorts: small refactor and speedup

Merge pull request #19449 from hongchaodeng/fitport
f9f4b74a · Mike Danese · 41df8749 · 91fa5f8c · f9f4b74a
Commit f9f4b74a authored Jan 14, 2016 by Mike Danese
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

predicates.go plugin/pkg/scheduler/algorithm/predicates/predicates.go +4 -1

No files found.
--- a/plugin/pkg/scheduler/algorithm/predicates/predicates.go
+++ b/plugin/pkg/scheduler/algorithm/predicates/predicates.go
@@ -513,8 +513,11 @@ func (s *ServiceAffinity) CheckServiceAffinity(pod *api.Pod, existingPods []*api
 }

 func PodFitsHostPorts(pod *api.Pod, existingPods []*api.Pod, node string) (bool, error) {
-	existingPorts := getUsedPorts(existingPods...)
 	wantPorts := getUsedPorts(pod)
+	if len(wantPorts) == 0 {
+		return true, nil
+	}
+	existingPorts := getUsedPorts(existingPods...)
 	for wport := range wantPorts {
 		if wport == 0 {
 			continue