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

Adding GCP Auth backend resource #198

Merged

Conversation

petems
Copy link
Contributor

@petems petems commented Oct 3, 2018

This is a largely a copy of the #99 backend config but for the GCP backend.

It only has 1 field, credentials. Probably needs more testing, I might need someone to pair with me on that 😄

@ghost ghost added the size/L label Oct 3, 2018
@petems petems force-pushed the add_gcp_auth_backend_resource branch 3 times, most recently from b505866 to 9c8dd3f Compare October 3, 2018 20:47
@ghost ghost added size/XL and removed size/L labels Oct 3, 2018
@petems petems force-pushed the add_gcp_auth_backend_resource branch 2 times, most recently from c8fcbd3 to 824205e Compare October 27, 2018 00:03
@petems petems force-pushed the add_gcp_auth_backend_resource branch 2 times, most recently from 3a9c994 to 172d778 Compare November 5, 2018 13:54
@tyrannosaurus-becks tyrannosaurus-becks self-assigned this Nov 7, 2018
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.

@petems this looks very good overall! The test is great, and it's passing. Just a couple of minor things and I'd be happy to merge it in.

vault/resource_gcp_auth_backend.go Show resolved Hide resolved
vault/resource_gcp_auth_backend.go Outdated Show resolved Hide resolved
vault/resource_gcp_auth_backend.go Outdated Show resolved Hide resolved
vault/resource_gcp_auth_backend_test.go Outdated Show resolved Hide resolved
vault/resource_gcp_auth_backend_test.go Show resolved Hide resolved
@petems petems force-pushed the add_gcp_auth_backend_resource branch 5 times, most recently from 26761d4 to 6c9abad Compare November 8, 2018 17:43
* Allows configuration of credentials for backend
@petems petems force-pushed the add_gcp_auth_backend_resource branch from 6c9abad to b767b03 Compare November 8, 2018 17:54
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! Thanks @petems!

@tyrannosaurus-becks tyrannosaurus-becks merged commit 42d55d7 into hashicorp:master Nov 9, 2018
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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants