Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ambari.apache.org
Content-Type: multipart/alternative;
 boundary="===============6412956652597927856=="
MIME-Version: 1.0
Subject: Review Request 37757: Installing Ambari-2.1.0 removes
 /etc/sudoers.d/ambari-agent file
From: "Andrew Onischuk" <aonishuk@hortonworks.com>
To: "Vitalyi Brodetskyi" <vbrodetskyi@hortonworks.com>
Cc: "Andrew Onischuk" <aonishuk@hortonworks.com>,
 "Ambari" <dev@ambari.apache.org>
Date: Tue, 25 Aug 2015 11:52:30 -0000
Message-ID: <20150825115230.13584.53629@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Andrew Onischuk" <noreply@reviews.apache.org>
Reply-To: "Andrew Onischuk" <aonishuk@hortonworks.com>

--===============6412956652597927856==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37757/
-----------------------------------------------------------

Review request for Ambari and Vitalyi Brodetskyi.


Bugs: AMBARI-12870
    https://issues.apache.org/jira/browse/AMBARI-12870


Repository: ambari


Description
-------

As part of a prior fix, the file got removed from the rpm package. This has
the side effect of user edited file getting removed during upgrade.

Modify Ambari upgrade logic to not remove `/etc/sudoers.d/ambari-agent` and if
that is not possible at least make a copy of the file and save a backup.


Diffs
-----

  ambari-agent/src/main/package/rpm/preinstall.sh fc6719b 

Diff: https://reviews.apache.org/r/37757/diff/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk


--===============6412956652597927856==--