github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-01 07:47:56 +00:00
Code Issues Projects Releases Wiki Activity

Fix onlylocal endpoint's healthcheck nodeport logic

Browse Source
This commit is contained in:
Zihong Zheng 2017-05-01 20:36:44 -07:00
parent 3ec39c35bf
commit 1cac5db856
3 changed files with 3 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/api/service/util.go
View File

@ -105,9 +105,6 @@ func NeedsHealthCheck(service *api.Service) bool {
// GetServiceHealthCheckNodePort Return health check node port annotation for service, if one exists // GetServiceHealthCheckNodePort Return health check node port annotation for service, if one exists
func GetServiceHealthCheckNodePort(service *api.Service) int32 { func GetServiceHealthCheckNodePort(service *api.Service) int32 {
if !NeedsHealthCheck(service) {
return 0
}
// First check the alpha annotation and then the beta. This is so existing // First check the alpha annotation and then the beta. This is so existing
// Services continue to work till the user decides to transition to beta. // Services continue to work till the user decides to transition to beta.
// If they transition to beta, there's no way to go back to alpha without // If they transition to beta, there's no way to go back to alpha without

3
pkg/api/v1/service/util.go
View File

@ -105,9 +105,6 @@ func NeedsHealthCheck(service *v1.Service) bool {
// GetServiceHealthCheckNodePort Return health check node port annotation for service, if one exists // GetServiceHealthCheckNodePort Return health check node port annotation for service, if one exists
func GetServiceHealthCheckNodePort(service *v1.Service) int32 { func GetServiceHealthCheckNodePort(service *v1.Service) int32 {
if !NeedsHealthCheck(service) {
return 0
}
// First check the alpha annotation and then the beta. This is so existing // First check the alpha annotation and then the beta. This is so existing
// Services continue to work till the user decides to transition to beta. // Services continue to work till the user decides to transition to beta.
// If they transition to beta, there's no way to go back to alpha without // If they transition to beta, there's no way to go back to alpha without

13
pkg/proxy/iptables/proxier.go
View File

@ -181,11 +181,10 @@ func newServiceInfo(serviceName proxy.ServicePortName, port *api.ServicePort, se
copy(info.loadBalancerSourceRanges, service.Spec.LoadBalancerSourceRanges) copy(info.loadBalancerSourceRanges, service.Spec.LoadBalancerSourceRanges)
copy(info.externalIPs, service.Spec.ExternalIPs) copy(info.externalIPs, service.Spec.ExternalIPs)
if info.onlyNodeLocalEndpoints { if apiservice.NeedsHealthCheck(service) {
p := apiservice.GetServiceHealthCheckNodePort(service) p := apiservice.GetServiceHealthCheckNodePort(service)
if p == 0 { if p == 0 {
glog.Errorf("Service does not contain necessary annotation %v", glog.Errorf("Service %q has no healthcheck nodeport", serviceName)
apiservice.BetaAnnotationHealthCheckNodePort)
} else { } else {
info.healthCheckNodePort = int(p) info.healthCheckNodePort = int(p)
} }
@ -644,16 +643,10 @@ func updateServiceMap(
// computing this incrementally similarly to serviceMap. // computing this incrementally similarly to serviceMap.
hcServices = make(map[types.NamespacedName]uint16) hcServices = make(map[types.NamespacedName]uint16)
for svcPort, info := range serviceMap { for svcPort, info := range serviceMap {
if info.onlyNodeLocalEndpoints { if info.healthCheckNodePort != 0 {
hcServices[svcPort.NamespacedName] = uint16(info.healthCheckNodePort) hcServices[svcPort.NamespacedName] = uint16(info.healthCheckNodePort)
} }
} }
for nsn, port := range hcServices {
if port == 0 {
glog.Errorf("Service %q has no healthcheck nodeport", nsn)
delete(hcServices, nsn)
}
}
return syncRequired, hcServices, staleServices return syncRequired, hcServices, staleServices
} }