Skip to content

Commit b83a37e

Browse files
Bug Fix - Delete evicted pods after de-registration. (#465)
Co-authored-by: David J. M. Karlsen <david@davidkarlsen.com>
1 parent 5adfd79 commit b83a37e

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

controllers/githubactionrunner_controller.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -364,6 +364,13 @@ func (r *GithubActionRunnerReconciler) handleFinalization(ctx context.Context, c
364364
if err := r.unregisterRunner(ctx, cr, item); err != nil {
365365
return err
366366
}
367+
if isEvicted(&item.pod) {
368+
logr.FromContextOrDiscard(ctx).Info("Deleting evicted pod", "podname", item.pod.Name)
369+
err := r.DeleteResourceIfExists(ctx, &item.pod)
370+
if err != nil {
371+
return err
372+
}
373+
}
367374
if isCompleted(&item.pod) {
368375
logr.FromContextOrDiscard(ctx).Info("Deleting succeeded pod", "podname", item.pod.Name)
369376
err := r.DeleteResourceIfExists(ctx, &item.pod)

0 commit comments

Comments
 (0)