Skip to content

feat: make controller requeue delays configurable#5509

Open
omerzifman wants to merge 2 commits intoakuity:mainfrom
omerzifman:feat/configurable-requeue-delay
Open

feat: make controller requeue delays configurable#5509
omerzifman wants to merge 2 commits intoakuity:mainfrom
omerzifman:feat/configurable-requeue-delay

Conversation

@omerzifman
Copy link

Closes #4969

This PR makes the reconcile requeue delay configurable for the following controllers:

  • Stage controller (regular_stages.go)
  • Project controller (projects.go)
  • ProjectConfig controller (project_configs.go)
  • ClusterConfig controller (cluster_configs.go)

Previously, these controllers used a hard-coded 5-minute requeue delay. Operators can now tune this interval via Helm values or environment variables, with the default remaining at 5 minutes for backward compatibility.

Changes

Go Code

  • Added RequeueInterval field to each controller's ReconcilerConfig struct with envconfig tags and a default of 5m
  • Updated Reconcile methods to use the configurable interval instead of hard-coded 5 * time.Minute

Helm Chart

  • Added new values for configuring requeue intervals:
    • controller.reconcilers.stages.requeueInterval
    • managementController.reconcilers.projects.requeueInterval
    • managementController.reconcilers.projectConfigs.requeueInterval
    • managementController.reconcilers.clusterConfigs.requeueInterval
  • Updated ConfigMap templates to pass environment variables when values are set

Documentation

  • Added "Tuning Requeue Intervals" section to the operator guide (docs/docs/40-operator-guide/20-advanced-installation/30-common-configurations.md)

Usage

# values.yaml
controller:
  reconcilers:
    stages:
      requeueInterval: 10m  # Custom interval for Stage controller

managementController:
  reconcilers:
    projects:
      requeueInterval: 10m
    projectConfigs:
      requeueInterval: 10m
    clusterConfigs:
      requeueInterval: 10m

Copilot AI review requested due to automatic review settings December 15, 2025 16:13
@omerzifman omerzifman requested review from a team as code owners December 15, 2025 16:13
@netlify
Copy link

netlify bot commented Dec 15, 2025

Deploy Preview for docs-kargo-io ready!

Name Link
🔨 Latest commit 87b758f
🔍 Latest deploy log https://app.netlify.com/projects/docs-kargo-io/deploys/69403398ae367e000828191c
😎 Deploy Preview https://deploy-preview-5509.docs.kargo.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Make Stage controller requeue delay configurable

1 participant