Return-Path: X-Original-To: apmail-ambari-dev-archive@www.apache.org Delivered-To: apmail-ambari-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 9997F18882 for ; Wed, 3 Feb 2016 14:47:29 +0000 (UTC) Received: (qmail 96418 invoked by uid 500); 3 Feb 2016 14:47:29 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 96383 invoked by uid 500); 3 Feb 2016 14:47:29 -0000 Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ambari.apache.org Delivered-To: mailing list dev@ambari.apache.org Received: (qmail 96358 invoked by uid 99); 3 Feb 2016 14:47:29 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 03 Feb 2016 14:47:29 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id B340B2948FB; Wed, 3 Feb 2016 14:47:28 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============3827032660508063960==" MIME-Version: 1.0 Subject: Re: Review Request 43045: AMBARI-14859 : Ranger - Keep all temporary files in the tmp directory defined in the Agent's configuration From: Jonathan Hurley To: Jaimin Jetly , Velmurugan Periasamy , Mahadev Konar , Alejandro Fernandez , Jonathan Hurley , Sumit Mohanty , Selvamohan Neethiraj Cc: Gautam Borad , Ambari Date: Wed, 03 Feb 2016 14:47:28 -0000 Message-ID: <20160203144728.1670.45460@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Jonathan Hurley X-ReviewGroup: Ambari X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/43045/ In-Reply-To: <20160202103750.1670.33597@reviews.apache.org> X-Sender: Jonathan Hurley X-ReviewBoard-ShipIt: 1 References: <20160202103750.1670.33597@reviews.apache.org> X-ReviewBoard-Diff-For: ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/status_params.py X-ReviewBoard-ShipIt-Only: 1 Reply-To: Jonathan Hurley X-ReviewRequest-Repository: ambari --===============3827032660508063960== 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/43045/#review117614 ----------------------------------------------------------- Ship it! Ship It! - Jonathan Hurley On Feb. 2, 2016, 5:37 a.m., Gautam Borad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43045/ > ----------------------------------------------------------- > > (Updated Feb. 2, 2016, 5:37 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Jonathan Hurley, Mahadev Konar, Sumit Mohanty, Selvamohan Neethiraj, and Velmurugan Periasamy. > > > Bugs: AMBARI-14859 > https://issues.apache.org/jira/browse/AMBARI-14859 > > > Repository: ambari > > > Description > ------- > > **Problem Statement** : > Move temporary files created by Ambari code, they should reside in the tmp directory defined in the Agent's configuration. (Refer: https://reviews.apache.org/r/34296 ) > > > ** Suggested Solution ** : > Removed the hard coding of /tmp/ directory for saving temperary files during the process of upgrade and use tmp directory defined in the Agent's configuration > > > Diffs > ----- > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py b6e5ee9 > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py a0007b3 > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/service_check.py 699e3c4 > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/status_params.py PRE-CREATION > > Diff: https://reviews.apache.org/r/43045/diff/ > > > Testing > ------- > > Tested Ranger Installation and Express upgrade from 2.2 to 2.3 > > > Thanks, > > Gautam Borad > > --===============3827032660508063960==--