Fix RedisAutoConfig Exclusion logic #10728
Conversation
|
going to update this to use this property instead |
| ## Comment out the line below to enable redis caching | ||
| spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.data.redis.RedisAutoConfiguration |
There was a problem hiding this comment.
is this going to cause an issue in our prod environment? I think we're still using .EXAMPLE for defaults?
There was a problem hiding this comment.
I guess it looks like we're overriding it everywhere anyway: https://github.com/search?q=repo%3Aknowledgesystems%2Fknowledgesystems-k8s-deployment%20spring.autoconfigure.exclude&type=code
There was a problem hiding this comment.
It shouldn't be needed anymore
There was a problem hiding this comment.
It should exlude RedisAutoConfig if the property is not present
|
inodb
changed the title
inodb
changed the title
Currently If users wanted to just configure Spring Redis Session it was impossible with out enabling redis cache type.
Updated conditional logic to look at the presence of spring.data.redis.host instead