Skip to content

Commit

Permalink
test: add exception for regen failure due to identity change
Browse files Browse the repository at this point in the history
It is possible for an endpoint's identity to change during policy
calculation. If this happens, we need to error-out and try again.
Unfortunately we don't have a good way to prevent this error message,
since it is logged far away from the context.

So, sadly, we have to ignore it.

Signed-off-by: Casey Callendrello <cdc@isovalent.com>
  • Loading branch information
squeed authored and nebril committed Aug 10, 2023
1 parent 9b4fdb7 commit b5900f5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/helpers/cons.go
Original file line number Diff line number Diff line change
Expand Up @@ -243,6 +243,7 @@ const (
failedToUpdateLock = "Failed to update lock:"
failedToReleaseLock = "Failed to release lock:"
errorCreatingInitialLeader = "error initially creating leader election record:"
secIdentityChanged = "SecurityIdentity changed during policy regeneration"

// HelmTemplate is the location of the Helm templates to install Cilium
HelmTemplate = "../install/kubernetes/cilium"
Expand Down Expand Up @@ -305,7 +306,7 @@ var badLogMessages = map[string][]string{
"DATA RACE": nil,
// Exceptions for level=error should only be added as a last resort, if the
// error cannot be fixed in Cilium or in the test.
"level=error": {opCantBeFulfilled, initLeaderElection, globalDataSupport, removeInexistentID, failedToListCRDs, retrieveResLock, failedToRelLockEmptyName, failedToUpdateLock, failedToReleaseLock, errorCreatingInitialLeader},
"level=error": {opCantBeFulfilled, initLeaderElection, globalDataSupport, removeInexistentID, failedToListCRDs, retrieveResLock, failedToRelLockEmptyName, failedToUpdateLock, failedToReleaseLock, errorCreatingInitialLeader, secIdentityChanged},
}

var ciliumCLICommands = map[string]string{
Expand Down

0 comments on commit b5900f5

Please sign in to comment.