Bug 1058018

Summary:	upgrade from 3.3 overwrites exports with acl None
Product:	[Retired] oVirt	Reporter:	Yedidyah Bar David <didi>
Component:	ovirt-engine-installer	Assignee:	Alon Bar-Lev <alonbl>
Status:	CLOSED CURRENTRELEASE	QA Contact:	bugs <bugs>
Severity:	high	Docs Contact:
Priority:	urgent
Version:	3.4	CC:	acathrow, alonbl, gklein, iheim, jon, sbonazzo, yeylon
Target Milestone:	---
Target Release:	3.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	integration
Fixed In Version:	ovirt-3.4.0-rc	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-03-31 12:25:14 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1024889

Description Yedidyah Bar David 2014-01-26 13:26:29 UTC

Description of problem:

On upgrade from 3.3, iso domain export line is overwritten with an ACL 'None'

Version-Release number of selected component (if applicable):

3.4.0-beta

How reproducible:

Always

Steps to Reproduce:
1. setup 3.3.2 with an iso domain
2. upgrade to 3.4.0-beta
3.

Actual results:

The nfs export line is overwritten with acl set to 'None'

Expected results:

Line should not change

Additional info:

Reported on users@ by Jonathan Archer.

Comment 1 Alon Bar-Lev 2014-02-13 23:11:39 UTC

commit 9a43a775bfb9a25a88c387bbfeebbca8f2388715
Author: Alon Bar-Lev <alonbl>
Date:   Thu Feb 13 01:24:34 2014 +0200

    packaging: setup: cleanup nfs exports usage
    
    1. if exports.d exists and we registered ourselves previously in
       exports, move the entry to our own file.
    
    2. if ACL specified enforce a new entry.
    
    3. do not touch anything on upgrade unless (2).
    
    Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1058018
    Change-Id: Ie3df61b1b52aead5ddf457da14f26324dbc2a56c
    Signed-off-by: Alon Bar-Lev <alonbl>

Comment 2 Sandro Bonazzola 2014-02-24 08:15:36 UTC

Shouldn't this be in modified?

Comment 3 Alon Bar-Lev 2014-02-24 10:05:36 UTC

yes

Comment 4 Sandro Bonazzola 2014-03-03 14:40:41 UTC

This BZ should be fixed in oVirt 3.4.0 RC

Comment 5 Sandro Bonazzola 2014-03-31 12:25:14 UTC

this is an automated message: moving to Closed CURRENT RELEASE since oVirt 3.4.0 has been released