Change in vdsm[master]: lvm.env: move to /usr/share/vdsm/storage

danken at redhat.com danken at redhat.com
Mon Dec 9 20:03:52 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: lvm.env: move to /usr/share/vdsm/storage
......................................................................


lvm.env: move to /usr/share/vdsm/storage

lvm.env is a tiny helper script, intended to make it a bit easier to
debug Vdsm-controlled lvm state. It is being created whenever the lvm
module is loaded, which generates an annoying warning message in unit
test.

This patch makes lvm.env ship statically with vdsm. It can be used by

    . /usr/share/vdsm/storage/lvm.env

Change-Id: I6ae6bc789efa22321eb06422e910bf272c575007
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21872
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
---
M .gitignore
M debian/vdsm.install
M vdsm.spec.in
M vdsm/storage/Makefile.am
A vdsm/storage/lvm.env.in
M vdsm/storage/lvm.py
6 files changed, 17 insertions(+), 14 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/21872
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6ae6bc789efa22321eb06422e910bf272c575007
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Gadi Ickowicz <gickowic at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list