-
Notifications
You must be signed in to change notification settings - Fork 40
Updating high-level architecture documentation #411
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
Conversation
|
Note that this will need Joe's contributions on SUMA and the LCM/Upgrade Controller stuff to merge first as we're currently failing tests due to missing sections. |
bc59bf6 to
9616ce0
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.
Small nits.
Co-authored-by: Eduardo Mínguez <[email protected]>
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
|
|
||
| * <<components-rancher,Rancher>> | ||
| * <<components-rancher-dashboard-extensions,Rancher Dashboard Extensions>> | ||
| * SUSE Manager |
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.
Should we also add the upgrade controller?
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.
Yes, for sure, we should.
Co-authored-by: Atanas Dinov <[email protected]>
* Updating high-level architecture documentation * Apply suggestions from code review Co-authored-by: Eduardo Mínguez <[email protected]> * Apply suggestions from code review Co-authored-by: Atanas Dinov <[email protected]> --------- Co-authored-by: Eduardo Mínguez <[email protected]> Co-authored-by: Atanas Dinov <[email protected]>
No description provided.