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 6150918FCE for ; Mon, 21 Dec 2015 23:03:54 +0000 (UTC) Received: (qmail 57942 invoked by uid 500); 21 Dec 2015 23:03:54 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 57898 invoked by uid 500); 21 Dec 2015 23:03:54 -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 57881 invoked by uid 99); 21 Dec 2015 23:03:53 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 21 Dec 2015 23:03:53 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 6ACC12959CD; Mon, 21 Dec 2015 23:03:53 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============6303136142063435061==" MIME-Version: 1.0 Subject: Re: Review Request 41607: Fix slider install failure and review comments for AMBARI-14430 From: "Sumit Mohanty" To: "Nate Cole" , "Dmitro Lisnichenko" , "Sumit Mohanty" , "Jonathan Hurley" Cc: "Andrew Onischuk" , "Ambari" Date: Mon, 21 Dec 2015 23:03:53 -0000 Message-ID: <20151221230353.664.98922@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Sumit Mohanty" X-ReviewGroup: Ambari X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/41607/ X-Sender: "Sumit Mohanty" References: <20151221143822.664.28519@reviews.apache.org> In-Reply-To: <20151221143822.664.28519@reviews.apache.org> Reply-To: "Sumit Mohanty" X-ReviewRequest-Repository: ambari --===============6303136142063435061== 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/41607/#review111549 ----------------------------------------------------------- Ship it! Ship It! - Sumit Mohanty On Dec. 21, 2015, 2:38 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41607/ > ----------------------------------------------------------- > > (Updated Dec. 21, 2015, 2:38 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, Nate Cole, and Sumit Mohanty. > > > Bugs: AMBARI-14447 > https://issues.apache.org/jira/browse/AMBARI-14447 > > > Repository: ambari > > > Description > ------- > > 1\. Add unit tests for getInstalledPackageVersion > 2\. Replace hdp specific names with more common > 3\. Don't install rpcbind on HDP-2.2 > 4\. Don't install atlas-metadata-${hdp_version}-hive-plugin on HDP-2.2 > 5\. Fix slider failure issue on centos/suse due to incorrect package name > 6\. Fix issue, when 'Install packages' is triggered and user still continue > using old version, but install commands try to install newer version. > > > Diffs > ----- > > ambari-agent/src/test/python/resource_management/TestPackagesAnalyzer.py PRE-CREATION > ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py 5628f33 > ambari-common/src/main/python/resource_management/libraries/script/script.py 3deb7a6 > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py 6b1c7f3 > ambari-server/src/main/resources/stacks/HDP/2.2/services/FALCON/metainfo.xml e0e4c63 > ambari-server/src/main/resources/stacks/HDP/2.2/services/FLUME/metainfo.xml 8d53e96 > ambari-server/src/main/resources/stacks/HDP/2.2/services/HBASE/metainfo.xml ccefaf2 > ambari-server/src/main/resources/stacks/HDP/2.2/services/HDFS/metainfo.xml f524685 > ambari-server/src/main/resources/stacks/HDP/2.2/services/HIVE/metainfo.xml 9d97946 > ambari-server/src/main/resources/stacks/HDP/2.2/services/KAFKA/metainfo.xml 78b3021 > ambari-server/src/main/resources/stacks/HDP/2.2/services/KNOX/metainfo.xml 1b97334 > ambari-server/src/main/resources/stacks/HDP/2.2/services/OOZIE/metainfo.xml d09d648 > ambari-server/src/main/resources/stacks/HDP/2.2/services/PIG/metainfo.xml 818640c > ambari-server/src/main/resources/stacks/HDP/2.2/services/RANGER/metainfo.xml 8a4b335 > ambari-server/src/main/resources/stacks/HDP/2.2/services/SLIDER/metainfo.xml 624deda > ambari-server/src/main/resources/stacks/HDP/2.2/services/SPARK/metainfo.xml 9f906a1 > ambari-server/src/main/resources/stacks/HDP/2.2/services/SQOOP/metainfo.xml b3c0e34 > ambari-server/src/main/resources/stacks/HDP/2.2/services/STORM/metainfo.xml a0144d7 > ambari-server/src/main/resources/stacks/HDP/2.2/services/TEZ/metainfo.xml 3bb9aea > ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/metainfo.xml 20de188 > ambari-server/src/main/resources/stacks/HDP/2.3/services/ACCUMULO/metainfo.xml d24d9b8 > ambari-server/src/main/resources/stacks/HDP/2.3/services/ATLAS/metainfo.xml 7e27659 > ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/metainfo.xml 8c992d2 > ambari-server/src/main/resources/stacks/HDP/2.3/services/HIVE/metainfo.xml 0c70dbb > ambari-server/src/main/resources/stacks/HDP/2.3/services/OOZIE/metainfo.xml 12c8b5f > ambari-server/src/main/resources/stacks/HDP/2.3/services/PIG/metainfo.xml fe05cff > ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER_KMS/metainfo.xml 03768f0 > ambari-server/src/main/resources/stacks/HDP/2.3/services/ZOOKEEPER/metainfo.xml 3e27928 > ambari-server/src/test/python/custom_actions/TestInstallPackages.py 39be4aa > ambari-server/src/test/python/custom_actions/configs/install_packages_config.json 24906cd > > Diff: https://reviews.apache.org/r/41607/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > > --===============6303136142063435061==--