Skip to content

Commit dbeffa8

Browse files
Zijian Zhangjoestringer
authored andcommitted
tests: remove allowlist entry for deleting no longer present service
The warning log "Deleting no longer present service" was emitted because cilium-agent in versions <= 1.13 waited on the wrong Kubernetes API group during startup. Specifically, it waited for K8sAPIGroupEndpointSliceV1Beta1Discovery instead of K8sAPIGroupEndpointSliceV1Discovery, causing SyncWithK8sFinished to run before all services were fully synced. This resulted in services restored from the datapath being temporarily deleted and later re-added, even though they were still present and active. This behavior was unintentionally fixed in v1.14 by refactoring the Kubernetes watcher initialization and resource synchronization logic. As a result, this warning is no longer expected. Remove the corresponding allowlist entry. More details of analysis in cilium#29679. Fixes: cilium#29679 Signed-off-by: Zijian Zhang <zijianzhang@bytedance.com>
1 parent 8887aac commit dbeffa8

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

test/helpers/cons.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,6 @@ const (
195195
failedToReleaseLock = "Failed to release lock:"
196196
errorCreatingInitialLeader = "error initially creating leader election record:"
197197
cantEnableJIT = "bpf_jit_enable: no such file or directory" // Because we run tests in Kind.
198-
delMissingService = "Deleting no longer present service" // cf. https://github.com/cilium/cilium/issues/29679
199198
podCIDRUnavailable = " PodCIDR not available" // cf. https://github.com/cilium/cilium/issues/29680
200199
unableGetNode = "Unable to get node resource" // cf. https://github.com/cilium/cilium/issues/29710
201200
objectHasBeenModified = "the object has been modified; please apply your changes" // cf. https://github.com/cilium/cilium/issues/29712
@@ -255,8 +254,8 @@ var badLogMessages = map[string][]string{
255254
logutils.ErrorLogs: {opCantBeFulfilled, initLeaderElection, globalDataSupport,
256255
failedToListCRDs, retrieveResLock, failedToRelLockEmptyName, failedToUpdateLock,
257256
failedToReleaseLock, errorCreatingInitialLeader},
258-
logutils.WarningLogs: {cantEnableJIT, delMissingService, podCIDRUnavailable,
259-
unableGetNode, objectHasBeenModified, etcdTimeout, endpointRestoreFailed,
257+
logutils.WarningLogs: {cantEnableJIT, podCIDRUnavailable, unableGetNode,
258+
objectHasBeenModified, etcdTimeout, endpointRestoreFailed,
260259
cantFindIdentityInCache, keyAllocFailedFoundMaster, cantRecreateMasterKey,
261260
cantUpdateCRDIdentity, cantDeleteFromPolicyMap, failedToListCRDs, mutationDetector},
262261
}

0 commit comments

Comments
 (0)