Skip to content

Fix refreshInterval type on group-wishlist auth-secret ExternalSecret - #357

Merged
bkonicek merged 1 commit into
mainfrom
fix/group-wishlist-auth-secret-refresh-interval-type
Oct 2, 2026
Merged

bkonicek merged 1 commit into
mainfrom
fix/group-wishlist-auth-secret-refresh-interval-type

Conversation

@bkonicek

@bkonicek bkonicek commented Oct 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Follow-up to Split AUTH_SECRET into its own ExternalSecret for group-wishlist #356, merged before this fix was pushed.
  • refreshInterval: 0 in externalsecret-auth-secret.yaml is parsed by YAML as an integer, but ESO's field is a duration string. ArgoCD sync was failing: failed to create typed patch object (group-wishlist/auth-secret; external-secrets.io/v1, Kind=ExternalSecret): .spec.refreshInterval: expected string, got &value.valueUnstructured{Value:0}.
  • Quoted it (refreshInterval: "0") so it serializes as a string.

Test plan

  • helm template charts/group-wishlist renders refreshInterval: "0"
  • ArgoCD sync succeeds and creates the auth-secret ExternalSecret/Secret

🤖 Generated with Claude Code

ESO's refreshInterval field is a duration string; unquoted 0 was
parsed by YAML as an integer, causing ArgoCD sync to fail with
"expected string, got valueUnstructured{Value:0}".

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@bkonicek
bkonicek merged commit 6775073 into main Oct 2, 2026
1 check failed
@bkonicek
bkonicek deleted the fix/group-wishlist-auth-secret-refresh-interval-type branch October 2, 2026 19:00
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.

1 participant