Merge ~raharper/curtin:fix/zfs-export-online-pool-only into curtin:master
Proposed by
Ryan Harper
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Ryan Harper | ||||||||
Approved revision: | a53333511d3119a80f1f3bf2b02a874dbef5b9eb | ||||||||
Merge reported by: | Ryan Harper | ||||||||
Merged at revision: | 736808c0b46dd0d677609694700b61ad748c384a | ||||||||
Proposed branch: | ~raharper/curtin:fix/zfs-export-online-pool-only | ||||||||
Merge into: | curtin:master | ||||||||
Diff against target: |
73 lines (+15/-2) 5 files modified
curtin/block/clear_holders.py (+3/-1) curtin/block/zfs.py (+1/-1) examples/tests/dirty_disks_config.yaml (+6/-0) tests/unittests/test_clear_holders.py (+1/-0) tests/vmtests/test_zfsroot.py (+4/-0) |
||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve | |
Scott Moser (community) | Approve | ||
Review via email: mp+345324@code.launchpad.net |
Commit message
clear_holders: only export zpools that have been imported
Curtin may encounter disks from a previous install which are
zfs_members but are part of a pool which isn't currently
imported. Attempting to export a pool that is not imported
results in a failure. Instead, only export a pool if it's
found in the list of currently imported pools.
Also enable Bionic ZfsRoot vmtest (BionicTestZfsR
LP: #1770280
To post a comment you must log in.
PASSED: Continuous integration, rev:d21284e3dd3 8f75b73c11c597b f06c1c7c53757e /jenkins. ubuntu. com/server/ job/curtin- ci/924/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-amd64/ 924 /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 924 /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 924 /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 924
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/924/ rebuild
https:/