Skip to content

[YUNIKORN-3226] Remove event creation for succeeded / deleted pods#1001

Closed
manirajv06 wants to merge 2 commits intoapache:masterfrom
manirajv06:YUNIKORN-3226
Closed

[YUNIKORN-3226] Remove event creation for succeeded / deleted pods#1001
manirajv06 wants to merge 2 commits intoapache:masterfrom
manirajv06:YUNIKORN-3226

Conversation

@manirajv06
Copy link
Contributor

@manirajv06 manirajv06 commented Feb 9, 2026

What is this PR for?

Send event to the pods and nodes before deleting it.

Flows cleaned up based on the similar approach are TaskFail, TaskCompleted and DecomissionNode.

What type of PR is it?

  • - Bug Fix

Todos

  • - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-3226

How should this be tested?

Existing tests should cover.

Screenshots (if appropriate)

Questions:

  • - The licenses files need update.
  • - There is breaking changes for older versions.
  • - It needs documentation.

@manirajv06 manirajv06 self-assigned this Feb 9, 2026
@codecov
Copy link

codecov bot commented Feb 9, 2026

Codecov Report

❌ Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.09%. Comparing base (b625a93) to head (8883d0d).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
pkg/cache/context.go 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1001      +/-   ##
==========================================
- Coverage   66.15%   66.09%   -0.07%     
==========================================
  Files          72       72              
  Lines        7715     7713       -2     
==========================================
- Hits         5104     5098       -6     
- Misses       2398     2401       +3     
- Partials      213      214       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@wilfred-s wilfred-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants