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 0BE1C17491 for ; Fri, 26 Sep 2014 03:10:18 +0000 (UTC) Received: (qmail 25605 invoked by uid 500); 26 Sep 2014 03:10:17 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 25456 invoked by uid 500); 26 Sep 2014 03:10:17 -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 25436 invoked by uid 99); 26 Sep 2014 03:10:17 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 26 Sep 2014 03:10:17 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id AF04E1DDA84; Fri, 26 Sep 2014 03:10:14 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8600579423528382116==" MIME-Version: 1.0 Subject: Re: Review Request 26065: Ambari DataNode shouldn't create dfs.data.dir paths after installation when path becomes unmounted From: "Alejandro Fernandez" To: "Mahadev Konar" , "Sid Wagle" , "Florian Barca" Cc: "Alejandro Fernandez" , "Ambari" Date: Fri, 26 Sep 2014 03:10:14 -0000 Message-ID: <20140926031014.15000.46999@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Alejandro Fernandez" X-ReviewGroup: Ambari X-ReviewRequest-URL: https://reviews.apache.org/r/26065/ X-Sender: "Alejandro Fernandez" References: <20140926025153.14999.37077@reviews.apache.org> In-Reply-To: <20140926025153.14999.37077@reviews.apache.org> Reply-To: "Alejandro Fernandez" X-ReviewRequest-Repository: ambari --===============8600579423528382116== 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/26065/ ----------------------------------------------------------- (Updated Sept. 26, 2014, 3:10 a.m.) Review request for Ambari, Florian Barca, Mahadev Konar, and Sid Wagle. Bugs: AMBARI-7506 https://issues.apache.org/jira/browse/AMBARI-7506 Repository: ambari Description ------- When a drive fails and it is unmounted for service, if the data node process is stopped/started using Ambari the dfs.data.dir path that was housed on that drive is re-created, but this time on the / partition leading to out of disk space issues and data being created on the wrong volume. In this case we only want the Ambari Agent to create dfs.data.dir's during installation, and not after as this makes drive replacements difficult. Diffs ----- ambari-agent/src/test/python/resource_management/TestFileSystem.py PRE-CREATION ambari-common/src/main/python/resource_management/core/logger.py e395bd7 ambari-common/src/main/python/resource_management/core/providers/mount.py dc6d7d9 ambari-common/src/main/python/resource_management/libraries/functions/dfs_datanode_helper.py PRE-CREATION ambari-common/src/main/python/resource_management/libraries/functions/file_system.py PRE-CREATION ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/configuration/hadoop-env.xml 5da6484 ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_datanode.py 2482f97 ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 245ad92 ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/configuration/hadoop-env.xml b3935d7 ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_datanode.py e38d9af ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py c56fac4 ambari-server/src/test/python/stacks/1.3.2/configs/default.json c5c162f ambari-server/src/test/python/stacks/1.3.2/configs/secured.json b04f6e1 ambari-server/src/test/python/stacks/2.0.6/configs/default.json ef279b4 ambari-server/src/test/python/stacks/2.0.6/configs/secured.json 8db2df2 ambari-web/app/data/HDP2/site_properties.js 1d2dbb4 ambari-web/app/data/site_properties.js 06bbe09 Diff: https://reviews.apache.org/r/26065/diff/ Testing (updated) ------- Created unit tests and simple end-to-end test on a sandbox VM. Still need to run more thorough functional tests on a host that has multiple drives. Thanks, Alejandro Fernandez --===============8600579423528382116==--