Merge ~dgadomski/ubuntu/+source/ceph:lp1908375 into ubuntu/+source/ceph:ubuntu/bionic-devel

Proposed by Dariusz Gadomski
Status: Needs review
Proposed branch: ~dgadomski/ubuntu/+source/ceph:lp1908375
Merge into: ubuntu/+source/ceph:ubuntu/bionic-devel
Diff against target: 832 lines (+780/-0)
8 files modified
debian/changelog (+6/-0)
debian/patches/lp1908375-0001-ceph-volume-add-methods-to-pass-filters-to-pvs-vgs-a.patch (+60/-0)
debian/patches/lp1908375-0002-ceph-volume-filter-based-on-tags-for-api.lvm.get_-me.patch (+144/-0)
debian/patches/lp1908375-0003-ceph-volume-add-helper-methods-to-get-only-first-LVM.patch (+172/-0)
debian/patches/lp1908375-0004-ceph-volume-sort-and-align-output.patch (+73/-0)
debian/patches/lp1908375-0005-ceph-volume-refactor-devices-lvm-listing.py.patch (+286/-0)
debian/patches/lp1908375-0006-ceph-volume-add-new-method-in-api-lvm.py.patch (+33/-0)
debian/patches/series (+6/-0)
Reviewer Review Type Date Requested Status
James Page (community) Approve
git-ubuntu developers Pending
Review via email: mp+397034@code.qastaging.launchpad.net

Commit message

d/p/lp1908375: remove blkid calls from ceph-volume lvm list (LP: #1908375)

Description of the change

Backported changes refactoring ceph-volume to drop blkid calls. The calls themselves don't seem justified enough to introduce potentially very long delays in cli execution. Hence, they have been removed upstream (and in later Ubuntu releases).

In some environments it may take minutes for a 'ceph-volume lvm list' call with numerous blkid calls. Without the unnecessary blkid calls it's just a matter for 20 s in the same environment producing identical output.

To post a comment you must log in.
Revision history for this message
James Page (james-page) wrote :

I've merged into the managed repository location rather than the auto imported repository.

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches