From df4705ad7829e669e1f907fab29cba789ce53482 Mon Sep 17 00:00:00 2001
From: Mykhailo Bobrovskyi
Date: Tue, 21 May 2024 14:48:58 +0300
Subject: [PATCH] Set DefaultRequeuingBackoffBaseSeconds to 60s.
---
apis/config/v1beta1/configuration_types.go | 4 ++--
apis/config/v1beta1/defaults.go | 2 +-
site/content/en/docs/reference/kueue-config.v1beta1.md | 4 ++--
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/apis/config/v1beta1/configuration_types.go b/apis/config/v1beta1/configuration_types.go
index ae09f244c7..107439fc1f 100644
--- a/apis/config/v1beta1/configuration_types.go
+++ b/apis/config/v1beta1/configuration_types.go
@@ -247,7 +247,7 @@ type RequeuingStrategy struct {
// - "Rand" represents the random jitter.
// During this time, the workload is taken as an inadmissible and
// other workloads will have a chance to be admitted.
- // By default, the consecutive requeue delays are around: (10s, 20s, 40s, ...).
+ // By default, the consecutive requeue delays are around: (60s, 120s, 240s, ...).
//
// Defaults to null.
// +optional
@@ -256,7 +256,7 @@ type RequeuingStrategy struct {
// BackoffBaseSeconds defines the base for the exponential backoff for
// re-queuing an evicted workload.
//
- // Defaults to 10.
+ // Defaults to 60.
// +optional
BackoffBaseSeconds *int32 `json:"backoffBaseSeconds,omitempty"`
}
diff --git a/apis/config/v1beta1/defaults.go b/apis/config/v1beta1/defaults.go
index f832850ffd..2444aa62fb 100644
--- a/apis/config/v1beta1/defaults.go
+++ b/apis/config/v1beta1/defaults.go
@@ -47,7 +47,7 @@ const (
DefaultMultiKueueGCInterval = time.Minute
DefaultMultiKueueOrigin = "multikueue"
DefaultMultiKueueWorkerLostTimeout = 15 * time.Minute
- DefaultRequeuingBackoffBaseSeconds = 10
+ DefaultRequeuingBackoffBaseSeconds = 60
)
func getOperatorNamespace() string {
diff --git a/site/content/en/docs/reference/kueue-config.v1beta1.md b/site/content/en/docs/reference/kueue-config.v1beta1.md
index 8a5e303af4..da66f53599 100644
--- a/site/content/en/docs/reference/kueue-config.v1beta1.md
+++ b/site/content/en/docs/reference/kueue-config.v1beta1.md
@@ -719,7 +719,7 @@ When it is null, the workloads will repeatedly and endless re-queueing.
"Rand" represents the random jitter.
During this time, the workload is taken as an inadmissible and
other workloads will have a chance to be admitted.
-By default, the consecutive requeue delays are around: (10s, 20s, 40s, ...).
+By default, the consecutive requeue delays are around: (60s, 120s, 240s, ...).
Defaults to null.
@@ -730,7 +730,7 @@ By default, the consecutive requeue delays are around: (10s, 20s, 40s, ...).
BackoffBaseSeconds defines the base for the exponential backoff for
re-queuing an evicted workload.
-Defaults to 10.
+Defaults to 60.