1Password/onepassword-operator
1
0
Fork 0
mirror of https://github.com/1Password/onepassword-operator.git synced 2025-10-22 07:28:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

Retry after 15 minutes, if get rate-limit error

Browse Source 
As currently service account are last for 1 hour https://developer.1password.com/docs/service-accounts/rate-limits/#hourly-limits
This commit is contained in:
Volodymyr Zotov
2025-06-09 16:04:23 -05:00
parent 4527336c37
commit c95078c34c
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
internal/controller/deployment_controller.go
View File

@@ -104,8 +104,8 @@ func (r *DeploymentReconciler) Reconcile(ctx context.Context, req ctrl.Request)
// Handles creation or updating secrets for deployment if needed // Handles creation or updating secrets for deployment if needed
if err = r.handleApplyingDeployment(deployment, deployment.Namespace, annotations, req); err != nil { if err = r.handleApplyingDeployment(deployment, deployment.Namespace, annotations, req); err != nil {
if strings.Contains(err.Error(), "rate limit") { if strings.Contains(err.Error(), "rate limit") {
reqLogger.V(logs.InfoLevel).Info("1Password rate limit hit. Requeuing after 1 minute.") reqLogger.V(logs.InfoLevel).Info("1Password rate limit hit. Requeuing after 15 minutes.")
return ctrl.Result{RequeueAfter: time.Minute}, nil return ctrl.Result{RequeueAfter: 15 * time.Minute}, nil
} else { } else {
return ctrl.Result{}, err return ctrl.Result{}, err
} }

4
internal/controller/onepassworditem_controller.go
View File

@@ -107,8 +107,8 @@ func (r *OnePasswordItemReconciler) Reconcile(ctx context.Context, req ctrl.Requ
err = r.handleOnePasswordItem(onepassworditem, req) err = r.handleOnePasswordItem(onepassworditem, req)
if err != nil { if err != nil {
if strings.Contains(err.Error(), "rate limit") { if strings.Contains(err.Error(), "rate limit") {
reqLogger.V(logs.InfoLevel).Info("1Password rate limit hit. Requeuing after 1 minute.") reqLogger.V(logs.InfoLevel).Info("1Password rate limit hit. Requeuing after 15 minutes.")
return ctrl.Result{RequeueAfter: time.Minute}, nil return ctrl.Result{RequeueAfter: 15 * time.Minute}, nil
} }
} }
if updateStatusErr := r.updateStatus(onepassworditem, err); updateStatusErr != nil { if updateStatusErr := r.updateStatus(onepassworditem, err); updateStatusErr != nil {