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 backup-retention-cronjob name for template in influxdb chart #494

Open
oxr463 opened this issue Aug 5, 2022 · 0 comments
Open

Comments

@oxr463
Copy link

oxr463 commented Aug 5, 2022

In _helpers.tpl, we have two places where we truncate names down to 63 characters. But then in the backup-retention-cronjob template, we append -backup-retention to the name; that's an extra 17 characters.

I suggest truncating the backup-retention-cronjob to 63 chars, we'll need to truncate it before we append the additional chars. Otherwise, the name won't make very much sense.

Or... We could provide a way to override the name in the values.yaml file?

Reference(s):

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

No branches or pull requests

1 participant