stacklok / toolhive / 24189574993

66%

Build Type

push

github

Committed by web-flow

Commit Message

Move MCPServer deletion check before validation and external API calls (#4650)

* move deletion check before validation in MCPServer reconciler

During deletion the controller was running 10+ validation steps and external
API calls before checking GetDeletionTimestamp(). This meant validation
failures could block deletion entirely and unnecessary work was done.

Moved the deletion timestamp check and finalizer logic right after fetching
the MCPServer resource, before any validation or config handling.

Closes #4631

* retrigger CI

---------

Co-authored-by: Chris Burns <29541485+ChrisJBurns@users.noreply.github.com>

Coverage Stats

5 of 17 new or added lines in 1 file covered. (29.41%)

10 existing lines in 3 files now uncovered.

55642 of 85584 relevant lines covered (65.01%)

61.26 hits per line