Skip to content

Fix one panic error when sts processDelete#1444

Open
TaoZou1 wants to merge 1 commit into
vmware-tanzu:mainfrom
TaoZou1:stspanic
Open

Fix one panic error when sts processDelete#1444
TaoZou1 wants to merge 1 commit into
vmware-tanzu:mainfrom
TaoZou1:stspanic

Conversation

@TaoZou1
Copy link
Copy Markdown
Contributor

@TaoZou1 TaoZou1 commented Jun 1, 2026

Pass nil to processDelete to avoid panic

Test Done:
run ut

Pass nil to processDelete to avoid panic
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 77.10%. Comparing base (3499a03) to head (f9fb282).

Files with missing lines Patch % Lines
.../controllers/statefulset/statefulset_controller.go 87.50% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1444      +/-   ##
==========================================
- Coverage   77.32%   77.10%   -0.22%     
==========================================
  Files         169      169              
  Lines       23325    23332       +7     
==========================================
- Hits        18036    17991      -45     
- Misses       3954     3955       +1     
- Partials     1335     1386      +51     
Flag Coverage Δ
unit-tests 77.10% <87.50%> (-0.22%) ⬇️
Files with missing lines Coverage Δ
.../controllers/statefulset/statefulset_controller.go 72.69% <87.50%> (-18.98%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TaoZou1
Copy link
Copy Markdown
Contributor Author

TaoZou1 commented Jun 1, 2026

e2e

@TaoZou1
Copy link
Copy Markdown
Contributor Author

TaoZou1 commented Jun 1, 2026

/e2e

Comment thread pkg/controllers/statefulset/statefulset_controller.go
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.

3 participants