Skip to content

Commit 3b75547

Browse files
committed
test(e2e): skip config-skyhook glob update step per #245
The `glob update` step in config-skyhook/chainsaw-test.yaml is the canonical reproducer for issue #245 (Owned ConfigMap can permanently desync from spec when a stale Status.ConfigUpdates entry drives a spurious interrupt cycle). The operator gets exactly one good reconcile window per spec change to satisfy the CM-write gate, and timing on kind regularly misses it. The issue itself documents this CI workaround: "Skip the `glob update` step in config-skyhook/chainsaw-test.yaml with a # TODO(<this-issue>) comment until the fix lands." This is unrelated to PR #258's CLI changes; the test flake was previously masked by other failures in the same file and surfaced after PR #242 closed the StageInterrupt trap. Signed-off-by: Alex Yuskauskas <ayuskauskas@nvidia.com>
1 parent 5c7f7cb commit 3b75547

1 file changed

Lines changed: 16 additions & 10 deletions

File tree

k8s-tests/chainsaw/skyhook/config-skyhook/chainsaw-test.yaml

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -385,13 +385,19 @@ spec:
385385
## parallel: configmaps are distinct steady-state resources
386386
- assert:
387387
file: assert-cm-update-no-interrupt.yaml
388-
- name: glob update
389-
description: Apply a config update using glob-based package selection and assert it completes
390-
try:
391-
- apply:
392-
file: update-glob.yaml
393-
## parallel: all docs checked simultaneously — the ConfigMap data ("changed via glob")
394-
## acts as a natural discriminator preventing matches against the previous step's state
395-
- assert:
396-
timeout: 270s
397-
file: assert-update-glob.yaml
388+
# TODO(#245): re-enable once HandleConfigUpdates CM-write gate is fixed.
389+
# The `glob update` step is the canonical reproducer for issue #245
390+
# (Owned ConfigMap can permanently desync from spec when a stale
391+
# Status.ConfigUpdates entry drives a spurious interrupt cycle). It flakes
392+
# in CI because the operator gets exactly one good reconcile window per
393+
# spec change to write the owned CM, and timing on kind can miss it.
394+
# - name: glob update
395+
# description: Apply a config update using glob-based package selection and assert it completes
396+
# try:
397+
# - apply:
398+
# file: update-glob.yaml
399+
# ## parallel: all docs checked simultaneously — the ConfigMap data ("changed via glob")
400+
# ## acts as a natural discriminator preventing matches against the previous step's state
401+
# - assert:
402+
# timeout: 270s
403+
# file: assert-update-glob.yaml

0 commit comments

Comments
 (0)