Skip to content

Commit

Permalink
Reduce ambiguity on nodePort
Browse files Browse the repository at this point in the history
At first sight, nodePort looks a lot like service.Spec.NodePort, though it just means a port allocated from "--service-node-port-range". Replacing "nodePort" with "node port" to add some clarity.
  • Loading branch information
Fangyuan Li committed Feb 10, 2020
1 parent eddbb00 commit c196466
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -127,10 +127,10 @@ IP and may cause a second hop to another node, but should have good overall
load-spreading. Local preserves the client source IP and avoids a second hop
for LoadBalancer and NodePort type services, but risks potentially imbalanced
traffic spreading.
* `service.spec.healthCheckNodePort` - specifies the health check nodePort
* `service.spec.healthCheckNodePort` - specifies the health check node port
(numeric port number) for the service. If not specified, `healthCheckNodePort` is
created by the service API backend with the allocated `nodePort`. It will use the
user-specified `nodePort` value if specified by the client. It only has an
created by the service API backend with the port allocated from range specified by `--service-node-port-range` in API server. It will use the
user-specified `healthCheckNodePort` value if specified by the client. It only has an
effect when `type` is set to LoadBalancer and `externalTrafficPolicy` is set
to Local.

Expand Down

0 comments on commit c196466

Please sign in to comment.