Skip to content

Commit

Permalink
kubelet: improve registryPullQPS and registryBurst
Browse files Browse the repository at this point in the history
Handlebars built-in IF helper treats "0" (zero) as false. We need
use `includeZero` flag to make sure IF HELPER treats "0" (zero) as
true, so "0" (zero) can be passed to registryPullQPS and registryBurst.
  • Loading branch information
gthao313 committed Apr 30, 2021
1 parent f65be1c commit a9d7890
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions packages/kubernetes-1.16/kubelet-config
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ evictionHard:
{{~#if settings.kubernetes.allowed-unsafe-sysctls}}
allowedUnsafeSysctls: {{settings.kubernetes.allowed-unsafe-sysctls}}
{{~/if}}
{{~#if settings.kubernetes.registry-qps}}
{{~#if settings.kubernetes.registry-qps includeZero=true}}
registryPullQPS: {{settings.kubernetes.registry-qps}}
{{~/if}}
{{~#if settings.kubernetes.registry-burst}}
{{~#if settings.kubernetes.registry-burst includeZero=true}}
registryBurst: {{settings.kubernetes.registry-burst}}
{{~/if}}
{{~#if settings.kubernetes.event-qps includeZero=true}}
Expand Down
4 changes: 2 additions & 2 deletions packages/kubernetes-1.17/kubelet-config
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ evictionHard:
{{~#if settings.kubernetes.allowed-unsafe-sysctls}}
allowedUnsafeSysctls: {{settings.kubernetes.allowed-unsafe-sysctls}}
{{~/if}}
{{~#if settings.kubernetes.registry-qps}}
{{~#if settings.kubernetes.registry-qps includeZero=true}}
registryPullQPS: {{settings.kubernetes.registry-qps}}
{{~/if}}
{{~#if settings.kubernetes.registry-burst}}
{{~#if settings.kubernetes.registry-burst includeZero=true}}
registryBurst: {{settings.kubernetes.registry-burst}}
{{~/if}}
{{~#if settings.kubernetes.event-qps includeZero=true}}
Expand Down
4 changes: 2 additions & 2 deletions packages/kubernetes-1.18/kubelet-config
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ evictionHard:
{{~#if settings.kubernetes.allowed-unsafe-sysctls}}
allowedUnsafeSysctls: {{settings.kubernetes.allowed-unsafe-sysctls}}
{{~/if}}
{{~#if settings.kubernetes.registry-qps}}
{{~#if settings.kubernetes.registry-qps includeZero=true}}
registryPullQPS: {{settings.kubernetes.registry-qps}}
{{~/if}}
{{~#if settings.kubernetes.registry-burst}}
{{~#if settings.kubernetes.registry-burst includeZero=true}}
registryBurst: {{settings.kubernetes.registry-burst}}
{{~/if}}
{{~#if settings.kubernetes.event-qps includeZero=true}}
Expand Down
4 changes: 2 additions & 2 deletions packages/kubernetes-1.19/kubelet-config
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ evictionHard:
{{~#if settings.kubernetes.allowed-unsafe-sysctls}}
allowedUnsafeSysctls: {{settings.kubernetes.allowed-unsafe-sysctls}}
{{~/if}}
{{~#if settings.kubernetes.registry-qps}}
{{~#if settings.kubernetes.registry-qps includeZero=true}}
registryPullQPS: {{settings.kubernetes.registry-qps}}
{{~/if}}
{{~#if settings.kubernetes.registry-burst}}
{{~#if settings.kubernetes.registry-burst includeZero=true}}
registryBurst: {{settings.kubernetes.registry-burst}}
{{~/if}}
{{~#if settings.kubernetes.event-qps includeZero=true}}
Expand Down
4 changes: 2 additions & 2 deletions packages/kubernetes-1.20/kubelet-config
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ evictionHard:
{{~#if settings.kubernetes.allowed-unsafe-sysctls}}
allowedUnsafeSysctls: {{settings.kubernetes.allowed-unsafe-sysctls}}
{{~/if}}
{{~#if settings.kubernetes.registry-qps}}
{{~#if settings.kubernetes.registry-qps includeZero=true}}
registryPullQPS: {{settings.kubernetes.registry-qps}}
{{~/if}}
{{~#if settings.kubernetes.registry-burst}}
{{~#if settings.kubernetes.registry-burst includeZero=true}}
registryBurst: {{settings.kubernetes.registry-burst}}
{{~/if}}
{{~#if settings.kubernetes.event-qps includeZero=true}}
Expand Down

0 comments on commit a9d7890

Please sign in to comment.