-
Notifications
You must be signed in to change notification settings - Fork 4
docs: architecture diagram #54
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
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.
Thanks for adding the architecture diagram! I'll make a couple of changes to some of the text that you didn't touch in Juju events and Charm code overview.
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.
One last nitpick from me :) let me know if you have any concerns about the commit I made to the ### Jenkins agent K8s section!
Co-authored-by: Erin Conley <erin.conley@canonical.com>
This reverts commit 6670c3c.
Test results for commit 18eaafaTest coverage for 18eaafa Static code analysis report |
Applicable spec:
Overview
Rationale
Juju Events Changes
Module Changes
Library Changes
Checklist
urgent,trivial,complex)