]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove warnings for long vgs and lvs calls
authorJohn Griffith <john.griffith@solidfire.com>
Thu, 19 Feb 2015 23:07:17 +0000 (16:07 -0700)
committerJohn Griffith <john.griffith@solidfire.com>
Mon, 23 Feb 2015 15:55:54 +0000 (08:55 -0700)
commit0768b1462d67c4f40c5e6c4e99ae402d71e23196
tree7f7217145289b90e49d2affaf1af52726fc97530
parent8bda008dd0594f006e21d2745b616a97d7119065
Remove warnings for long vgs and lvs calls

We put some warning statements in a while back for
lvs and vgs commands that lasted more than 60 seconds
so we could easily query issues in the gate.

Since then we've made a number of changes to improve our
LVM interactions including the use of an LVM conf file with
a filter for only the VG's Cinder uses.

There's one issue remaining however where during the instance
rescue test in Nova where LVM get's stuck trying to open
devices that no longer exist.  Part of this I think is due
to some manipulation that Nova/KVM does on the attached volume
when the test is run, but also the rescue process in Nova
uses LVM volumes as well and does some moving around and mounting
of devices.

The result is that LVM calls can take quite a while
(greater than 100 seconds) to complete during the rescue process.
This doesn't result in any failures, just that the rescue process
slows things down dramatically and is a pretty heavy process.

Also, the newer versions of process utils include completion
timing in the logs so we don't really need this any longer eve
to query timings and get info.

This patch removes those log statements, and proposes we
document somewhere that rescue operations on a setup that
includes c-vol service and nova-compute service on the same
node will temporarily impact LVM performance.

Change-Id: Iddc8318ba23725990512f5a5945406561818c5ba
cinder/brick/local_dev/lvm.py