fix: reconnect port forwarding if pod is restarted (crw-11119)#314
fix: reconnect port forwarding if pod is restarted (crw-11119)#314adietish wants to merge 5 commits into
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #314 +/- ##
==========================================
+ Coverage 0.00% 25.84% +25.84%
==========================================
Files 4 104 +100
Lines 26 4414 +4388
Branches 0 830 +830
==========================================
+ Hits 0 1141 +1141
- Misses 26 3108 +3082
- Partials 0 165 +165 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Signed-off-by: Andre Dietisheim <adietish@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Andre Dietisheim <adietish@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Andre Dietisheim <adietish@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
…methods Signed-off-by: Andre Dietisheim <adietish@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
…pod roll - Add retry loop (maxRetries, retryDelay) for transient failures during pod roll reconnect; permanent failures fail immediately. - Distinguish TimeoutCancellationException from cancellation exceptions: retry it, propagate cancellation exceptions. - Updated logging to reflect retry attempt count and omit stack trace for timeout failures. - Rename readyTimeout -> READY_TIMEOUT (constant), double value 60->120s. - Add cachePod() to seed cache without cluster query during pod roll. - Update findFirstRunning() to return newest pod by creationTimestamp and exclude terminating pods (deletionTimestamp). - Add comprehensive tests for retry, timeout, permanent failure, and cancellation scenarios.
fixes https://redhat.atlassian.net/browse/CRW-11119