Skip to content

Commit 0847d07

Browse files
authored
Merge pull request #5064 from aroradaman/design-update
KEP-784 Design update
2 parents 464d254 + e4bd30e commit 0847d07

File tree

1 file changed

+1
-5
lines changed
  • keps/sig-network/784-kube-proxy-component-config

1 file changed

+1
-5
lines changed

keps/sig-network/784-kube-proxy-component-config/README.md

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ This resulted in a configuration format, that had various different options grou
7777
- Backend agnostic options are marked as backed specific options (eg. syncPeriod, minSyncPeriod).
7878
- Options specific to a backend are used by other backends (eg. masqueradeBit and masqueradeAll).
7979

80-
[kubernetes/issues/117909](https://github.com/kubernetes/kubernetes/issues/117909) captures all the mis configurations in details.
80+
[kubernetes/issues/117909](https://github.com/kubernetes/kubernetes/issues/117909) captures all the misconfigurations in details.
8181

8282
Clearly, this made the configuration both hard to use and to maintain. Therefore, a plan to restructure and stabilize the config format is needed.
8383

@@ -157,8 +157,6 @@ The mitigations to those risks:
157157
| v1alpha1 | v1alpha2 | DataType | Comments |
158158
|--------------------|--------------------------|--------------|----------------------------------------------------------------------------------------------------------------|
159159
| ClusterCIDR | DetectLocal.ClusterCIDRs | list[string] | list of CIDR ranges for detecting local traffic |
160-
| MetricsBindAddress | MetricsBindAddresses | list[string] | list of CIDR ranges that contain valid node IPs to expose metrics server, instead of host port(ip:port) format |
161-
| HealthzBindAddress | HealthzBindAddresses | list[string] | list of CIDR ranges that contain valid node IPs to expose healthz server, instead of host port(ip:port) format | |
162160

163161
### Following fields will be added
164162
| Field | DataType | Default Value | Comments |
@@ -168,8 +166,6 @@ The mitigations to those risks:
168166
| ConfigHardFail | boolean | true | if set to true, kube-proxy will exit rather than just warning on config errors |
169167
| NodeIPOverride | list[string] | | list of primary node IPs |
170168
| IPFamilyPolicy | string | | controls nodeIP(s) detection, allowed values: [`SingleStack` \| `PreferDualStack` \| `RequireDualStack`] |
171-
| MetricsBindPort | integer (32-bit) | 10249 | port on which metrics server will be exposed |
172-
| HealthzBindPort | integer (32-bit) | 10256 | port on which helathz server will be exposed |
173169

174170
### Following fields will have different default values
175171
| Field | v1alpha1 (default) | v1alpha2 (default) |

0 commit comments

Comments
 (0)