Skip to content

controller: improve error for missed start times in cronjob#374

Merged
mergify[bot] merged 1 commit into
csi-addons:mainfrom
Rakshith-R:schedule-change
Jun 21, 2023
Merged

controller: improve error for missed start times in cronjob#374
mergify[bot] merged 1 commit into
csi-addons:mainfrom
Rakshith-R:schedule-change

Conversation

@Rakshith-R

Copy link
Copy Markdown
Member

The reclaimspacecronjob may miss too many runs due to the following reasons:

  • low startingDeadlineSeconds
  • clock skew
  • (added now) when user has reduced schedule which in turn leads to the new interval being < 100 times of old interval, aka >100 missed runs.

This issue only occurs when user changes schedule which drastically reduces interval by more than 100 times. Error message is improved to ask user to delete and recreate cronjob to overcome this issue.

@mergify mergify Bot requested review from nixpanic and yati1998 June 21, 2023 06:14
@Rakshith-R Rakshith-R requested a review from Madhu-1 June 21, 2023 06:16
The reclaimspacecronjob may miss too many runs due to the
following reasons:
- low startingDeadlineSeconds
- clock skew
- (added now) when user has reduced schedule which in turn
  leads to the new interval being < 100 times of old interval,
  aka >100 missed runs.

This issue only occurs when user changes schedule which
drastically reduces interval by more than 100 times.
Error message is improved to ask user to delete and
recreate cronjob to overcome this issue.

Signed-off-by: Rakshith R <rar@redhat.com>
@mergify mergify Bot merged commit 91d1cac into csi-addons:main Jun 21, 2023
black-dragon74 pushed a commit to black-dragon74/kubernetes-csi-addons that referenced this pull request Jan 27, 2026
DFBUGS-5196: [release-4.21] Fix for cases where CSIAddonsNode was stuck in deletion phase
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