Ubiquity 20.04 exports existing ZFS pools
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ubiquity (Ubuntu) |
Fix Released
|
Low
|
Jean-Baptiste Lallement | ||
Focal |
Fix Released
|
Low
|
Unassigned |
Bug Description
[Impact]
* Ubiquity unmounts everything that could be mounted on the target file system when it starts and on tear down. For ZFS it exports all the pools.
* If a user had existing pools, they are exported too.
* This fix only unmount pool that are used as a target for the installation, leaving alone any other pool.
[Test Case]
1. Boot a live session
2. Create a new pool with:
$ zpool create -R /mnt tpool /dev/vda
3. Verify that the pool is created with zpool list
4. Start ubiquity
5. Quit ubiquity
=> Verify that the pool is still there with zpool list. Without the fix, tpool is exported.
[Regression Potential]
* Low risk of regression. Worst case, the bug is not fixed and pools are still unmounted or none of the pools are exported. This case would prevent to do 2 installations in a row from the same live session.
[Other info]
* Already shipped in Groovy.
===
For unclear reasons (18.04 didn't have this issue), Ubiquity on 20.04 exports existing ZFS pools, (very) shortly after execution.
To repeat (assumes a `/dev/sda` disk):
- start a 20.04 Ubuntu Desktop live media
- open a terminal
- create a zfs pool: `zpool create test /dev/sda`
- verify that it's been created: `zpool list`
- launch ubiquity: `ubiquity`
- open a separate terminal
- list the ZFS pools: `zpool list`
no pools are now listed.
ProblemType: Bug
DistroRelease: Ubuntu 20.04
Package: ubiquity 20.04.15
ProcVersionSign
Uname: Linux 5.4.0-26-generic x86_64
NonfreeKernelMo
ApportVersion: 2.20.11-0ubuntu27
Architecture: amd64
CasperMD5CheckM
CasperMD5CheckR
CasperVersion: 1.445
CurrentDesktop: MATE
Date: Sat Apr 25 15:21:25 2020
InstallCmdLine: BOOT_IMAGE=
LiveMediaBuild: Ubuntu-MATE 20.04 LTS "Focal Fossa" - Release amd64 (20200423)
ProcEnviron:
TERM=xterm-
PATH=(custom, no user)
XDG_RUNTIME_
LANG=C.UTF-8
SHELL=/bin/bash
SourcePackage: ubiquity
UpgradeStatus: No upgrade log present (probably fresh install)
Related branches
- Iain Lane: Pending requested
-
Diff: 7002 lines (+2462/-4) (has conflicts)140 files modifiedbin/ubiquity (+3/-0)
d-i/manifest (+9/-0)
debian/changelog (+47/-0)
debian/real-po/aa.po (+9/-0)
debian/real-po/ace.po (+9/-0)
debian/real-po/af.po (+24/-0)
debian/real-po/am.po (+9/-0)
debian/real-po/an.po (+9/-0)
debian/real-po/ar.po (+14/-0)
debian/real-po/as.po (+9/-0)
debian/real-po/ast.po (+23/-0)
debian/real-po/az.po (+9/-0)
debian/real-po/be.po (+23/-0)
debian/real-po/bem.po (+9/-0)
debian/real-po/bg.po (+9/-0)
debian/real-po/bn.po (+34/-0)
debian/real-po/bo.po (+21/-0)
debian/real-po/br.po (+9/-0)
debian/real-po/bs.po (+24/-0)
debian/real-po/ca.po (+27/-0)
debian/real-po/ca@valencia.po (+9/-0)
debian/real-po/ce.po (+9/-0)
debian/real-po/ckb.po (+23/-0)
debian/real-po/cs.po (+23/-0)
debian/real-po/csb.po (+9/-0)
debian/real-po/cv.po (+9/-0)
debian/real-po/cy.po (+22/-0)
debian/real-po/da.po (+24/-0)
debian/real-po/de.po (+24/-0)
debian/real-po/dv.po (+9/-0)
debian/real-po/dz.po (+9/-0)
debian/real-po/el.po (+25/-0)
debian/real-po/en_AU.po (+27/-0)
debian/real-po/en_CA.po (+23/-0)
debian/real-po/en_GB.po (+23/-0)
debian/real-po/eo.po (+9/-0)
debian/real-po/es.po (+24/-0)
debian/real-po/et.po (+23/-0)
debian/real-po/eu.po (+24/-0)
debian/real-po/fa.po (+40/-0)
debian/real-po/fi.po (+24/-0)
debian/real-po/fil.po (+9/-0)
debian/real-po/fo.po (+9/-0)
debian/real-po/fr.po (+30/-0)
debian/real-po/fr_CA.po (+27/-0)
debian/real-po/frp.po (+9/-0)
debian/real-po/fy.po (+9/-0)
debian/real-po/ga.po (+9/-0)
debian/real-po/gd.po (+9/-0)
debian/real-po/gl.po (+23/-0)
debian/real-po/gu.po (+9/-0)
debian/real-po/gv.po (+9/-0)
debian/real-po/he.po (+23/-0)
debian/real-po/hi.po (+34/-0)
debian/real-po/hr.po (+23/-0)
debian/real-po/hu.po (+23/-0)
debian/real-po/hy.po (+9/-0)
debian/real-po/ia.po (+27/-0)
debian/real-po/id.po (+23/-0)
debian/real-po/is.po (+9/-0)
debian/real-po/it.po (+23/-0)
debian/real-po/ja.po (+42/-0)
debian/real-po/ka.po (+9/-0)
debian/real-po/kab.po (+13/-0)
debian/real-po/kk.po (+16/-0)
debian/real-po/kl.po (+9/-0)
debian/real-po/km.po (+21/-0)
debian/real-po/kn.po (+9/-0)
debian/real-po/ko.po (+41/-0)
debian/real-po/ku.po (+23/-0)
debian/real-po/kw.po (+9/-0)
debian/real-po/ky.po (+9/-0)
debian/real-po/lb.po (+9/-0)
debian/real-po/lo.po (+9/-0)
debian/real-po/lt.po (+28/-0)
debian/real-po/lv.po (+23/-0)
debian/real-po/mg.po (+9/-0)
debian/real-po/mhr.po (+9/-0)
debian/real-po/mjw.po (+9/-0)
debian/real-po/mk.po (+9/-0)
debian/real-po/ml.po (+9/-0)
debian/real-po/mnw.po (+9/-0)
debian/real-po/mr.po (+19/-0)
debian/real-po/ms.po (+24/-0)
debian/real-po/my.po (+29/-0)
debian/real-po/nb.po (+24/-0)
debian/real-po/nds.po (+9/-0)
debian/real-po/ne.po (+20/-0)
debian/real-po/nl.po (+23/-0)
debian/real-po/nl_NL.po (+13/-0)
debian/real-po/nn.po (+24/-0)
debian/real-po/ny.po (+9/-0)
debian/real-po/oc.po (+23/-0)
debian/real-po/oj.po (+9/-0)
debian/real-po/om.po (+9/-0)
debian/real-po/or.po (+9/-0)
debian/real-po/os.po (+9/-0)
debian/real-po/pa.po (+9/-0)
debian/real-po/pl.po (+23/-0)
debian/real-po/pt.po (+28/-0)
debian/real-po/pt_BR.po (+9/-0)
debian/real-po/ro.po (+9/-0)
debian/real-po/ru.po (+27/-0)
debian/real-po/sa.po (+9/-0)
debian/real-po/sco.po (+9/-0)
debian/real-po/sd.po (+9/-0)
debian/real-po/se.po (+9/-0)
debian/real-po/shn.po (+9/-0)
debian/real-po/si.po (+19/-0)
debian/real-po/sk.po (+9/-0)
debian/real-po/sl.po (+33/-0)
debian/real-po/sn.po (+9/-0)
debian/real-po/sq.po (+27/-0)
debian/real-po/sr.po (+28/-0)
debian/real-po/sv.po (+27/-0)
debian/real-po/sw.po (+9/-0)
debian/real-po/szl.po (+24/-0)
debian/real-po/ta.po (+20/-0)
debian/real-po/ta_LK.po (+9/-0)
debian/real-po/te.po (+21/-0)
debian/real-po/tet.po (+9/-0)
debian/real-po/tg.po (+27/-0)
debian/real-po/th.po (+33/-0)
debian/real-po/tl.po (+9/-0)
debian/real-po/tr.po (+63/-0)
debian/real-po/tt.po (+9/-0)
debian/real-po/ug.po (+13/-0)
debian/real-po/uk.po (+34/-0)
debian/real-po/ur.po (+9/-0)
debian/real-po/uz.po (+9/-0)
debian/real-po/vec.po (+9/-0)
debian/real-po/vi.po (+9/-0)
debian/real-po/wae.po (+9/-0)
debian/real-po/wo.po (+9/-0)
debian/real-po/xh.po (+9/-0)
debian/real-po/zh_CN.po (+32/-0)
debian/real-po/zh_HK.po (+34/-0)
debian/real-po/zh_TW.po (+46/-0)
debian/ubiquity.templates (+6/-0)
ubiquity/plugins/ubi-prepare.py (+34/-4)
- 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)
- Didier Roche-Tolomelli (community): Approve
-
Diff: 42 lines (+8/-4)2 files modifiedbin/ubiquity (+6/-4)
debian/changelog (+2/-0)
summary: |
- Ubiquit 20.04 interferes with existing ZFS pools, making them + Ubiquity 20.04 interferes with existing ZFS pools, making them "disappear" |
tags: | added: zfs |
summary: |
- Ubiquity 20.04 interferes with existing ZFS pools, making them - "disappear" + Ubiquity 20.04 exports existing ZFS pools |
description: | updated |
Changed in ubiquity (Ubuntu): | |
status: | New → Triaged |
importance: | Undecided → Low |
Changed in ubiquity (Ubuntu): | |
assignee: | nobody → Jean-Baptiste Lallement (jibel) |
Changed in ubiquity (Ubuntu): | |
status: | In Progress → Fix Committed |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
Changed in ubiquity (Ubuntu Focal): | |
status: | New → Triaged |
importance: | Undecided → Low |
Something odd is going on, which I don't know if it's expected or not, as Ubiquity may be intentionally messing with udev.
Specifically, if I have zvols, and run Ubiquity (which exports them), once I reimport the pool, the zvols have the block devices swapped. I can reproduce this all the time, and it doesn't happen when I export/import without Ubiquity involved.