search

skupperproject / skupper

Skupper is an implementation of a Virtual Application Network, enabling rich hybrid cloud communication.
http://skupper.io
Apache License 2.0
595 stars 74 forks source link

[v2] Controller retries forever resolving pod when connector deleted #1733

Closed c-kruse closed 1 month ago

c-kruse commented 1 month ago

Describe the bug

time=2024-10-17T18:05:33.191Z level=INFO msg="[ControllerError] Error while handling Pod default/vet-5f59f050-5f6fb696cc-fm4z5: Error looking up connector for default/vet-5f59f050"
time=2024-10-17T18:05:33.577Z level=INFO msg="[ControllerError] Error while handling Pod default/vet-5f59f050-5f6fb696cc-fm4z5: Error looking up connector for default/vet-5f59f050"
time=2024-10-17T18:05:33.991Z level=INFO msg="[ControllerError] Error while handling Pod default/vet-5f59f050-5f6fb696cc-fm4z5: Error looking up connector for default/vet-5f59f050"
time=2024-10-17T18:05:34.391Z level=INFO msg="[ControllerError] Error while handling Pod default/vet-5f59f050-5f6fb696cc-fm4z5: Error looking up connector for default/vet-5f59f050"
...

How To Reproduce Add a selector connector and a pod target workload then delete both at the same time.

reproducer script: https://gist.github.com/c-kruse/d146e2caccfa9d2cedfec61b42f6f8e7

grs commented 1 month ago

This is fixed by #1735, the cause is that the pod watcher for the connector is not cleaned up properly.