Skip to content

[DATA RACE]: github.com/cilium/cilium/pkg/ipcache.(*IPIdentitySynchronizer).Upsert() #39294

@aanm

Description

@aanm
2025-05-02T12:27:20.257208035Z WARNING: DATA RACE
2025-05-02T12:27:20.257217443Z Read at 0x00c001ade840 by goroutine 4843:
2025-05-02T12:27:20.257223123Z   github.com/cilium/cilium/pkg/ipcache.(*IPIdentitySynchronizer).Upsert()
2025-05-02T12:27:20.257228633Z       /go/src/github.com/cilium/cilium/pkg/ipcache/kvstore.go:64 +0x93
2025-05-02T12:27:20.257234184Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).runIPIdentitySync.func1()
2025-05-02T12:27:20.257239934Z       /go/src/github.com/cilium/cilium/pkg/endpoint/policy.go:1043 +0x452
2025-05-02T12:27:20.257244583Z   github.com/cilium/cilium/pkg/controller.(*controller).runController()
2025-05-02T12:27:20.257249182Z       /go/src/github.com/cilium/cilium/pkg/controller/controller.go:312 +0x1af
2025-05-02T12:27:20.257263358Z   github.com/cilium/cilium/pkg/controller.(*Manager).createControllerLocked.gowrap1()
2025-05-02T12:27:20.257267626Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:115 +0x33
2025-05-02T12:27:20.257271593Z 
2025-05-02T12:27:20.257275671Z Previous write at 0x00c001ade840 by goroutine 4834:
2025-05-02T12:27:20.257279769Z   github.com/cilium/cilium/pkg/ipcache.(*IPIdentitySynchronizer).Upsert()
2025-05-02T12:27:20.257284087Z       /go/src/github.com/cilium/cilium/pkg/ipcache/kvstore.go:65 +0x119
2025-05-02T12:27:20.257288255Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).runIPIdentitySync.func1()
2025-05-02T12:27:20.257292222Z       /go/src/github.com/cilium/cilium/pkg/endpoint/policy.go:1043 +0x452
2025-05-02T12:27:20.257296730Z   github.com/cilium/cilium/pkg/controller.(*controller).runController()
2025-05-02T12:27:20.257301830Z       /go/src/github.com/cilium/cilium/pkg/controller/controller.go:312 +0x1af
2025-05-02T12:27:20.257307670Z   github.com/cilium/cilium/pkg/controller.(*Manager).createControllerLocked.gowrap1()
2025-05-02T12:27:20.257312169Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:115 +0x33
2025-05-02T12:27:20.257316948Z 
2025-05-02T12:27:20.257322318Z Goroutine 4843 (running) created at:
2025-05-02T12:27:20.257328028Z   github.com/cilium/cilium/pkg/controller.(*Manager).createControllerLocked()
2025-05-02T12:27:20.257333349Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:115 +0x630
2025-05-02T12:27:20.257338989Z   github.com/cilium/cilium/pkg/controller.(*Manager).updateController()
2025-05-02T12:27:20.257344029Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:88 +0x48f
2025-05-02T12:27:20.257348737Z   github.com/cilium/cilium/pkg/controller.(*Manager).UpdateController()
2025-05-02T12:27:20.257353487Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:56 +0x50f
2025-05-02T12:27:20.257358395Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).runIPIdentitySync()
2025-05-02T12:27:20.257363845Z       /go/src/github.com/cilium/cilium/pkg/endpoint/policy.go:1014 +0x1c3
2025-05-02T12:27:20.257369376Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).SetIdentity()
2025-05-02T12:27:20.257375177Z       /go/src/github.com/cilium/cilium/pkg/endpoint/policy.go:1090 +0x2cf
2025-05-02T12:27:20.257379345Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).restoreIdentity()
2025-05-02T12:27:20.257383372Z       /go/src/github.com/cilium/cilium/pkg/endpoint/restore.go:405 +0xc8e
2025-05-02T12:27:20.257470946Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).RegenerateAfterRestore()
2025-05-02T12:27:20.257481234Z       /go/src/github.com/cilium/cilium/pkg/endpoint/restore.go:205 +0x6f
2025-05-02T12:27:20.257487346Z   github.com/cilium/cilium/daemon/cmd.(*Daemon).regenerateRestoredEndpoints.func1()
2025-05-02T12:27:20.257492335Z       /go/src/github.com/cilium/cilium/daemon/cmd/state.go:342 +0xd2
2025-05-02T12:27:20.257496994Z   github.com/cilium/cilium/daemon/cmd.(*Daemon).regenerateRestoredEndpoints.gowrap1()
2025-05-02T12:27:20.257502133Z       /go/src/github.com/cilium/cilium/daemon/cmd/state.go:348 +0x4f
2025-05-02T12:27:20.257506552Z 
2025-05-02T12:27:20.257510700Z Goroutine 4834 (running) created at:
2025-05-02T12:27:20.257514707Z   github.com/cilium/cilium/pkg/controller.(*Manager).createControllerLocked()
2025-05-02T12:27:20.257518694Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:115 +0x630
2025-05-02T12:27:20.257522662Z   github.com/cilium/cilium/pkg/controller.(*Manager).updateController()
2025-05-02T12:27:20.257526649Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:88 +0x48f
2025-05-02T12:27:20.257537359Z   github.com/cilium/cilium/pkg/controller.(*Manager).UpdateController()
2025-05-02T12:27:20.257541386Z       /go/src/github.com/cilium/cilium/pkg/controller/manager.go:56 +0x50f
2025-05-02T12:27:20.257545855Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).runIPIdentitySync()
2025-05-02T12:27:20.257551225Z       /go/src/github.com/cilium/cilium/pkg/endpoint/policy.go:1014 +0x1c3
2025-05-02T12:27:20.257557055Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).SetIdentity()
2025-05-02T12:27:20.257562015Z       /go/src/github.com/cilium/cilium/pkg/endpoint/policy.go:1089 +0x295
2025-05-02T12:27:20.257569499Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).restoreIdentity()
2025-05-02T12:27:20.257574298Z       /go/src/github.com/cilium/cilium/pkg/endpoint/restore.go:405 +0xc8e
2025-05-02T12:27:20.257578466Z   github.com/cilium/cilium/pkg/endpoint.(*Endpoint).RegenerateAfterRestore()
2025-05-02T12:27:20.257582874Z       /go/src/github.com/cilium/cilium/pkg/endpoint/restore.go:205 +0x6f
2025-05-02T12:27:20.257588234Z   github.com/cilium/cilium/daemon/cmd.(*Daemon).regenerateRestoredEndpoints.func1()
2025-05-02T12:27:20.257593764Z       /go/src/github.com/cilium/cilium/daemon/cmd/state.go:342 +0xd2
2025-05-02T12:27:20.257599265Z   github.com/cilium/cilium/daemon/cmd.(*Daemon).regenerateRestoredEndpoints.gowrap1()
2025-05-02T12:27:20.257604765Z       /go/src/github.com/cilium/cilium/daemon/cmd/state.go:348 +0x4f

Potentially created by f7e3eaa /cc @giorio94

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/kvstoreImpacts the KVStore package interactions.kind/bugThis is a bug in the Cilium logic.kind/bug/race-detectorThis is a bug identified by concurrency tests.release-blocker/1.18This issue will prevent the release of the next version of Cilium.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions