Minor fixup to auto egress IP deletion fix

[danwinship]

Fix to #18720. That added a missing node.assignedIPs.Delete(egressIP) to deleteEgressIP, but it added it in the wrong place, inside the if node.nodeIP == eip.localIP check, so only the node hosting the egress IP would clean things up properly and the other nodes would still stay broken. The test added in #18720 didn't pick this up because it only tested add-remove-add in the egress-on-local-node case, not the remote case.

(This is fixed in master as part of the larger changes in #18808, but that's not going to be backported immediately, whereas this could be.)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1547899

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danwinship

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

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/network/OWNERS [danwinship]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[pravisankar]

LGTM

[openshift-ci-robot]

@danwinship: The following test failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
ci/openshift-jenkins/cmd	372e914	link	/test cmd

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[danwinship]

This didn't merge in time and will eventually be fixed by #18808 instead