first pass edit on architecture doc #212
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
Yes ELK should be changed to the Elastic Stack. |
Member
|
Changes look good here. Are you going to remove tech preview description in this PR or another. I need some way to know when the PR switches from a work-in-progress to ready to commit 😀 |
Contributor
Author
|
I had already removed the reference to the Technology Preview before Monica made her comment. But, I understand your need to know when I am done incorporating edits. I will make a note of my status after each review comment. So, unless others have further input, from my point of view, this doc is done. |
mriccell
approved these changes
Apr 27, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The architecture doc refers to ELK which is now referred to as the Elastic Stack. Do you want to keep the older term (throughout) or update it?
Under, "This diagram shows the following details" please review the bulleted item regarding, "An Ingress is created for each WebLogic cluster." In particular, this sentence, "In the technology preview release, there is one instance of the load balancer running for each WebLogic cluster, and the load balancers are configured with the root URL path (“/”). More flexible load balancer configuration is planned for a future release."