Change in vdsm[master]: libvirt_configure: move unpersist up

danken at redhat.com danken at redhat.com
Mon Mar 24 16:23:18 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: libvirt_configure: move unpersist up
......................................................................


libvirt_configure: move unpersist up

On commit b4efa0036e065 we have fixed the failure on
mv command to logrotate file but a few lines above vdsm
executes a sed which also requires the file be unpersist before it.
This patch move the unpersist command some lines above.

Change-Id: Iea6ae62990a414f52225ad690ec9c21060c2a48d
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25990
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/libvirt_configure.sh.in
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Verified
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iea6ae62990a414f52225ad690ec9c21060c2a48d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list