Good catch!

/assign
/retest

This is a good change. I'd like to see a unit test so that we protect against future regressions. Something like this, perhaps:

diff --git a/pkg/controller/endpoint/endpoints_controller_test.go b/pkg/controller/endpoint/endpoints_controller_test.go
--- a/pkg/controller/endpoint/endpoints_controller_test.go
+++ b/pkg/controller/endpoint/endpoints_controller_test.go
@@ -1039,6 +1039,13 @@ func TestPodChanged(t *testing.T) {
 		t.Errorf("Expected pod to be changed with pod readiness change")
 	}
 	oldPod.Status.Conditions = saveConditions
+
+	now := metav1.NewTime(time.Now().UTC())
+	newPod.ObjectMeta.DeletionTimestamp = &now
+	if !podChanged(oldPod, newPod) {
+		t.Errorf("Expected pod to be changed with DeletionTimestamp change")
+	}
+	newPod.ObjectMeta.DeletionTimestamp = oldPod.ObjectMeta.DeletionTimestamp.DeepCopy()
 }
 
 func TestDetermineNeededServiceUpdates(t *testing.T) {

Should this be cherry-picked to 1.8?

@yujuhong yes, it should. @derekwaynecarr or anybody else with rights to do so who happens across this PR, could you please add the cherry-pick-candidate label and the 1.8 milestone?

Added unit test

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: freehan, joelsmith
We suggest the following additional approver: deads2k

Assign the PR to them by writing /assign @deads2k in a comment when ready.

Associated issue: 54723

The full list of commands accepted by this bot can be found here.

@freehan need to add release note for cherrypicks.

@freehan: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

@freehan please add release note or release-note-none label as appropriate.

Why have the tests still not run? Was the bot asleep when the code was updated?
/retest

/test pull-kubernetes-unit

/retest
This bot automatically retries jobs that failed/flaked on approved PRs (send feedback to @fejta).

Review the full test history for this PR.

/test pull-kubernetes-e2e-kops-aws

/retest
This bot automatically retries jobs that failed/flaked on approved PRs (send feedback to @fejta).

Review the full test history for this PR.

Automatic merge from submit-queue (batch tested with PRs 54894, 54630, 54828, 54926, 54865). If you want to cherry-pick this change to another branch, please follow the instructions here.