Skip to content
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

truncate release names so resource names never exceed 63 chars #499

Closed
wants to merge 7 commits into from

Conversation

kschoche
Copy link
Contributor

@kschoche kschoche commented Dec 6, 2023

Our largest resource name suffix is 36 characters, so truncating the release name to 27 characters ensures we don't ever exceed 63 characters.

Limit the release name and fullNameOverride to 27 characters so we never exceed 63 characters with a resource name.

Fixes #260
Closes #476

@kschoche kschoche added the bug Something isn't working label Dec 6, 2023
@kschoche kschoche marked this pull request as ready for review December 6, 2023 18:54
@kschoche kschoche requested a review from a team as a code owner December 6, 2023 18:54
@kschoche kschoche modified the milestones: v0.4.2, v0.5.0 Dec 6, 2023
@benashz benashz modified the milestones: v0.5.0, v0.4.3 Jan 8, 2024
@kschoche kschoche removed this from the v0.4.3 milestone Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Chart] Pre-delete hook name can be >63 chars
3 participants