Refactor gameserverallocation to its components (#1015)

gameserverallocation will serve as an extension API that uses the common library for allocation. agones-allocator service will reference the same allocation library that is refactored out.
googleforgames · Aug 28, 2019 · 7b5e618 · 7b5e618
1 parent 9228d20
commit 7b5e618
Show file tree

Hide file tree

Showing 7 changed files with 1,121 additions and 887 deletions.
diff --git a/cmd/controller/main.go b/cmd/controller/main.go
@@ -71,9 +71,6 @@ const (
 	logSizeLimitMBFlag           = "log-size-limit-mb"
 	kubeconfigFlag               = "kubeconfig"
 	defaultResync                = 30 * time.Second
-	// topNGSForAllocation is used by the GameServerAllocation controller
-	// to reduce the contention while allocating gameservers.
-	topNGSForAllocation = 100
 )
 
 var (
@@ -193,8 +190,7 @@ func main() {
 	gsSetController := gameserversets.NewController(wh, health, gsCounter,
 		kubeClient, extClient, agonesClient, agonesInformerFactory)
 	fleetController := fleets.NewController(wh, health, kubeClient, extClient, agonesClient, agonesInformerFactory)
-	gasController := gameserverallocations.NewController(api, health, gsCounter, topNGSForAllocation,
-		kubeClient, kubeInformerFactory, agonesClient, agonesInformerFactory)
+	gasController := gameserverallocations.NewController(api, health, gsCounter, kubeClient, kubeInformerFactory, agonesClient, agonesInformerFactory)
 	fasController := fleetautoscalers.NewController(wh, health,
 		kubeClient, extClient, agonesClient, agonesInformerFactory)