Merge ~rafaeldtinoco/ubuntu/+source/ndctl:lp1811785-bionic into ubuntu/+source/ndctl:ubuntu/bionic-devel
Proposed by
Rafael David Tinoco
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Christian Ehrhardt | ||||
Proposed branch: | ~rafaeldtinoco/ubuntu/+source/ndctl:lp1811785-bionic | ||||
Merge into: | ubuntu/+source/ndctl:ubuntu/bionic-devel | ||||
Diff against target: |
119 lines (+99/-0) 3 files modified
debian/changelog (+6/-0) debian/patches/ndctl-init-labels-Fix-label-slot-accounting-per-UEFI.patch (+92/-0) debian/patches/series (+1/-0) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Rafael David Tinoco (community) | Needs Fixing | ||
Christian Ehrhardt (community) | Needs Fixing | ||
Review via email:
|
Commit message
An XML file for a KVM guest (example) can be found here:
https:/
And an example on how to test ndctl can be found here:
https:/
PPA: https:/
To post a comment you must log in.
I think the fix itself is good, but a few things need cleanup.
changelog: we usually name the patch before the colon: init-labels- Fix-label- slot-accounting -per-UEFI. patch: Fix label slot accounting per UEFI 2.7 (LP: #1811785)
so instead
* ndctl/init-labels: Fix label slot accounting per UEFI 2.7 (LP: #1811785)
maybe:
* d/p/ndctl-
Furthermore the bug should get a proper SRU Template before sponsoring it into -unapproved.