-
Notifications
You must be signed in to change notification settings - Fork 956
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
Optimize the jobflow architecture design diagram #3016
Optimize the jobflow architecture design diagram #3016
Conversation
f4a7c07
to
992cd67
Compare
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.
@wangyang0616 Thanks for your hard work! The new architecture turns to be more clear than the older ones. They not only highlight the added compontents but also mark the resource relationship.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Thor-wl The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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.
/hold
docs/design/jobflow/README.md
Outdated
![jobflow-1.png](../images/jobflow-1.png) | ||
|
||
![jobflow-3.png](../images/jobflow-3.png) | ||
![jobflow-1.png](../images/jobflow-2.png) |
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.
![jobflow-1.png](../images/jobflow-2.png) | |
![jobflow-2.png](../images/jobflow-2.png) |
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.
done
docs/design/jobflow/README.md
Outdated
@@ -58,7 +56,7 @@ Deleting a jobflow when the jobflow is in a non-complete state will be intercept | |||
|
|||
### Controller | |||
|
|||
![jobflow-4.png](../images/jobflow-4.png) | |||
![jobflow-4.png](../images/jobflow-3.png) |
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.
![jobflow-4.png](../images/jobflow-3.png) | |
![jobflow-3.png](../images/jobflow-3.png) |
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.
done
…rly and concisely Signed-off-by: wangyang <wangyang8126@gmail.com>
992cd67
to
ae3c9b3
Compare
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.
/lgtm
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.
/lgtm
/hold cancel
![jobflow-3.png](../images/jobflow-3.png) | ||
|
||
The blue part is the component of k8s itself, the green and brown are the components of volcano, and the yellow is the crd resource of volcano. | ||
The blue part is the components of k8s itself, the orange is the existing definition of Volcano, and the red is the new definition of JobFlow. |
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.
please update the description. It is out of date after refreshing the jobflow-2.jpg
Adopt a more concise color scheme and optimize the color and style of the architecture diagram.