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

Keyring constantly changing #52

Closed
apenney opened this issue Mar 5, 2018 · 7 comments · Fixed by #193
Closed

Keyring constantly changing #52

apenney opened this issue Mar 5, 2018 · 7 comments · Fixed by #193

Comments

@apenney
Copy link

apenney commented Mar 5, 2018

Hi,

Is there a way to force an attribute for all instances of helm_release{}? Every time we run terraform as a different user it wants to change keyring in every helm_release, which is kind of annoying. It's a value we don't set, but it defaults to your homedirectory. Failing that, if it stored it as ~/.gnupg instead of the expanded path, it wouldn't need to change for different users.

@burdiyan
Copy link
Contributor

I think this was addressed in some of the PRs that was adding verify option to the release, which is now by default is set to false.

https://github.com/mcuadros/terraform-provider-helm/blob/master/helm/resource_release.go#L95

Is it still happening in version 0.5 of the provider? @apenney

@apenney
Copy link
Author

apenney commented Mar 22, 2018

I believe it still happens in 0.5! There's two of us that have verified we're on 0.5 and we've seen it flip between us. I'll check in with the third person and update this ticket again to make sure he's not a straggler on 0.4!

Oh, and one more thing we noticed:

+ provider.helm (unversioned)

We had to guesstimate by release date :)

@apenney
Copy link
Author

apenney commented Mar 22, 2018

Checked, all 3 of us were on 0.5.

@burdiyan
Copy link
Contributor

Yes it still happening, although validate is set to false, the default value for keyring is changing constantly on different environments. Maybe the provider shouldn't store keyring in Terraform state file at all and use what's setup in the current environment.

@apenney
Copy link
Author

apenney commented Mar 22, 2018

We'd be 👍 on that!

@vcardenas
Copy link

As a workaround, I'm setting the keyring empty.

resource "helm_release" "my_release" {
    ...
    keyring = ""
    ...
}

That cause the provider to ignore it hence not causing the updates (in 0.5.0)

@walkafwalka
Copy link

This is still present in 0.7.

@ghost ghost locked and limited conversation to collaborators Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants