Skip to content

fix(openhands): dedupe RESEND_API_KEY in resend-sync cronjob#743

Merged
jlav merged 2 commits into
mainfrom
jl/fix-resend-env-dup
Jun 23, 2026
Merged

fix(openhands): dedupe RESEND_API_KEY in resend-sync cronjob#743
jlav merged 2 commits into
mainfrom
jl/fix-resend-env-dup

Conversation

@jlav

@jlav jlav commented Jun 23, 2026

Copy link
Copy Markdown
Member

Description

The resend-sync cronjob declares RESEND_API_KEY twice: hardcoded in the cronjob template, and again via the shared openhands.env block (when resend.enabled). Kubernetes tolerates duplicate env names, so helm/CI never flagged it. But ArgoCD's server-side-apply diff treats env as a map keyed by name and rejects the duplicate, which blocks syncing the production openhands app.

This drops the redundant hardcoded entry; the cronjob gets RESEND_API_KEY from openhands.env. resend is enabled in every environment that runs this cronjob, so there's no case that needs the hardcoded fallback. No runtime change: the openhands.env copy already wins today.

Helm Chart Checklist

  • I have updated the version field in Chart.yaml for each modified chart
  • I have tested the chart upgrade path from the previous version
  • I have verified backwards compatibility with existing values.yaml configurations
  • I have updated the chart's README.md if there are any breaking changes or new required values

Additional Notes

Validated by rendering the resend-sync cronjob: exactly one RESEND_API_KEY, same secret ref as before. No live helm upgrade run, and no new/changed values (so no README change). Consuming this in saas-deploy (prod dependency bump to 0.7.67) is being handled separately.

@jlav jlav enabled auto-merge (squash) June 23, 2026 14:04
@jlav jlav requested review from dylan-openhands and mamoodi June 23, 2026 14:05
@jlav jlav merged commit c427ca5 into main Jun 23, 2026
27 of 28 checks passed
@jlav jlav deleted the jl/fix-resend-env-dup branch June 23, 2026 14:05
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.

2 participants