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

"20241115152137" is out of range for type integer (SQLSTATE 22003) #4146

Closed
sathieu opened this issue Nov 15, 2024 · 2 comments · Fixed by #4154
Closed

"20241115152137" is out of range for type integer (SQLSTATE 22003) #4146

sathieu opened this issue Nov 15, 2024 · 2 comments · Fixed by #4154
Labels

Comments

@sathieu
Copy link

sathieu commented Nov 15, 2024

Describe the bug
Using long int in version is not accepted:

error registering package keycloak-operator version 1.0.20241115152137: ERROR: value "20241115152137" is out of range for type integer (SQLSTATE 22003)

To Reproduce
Steps to reproduce the behavior:

  1. Create an helm chart with version 1.0.20241115152137
  2. publish the chart
  3. You'll receive a mail with the error

Expected behavior
This is valid semantic versioning, should be accepted

Screenshots

image

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@sathieu sathieu added the bug label Nov 15, 2024
tegioz added a commit that referenced this issue Nov 18, 2024
Closes #4146

Signed-off-by: Sergio Castaño Arteaga <[email protected]>
@tegioz
Copy link
Collaborator

tegioz commented Nov 18, 2024

Hi @sathieu 👋

#4154 should fix this.

Even though it'll be possible to add those versions once we deploy this fix, we'd recommend using shorter ones if possible. The reason is that such long versions won't fit well in the UI in some places (see screenshots below) and it may be confusing for users.

long_version long_version2

tegioz added a commit that referenced this issue Nov 18, 2024
@sathieu
Copy link
Author

sathieu commented Nov 18, 2024

Thanks @tegioz!

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

Successfully merging a pull request may close this issue.

2 participants