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 E9E3018CB9 for ; Wed, 10 Feb 2016 22:14:10 +0000 (UTC) Received: (qmail 31252 invoked by uid 500); 10 Feb 2016 22:14:10 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 31218 invoked by uid 500); 10 Feb 2016 22:14:10 -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 31202 invoked by uid 99); 10 Feb 2016 22:14:10 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Feb 2016 22:14:10 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 5CC6D29B754; Wed, 10 Feb 2016 22:14:09 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8219269080835375920==" MIME-Version: 1.0 Subject: Re: Review Request 43377: Create Upgrade Packs for Patch Upgrades From: Alejandro Fernandez To: Jonathan Hurley , Dmitro Lisnichenko Cc: Alejandro Fernandez , Nate Cole , Ambari Date: Wed, 10 Feb 2016 22:14:09 -0000 Message-ID: <20160210221409.24150.95532@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Alejandro Fernandez X-ReviewGroup: Ambari X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/43377/ In-Reply-To: <20160210210111.24150.17820@reviews.apache.org> X-Sender: Alejandro Fernandez X-ReviewBoard-ShipIt: 1 References: <20160210210111.24150.17820@reviews.apache.org> X-ReviewBoard-Diff-For: ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/UpgradeScope.java X-ReviewBoard-Diff-For: ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_partial.xml X-ReviewBoard-ShipIt-Only: 1 Reply-To: Alejandro Fernandez X-ReviewRequest-Repository: ambari --===============8219269080835375920== 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/43377/#review118734 ----------------------------------------------------------- Ship it! Ship It! - Alejandro Fernandez On Feb. 10, 2016, 9:01 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43377/ > ----------------------------------------------------------- > > (Updated Feb. 10, 2016, 9:01 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Jonathan Hurley. > > > Bugs: AMBARI-14978 > https://issues.apache.org/jira/browse/AMBARI-14978 > > > Repository: ambari > > > Description > ------- > > Created separate upgrade packs for patches. The necessitated a new enum UpgradeScope to indicate that the upgrade was for partial cluster upgrades (which will help when doing Component (Service) Upgrades). > > Also added a fix for orchestration of ClusterGroupings that target specific services that are not getting upgraded. > > > Diffs > ----- > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java db8c079 > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeContext.java 05aecac > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 0d9176d > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java 5e21da5 > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ColocatedGrouping.java 11e9267 > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Grouping.java 5af893e > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapperBuilder.java e96c7ff > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Task.java f365a7f > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/UpgradeScope.java PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.2.xml 4fd3316 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 38ee39b > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.3.xml ac84443 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml 508483e > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.4.xml 3863877 > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.4.xml e45e851 > ambari-server/src/main/resources/version_definition.xsd 77b4203 > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java a0ac966 > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java b15157e > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_partial.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/43377/diff/ > > > Testing > ------- > > Manual. Automated: > > > Tests run: 3822, Failures: 0, Errors: 0, Skipped: 31 > > [INFO] ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] ------------------------------------------------------------------------ > [INFO] Total time: 33:52.026s > [INFO] Finished at: Tue Feb 09 17:08:46 EST 2016 > [INFO] Final Memory: 33M/677M > [INFO] ------------------------------------------------------------------------ > > > Thanks, > > Nate Cole > > --===============8219269080835375920==--