minor fixes in comments of developeraccountCRD reconcilliation logic

3scale · Feb 8, 2021 · e0dc60a · e0dc60a
1 parent 8530a91
commit e0dc60a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/controllers/capabilities/developeraccount_status_reconciler.go b/controllers/capabilities/developeraccount_status_reconciler.go
@@ -86,8 +86,6 @@ func (s *DeveloperAccountStatusReconciler) calculateStatus() (*capabilitiesv1bet
 		ObservedGeneration:  s.resource.Status.ObservedGeneration,
 	}
 
-	// If there is an error and s.remoteDeveloperAccount is nil
-	// ID is not changed, as it is initialized with previous values d
 	if s.remoteDeveloperAccount != nil {
 		newStatus.ID = s.remoteDeveloperAccount.Element.ID
 		newStatus.AccountState = s.remoteDeveloperAccount.Element.State

diff --git a/controllers/capabilities/developeraccount_threescale_reconciler.go b/controllers/capabilities/developeraccount_threescale_reconciler.go
@@ -183,7 +183,7 @@ func (s *DeveloperAccountThreescaleReconciler) syncDeveloperAccount(devAccount *
 		deltaAccount.Element.MonthlyBillingEnabled = &desiredMonthlyBillingEnabled
 	}
 
-	// MonthlyChargingEnabled defaults to False
+	// MonthlyChargingEnabled defaults to True
 	desiredMonthlyChargingEnabled := true
 	if s.resource.Spec.MonthlyChargingEnabled != nil {
 		desiredMonthlyChargingEnabled = *s.resource.Spec.MonthlyChargingEnabled