Return-Path: X-Original-To: apmail-aurora-reviews-archive@minotaur.apache.org Delivered-To: apmail-aurora-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B6B5A10698 for ; Thu, 8 Jan 2015 01:02:53 +0000 (UTC) Received: (qmail 77947 invoked by uid 500); 8 Jan 2015 01:02:55 -0000 Delivered-To: apmail-aurora-reviews-archive@aurora.apache.org Received: (qmail 77906 invoked by uid 500); 8 Jan 2015 01:02:55 -0000 Mailing-List: contact reviews-help@aurora.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@aurora.incubator.apache.org Delivered-To: mailing list reviews@aurora.incubator.apache.org Received: (qmail 77895 invoked by uid 99); 8 Jan 2015 01:02:54 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 Jan 2015 01:02:54 +0000 X-ASF-Spam-Status: No, hits=-1997.8 required=5.0 tests=ALL_TRUSTED,HTML_MESSAGE,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO mail.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with SMTP; Thu, 08 Jan 2015 01:02:52 +0000 Received: (qmail 76828 invoked by uid 99); 8 Jan 2015 01:02:32 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 Jan 2015 01:02:32 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 1DA301CCCF2; Thu, 8 Jan 2015 01:02:30 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8447933760404630267==" MIME-Version: 1.0 Subject: Re: Review Request 29216: Removing client v1 code. From: "Aurora ReviewBot" To: "Zameer Manji" , "Kevin Sweeney" Cc: "Aurora ReviewBot" , "Aurora" , "Maxim Khutornenko" Date: Thu, 08 Jan 2015 01:02:30 -0000 Message-ID: <20150108010230.26560.84854@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Aurora ReviewBot" X-ReviewGroup: Aurora X-ReviewRequest-URL: https://reviews.apache.org/r/29216/ X-Sender: "Aurora ReviewBot" References: <20150108005552.26561.95585@reviews.apache.org> In-Reply-To: <20150108005552.26561.95585@reviews.apache.org> Reply-To: "Aurora ReviewBot" X-ReviewRequest-Repository: aurora X-Virus-Checked: Checked by ClamAV on apache.org --===============8447933760404630267== 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/29216/#review67137 ----------------------------------------------------------- Master (7449e34) is green with this patch. ./build-support/jenkins/build.sh However, it appears that it might lack test coverage. I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Jan. 8, 2015, 12:55 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29216/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2015, 12:55 a.m.) > > > Review request for Aurora, Kevin Sweeney and Zameer Manji. > > > Bugs: AURORA-775 > https://issues.apache.org/jira/browse/AURORA-775 > > > Repository: aurora > > > Description > ------- > > Despite the diff size, the changes are very limited in scope. The majority of green/yellow is consolidating admin-related code under /apache/aurora/admin. The rest is removals and renamings to get rid of "v2". > > This diff has minor overlaps with https://reviews.apache.org/r/28971 in vagrant and target renaming parts. > > > Diffs > ----- > > build-support/release/make-python-sdists bf8d9604672ba286d70207e8101a221dfa5bb407 > examples/vagrant/aurorabuild.sh b7ea41719a8f41bb23d0254e732926d89399c77c > src/main/python/apache/aurora/admin/BUILD f874264bdf07a9cbb2f0990739be3c95f851b040 > src/main/python/apache/aurora/client/BUILD 1a91ff64ce2654b16f8932d445626200c2cb5b00 > src/main/python/apache/aurora/client/api/BUILD 65e5a85e23c4c698356c8b45c45943e560c1bcd5 > src/main/python/apache/aurora/client/api/command_runner.py 48cb567c2098620e0ee322fe9528e167ce7c7c62 > src/main/python/apache/aurora/client/api/disambiguator.py 6a78ccd44533ef327f751a08c9e2e16555354d97 > src/main/python/apache/aurora/client/base.py 0f7436a1172eac8369c68c1181720a0f837f98a4 > src/main/python/apache/aurora/client/bin/BUILD a69807ff91fde2bba7874f46f6c82895bdac19ff > src/main/python/apache/aurora/client/bin/__init__.py 0663a9ad8ae194b63ce6d77dabfe65564e2d26dd > src/main/python/apache/aurora/client/bin/aurora_admin.py 136cf601f0cc137f5111f1ddec5fd823375626b6 > src/main/python/apache/aurora/client/bin/aurora_client.py 4999265af358d06d236e819bd600426f4574742c > src/main/python/apache/aurora/client/cli/BUILD 8bcaccc08032d853a602c49a7f3cbf1b5f94a2f4 > src/main/python/apache/aurora/client/cli/bridge.py 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 > src/main/python/apache/aurora/client/cli/context.py 5edaf08e4cf2b3bc88f716fa4daa8271062151b0 > src/main/python/apache/aurora/client/cli/jobs.py e65c80b284f10252185f380e508fa821526403ec > src/main/python/apache/aurora/client/cli/update.py fa0f00d4cff5400fe4fff6f2fbcc0a41beb1fe3a > src/main/python/apache/aurora/client/commands/BUILD 33cd91f02e3b323ffd55e3f1841ecb15cb003ae6 > src/main/python/apache/aurora/client/commands/__init__.py > src/main/python/apache/aurora/client/commands/admin.py b7cbba02860af576d9d09e54641d261d7e32c56e > src/main/python/apache/aurora/client/commands/core.py 7c9505bb8b3fcc405ce477f99d0e4b43bac52593 > src/main/python/apache/aurora/client/commands/help.py f79e9718fa9fd8f69331d60e95988f89105089f4 > src/main/python/apache/aurora/client/commands/maintenance.py 5321eb6e8d8a15a5e2115e3664d5f12990b67617 > src/main/python/apache/aurora/client/commands/run.py 6844ac3eb977f5debfad003eb9e6590d3bb2022e > src/main/python/apache/aurora/client/commands/ssh.py ab6832ccb53335ed044b2708abfdac041253f676 > src/main/python/apache/aurora/client/options.py f43aced0131cea3adac018d8abcd938ad45af35f > src/test/python/apache/aurora/admin/BUILD 3a216809d1e31247f7d01451fcc7fd877a4c1fb2 > src/test/python/apache/aurora/client/BUILD e40c7a6cb0e86d0436aff18341a5fc8f3bf9faf8 > src/test/python/apache/aurora/client/api/BUILD 2c0c4070cc1f1784b1d4e7f9cd8aac236e97be75 > src/test/python/apache/aurora/client/api/api_util.py > src/test/python/apache/aurora/client/api/test_api.py 7390d1c4efc5f7b2ec140590de4a293958f73a87 > src/test/python/apache/aurora/client/api/test_disambiguator.py 72e98d266e96104552e6891dad737701a04f29c1 > src/test/python/apache/aurora/client/api/test_job_monitor.py 1e7a8793e821da3ed53e0ea2b073ea434993a2c6 > src/test/python/apache/aurora/client/api/test_quota_check.py 2c9bef1e8b4a9e65cbb7af7c07dc4fc23b224d16 > src/test/python/apache/aurora/client/api/test_task_util.py 048aff6874259810efea463df1ca2a1fdc419ca1 > src/test/python/apache/aurora/client/cli/BUILD 4b5817e667281525295730664f96dedeb3e02779 > src/test/python/apache/aurora/client/cli/test_bridge.py 6c40bf74677aa6306657dd30016cc380ac20f989 > src/test/python/apache/aurora/client/cli/test_cron.py c748212febf5867f5f7cc54e34bf91a8890d9999 > src/test/python/apache/aurora/client/cli/util.py b51ca2fd00c433a5c141c4445e2e21ee6d803390 > src/test/python/apache/aurora/client/commands/BUILD 86ab65e40536a28bddbf8efc9148111ba1518465 > src/test/python/apache/aurora/client/commands/__init__.py > src/test/python/apache/aurora/client/commands/test_admin.py b093682d3ac22f00eb88e98dec4c33dca84fb574 > src/test/python/apache/aurora/client/commands/test_admin_sla.py ec558f8843dc7e9b18b7d292e0869c9e653dd103 > src/test/python/apache/aurora/client/commands/test_cancel_update.py e827b386323906ba827c8f1ebc7745d2303ddc23 > src/test/python/apache/aurora/client/commands/test_create.py 2a61d6e81ba021d9eedb609c43bd4e4a0fbe7943 > src/test/python/apache/aurora/client/commands/test_diff.py 79e02b21a2b854c3bb0c1876a98c3734dc670923 > src/test/python/apache/aurora/client/commands/test_hooks.py de1781eecf28c846404849c9cb3b87372fd93276 > src/test/python/apache/aurora/client/commands/test_kill.py 52732eaf6cec5fa37fd8596c3dbdc31e217dc116 > src/test/python/apache/aurora/client/commands/test_listjobs.py a7cb98f8638aac1372419cd2db8c1ce610fdf92f > src/test/python/apache/aurora/client/commands/test_maintenance.py 20eecc03504277a2178295510efe0db5464fcfde > src/test/python/apache/aurora/client/commands/test_restart.py c71f8180b0ad0fac372e66d1dbc7db518f0ee44f > src/test/python/apache/aurora/client/commands/test_run.py 528c3e9d93cb180eda9b2a7e0163737b95b9226e > src/test/python/apache/aurora/client/commands/test_ssh.py 5af91336da2fc4eac268a6e351863b6f5403d060 > src/test/python/apache/aurora/client/commands/test_status.py 003af25b99ef3ec4b087df4c37e7626873cafbb8 > src/test/python/apache/aurora/client/commands/test_update.py 0439f60e4925ef8a3d3f9732fe822c1165fcb880 > src/test/python/apache/aurora/client/commands/test_version.py cb0f4103fbcff93b506c6faf2b229ec9829f7caa > src/test/python/apache/aurora/client/commands/util.py 0d93e76878613073812e96dcc98b95b6b0beafbe > src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh 2a436b40ebb2e13f88c1cabc610e41e0eea95dda > > Diff: https://reviews.apache.org/r/29216/diff/ > > > Testing > ------- > > ./pants src/test/python:all > ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > > > Thanks, > > Maxim Khutornenko > > --===============8447933760404630267==--