Skip to content

chore(auto-upgrade): run cronjob hourly at :23#112

Merged
saadqbal merged 1 commit intodevelopfrom
chore/auto-upgrade-hourly
May 7, 2026
Merged

chore(auto-upgrade): run cronjob hourly at :23#112
saadqbal merged 1 commit intodevelopfrom
chore/auto-upgrade-hourly

Conversation

@saadqbal
Copy link
Copy Markdown
Contributor

@saadqbal saadqbal commented May 7, 2026

Summary

  • Switches default autoUpgrade.schedule from "23 2 * * *" (daily 02:23 UTC) to "23 * * * *" (hourly at :23) so deployed clients converge on the latest chart within ~1h instead of ~24h.
  • Companion PR will cherry-pick this single commit onto a branch off main so prod gets only this change without pulling unshipped develop commits along with it.

Closes #111 (dev side — the prod-side PR will reference the same ticket).

Test plan

  • helm template ... | grep -A1 schedule: shows 23 * * * * on a fresh render
  • client/tests/auto_upgrade_test.yaml still passes (helm unittest client/)
  • After deploy, kubectl get cronjob -n <ns> -l app.kubernetes.io/component=auto-upgrade shows the hourly schedule

🤖 Generated with Claude Code


Note

Low Risk
Low risk: only changes the default CronJob schedule and its unit test expectation, with no changes to the upgrade logic itself. The main impact is increased upgrade/check frequency (more cluster activity and repo polling).

Overview
Switches the default autoUpgrade.schedule from daily "23 2 * * *" to hourly "23 * * * *", so the self-upgrade CronJob checks/applies chart updates up to once per hour.

Updates the Helm unittest (client/tests/auto_upgrade_test.yaml) to assert the new default schedule.

Reviewed by Cursor Bugbot for commit 66252c7. Bugbot is set up for automated code reviews on this repo. Configure here.

@saadqbal saadqbal self-assigned this May 7, 2026
@LukasWodka
Copy link
Copy Markdown
Contributor

👋 Heads-up — Code review queue is at 11 / 8

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3cbde1c. Configure here.

Comment thread client/values.yaml
Switches the auto-upgrade CronJob default schedule from
"23 2 * * *" (daily 02:23 UTC) to "23 * * * *" (hourly at :23).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@saadqbal saadqbal force-pushed the chore/auto-upgrade-hourly branch from 3cbde1c to 66252c7 Compare May 7, 2026 10:20
@saadqbal saadqbal merged commit bb1ad55 into develop May 7, 2026
1 check passed
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