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

unknown flag when chart uninstalled #212

Closed
vmanyushin opened this issue Mar 12, 2020 · 5 comments · Fixed by #402
Closed

unknown flag when chart uninstalled #212

vmanyushin opened this issue Mar 12, 2020 · 5 comments · Fixed by #402

Comments

@vmanyushin
Copy link

BUG REPORT

ct version 3.0.0-beta2

What happened:

ct lint-and-install --chart-dirs $(pwd) --validate-maintainers=false --helm-extra-args="--values redis/ci/vault-values.yaml" --charts redis  --debug

...
Deleting release 'redis-5xuikf4s43'...
>>> helm uninstall redis-5xuikf4s43 --namespace redis-5xuikf4s43 --values redis/ci/vault-values.yaml
Error: unknown flag: --values
Error deleting Helm release: Error waiting for process: exit status 1
Deleting namespace 'redis-5xuikf4s43'...
>>> kubectl delete namespace redis-5xuikf4s43 --timeout 180s
namespace "redis-5xuikf4s43" deleted
>>> kubectl get namespace redis-5xuikf4s43
Namespace 'redis-5xuikf4s43' terminated.
Error: Error linting and installing charts: Error processing charts

i think extra args must not be present in uninstall part.

@unguiculus
Copy link
Member

This is because you pass that in as --helm-axtra-args. Values files for testing chart must be in the ci folder within the chart's directory. Extra args are not meant to be used to pass in values files. This flag was originally added so one could pass in TLS args with Helm 2.

@rawkode
Copy link

rawkode commented Jul 8, 2020

@unguiculus That's unfortunate, as we're hitting this problem too. We need to be able to use --set, which fails during uninstall, to pass through a secret that is required for one of the charts to pass successfully.

@itewk
Copy link
Contributor

itewk commented Nov 20, 2020

related but not exactly the same...#301

@ahmetkaftan
Copy link

I've also hit the same issue when I used --set flag in extraArgs

Deleting release 'mychart-5uade0o099'...
>>> helm uninstall mychart-5uade0o099 --namespace mychart-5uade0o099  --set kafka.replicas=1 --set cassandra.jvm.maxHeapSize=1G --set cassandra.jvm.newHeapSize=200M --timeout 10m
Error: unknown flag: --set

@smlx
Copy link
Contributor

smlx commented Dec 14, 2020

I think this is a duplicate of #171

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