Skip to content

Commit 3df2947

Browse files
committed
vrrp: fix track process reinitialize fork delay timer
Github user Bbulatov identified that terminate_delay was being used when fork_delay should have been used. While investigating, it was also found, albeit in a debug message that fork_delay was used where terminate_delay should have been used. Further, the process state was being updated immediately even if the fork_delay was being invoked. Signed-off-by: Quentin Armitage <[email protected]>
1 parent cf61539 commit 3df2947

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

keepalived/core/track_process.c

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -687,7 +687,7 @@ check_process_termination(pid_t pid)
687687
tpr->terminate_timer_thread = thread_add_timer(master, process_lost_quorum_timer_thread, tpr, tpr->terminate_delay);
688688
#ifdef _TRACK_PROCESS_DEBUG_
689689
if (do_track_process_debug_detail)
690-
log_message(LOG_INFO, "Adding timer %d for %s termination", tpr->fork_delay, tpr->pname);
690+
log_message(LOG_INFO, "Adding timer %d for %s termination", tpr->terminate_delay, tpr->pname);
691691
#endif
692692
continue;
693693
}
@@ -893,8 +893,9 @@ reinitialise_track_processes(void)
893893
, tpr->sav_num_cur_proc
894894
, tpr->num_cur_proc);
895895
if (tpr->fork_delay)
896-
tpr->fork_timer_thread = thread_add_timer(master, process_gained_quorum_timer_thread, tpr, tpr->terminate_delay);
897-
process_update_track_process_status(tpr, true);
896+
tpr->fork_timer_thread = thread_add_timer(master, process_gained_quorum_timer_thread, tpr, tpr->fork_delay);
897+
else
898+
process_update_track_process_status(tpr, true);
898899
} else {
899900
if (__test_bit(LOG_DETAIL_BIT, &debug))
900901
log_message(LOG_INFO, "Process %s, number of current processes changed"
@@ -909,8 +910,6 @@ reinitialise_track_processes(void)
909910
}
910911
}
911912
}
912-
913-
return;
914913
}
915914

916915
static void

0 commit comments

Comments
 (0)