1Password / connect-helm-charts

Official 1Password Helm Charts
https://developer.1password.com
MIT License
93 stars 74 forks source link

operator connect fails #16

Closed mmarkell closed 3 years ago

mmarkell commented 3 years ago

When I run this command for an already existing service

helm upgrade --set operator.token=<token> connect 1password/connect

I get this error:

coalesce.go:200: warning: cannot overwrite table with non table for token (map[key:token name:onepassword-token value:<nil>])
Error: UPGRADE FAILED: template: connect/templates/operator-token.yaml:1:45: executing "connect/templates/operator-token.yaml" at <.Values.operator.token.value>: can't evaluate field value in type interface {}
mmarkell commented 3 years ago

I think it's supposed to be operator.token.value=<token>

Can we please fix the readme to have this syntax?

jpcoenen commented 3 years ago

You are very right, that seems to be a typo. I've opened a PR to make this clearer :grinning:

Thanks for your feedback!