get_package_version fails to parse latest multipath version structure
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
curtin |
Fix Released
|
Medium
|
Scott Moser |
Bug Description
Running an mutipath install in vmtest, we see this output:
[ 153.737956] cloud-init[1304]: Running command ['unshare', '--fork', '--pid', '--', 'chroot', '/tmp/tmpoq5zr8
[ 153.763045] cloud-init[1304]: get_package_
[ 153.763908] cloud-init[1304]: {'micro': '0+git1.656f8865', 'raw': '0.5.0+
[ 153.765984] cloud-init[1304]: multipath version is None (major=0 minor=5 micro=0+
[ 153.772042] cloud-init[1304]: failed reading multipath-tools version, assuming it wants no spaces in wwids: unorderable types: NoneType() < int()
Looks like the logic for determing the semantic version.
Related branches
- Server Team CI bot: Approve (continuous-integration)
- curtin developers: Pending requested
-
Diff: 40 lines (+9/-3)2 files modifieddebian/changelog (+3/-2)
tests/unittests/test_block_zfs.py (+6/-1)
- Scott Moser (community): Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 1195 lines (+626/-86)23 files modifiedcurtin/block/__init__.py (+14/-0)
curtin/block/clear_holders.py (+11/-9)
curtin/block/lvm.py (+23/-5)
curtin/block/mdadm.py (+2/-3)
curtin/block/zfs.py (+18/-7)
curtin/commands/block_meta.py (+5/-3)
curtin/commands/install.py (+2/-1)
curtin/log.py (+43/-0)
curtin/udev.py (+2/-0)
curtin/util.py (+33/-8)
debian/changelog (+11/-0)
examples/tests/dirty_disks_config.yaml (+30/-3)
examples/tests/lvmoverraid.yaml (+98/-0)
examples/tests/vmtest_defaults.yaml (+14/-0)
tests/unittests/test_block.py (+35/-0)
tests/unittests/test_block_lvm.py (+13/-13)
tests/unittests/test_block_mdadm.py (+4/-5)
tests/unittests/test_block_zfs.py (+80/-24)
tests/unittests/test_clear_holders.py (+60/-3)
tests/unittests/test_util.py (+62/-0)
tests/vmtests/__init__.py (+15/-1)
tests/vmtests/test_lvm_raid.py (+50/-0)
tests/vmtests/test_lvm_root.py (+1/-1)
- Chad Smith: Disapprove
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 772 lines (+412/-43)19 files modifiedcurtin/block/clear_holders.py (+3/-0)
curtin/block/lvm.py (+23/-5)
curtin/block/mdadm.py (+2/-3)
curtin/commands/block_meta.py (+2/-1)
curtin/commands/install.py (+2/-1)
curtin/log.py (+43/-0)
curtin/udev.py (+2/-0)
curtin/util.py (+33/-8)
debian/changelog (+10/-0)
examples/tests/dirty_disks_config.yaml (+30/-3)
examples/tests/lvmoverraid.yaml (+98/-0)
examples/tests/vmtest_defaults.yaml (+14/-0)
tests/unittests/test_block_lvm.py (+13/-13)
tests/unittests/test_block_mdadm.py (+4/-5)
tests/unittests/test_clear_holders.py (+5/-2)
tests/unittests/test_util.py (+62/-0)
tests/vmtests/__init__.py (+15/-1)
tests/vmtests/test_lvm_raid.py (+50/-0)
tests/vmtests/test_lvm_root.py (+1/-1)
- Server Team CI bot: Approve (continuous-integration)
- Ryan Harper (community): Approve
-
Diff: 137 lines (+93/-6)2 files modifiedcurtin/util.py (+31/-6)
tests/unittests/test_util.py (+62/-0)
Changed in curtin: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
assignee: | nobody → Scott Moser (smoser) |
status: | Confirmed → In Progress |
This bug is fixed with commit 1a201e5b to curtin on branch master. /git.launchpad. net/curtin/ commit/ ?id=1a201e5b
To view that commit see the following URL:
https:/