-
Notifications
You must be signed in to change notification settings - Fork 10
reference: snap lifecycle documentation #78
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
base: main
Are you sure you want to change the base?
Conversation
4c0ce31 to
8e13305
Compare
Signed-off-by: Andy Whitcroft <[email protected]>
8e13305 to
fbd0f3e
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.
Hi @awhitcroft , apologies for the delay.
I left some comments but they are mostly suggestions about the formatting; less concerns about the content.
Thanks!
| ``` | ||
|
|
||
| ```{toctree} | ||
| :maxdepth: 2 |
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.
Hi @awhitcroft , was there a reason for setting :maxdepth: 2 here?
For landing pages I would suggest keeping it clean and just having the title displayed.
| @@ -0,0 +1,150 @@ | |||
| ************** | |||
| Snap Lifecycle | |||
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.
| Snap Lifecycle | |
| Snap lifecycle |
Ubuntu style guide recommends we use sentence case headings. Same comment for the rest of the doc.
https://docs.ubuntu.com/styleguide/en/#headings
Thanks.
| Snap Lifecycle | ||
| ************** | ||
|
|
||
| Snap Build Lifecycle |
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.
| Snap Build Lifecycle | |
| Snap build lifecycle |
| ************** | ||
| Snap Lifecycle | ||
| ************** | ||
|
|
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.
| This guide describes the build and workflow lifecycle of kernel snaps. |
Ubuntu style guide also advises against stacked headings, e.g. headings need to be followed by content.
https://docs.ubuntu.com/styleguide/en/#headings
| Background | ||
| ---------- | ||
| The majority of kernels with snaps are consumed both as Debian packages and | ||
| those snaps. To reduce testing requirements and streamline production the same |
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.
| those snaps. To reduce testing requirements and streamline production the same | |
| snaps. To reduce testing requirements and streamline production, the same |
| Testing | ||
| ------- |
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.
| Testing | |
| ------- | |
| Testing | |
| ------- | |
| QA Testing | ||
| ---------- |
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.
| QA Testing | |
| ---------- | |
| QA testing | |
| ---------- | |
|
|
||
| QA Testing | ||
| ---------- | ||
| Once we have a formal canidate snap this may be sent for further acceptance |
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.
| Once we have a formal canidate snap this may be sent for further acceptance | |
| Once we have a formal candidate snap this may be sent for further acceptance |
| Release | ||
| ------- |
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.
| Release | |
| ------- | |
| Release | |
| ------- | |
| with the promotion of the Debian package to ``updates``. The Debian package my | ||
| then promote further to ``security`` but there is no equivalent channel for |
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.
| with the promotion of the Debian package to ``updates``. The Debian package my | |
| then promote further to ``security`` but there is no equivalent channel for | |
| with the promotion of the Debian package to ``updates``. The Debian package | |
| may then promote further to ``security`` but there is no equivalent channel for |
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.
Hi @awhitcroft , apologies for the delay.
I left some comments but they are mostly suggestions about the formatting; less concerns about the content.
Thanks!
|
There are also a few words that need to be added to Thanks! |
| parameterises the snapcraft.yaml configuration, and kicks off builds against | ||
| the appropriate snap build recipe. This causes the kernel to be processed into | ||
| a snap and uploaded to the snap-store. The store will automatically publish this | ||
| onto the ``edge`` channel for testing. |
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.
If there are test failures for the snap in "edge", does that gate the progress of the Debian package to "proposed"?
Initial documentation for the Snap lifecycles. This include the build lifecycle and the workflow lifecycle.