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

Data source secrethub_secret #6

Merged
merged 6 commits into from
Feb 25, 2019
Merged

Data source secrethub_secret #6

merged 6 commits into from
Feb 25, 2019

Conversation

florisvdg
Copy link
Member

New data source secrethub_secret to be able to use secrets that are created elsewhere.

The implementation is unfortunately not ideal because it will flood the secret's audit trail due to the way Terraform reads data sources hashicorp/terraform#13044 (comment).

There are plans to change the data source lifecycle, which can be a big improvement for our use case:
hashicorp/terraform#17034.

@florisvdg
Copy link
Member Author

I also opened a feature request issue on the Terraform repo https://github.com/hashicorp/terraform/issues/20391. Maybe they have a workaround for it.

Copy link
Member

@SimonBarendse SimonBarendse left a comment

Choose a reason for hiding this comment

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

The version arg does not work yet. Other than that good to go imo 👍

secrethub/data_source_secret.go Outdated Show resolved Hide resolved
Copy link
Member

@SimonBarendse SimonBarendse left a comment

Choose a reason for hiding this comment

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

Confirmed that versioning now works

One tiny comment about a redundant check

secrethub/resource_secret.go Outdated Show resolved Hide resolved
@florisvdg florisvdg merged commit 4b78ec1 into develop Feb 25, 2019
@florisvdg florisvdg deleted the secret-datasource branch February 25, 2019 12:06
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.

2 participants