fix: explicitly delete executor pods during invalidation

[Denyme24]

fixes #2803

Proposed changes:

• Added deleteExecutorPods() function that:

  • Retrieves all executor pods using getExecutorPods()
  • Iterates and deletes each executor pod explicitly
  • Logs deletion operations for visibility

• Updated cleanup order in deleteSparkResources():

  • Delete executor pods first (deterministic cleanup)
  • Then delete driver pod
  • Finally delete WebUI service and ingress

Change Category

• Bugfix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that could affect existing functionality)
• Documentation update

Rationale

Checklist

• I have conducted a self-review of my own code.
• I have updated documentation accordingly.
• I have added tests that prove my changes are effective or that my feature works.
• Existing unit tests pass locally with my changes.

Additional Notes

[google-oss-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign jacobsalway for approval. For more information see the Kubernetes Code Review Process.

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

Details

Needs approval from an approver in each of these files:

• OWNERS

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

[Denyme24]

hey team, @nabuskey @ImpSy. PTAL

[Denyme24]

hey @ChenYi015 @Shekharrajak, could you please review these changes.
thank you