-
Notifications
You must be signed in to change notification settings - Fork 299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Condense backendPool and defaultBackendPool #242
Condense backendPool and defaultBackendPool #242
Conversation
pkg/loadbalancers/l7s.go
Outdated
return &L7s{cloud, storage.NewInMemoryPool(), nil, defaultBackendPool, defaultBackendNodePort, namer} | ||
// - backendPool: Used to sync BackendServices with the cloud. | ||
func NewLoadBalancerPool(cloud LoadBalancers, backendPool backends.BackendPool, namer *utils.Namer) LoadBalancerPool { | ||
return &L7s{cloud, storage.NewInMemoryPool(), backendPool, namer} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My goal was that the load balancer pool doesn't need to have a reference to the backendpool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, gonna work on this more to see if I can remove it from LB pool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to remove the backend pool reference from the load balancer pool. This required some semi-significant refactoring but I think everything is a lot cleaner now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the tests files just for clarity.
582717e
to
c1dfbbd
Compare
044c947
to
468956c
Compare
468956c
to
1fc6b2d
Compare
1fc6b2d
to
753e1f3
Compare
Test panicked. |
753e1f3
to
f368eb5
Compare
/lgtm |
This PR attempts to condense management of all backends into one backend pool. Users of the condensed backend pool can now access the default backend's NodePort through a method in the BackendPool interface.
/assign @nicksardo
Should fix #184 & #127