zfs should enable autotrim
Bug #1883686 reported by
Dimitri John Ledkov
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| ubiquity (Ubuntu) |
Fix Released
|
Low
|
Unassigned | ||
| Focal |
Fix Released
|
Low
|
Unassigned | ||
Bug Description
[Impact]
* Ubuntu enables automatic TRIM / DISCARD on all filesystems that support it, such that on SSD-like storage devices the performance over time is improved. ZFS, as setup by ubiquity, did not used to enable autotrim option. But it should have it enabled.
[Test Case]
1. Complete install using zfs
2. Reboot and login
3. Check that the zfs pools have autotrim feature enabled with the following command:
# zpool get autotrim
Both pools must have the autotrim property set to 'on'.
NAME PROPERTY VALUE SOURCE
bpool autotrim on local
rpool autotrim on local
You can also check the status of trimming with:
# zpool status -t rpool
pool: rpool
state: ONLINE
scan: none requested
config:
NAME STATE READ WRITE CKSUM
rpool ONLINE 0 0 0
nvme0n1p4 ONLINE 0 0 0 (100% trimmed, completed at mer. 03 juin 2020 07:50:22)
[Regression Potential]
* When autotrim is in progress, i/o performance may be observed as stalling. However, periodic trim outweighs the benefits. Advanced operators may choose to disable autotrim, and execute it manually during regularly scheduled maintenance windows.
[Other Info]
* Shipped in groovy already
Related branches
~xnox/ubiquity:focal
- Ubuntu Stable Release Updates Team: Pending requested
- Ubuntu Installer Team: Pending requested
-
Diff: 28265 lines (+6214/-5241)140 files modifiedbin/ubiquity (+6/-4)
bin/ubiquity-dm (+1/-0)
debian/changelog (+20/-0)
debian/real-po/aa.po (+24/-23)
debian/real-po/ace.po (+20/-20)
debian/real-po/af.po (+53/-34)
debian/real-po/am.po (+20/-20)
debian/real-po/an.po (+19/-17)
debian/real-po/ar.po (+39/-35)
debian/real-po/as.po (+20/-20)
debian/real-po/ast.po (+51/-34)
debian/real-po/az.po (+20/-20)
debian/real-po/be.po (+41/-32)
debian/real-po/bem.po (+20/-20)
debian/real-po/bg.po (+23/-23)
debian/real-po/bn.po (+160/-135)
debian/real-po/bo.po (+180/-126)
debian/real-po/br.po (+26/-25)
debian/real-po/bs.po (+36/-26)
debian/real-po/ca.po (+36/-27)
debian/real-po/ca@valencia.po (+27/-25)
debian/real-po/ce.po (+19/-17)
debian/real-po/ckb.po (+54/-40)
debian/real-po/cs.po (+48/-33)
debian/real-po/csb.po (+26/-24)
debian/real-po/cv.po (+19/-17)
debian/real-po/cy.po (+38/-29)
debian/real-po/da.po (+46/-30)
debian/real-po/de.po (+45/-28)
debian/real-po/dv.po (+19/-17)
debian/real-po/dz.po (+19/-17)
debian/real-po/el.po (+52/-34)
debian/real-po/en_AU.po (+44/-33)
debian/real-po/en_CA.po (+44/-33)
debian/real-po/en_GB.po (+54/-35)
debian/real-po/eo.po (+36/-33)
debian/real-po/es.po (+53/-34)
debian/real-po/et.po (+42/-33)
debian/real-po/eu.po (+55/-39)
debian/real-po/fa.po (+92/-81)
debian/real-po/fi.po (+43/-26)
debian/real-po/fil.po (+28/-26)
debian/real-po/fo.po (+20/-20)
debian/real-po/fr.po (+48/-32)
debian/real-po/fr_CA.po (+40/-32)
debian/real-po/frp.po (+22/-21)
debian/real-po/fy.po (+28/-28)
debian/real-po/ga.po (+20/-20)
debian/real-po/gd.po (+40/-34)
debian/real-po/gl.po (+43/-28)
debian/real-po/gu.po (+30/-26)
debian/real-po/gv.po (+19/-17)
debian/real-po/he.po (+44/-29)
debian/real-po/hi.po (+157/-130)
debian/real-po/hr.po (+58/-42)
debian/real-po/hu.po (+49/-32)
debian/real-po/hy.po (+22/-22)
debian/real-po/ia.po (+40/-31)
debian/real-po/id.po (+57/-39)
debian/real-po/is.po (+32/-30)
debian/real-po/it.po (+52/-35)
debian/real-po/ja.po (+122/-200)
debian/real-po/ka.po (+20/-20)
debian/real-po/kab.po (+20/-20)
debian/real-po/kk.po (+36/-31)
debian/real-po/kl.po (+19/-17)
debian/real-po/km.po (+206/-130)
debian/real-po/kn.po (+35/-32)
debian/real-po/ko.po (+111/-175)
debian/real-po/ku.po (+44/-35)
debian/real-po/kw.po (+26/-25)
debian/real-po/ky.po (+20/-20)
debian/real-po/lb.po (+20/-20)
debian/real-po/lo.po (+19/-17)
debian/real-po/lt.po (+28/-21)
debian/real-po/lv.po (+43/-32)
debian/real-po/mg.po (+20/-20)
debian/real-po/mhr.po (+20/-20)
debian/real-po/mjw.po (+22/-22)
debian/real-po/mk.po (+21/-18)
debian/real-po/ml.po (+58/-45)
debian/real-po/mnw.po (+51/-37)
debian/real-po/mr.po (+142/-123)
debian/real-po/ms.po (+35/-27)
debian/real-po/my.po (+167/-112)
debian/real-po/nb.po (+43/-33)
debian/real-po/nds.po (+19/-17)
debian/real-po/ne.po (+157/-131)
debian/real-po/nl.po (+53/-35)
debian/real-po/nl_NL.po (+19/-18)
debian/real-po/nn.po (+39/-28)
debian/real-po/ny.po (+22/-21)
debian/real-po/oc.po (+59/-38)
debian/real-po/oj.po (+19/-17)
debian/real-po/om.po (+19/-17)
debian/real-po/or.po (+19/-17)
debian/real-po/os.po (+24/-22)
debian/real-po/pa.po (+41/-34)
debian/real-po/pl.po (+48/-36)
debian/real-po/pt.po (+37/-28)
debian/real-po/pt_BR.po (+26/-24)
debian/real-po/ro.po (+27/-24)
debian/real-po/ru.po (+30/-23)
debian/real-po/sa.po (+32/-27)
debian/real-po/sco.po (+20/-20)
debian/real-po/sd.po (+22/-22)
debian/real-po/se.po (+20/-20)
debian/real-po/shn.po (+19/-17)
debian/real-po/si.po (+123/-103)
debian/real-po/sk.po (+26/-25)
debian/real-po/sl.po (+55/-33)
debian/real-po/sn.po (+24/-23)
debian/real-po/sq.po (+36/-29)
debian/real-po/sr.po (+79/-49)
debian/real-po/sv.po (+33/-26)
debian/real-po/sw.po (+19/-17)
debian/real-po/szl.po (+39/-31)
debian/real-po/ta.po (+169/-140)
debian/real-po/ta_LK.po (+23/-23)
debian/real-po/te.po (+156/-114)
debian/real-po/tet.po (+19/-17)
debian/real-po/tg.po (+38/-29)
debian/real-po/th.po (+128/-80)
debian/real-po/tl.po (+21/-18)
debian/real-po/tr.po (+47/-37)
debian/real-po/tt.po (+19/-17)
debian/real-po/ug.po (+37/-36)
debian/real-po/uk.po (+49/-32)
debian/real-po/ur.po (+20/-20)
debian/real-po/uz.po (+38/-35)
debian/real-po/vec.po (+31/-31)
debian/real-po/vi.po (+29/-28)
debian/real-po/wae.po (+19/-17)
debian/real-po/wo.po (+19/-17)
debian/real-po/xh.po (+19/-17)
debian/real-po/zh_CN.po (+66/-101)
debian/real-po/zh_HK.po (+70/-109)
debian/real-po/zh_TW.po (+82/-125)
scripts/zsys-setup (+10/-0)
ubiquity/misc.py (+13/-1)
| Changed in ubiquity (Ubuntu): | |
| status: | New → Fix Released |
| importance: | Undecided → Low |
| Changed in ubiquity (Ubuntu Focal): | |
| status: | New → Triaged |
| importance: | Undecided → Low |
Revision history for this message
| Brian Murray (brian-murray) wrote Please test proposed package | #1 |
| Changed in ubiquity (Ubuntu Focal): | |
| status: | Triaged → Fix Committed |
| tags: | added: verification-needed verification-needed-focal |
Revision history for this message
| Steve Langasek (vorlon) wrote | #2 |
| description: | updated |
| description: | updated |
Revision history for this message
| Jean-Baptiste Lallement (jibel) wrote | #3 |
| tags: |
added: verification-done verification-done-focal removed: verification-needed verification-needed-focal |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in ubiquity (Ubuntu Focal): | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Łukasz Zemczak (sil2100) wrote Update Released | #5 |
To post a comment you must log in.
Hello Dimitri, or anyone else affected,
Accepted ubiquity into focal-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-
Further information regarding the verification process can be found at https:/
N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.