Snappy

Merge lp://qastaging/~mvo/snappy/snappy-snapfs-cleanup-kernel-assets into lp://qastaging/~snappy-dev/snappy/snappy-moved-to-github

Proposed by Michael Vogt on 2015-10-20

Status:	Needs review
Proposed branch:	lp://qastaging/~mvo/snappy/snappy-snapfs-cleanup-kernel-assets
Merge into:	lp://qastaging/~snappy-dev/snappy/snappy-moved-to-github
Prerequisite:	lp://qastaging/~mvo/snappy/snappy-snapfs-needs-reboot
Diff against target:	64 lines (+38/-0) 3 files modified snappy/kernel.go (+11/-0) snappy/snapp.go (+7/-0) snappy/snapp_snapfs_test.go (+20/-0)
To merge this branch:	bzr merge lp://qastaging/~mvo/snappy/snappy-snapfs-cleanup-kernel-assets
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
John Lenton (community)		2015-10-20	Approve on 2015-10-20
Review via email: mp+274982@code.qastaging.launchpad.net

Cleanup kernel assets on remove

782. By Michael Vogt on 2015-10-20: remove kernel assets in /boot/ on snap removal

Revision history for this message

John Lenton (chipaca) wrote on 2015-10-20:

Looks good. But a question.

review: Approve

Revision history for this message

Michael Vogt (mvo) wrote on 2015-10-29:

Thanks! Fixed the code as suggested.

782. By Michael Vogt on 2015-10-20: remove kernel assets in /boot/ on snap removal
781. By Michael Vogt on 2015-10-20: show a reboot required message (not complete yet)
780. By Michael Vogt on 2015-10-20: go fmt
779. By Michael Vogt on 2015-10-20: partition/partition.go: IsNextBootOther() is always false on all-snap systems
778. By Michael Vogt on 2015-10-20: add missing tests for reboot-required reporting
777. By Michael Vogt on 2015-10-20: partition/bootloader.go: its ok to not have an "other" partition
776. By Michael Vogt on 2015-10-20: snappy/snapp.go: detect if kernel/os needs reboot
775. By Michael Vogt on 2015-10-20: partition/bootloader.go: do not crash if "/" is not system-{a,b}
774. By Michael Vogt on 2015-10-19: fix tests
773. By Michael Vogt on 2015-10-19: mered lp:snappy