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

Helm chart errors if using a SemVer2 chart version #14032

Closed
3 tasks done
thomasv314 opened this issue Apr 8, 2021 · 0 comments · Fixed by #14031
Closed
3 tasks done

Helm chart errors if using a SemVer2 chart version #14032

thomasv314 opened this issue Apr 8, 2021 · 0 comments · Fixed by #14031
Labels
#bug Bug report

Comments

@thomasv314
Copy link
Contributor

A clear and concise description of what the bug is.

Expected results

Helm chart deploys when the chart version uses SemVer2

Actual results

 Error: release superset failed, and has been uninstalled due to atomic being set: Secret "superset-config" is invalid: metadata.labels: Invalid value: "superset-2021.13.1-79+44068e96a": a valid label must be an empty string or consist of alphanumeric characters, '-', '_' or '.', and must start and end with an alphanumeric character (e.g. 'MyValue',  or 'my_value',  or '12345', regex used for validation is '(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])?')

Screenshots

If applicable, add screenshots to help explain your problem.

How to reproduce the bug

Set the chart version.yaml to include a +, i.e. 2021.13.1-79+44068e96a

The superset-config resource errors out because it does not use the superset.chart function that all the other resources use

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

Add any other context about the problem here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant