Skip to content

Conversation

@rdoxenham
Copy link
Contributor

No description provided.

@rdoxenham
Copy link
Contributor Author

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.

Copy link
Collaborator

@e-minguez e-minguez left a 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]>
e-minguez
e-minguez previously approved these changes Oct 10, 2024
Copy link
Collaborator

@e-minguez e-minguez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

atanasdinov
atanasdinov previously approved these changes Oct 10, 2024

* <<components-rancher,Rancher>>
* <<components-rancher-dashboard-extensions,Rancher Dashboard Extensions>>
* SUSE Manager
Copy link
Contributor

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?

Copy link
Contributor Author

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.

@rdoxenham rdoxenham dismissed stale reviews from atanasdinov and e-minguez via 10a6c2a October 11, 2024 08:46
@rdoxenham rdoxenham merged commit ce3658b into suse-edge:main Oct 11, 2024
1 check passed
@rdoxenham rdoxenham deleted the arch-overview branch October 11, 2024 10:14
hardys pushed a commit to hardys/suse-edge.github.io that referenced this pull request Nov 21, 2024
* 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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants