cilium / cilium / 10947

39%

push

travis-ci-com

daemon: remove `uniqueID` map

We encode whether a given Endpoint needs to be regenerated or not with changes
to its state; regenerations are not attempted by callers if the endpoint is in
`StateWaitingToRegenerate` already. The `uniqueID` map encoded this same state,
but outside of the Endpoint package; therefore, it is now redundant. It makes
more logical sense to have the logic of whether a regeneration needs to be
performed for an Endpoint to be part of `pkg/endpoint` itself.

If an Endpoint is waiting on the semaphore from the Daemon to be acquired which
meters the number of concurrent regenerations, provide its `aliveCtx` to the
call to `QueueEndpointBuild` if no context is provided in the call to
`regenerate`. This ensures that when an Endpoint is deleted, it will release all
resources associated with it, including if it is waiting on this semaphore.

Signed-off by: Ian Vernon <ian@cilium.io>

4 of 4 new or added lines in 2 files covered. (100.0%)

25526 of 57107 relevant lines covered (44.7%)

1179.73 hits per line