fix(just): exclude noisy connectivity-test checks from smoke-cilium#215
Merged
Conversation
After #212 made `cilium connectivity test` runnable, `just smoke` still exited 1 on two checks that are unreliable on this homelab: - no-unexpected-packet-drops trips on ambient VLAN-tagged LAN traffic that Cilium drops by design ("VLAN traffic disallowed by VLAN filter") — not cluster-pod traffic, and the counter is cumulative. - check-log-errors scans the full agent log and re-flags benign agent-startup transients (e.g. a one-off cilium-health socket poll racing agent start). Exclude both via `--test '!...'` so `just smoke` is a clean pass/fail gate. A full conformance pass remains available by running `cilium connectivity test` directly without the excludes. Closes: #214 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This was referenced May 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
#212 made
smoke-ciliumable to runcilium connectivity test(test namespaces are now pre-createdprivileged). The test now runs the full matrix, butjust smokestill exits 1 — two of cilium-cli's checks fail on environmental noise rather than real defects.This excludes both via
--test '!...':Why these two
no-unexpected-packet-drops— flagscilium_drop_count_totalwith reasonVLAN traffic disallowed by VLAN filter(INGRESS). That's ambient VLAN-tagged traffic on the LAN that Cilium drops by design, not cluster-pod traffic. The counter is lifetime-cumulative, so the check trips on essentially every run.check-log-errors— scans the full agent log and re-flags benign agent-startup transients (e.g. a one-offcilium-healthsocket poll racing agent start) for as long as that line stays in the log.Both surfaced verifying the Cilium 1.19.4 upgrade (#198): 78/80 connectivity tests passed — pod-to-pod, services, DNS, network policy all green — and the only 2 failures were these noise checks. A full conformance pass remains available by running
cilium connectivity testdirectly without the excludes.Testing
just --liststill renderssmoke-ciliumwith its one-line description.just smokerun against therockinghamcluster to confirm a clean exit 0 — to be done after merge.Closes: #214
Refs: #211, #198
🤖 Generated with Claude Code