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

feat(fastly_secretstore): implement resource and documentation #707

Merged
merged 6 commits into from
Aug 18, 2023

Conversation

Integralist
Copy link
Collaborator

@Integralist Integralist commented Jun 9, 2023

This PR implements a new fastly_secretstore resource and fastly_secretstores data source.

✅ The full end-to-end integration test suite has been run and is passing successfully.

@Integralist Integralist added the enhancement New feature or request label Jun 9, 2023
@Integralist Integralist marked this pull request as ready for review June 9, 2023 13:32
@Integralist Integralist requested a review from mccurdyc June 9, 2023 13:32
Copy link
Collaborator

@mccurdyc mccurdyc left a comment

Choose a reason for hiding this comment

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

One non-blocking question

fastly/resource_fastly_secretstore_test.go Show resolved Hide resolved
@jfctfl
Copy link

jfctfl commented Jul 14, 2023

Hi, glad to see this coming! We could really use this for our terraform deployment, are there any plans on when this will be merged and available?

@Integralist
Copy link
Collaborator Author

Thanks @jfctfl

I'm just at this moment chasing up feedback on when I'm able to merge this PR and publish it into a new release.

Unfortunately I don't have a timeline yet. Hopefully not too long.

@jfctfl
Copy link

jfctfl commented Jul 14, 2023

Awesome 🚀
If you have any updates (also on the timeline), would appreciate if you could share those!

@Integralist Integralist force-pushed the integralist/secret-store branch 3 times, most recently from cd435f3 to 6404f7a Compare August 4, 2023 16:20
@Integralist Integralist merged commit 5bcdcd4 into main Aug 18, 2023
7 checks passed
@Integralist Integralist deleted the integralist/secret-store branch August 18, 2023 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants