Revision - bb49619 - endpoint: Avoid benign error messages on restoration - origin: https://github.com/cilium/cilium

visit type:

https://github.com/cilium/cilium

09 April 2024, 21:14:05 UTC

Revision bb49619febba44d56ca090792ccf13c5aece2219 authored by Paul Chaignon on 21 October 2020, 13:38:24 UTC, committed by Joe Stringer on 22 October 2020, 22:04:10 UTC

endpoint: Avoid benign error messages on restoration

[ upstream commit 228a485f2a5441f506ba9c0f357321c060f93590 ]

During the endpoint restoration process, when we parse the endpoints, we
assign them a reserved init identity if they don't already have an
identity [0]. If we later remove the endpoint (because the corresponding
K8s pod or interface are missing), we attempt to remove the identity
from the identity manager. That last operation results in the following
error message because the init identity was never added to the manager.

  level=error msg="removing identity not added to the identity manager!" identity=5 subsys=identitymanager

This commit fixes it by skipping the removal attempt from the manager in
the case of identity init.

0 - https://github.com/cilium/cilium/blob/80a71791320df34df5b6252b9680553e38d88d20/pkg/endpoint/endpoint.go#L819
Signed-off-by: Paul Chaignon <paul@cilium.io>

1 parent a2c9ee2

Files
Changes

Permalinks

Tip revision: bb49619febba44d56ca090792ccf13c5aece2219 authored by Paul Chaignon on 21 October 2020, 13:38:24 UTC
endpoint: Avoid benign error messages on restoration

Tip revision: bb49619

File	Mode	Size
.github
.travis
Documentation
api
bpf
bugtool
cilium
cilium-health
common
contrib
daemon
envoy
examples
hack
install
jenkinsfiles
operator
pkg
plugins
proxylib
test
tests
tools
vendor
.authors.aux	-rw-r--r--	416 bytes
.dockerignore	-rw-r--r--	884 bytes
.gitignore	-rw-r--r--	818 bytes
.gitmodules	-rw-r--r--	0 bytes
.mailmap	-rw-r--r--	1.5 KB
.travis.yml	-rw-r--r--	367 bytes
.travis.yml.tmpl	-rw-r--r--	387 bytes
AUTHORS	-rw-r--r--	11.6 KB
CHANGELOG.md	-rw-r--r--	85.2 KB
CODEOWNERS	-rw-r--r--	1.3 KB
CONTRIBUTING.md	-rw-r--r--	208 bytes
Dockerfile	-rw-r--r--	2.3 KB
Dockerfile.builder	-rw-r--r--	1.7 KB
FURTHER_READINGS.rst	-rw-r--r--	4.4 KB
GO_VERSION	-rw-r--r--	8 bytes
Jenkinsfile.nightly	l---------	32 bytes
LICENSE	-rw-r--r--	11.1 KB
MAINTAINERS.rst	-rw-r--r--	1.4 KB
Makefile	-rw-r--r--	17.9 KB
Makefile.defs	-rw-r--r--	2.5 KB
Makefile.quiet	-rw-r--r--	626 bytes
README.rst	-rw-r--r--	12.7 KB
SECURITY.md	-rw-r--r--	615 bytes
USERS.md	-rw-r--r--	3.5 KB
VERSION	-rw-r--r--	7 bytes
Vagrantfile	-rw-r--r--	11.8 KB
cilium-docker-plugin.Dockerfile	-rw-r--r--	553 bytes
cilium-operator.Dockerfile	-rw-r--r--	706 bytes
docs.Jenkinsfile	l---------	29 bytes
flannel.Jenkinsfile	l---------	32 bytes
ginkgo-kubernetes-all.Jenkinsfile	l---------	46 bytes
ginkgo.Jenkinsfile	l---------	31 bytes
go.mod	-rw-r--r--	41.2 KB
go.sum	-rw-r--r--	67.7 KB
kubernetes-upstream.Jenkinsfile	l---------	44 bytes
vagrant_box_defaults.rb	-rw-r--r--	270 bytes

Showing with 0 additions and 0 deletions (0 / 0 diffs computed)

Computing file changes ...

https://github.com/cilium/cilium

endpoint: Avoid benign error messages on restoration

README.rst