azurerm_redis_cache - add support for notify-keyspace-events
#949
Conversation
bailantaotao
force-pushed
the
feature/add-notify-keyspace-events-for-redis-configuration
branch
from
ea2b9df
to
ab68ea8
Compare
bailantaotao
force-pushed
the
feature/add-notify-keyspace-events-for-redis-configuration
branch
from
ab68ea8
to
86c4b1b
Compare
There was a problem hiding this comment.
hey @bailantaotao
Thanks for this PR - I've taken a look through and this LGTM - thanks for this 👍
Thanks!
tombuildsstuff
changed the title
azurerm_redis_cache - add support for notify-keyspace-events
| name = "unlikely23exst2acct%s" | ||
| resource_group_name = "${azurerm_resource_group.test.name}" | ||
| location = "${azurerm_resource_group.test.location}" | ||
| account_type = "Standard_GRS" |
There was a problem hiding this comment.
in running the tests I noticed this field needs to be split into the two sub-fields - I'll push a commit to fix this :)
account_tier = "Standard"
account_replication_type = "GRS"
There was a problem hiding this comment.
Oh...thanks for your help :D
|
Tests pass:
|
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
add new configuration about
notify-keyspace-eventson Redis