Skip to content

Conversation

kurtmc
Copy link

@kurtmc kurtmc commented Jan 19, 2021

what

This adds the ability to restore redis to snapshots created using the AWS implementation of backups for this service.

why

Need a way to restore redis to from snapshots.

references

Relates to #96

@kurtmc kurtmc requested review from a team as code owners January 19, 2021 20:39
@kurtmc kurtmc requested review from Makeshift and brcnblc and removed request for a team January 19, 2021 20:39
@kurtmc kurtmc force-pushed the feature/restore-snapshot-name branch from 35a684e to 077099f Compare January 19, 2021 20:55
@kurtmc kurtmc requested a review from a team as a code owner January 19, 2021 20:55
@SweetOps
Copy link
Contributor

/rebuild-readme

@SweetOps
Copy link
Contributor

LGTM, just one thing need to fix

Co-authored-by: Vladimir <[email protected]>
@mergify mergify bot dismissed SweetOps’s stale review January 23, 2021 22:05

This Pull Request has been updated, so we're dismissing all reviews.

* Package terraform-docs already installed
Generated README.md from /home/kurt/halter/platform/terraform-modules/terraform-aws-elasticache-redis/build-harness/templates/README.md.gotmpl using data from /home/kurt/halter/platform/terraform-modules/terraform-aws-elasticache-redis/build-harness/templates/README.yaml
@kurtmc kurtmc requested review from SweetOps and removed request for a team January 24, 2021 19:19
@SweetOps
Copy link
Contributor

/test all

@SweetOps SweetOps merged commit 8fc0d98 into cloudposse:master Jan 25, 2021
@kurtmc kurtmc deleted the feature/restore-snapshot-name branch January 25, 2021 19:13
@Nuru Nuru mentioned this pull request Jul 12, 2021
brian-weis-msr pushed a commit to Measurabl/terraform-aws-elasticache-redis that referenced this pull request Apr 2, 2024
* Add snapshot name argument

* Update variables.tf

Co-authored-by: Vladimir <[email protected]>
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 this pull request may close these issues.

2 participants