Skip to content

Commit 8b0294d

Browse files
committed
Fix typo and pass the environment variable required to enable watchcache consistency checking in GCE tests
1 parent 2e9bb32 commit 8b0294d

File tree

3 files changed

+9
-2
lines changed

3 files changed

+9
-2
lines changed

cluster/gce/gci/configure-kubeapiserver.sh

+6
Original file line numberDiff line numberDiff line change
@@ -348,6 +348,12 @@ function start-kube-apiserver {
348348
fi
349349
container_env+="{\"name\": \"KUBE_LIST_FROM_CACHE_INCONSISTENCY_DETECTOR\", \"value\": \"${ENABLE_KUBE_LIST_FROM_CACHE_INCONSISTENCY_DETECTOR}\"}"
350350
fi
351+
if [[ -n "${ENABLE_KUBE_WATCHCACHE_CONSISTENCY_CHECKER:-}" ]]; then
352+
if [[ -n "${container_env}" ]]; then
353+
container_env="${container_env}, "
354+
fi
355+
container_env+="{\"name\": \"KUBE_WATCHCACHE_CONSISTENCY_CHECKER\", \"value\": \"${ENABLE_KUBE_WATCHCACHE_CONSISTENCY_CHECKER}\"}"
356+
fi
351357
if [[ -n "${ENABLE_PATCH_CONVERSION_DETECTOR:-}" ]]; then
352358
if [[ -n "${container_env}" ]]; then
353359
container_env="${container_env}, "

cluster/gce/util.sh

+1
Original file line numberDiff line numberDiff line change
@@ -1197,6 +1197,7 @@ ENABLE_VOLUME_SNAPSHOTS: $(yaml-quote "${ENABLE_VOLUME_SNAPSHOTS:-}")
11971197
ENABLE_APISERVER_ADVANCED_AUDIT: $(yaml-quote "${ENABLE_APISERVER_ADVANCED_AUDIT:-}")
11981198
ENABLE_APISERVER_DYNAMIC_AUDIT: $(yaml-quote "${ENABLE_APISERVER_DYNAMIC_AUDIT:-}")
11991199
ENABLE_CACHE_MUTATION_DETECTOR: $(yaml-quote "${ENABLE_CACHE_MUTATION_DETECTOR:-false}")
1200+
ENABLE_KUBE_WATCHCACHE_CONSISTENCY_CHECKER: $(yaml-quote "${ENABLE_KUBE_WATCHCACHE_CONSISTENCY_CHECKER:-false}")
12001201
ENABLE_KUBE_WATCHLIST_INCONSISTENCY_DETECTOR: $(yaml-quote "${ENABLE_KUBE_WATCHLIST_INCONSISTENCY_DETECTOR:-false}")
12011202
ENABLE_KUBE_LIST_FROM_CACHE_INCONSISTENCY_DETECTOR: $(yaml-quote "${ENABLE_KUBE_LIST_FROM_CACHE_INCONSISTENCY_DETECTOR:-false}")
12021203
ENABLE_PATCH_CONVERSION_DETECTOR: $(yaml-quote "${ENABLE_PATCH_CONVERSION_DETECTOR:-false}")

staging/src/k8s.io/apiserver/pkg/storage/cacher/delegator.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,12 @@ var (
4949
// List latency SLO (30 seconds) and timeout (1 minute).
5050
ConsistencyCheckPeriod = 5 * time.Minute
5151
// ConsistencyCheckerEnabled enables the consistency checking mechanism for cache.
52-
// Based on KUBE_WATCHCACHE_CONSISTANCY_CHECKER environment variable.
52+
// Based on KUBE_WATCHCACHE_CONSISTENCY_CHECKER environment variable.
5353
ConsistencyCheckerEnabled = false
5454
)
5555

5656
func init() {
57-
ConsistencyCheckerEnabled, _ = strconv.ParseBool(os.Getenv("KUBE_WATCHCACHE_CONSISTANCY_CHECKER"))
57+
ConsistencyCheckerEnabled, _ = strconv.ParseBool(os.Getenv("KUBE_WATCHCACHE_CONSISTENCY_CHECKER"))
5858
}
5959

6060
func NewCacheDelegator(cacher *Cacher, storage storage.Interface) *CacheDelegator {

0 commit comments

Comments
 (0)