Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

The pilot-discovery has compile error because the component of mixer #12

Closed
casionone opened this issue Aug 17, 2018 · 1 comment
Closed
Labels
Milestone

Comments

@casionone
Copy link

casionone commented Aug 17, 2018

the [pilot/pkg/networking/plugin/mixer.go] file ( https://github.com/alipay/sofa-mesh/blob/master/pilot/pkg/networking/plugin/mixer/mixer.go ) has error because of too many arguments in call to addServiceConfig.

the compile  error info :
 pilot\pkg\networking\plugin\mixer\mixer.go:178:45: too many arguments in call to addServiceConfig
	have (*model.Proxy, map[string]*types.Struct, *client.ServiceConfig)
	want (map[string]*types.Struct, *client.ServiceConfig)
pilot\pkg\networking\plugin\mixer\mixer.go:251:46: too many arguments in call to addServiceConfig
	have (*model.Proxy, map[string]*types.Struct, *client.ServiceConfig)
	want (map[string]*types.Struct, *client.ServiceConfig)
pilot\pkg\networking\plugin\mixer\mixer.go:260:48: too many arguments in call to addServiceConfig
	have (*model.Proxy, map[string]*types.Struct, *client.ServiceConfig)
	want (map[string]*types.Struct, *client.ServiceConfig)
pilot\pkg\networking\plugin\mixer\mixer.go:269:48: too many arguments in call to addServiceConfig
	have (*model.Proxy, map[string]*types.Struct, *client.ServiceConfig)
	want (map[string]*types.Struct, *client.ServiceConfig)
pilot\pkg\networking\plugin\mixer\mixer.go:355:17: undefined: node

@casionone casionone changed the title the pilot-discovey hava compile error because the component of mixer the pilot-discovery has compile error because the component of mixer Aug 17, 2018
@casionone casionone changed the title the pilot-discovery has compile error because the component of mixer The pilot-discovery has compile error because the component of mixer Aug 17, 2018
@junxiong
Copy link

PR #11 fixed this issue.

@junxiong junxiong added the kind/bug bug need to fix label Aug 17, 2018
@junxiong junxiong added this to the 0.2.0 milestone Aug 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants