feat(redis): expose logging configuration

[pivotal-marcela-campo]

slow logs and engine logs can be sent to cloudwatch

#184471699

Checklist:

• Have you added Release Notes in the docs repositories?
• Have you followed the Conventional Commits specification?

[blgm]

Happy for this to be merged, but I would suggest improving the initialisation of values in the Terraform tests to avoid unexpected issues: https://onsi.github.io/ginkgo/#avoid-spec-pollution-dont-initialize-variables-in-container-nodes

Another thought is that I don't think you necessarily get much value from a test that does:

Context("...", Ordered, func(){
  BeforeAll(func(){
    ...setup
  })

  It("test1",func(){
   ...
  })

  It("test2",func() {
    ...
  })
})

Over:

It("...",func() {
  ..setup

  By("step 1")
  ....

  By("step 2")
  ...
})

I wonder if these tests would be simpler to read and maintain if written in the latter style.

[blgm]

Because this is an ordered test, setting values outside of a BeforeEach() works, but I'd strongly recommend against it because the test starting on line 381 has:

"retention_in_days": BeNumerically("==", retentionInDays),

If another piece of code were to update the value of retentionInDays then the test would continue to pass, but it would not be testing what you think it's testing.

[pivotal-marcela-campo]

Thanks, fixing this