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

Update Vendored Vault Dependency to 1.1.x #333

Closed
lawliet89 opened this issue Mar 1, 2019 · 6 comments · Fixed by #413
Closed

Update Vendored Vault Dependency to 1.1.x #333

lawliet89 opened this issue Mar 1, 2019 · 6 comments · Fixed by #413

Comments

@lawliet89
Copy link
Contributor

lawliet89 commented Mar 1, 2019

I was trying to implement some new features for Vault 1.0 but I realised that the vendored Vault dependency is at 0.11.1 (!!!).

I tried taking a stab at this and I managed to build Vault but I am running into some nasty error running tests where Vault would report

2019-03-01T10:10:02.275+0800 [DEBUG] core: non-timeout error accepting on cluster port: error="accept tcp 127.0.0.1:8201: accept4: too many open files"

and the tests never complete running. Somewhere along the way, some misbehaving dependency might be causing this.

I am new to Go and I have no clue how to get govendor to update Vault and all of its transitive dependencies. Could I get some help on this please?

@tyrannosaurus-becks

@lawliet89 lawliet89 changed the title Update Vendored Vault Dependency to 1.0 Update Vendored Vault Dependency to 1.0.x Mar 1, 2019
@cvbarros
Copy link
Contributor

cvbarros commented Mar 15, 2019

Resolved via #292

@lawliet89
Copy link
Contributor Author

It's not resolved by that. The vendored libraries were not updated.

@cvbarros
Copy link
Contributor

Thats correct, just confirmed vendored dependency to 0.11. Got mistaken because I actually run the tests against 1.0.3 container and they work 🤷‍♂️

@lawliet89
Copy link
Contributor Author

lawliet89 commented Mar 15, 2019 via email

@cvbarros
Copy link
Contributor

Better wait for #356 , it will make it a lot easier to manage dependencies moving forward.

@lawliet89 lawliet89 changed the title Update Vendored Vault Dependency to 1.0.x Update Vendored Vault Dependency to 1.1.x Mar 30, 2019
@lawliet89
Copy link
Contributor Author

lawliet89 commented May 8, 2019

With the switch to go modules it's slightly easier to do this probably.

In any case, I tried to upgrade to v1.1.2 go get github.com/hashicorp/vault@v1.1.2 and I am getting

build github.com/terraform-providers/terraform-provider-vault: cannot load github.com/hashicorp/vault/api: ambiguous import: found github.com/hashicorp/vault/api in multiple modules:
	github.com/hashicorp/vault v1.1.2 (/home/yongwen/go/pkg/mod/github.com/hashicorp/vault@v1.1.2/api)
	github.com/hashicorp/vault/api v1.0.1 (/home/yongwen/go/pkg/mod/github.com/hashicorp/vault/api@v1.0.1)

Is @tyrannosaurus-becks able to provide any advice for this?

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

Successfully merging a pull request may close this issue.

2 participants