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

Setting version of chart to latest doesn't work #925

Open
AndyBurdett opened this issue Oct 24, 2024 · 0 comments
Open

Setting version of chart to latest doesn't work #925

AndyBurdett opened this issue Oct 24, 2024 · 0 comments

Comments

@AndyBurdett
Copy link

AndyBurdett commented Oct 24, 2024

An issue was raised App Forward Versioning #712
A fix was applied but it doesn't work.
I'm no expert but line 58 of helmsman/internal/app/helm_helpers.go fails.
The constraint check doesn't allow for versions of latest.

the desired state file

cesa-futures-web:
    namespace: mynamespace
    enabled: true
    chart: nginx/nginx
    version: "latest"
    valuesFiles:
      - values/nginx.yaml

results in the following

 _          _ 
| |        | | 
| |__   ___| |_ __ ___  ___ _ __ ___   __ _ _ __
| '_ \ / _ \ | '_ ` _ \/ __| '_ ` _ \ / _` | '_ \ 
| | | |  __/ | | | | | \__ \ | | | | | (_| | | | | 
|_| |_|\___|_|_| |_| |_|___/_| |_| |_|\__,_|_| |_| version: v3.17.1
A Helm-Charts-as-Code tool.
2024-10-23 12:48:46 INFO: Parsed [[ tenant-asas-dev.yaml ]] successfully and found [ 2 ] apps
2024-10-23 12:48:46 INFO: Validating desired state definition
2024-10-23 12:48:46 INFO: Setting up kubectl
2024-10-23 12:48:46 INFO: Setting up helm
2024-10-23 12:48:48 INFO: Getting chart information
2024-10-23 12:48:48 ERROR: improper constraint: latest
2024-10-23 12:48:48 CRITICAL: chart validation failed
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