Skip to content

Conversation

jim80net
Copy link
Contributor

@jim80net jim80net commented Feb 5, 2020

This PR adds the following functionality when setting cluster_mode_enabled == true:

  • Automatically set number of nodes in cluster_size, enabling scale-out with a single parameter change.
  • Automatically set the cluster-enabled parameter as an override-able default.

This PR makes the following fixes when setting cluster_mode_enabled == true:

  • correct the endpoint keyname to fix a null value error.

@jim80net jim80net requested a review from aknysh February 5, 2020 04:21
@aknysh
Copy link
Member

aknysh commented Feb 6, 2020

/codefresh run test

@aknysh
Copy link
Member

aknysh commented Feb 6, 2020

/codefresh run test

@aknysh
Copy link
Member

aknysh commented Feb 6, 2020

@jim80net

please rebuild README by executing these commands:

make init
make readme/deps
make readme

@jim80net
Copy link
Contributor Author

jim80net commented Feb 6, 2020

Could it be a failing test?

2020-02-06T03:53:36Z |ruby-2.6.3| MacBook-Pro-(4) in ~/workspace/github.com/scribd/terraform-aws-elasticache-redis
±  |cluster_enabled_in_default_parameter_group ✓| → git co 8afc1f80978e146d6bbc728976e37b2702f64621
Note: checking out '8afc1f80978e146d6bbc728976e37b2702f64621'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 8afc1f8 chore: terraform fmt

2020-02-06T03:54:42Z |ruby-2.6.3| MacBook-Pro-(4) in ~/workspace/github.com/scribd/terraform-aws-elasticache-redis
±  |detached:cluster_enabled_in_default_parameter_group ✓| → make init
exit 0
Removing existing build-harness
Cloning https://github.com/cloudposse/build-harness.git#master...
Cloning into 'build-harness'...
remote: Enumerating objects: 22, done.
remote: Counting objects: 100% (22/22), done.
remote: Compressing objects: 100% (17/17), done.
remote: Total 1249 (delta 7), reused 10 (delta 5), pack-reused 1227
Receiving objects: 100% (1249/1249), 304.62 KiB | 1.51 MiB/s, done.
Resolving deltas: 100% (662/662), done.

2020-02-06T03:55:02Z |ruby-2.6.3| MacBook-Pro-(4) in ~/workspace/github.com/scribd/terraform-aws-elasticache-redis
±  |detached:cluster_enabled_in_default_parameter_group ✓| → make readme/deps
Installing packages 0.95.0...
Cloning into '/Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/vendor/packages'...
remote: Enumerating objects: 293, done.
remote: Counting objects: 100% (293/293), done.
remote: Compressing objects: 100% (144/144), done.
remote: Total 293 (delta 52), reused 245 (delta 43), pack-reused 0
Receiving objects: 100% (293/293), 52.60 KiB | 681.00 KiB/s, done.
Resolving deltas: 100% (52/52), done.
curl --retry 3 --retry-delay 5 --fail -sSL -o /Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/vendor/gomplate https://github.com/hairyhenderson/gomplate/releases/download/v3.1.0/gomplate_darwin-amd64-slim && chmod +x /Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/vendor/gomplate

2020-02-06T03:55:24Z |ruby-2.6.3| MacBook-Pro-(4) in ~/workspace/github.com/scribd/terraform-aws-elasticache-redis
±  |detached:cluster_enabled_in_default_parameter_group ✓| → make readme
curl --retry 3 --retry-delay 5 --fail -sSL -o /Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/vendor/terraform-docs https://github.com/segmentio/terraform-docs/releases/download/v0.4.5/terraform-docs-v0.4.5-darwin-amd64 && chmod +x /Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/vendor/terraform-docs
Generated README.md from /Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/templates/README.md.gotmpl using data from /Users/jimp/workspace/github.com/scribd/terraform-aws-elasticache-redis/build-harness/templates/README.yaml

2020-02-06T03:56:02Z |ruby-2.6.3| MacBook-Pro-(4) in ~/workspace/github.com/scribd/terraform-aws-elasticache-redis
±  |detached:cluster_enabled_in_default_parameter_group ✓| → git status --porcelain && echo clean || echo dirty
clean

@maximmi
Copy link
Contributor

maximmi commented Feb 11, 2020

/codefresh run test

Copy link
Contributor

@maximmi maximmi left a comment

Choose a reason for hiding this comment

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

thanks @jim80net

@maximmi maximmi merged commit ea7760b into cloudposse:master Feb 12, 2020
brian-weis-msr pushed a commit to Measurabl/terraform-aws-elasticache-redis that referenced this pull request Apr 2, 2024
* feat: add ClusterEnabled: true default parameter (unless overwritten)

* fix: endpoint key changes when cluster mode enabled

* feat: set number_cache_clusters when in cluster_mode_enabled

* chore: terraform fmt
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.

3 participants