Skip to content

Commit 5ee5c3f

Browse files
committed
webhosting-operator: drop special handling for conflict errors
Rely on the usual exponential backoff and ignore the noisy logs.
1 parent e42bdb4 commit 5ee5c3f

File tree

3 files changed

+3
-43
lines changed

3 files changed

+3
-43
lines changed

webhosting-operator/pkg/controllers/webhosting/common.go

Lines changed: 0 additions & 40 deletions
This file was deleted.

webhosting-operator/pkg/controllers/webhosting/website_controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -482,7 +482,7 @@ func (r *WebsiteReconciler) SetupWithManager(mgr manager.Manager, enableSharding
482482
var (
483483
// trigger on spec change and annotation changes (manual trigger for testing purposes)
484484
websitePredicate = predicate.Or(predicate.GenerationChangedPredicate{}, predicate.AnnotationChangedPredicate{})
485-
reconciler = SilenceConflicts(reconcile.AsReconciler[*webhostingv1alpha1.Website](r.Client, r))
485+
reconciler = reconcile.AsReconciler[*webhostingv1alpha1.Website](r.Client, r)
486486
)
487487

488488
if enableSharding {

webhosting-operator/test/e2e/e2e_suite_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,12 +124,12 @@ var _ = BeforeEach(func(ctx SpecContext) {
124124
By("Set up test ControllerRing")
125125
controllerRing = &shardingv1alpha1.ControllerRing{ObjectMeta: metav1.ObjectMeta{Name: webhostingv1alpha1.WebhostingOperatorName}}
126126

127-
By("Scaling checksum-controller")
127+
By("Scaling webhosting-operator")
128128
controllerDeployment = &appsv1.Deployment{ObjectMeta: metav1.ObjectMeta{Name: webhostingv1alpha1.WebhostingOperatorName, Namespace: webhostingv1alpha1.NamespaceSystem}}
129129
scaleController(ctx, 3)
130130

131131
DeferCleanup(func(ctx SpecContext) {
132-
By("Scaling checksum-controller")
132+
By("Scaling webhosting-operator")
133133
scaleController(ctx, 3)
134134
}, NodeTimeout(ShortTimeout))
135135

0 commit comments

Comments
 (0)