-
Notifications
You must be signed in to change notification settings - Fork 23
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
revisions as a training exercise #329
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marking this as "request changes" but we can discuss on standup. Lots of improvements here and some things for us to sort out together.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few small changes but looks close and definitely an improvement. I only commented on next but same will apply to 6.3.2
|
||
![Redis Caching](./redis-caching.png) | ||
|
||
The Redis cache is not deployed by the Entando Operator and must be managed by the implementing teams dev ops or Kubernetes cluster administrators. | ||
The Redis cache is not deployed by the Entando Operator and must be managed by the DevOps of implementation teams or Kubernetes cluster administrators. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe "DevOps team member" instead of "DevOps of implementation teams" which reads a bit awkward.
stumbling to the finish line...