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 Valkey engine support for memorydb resources #40224

Open
wants to merge 35 commits into
base: main
Choose a base branch
from

Conversation

atkretsch
Copy link
Contributor

@atkretsch atkretsch commented Nov 20, 2024

Description

Adding support for the engine parameter for MemoryDB clusters, including allowing valkey. Picks up where @sasidhar-aws left off with #39939.

Relations

Closes #39939.
Closes #39644.

References

AWS CreateCluster API

Output from Acceptance Testing

make testacc TESTS=TestAccMemoryDBSnapshot_basic PKG=memorydb
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/memorydb/... -v -count 1 -parallel 20 -run='TestAccMemoryDBSnapshot_basic' -timeout 360m
2024/11/20 16:11:59 Initializing Terraform AWS Provider...
=== RUN TestAccMemoryDBSnapshot_basic
=== PAUSE TestAccMemoryDBSnapshot_basic
=== CONT TestAccMemoryDBSnapshot_basic
--- PASS: TestAccMemoryDBSnapshot_basic (2645.29s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/memorydb 2661.259s

make testacc TESTS=TestAccMemoryDBClusterDataSource_basic PKG=memorydb
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/memorydb/... -v -count 1 -parallel 20 -run='TestAccMemoryDBClusterDataSource_basic' -timeout 360m
2024/11/20 16:11:51 Initializing Terraform AWS Provider...
=== RUN TestAccMemoryDBClusterDataSource_basic
=== PAUSE TestAccMemoryDBClusterDataSource_basic
=== CONT TestAccMemoryDBClusterDataSource_basic
--- PASS: TestAccMemoryDBClusterDataSource_basic (1621.70s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/memorydb 1629.811s

make testacc TESTS=TestAccMemoryDBCluster_basic PKG=memorydb
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/memorydb/... -v -count 1 -parallel 20 -run='TestAccMemoryDBCluster_basic' -timeout 360m
2024/11/20 20:11:03 Initializing Terraform AWS Provider...
=== RUN TestAccMemoryDBCluster_basic
=== PAUSE TestAccMemoryDBCluster_basic
=== CONT TestAccMemoryDBCluster_basic
--- PASS: TestAccMemoryDBCluster_basic (1499.63s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/memorydb 1503.446s

make testacc TESTS=TestAccMemoryDBCluster_valkey PKG=memorydb
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/memorydb/... -v -count 1 -parallel 20 -run='TestAccMemoryDBCluster_valkey' -timeout 360m
2024/11/20 16:11:47 Initializing Terraform AWS Provider...
=== RUN TestAccMemoryDBCluster_valkey
=== PAUSE TestAccMemoryDBCluster_valkey
=== CONT TestAccMemoryDBCluster_valkey
--- PASS: TestAccMemoryDBCluster_valkey (1687.57s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/memorydb 1691.732s

make testacc TESTS=TestAccMemoryDBCluster_Update_engine PKG=memorydb
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/memorydb/... -v -count 1 -parallel 20 -run='TestAccMemoryDBCluster_Update_engine' -timeout 360m
2024/11/20 16:11:55 Initializing Terraform AWS Provider...
=== RUN TestAccMemoryDBCluster_Update_engine
=== PAUSE TestAccMemoryDBCluster_Update_engine
=== CONT TestAccMemoryDBCluster_Update_engine
--- PASS: TestAccMemoryDBCluster_Update_engine (3191.70s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/memorydb 3203.758s

@atkretsch atkretsch requested a review from a team as a code owner November 20, 2024 22:26
Copy link

Community Note

Voting for Prioritization

  • Please vote on this pull request by adding a 👍 reaction to the original post to help the community and maintainers prioritize this pull request.
  • Please see our prioritization guide for information on how we prioritize.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

For Submitters

  • Review the contribution guide relating to the type of change you are making to ensure all of the necessary steps have been taken.
  • For new resources and data sources, use skaff to generate scaffolding with comments detailing common expectations.
  • Whether or not the branch has been rebased will not impact prioritization, but doing so is always a welcome surprise.

@github-actions github-actions bot added documentation Introduces or discusses updates to documentation. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. service/memorydb Issues and PRs that pertain to the memorydb service. needs-triage Waiting for first response or review from a maintainer. labels Nov 20, 2024
@atkretsch atkretsch changed the title F memorydb valkey Add Valkey engine support for memorydb resources Nov 21, 2024
@ewbankkit ewbankkit added enhancement Requests to existing resources that expand the functionality or scope. and removed needs-triage Waiting for first response or review from a maintainer. labels Nov 21, 2024
@ewbankkit ewbankkit self-assigned this Nov 21, 2024
@github-actions github-actions bot added the prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Introduces or discusses updates to documentation. enhancement Requests to existing resources that expand the functionality or scope. prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. service/memorydb Issues and PRs that pertain to the memorydb service. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Amazon MemoryDB for Valkey
3 participants