Skip to content

Commit 95f49ea

Browse files
fristoniojoestringer
authored andcommitted
cli: ignore Link not Found errors in cilium connectivity tests check
This commit introduces the change to ignore `Link not Found` errors in cilium agent logs when running connectivity tests. The underlying issue was fixed in cilium version >= 1.18.0 but has not been backported to older stable release branches yet. cilium#39370 Signed-off-by: Deepesh Pathak <deepesh.pathak@isovalent.com>
1 parent b7a16fa commit 95f49ea

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

cilium-cli/connectivity/tests/errors.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,12 @@ func NoErrorsInLogs(ciliumVersion semver.Version, checkLevels []string, external
7171
envoyExternalTargetTLSWarning, envoyExternalOtherTargetTLSWarning, ciliumNodeConfigDeprecation,
7272
hubbleUIEnvVarFallback, k8sClientNetworkStatusError, bgpAlphaResourceDeprecation, ccgAlphaResourceDeprecation,
7373
k8sEndpointDeprecatedWarn, proxylibDeprecatedWarn}
74+
75+
if ciliumVersion.LT(semver.MustParse("1.18.0")) {
76+
errorLogExceptions = append(errorLogExceptions, linkNotFound)
77+
warningLogExceptions = append(warningLogExceptions, linkNotFound)
78+
}
79+
7480
// The list is adopted from cilium/cilium/test/helper/utils.go
7581
var errorMsgsWithExceptions = map[string][]logMatcher{
7682
panicMessage: nil,
@@ -479,4 +485,6 @@ var (
479485
bgpAlphaResourceDeprecation = regexMatcher{regexp.MustCompile(`cilium.io/v2alpha1 CiliumBGP\w+ is deprecated`)}
480486
// ccgAlphaResourceDeprecation is the same as bgpAlphaResourceDeprecation but for the CiliumCIDRGroup.
481487
ccgAlphaResourceDeprecation = regexMatcher{regexp.MustCompile(`cilium.io/v2alpha1 CiliumCIDRGroup is deprecated`)}
488+
// For https://github.com/cilium/cilium/issues/39370: Fixed only in cilium version >= 1.18
489+
linkNotFound = regexMatcher{regexp.MustCompile(`retrieving device .+\: Link not found`)}
482490
)

0 commit comments

Comments
 (0)