-
Notifications
You must be signed in to change notification settings - Fork 6.7k
docs: add faq entry for marshaling error #25851
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: nitishfy <[email protected]>
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
| Kubernetes environment variables must be valid UTF-8 strings. In affected clusters, the argocd-redis Secret contained non-UTF-8 (binary) data, while other clusters used | ||
| ASCII-only values. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know why the auto generated redis secret is containing binary data?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've not researched about it yet, will do it and let you know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Redis password generator only uses printable ASCII characters, so it should not generate binary / non-UTF-8 data.
The clusters where this breaks likely already had an existing Redis secret with non-UTF-8 bytes (possibly created by an older Argo CD version). IMO, Kubernetes 1.34 surfaces this more strictly, causing pods to fail when a secret with non-UTF-8 data is referenced. So this doesn’t appear to be a new secret generation producing binary data, but rather existing invalid secret contents being exposed by stricter validation.
Signed-off-by: nitishfy <[email protected]>
Signed-off-by: nitishfy <[email protected]> Signed-off-by: Vishwa Gandhi <[email protected]>
Closes #25815
Checklist: