Bug 1058018

Summary: upgrade from 3.3 overwrites exports with acl None
Product: [Retired] oVirt Reporter: Yedidyah Bar David <didi>
Component: ovirt-engine-installerAssignee: Alon Bar-Lev <alonbl>
Status: CLOSED CURRENTRELEASE QA Contact: bugs <bugs>
Severity: high Docs Contact:
Priority: urgent    
Version: 3.4CC: 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:
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