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

Add vault_gcp_secret_roleset #312

Merged
merged 4 commits into from
Mar 29, 2019

Conversation

lawliet89
Copy link
Contributor

@lawliet89 lawliet89 commented Feb 21, 2019

To test this, we need to set GOOGLE_CREDENTIALS as a path to the JSON file containing GCP credentials, and GOOGLE_PROJECT as the name of the GCP project to run the tests against.

I have tested it locally and the test pass.

The bindings could have been passed to Vault as JSON, but it is currently bugged.

make testacc TESTARGS='-run=TestGCPSecretRoleset'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -run=TestGCPSecretRoleset -timeout 120m
?   	github.com/terraform-providers/terraform-provider-vault	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/terraform-providers/terraform-provider-vault/util	(cached) [no tests to run]
=== RUN   TestGCPSecretRoleset
--- PASS: TestGCPSecretRoleset (26.46s)
PASS
ok  	github.com/terraform-providers/terraform-provider-vault/vault	26.473s

@lawliet89 lawliet89 force-pushed the gcp-secret-roleset branch 3 times, most recently from e2efabf to bc74e56 Compare February 25, 2019 05:40
@lawliet89 lawliet89 force-pushed the gcp-secret-roleset branch 2 times, most recently from adec7f2 to bc1fdb5 Compare February 25, 2019 06:36
@tyrannosaurus-becks
Copy link
Contributor

tyrannosaurus-becks commented Mar 5, 2019

Thanks @lawliet89 ! This looks great! Should we wait until hashicorp/vault-plugin-secrets-gcp#28 lands and then use project instead of service_account_project?

@lawliet89
Copy link
Contributor Author

@tyrannosaurus-becks I've added code to handle both cases since that change might take a while to land in Vault and not everyone will upgrade right away.

@lawliet89
Copy link
Contributor Author

Failure is due to unrelated TestDataSourcePolicyDocument

Copy link
Contributor

@tyrannosaurus-becks tyrannosaurus-becks left a comment

Choose a reason for hiding this comment

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

This looks fantastic! I merged master and found the tests pass. Thank you.

@tyrannosaurus-becks tyrannosaurus-becks merged commit 1162222 into hashicorp:master Mar 29, 2019
@fatmcgav
Copy link
Contributor

fatmcgav commented Apr 4, 2019

@tyrannosaurus-becks Is there a feel for when 1.7.0 will be released that includes this really useful new resource?

Edit: Ignore that comment, I've just refreshed the repo and noticed that 1.7.0 was released yesterday!!! Sorry for the noise!

dandandy pushed a commit to dandandy/terraform-provider-vault that referenced this pull request Jun 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants