Register plugins with the scheduling framework properly

y-taka-23 · y-taka-23 · commit ec6a9881a48e · 2018-12-28T19:10:01.000+09:00
diff --git a/pkg/scheduler/plugins/registrar.go b/pkg/scheduler/plugins/registrar.go
@@ -58,7 +58,7 @@ func NewDefaultPluginSet(ctx *plugins.PluginContext, schedulerCache *cache.Cache
 	return &defaultRegistrar
 }
 
-func (r DefaultPluginSet) registerReservePlugins() {
+func (r *DefaultPluginSet) registerReservePlugins() {
 	r.reservePlugins = []plugins.ReservePlugin{
 		// Init functions of all reserve plugins go here. They are called in the
 		// same order that they are registered.
@@ -67,7 +67,7 @@ func (r DefaultPluginSet) registerReservePlugins() {
 	}
 }
 
-func (r DefaultPluginSet) registerPrebindPlugins() {
+func (r *DefaultPluginSet) registerPrebindPlugins() {
 	r.prebindPlugins = []plugins.PrebindPlugin{
 		// Init functions of all prebind plugins go here. They are called in the
 		// same order that they are registered.

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ func NewDefaultPluginSet(ctx plugins.PluginContext, schedulerCache cache.Cache`
`58`	`58`	`return &defaultRegistrar`
`59`	`59`	`}`
`60`	`60`
`61`		`-func (r DefaultPluginSet) registerReservePlugins() {`
	`61`	`+func (r *DefaultPluginSet) registerReservePlugins() {`
`62`	`62`	`r.reservePlugins = []plugins.ReservePlugin{`
`63`	`63`	`// Init functions of all reserve plugins go here. They are called in the`
`64`	`64`	`// same order that they are registered.`
`@@ -67,7 +67,7 @@ func (r DefaultPluginSet) registerReservePlugins() {`
`67`	`67`	`}`
`68`	`68`	`}`
`69`	`69`
`70`		`-func (r DefaultPluginSet) registerPrebindPlugins() {`
	`70`	`+func (r *DefaultPluginSet) registerPrebindPlugins() {`
`71`	`71`	`r.prebindPlugins = []plugins.PrebindPlugin{`
`72`	`72`	`// Init functions of all prebind plugins go here. They are called in the`
`73`	`73`	`// same order that they are registered.`