Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

update dependencies #11

Merged
merged 1 commit into from
Aug 15, 2022
Merged

update dependencies #11

merged 1 commit into from
Aug 15, 2022

Conversation

hirosassa
Copy link

@hirosassa hirosassa commented Aug 14, 2022

Updated dependencies for following reasons:

  • To follow newer go version
  • To use workload identity federation by using newer version of golang.org/x/oauth2

Using current latest berglas provider with a credential obtained from workload identity federation, we saw following error. I would like to resolve it.

Error: failed to configure provider: failed to parse credentials: unknown credential type: "external_account"
with provider["registry.terraform.io/sethvargo/berglas"]

cloud.google.com/go/secretmanager v0.1.0 // indirect
cloud.google.com/go/storage v1.16.0 // indirect
github.com/GoogleCloudPlatform/berglas v0.6.0
github.com/GoogleCloudPlatform/berglas v0.6.3-0.20220726215926-83e4b9214bea
Copy link
Author

@hirosassa hirosassa Aug 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could not build this provider by using current latest tagged version of berglas client due to following error:

../../go/pkg/mod/github.com/!google!cloud!platform/berglas@v0.6.1/pkg/berglas/iam_storage.go:201:12: assignment mismatch: 2 variables but retry.NewFibonacci returns 1 value

This error comes from this change, so I referred latest commit of the client for now.

@sethvargo sethvargo merged commit 6a05617 into sethvargo:main Aug 15, 2022
@hirosassa hirosassa deleted the update-deps branch August 15, 2022 21:27
@hirosassa
Copy link
Author

@sethvargo Thank you for the quick review and merge!

@hirosassa
Copy link
Author

@sethvargo v0.2.1 is not released to terraform registry yet?
https://registry.terraform.io/providers/sethvargo/berglas/0.2.0

@sethvargo
Copy link
Owner

@hirosassa so - I've manually refreshed it at least 3x now...

screenshot-20220815-OYxnBteM@2x

I think the problem might be that I have ECC GPG signing keys and it looks like the Terraform registry doesn't support ECC keys. I don't have RSA/DSA keys and don't have intentions of managing multiple GPG keys right now.

@hirosassa
Copy link
Author

hirosassa commented Aug 16, 2022

@sethvargo Thanks for switching back commit.
So can we re-run release job (by adding new tag?) to release to terraform registry?

@sethvargo
Copy link
Owner

That won't fix it - the Terraform registry doesn't support the signature I'm using.

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

Successfully merging this pull request may close these issues.

None yet

2 participants