Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/
---

(Updated Jan. 8, 2015, 12:44 a.m.)


Review request for Aurora, Kevin Sweeney and Zameer Manji.


Changes
---

Rebased.


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 (updated)
-

  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 
c748212febf5867f5f7cc54e34bf91a8890d 
  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 
  

Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Maxim Khutornenko

---
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.


Changes
---

Adding new admin-packaged dist into make-python-sdists.


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 (updated)
-

  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 
c748212febf5867f5f7cc54e34bf91a8890d 
  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 

Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Aurora ReviewBot

---
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 
 c748212febf5867f5f7cc54e34bf91a8890d 
   

Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Kevin Sweeney

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review67067
---

Ship it!


Ship It!

- Kevin Sweeney


On Dec. 22, 2014, 5:35 p.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Dec. 22, 2014, 5:35 p.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
 -
 
   examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
   src/main/python/apache/aurora/admin/BUILD 
 f874264bdf07a9cbb2f0990739be3c95f851b040 
   src/main/python/apache/aurora/client/BUILD 
 a2a29ff53ffb0088909a10e1673f9578425df5ce 
   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 
 258c1fa87645ec48854198529392c84ae08c93be 
   src/main/python/apache/aurora/client/bin/BUILD 
 671bd78148788d885e50a93bdc006361297a63a2 
   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 
 e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
   src/main/python/apache/aurora/client/cli/bridge.py 
 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
   src/main/python/apache/aurora/client/cli/context.py 
 f062afc8de1769f6796fc307107c61a59e9366c5 
   src/main/python/apache/aurora/client/cli/jobs.py 
 8bdb7f488f68b10b767ced627e55e57698e4f5f2 
   src/main/python/apache/aurora/client/cli/update.py 
 d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
   src/main/python/apache/aurora/client/commands/BUILD 
 78a2f57b4b42edf363f40e2988cf9a69c36ad003 
   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 
 cbaa67a78533fedf679eb993c9deb0f09b705616 
   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 
 2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
   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 
 c748212febf5867f5f7cc54e34bf91a8890d 
   src/test/python/apache/aurora/client/cli/util.py 
 6dba1855f8951d38882c0d5b01a5a609b774d00f 
   src/test/python/apache/aurora/client/commands/BUILD 
 86ab65e40536a28bddbf8efc9148111ba1518465 
   src/test/python/apache/aurora/client/commands/__init__.py  
   

Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/
---

(Updated Jan. 7, 2015, 9:42 p.m.)


Review request for Aurora, Kevin Sweeney and Zameer Manji.


Changes
---

Bill's comments.


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 (updated)
-

  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 
258c1fa87645ec48854198529392c84ae08c93be 
  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 
f062afc8de1769f6796fc307107c61a59e9366c5 
  src/main/python/apache/aurora/client/cli/jobs.py 
8bdb7f488f68b10b767ced627e55e57698e4f5f2 
  src/main/python/apache/aurora/client/cli/update.py 
d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
  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 
cbaa67a78533fedf679eb993c9deb0f09b705616 
  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 
c748212febf5867f5f7cc54e34bf91a8890d 
  src/test/python/apache/aurora/client/cli/util.py 
6dba1855f8951d38882c0d5b01a5a609b774d00f 
  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 
  

Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Maxim Khutornenko


 On Jan. 6, 2015, 10:13 p.m., Bill Farner wrote:
  src/main/python/apache/aurora/client/base.py, line 247
  https://reviews.apache.org/r/29216/diff/2/?file=799048#file799048line247
 
  s/ v2//, s/V2_//?

no preference, changed.


 On Jan. 6, 2015, 10:13 p.m., Bill Farner wrote:
  src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh, line 73
  https://reviews.apache.org/r/29216/diff/2/?file=799098#file799098line73
 
  s/mchucarroll/maxim/

changed


 On Jan. 6, 2015, 10:13 p.m., Bill Farner wrote:
  src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh, line 79
  https://reviews.apache.org/r/29216/diff/2/?file=799098#file799098line79
 
  Remove extra space after `killall`

done


 On Jan. 6, 2015, 10:13 p.m., Bill Farner wrote:
  src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh, line 91
  https://reviews.apache.org/r/29216/diff/2/?file=799098#file799098line91
 
  Is there a reason we can't grep for the full URL any more?

Relying on URL is flaky as it may contain a machine name instead of IP (that's 
exactly what happens in my case).


- Maxim


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review66928
---


On Dec. 23, 2014, 1:35 a.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Dec. 23, 2014, 1:35 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
 -
 
   examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
   src/main/python/apache/aurora/admin/BUILD 
 f874264bdf07a9cbb2f0990739be3c95f851b040 
   src/main/python/apache/aurora/client/BUILD 
 a2a29ff53ffb0088909a10e1673f9578425df5ce 
   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 
 258c1fa87645ec48854198529392c84ae08c93be 
   src/main/python/apache/aurora/client/bin/BUILD 
 671bd78148788d885e50a93bdc006361297a63a2 
   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 
 e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
   src/main/python/apache/aurora/client/cli/bridge.py 
 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
   src/main/python/apache/aurora/client/cli/context.py 
 f062afc8de1769f6796fc307107c61a59e9366c5 
   src/main/python/apache/aurora/client/cli/jobs.py 
 8bdb7f488f68b10b767ced627e55e57698e4f5f2 
   src/main/python/apache/aurora/client/cli/update.py 
 d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
   src/main/python/apache/aurora/client/commands/BUILD 
 78a2f57b4b42edf363f40e2988cf9a69c36ad003 
   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 
 cbaa67a78533fedf679eb993c9deb0f09b705616 
   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 
 

Re: Review Request 29216: Removing client v1 code.

2015-01-07 Thread Aurora ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review67105
---


This patch does not apply cleanly on master (5d8424c), do you need to rebase?

I will refresh this build result if you post a review containing @ReviewBot 
retry

- Aurora ReviewBot


On Jan. 7, 2015, 9:42 p.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Jan. 7, 2015, 9:42 p.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
 -
 
   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 
 258c1fa87645ec48854198529392c84ae08c93be 
   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 
 f062afc8de1769f6796fc307107c61a59e9366c5 
   src/main/python/apache/aurora/client/cli/jobs.py 
 8bdb7f488f68b10b767ced627e55e57698e4f5f2 
   src/main/python/apache/aurora/client/cli/update.py 
 d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
   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 
 cbaa67a78533fedf679eb993c9deb0f09b705616 
   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 
 c748212febf5867f5f7cc54e34bf91a8890d 
   src/test/python/apache/aurora/client/cli/util.py 
 6dba1855f8951d38882c0d5b01a5a609b774d00f 
   src/test/python/apache/aurora/client/commands/BUILD 
 

Re: Review Request 29216: Removing client v1 code.

2015-01-06 Thread Bill Farner

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review66928
---

Ship it!



src/main/python/apache/aurora/client/base.py
https://reviews.apache.org/r/29216/#comment110644

s/ v2//, s/V2_//?



src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
https://reviews.apache.org/r/29216/#comment110648

s/mchucarroll/maxim/



src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
https://reviews.apache.org/r/29216/#comment110649

Remove extra space after `killall`



src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
https://reviews.apache.org/r/29216/#comment110650

Is there a reason we can't grep for the full URL any more?


- Bill Farner


On Dec. 23, 2014, 1:35 a.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Dec. 23, 2014, 1:35 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
 -
 
   examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
   src/main/python/apache/aurora/admin/BUILD 
 f874264bdf07a9cbb2f0990739be3c95f851b040 
   src/main/python/apache/aurora/client/BUILD 
 a2a29ff53ffb0088909a10e1673f9578425df5ce 
   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 
 258c1fa87645ec48854198529392c84ae08c93be 
   src/main/python/apache/aurora/client/bin/BUILD 
 671bd78148788d885e50a93bdc006361297a63a2 
   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 
 e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
   src/main/python/apache/aurora/client/cli/bridge.py 
 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
   src/main/python/apache/aurora/client/cli/context.py 
 f062afc8de1769f6796fc307107c61a59e9366c5 
   src/main/python/apache/aurora/client/cli/jobs.py 
 8bdb7f488f68b10b767ced627e55e57698e4f5f2 
   src/main/python/apache/aurora/client/cli/update.py 
 d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
   src/main/python/apache/aurora/client/commands/BUILD 
 78a2f57b4b42edf363f40e2988cf9a69c36ad003 
   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 
 cbaa67a78533fedf679eb993c9deb0f09b705616 
   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 
 2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
   src/test/python/apache/aurora/client/api/test_task_util.py 
 048aff6874259810efea463df1ca2a1fdc419ca1 
   

Re: Review Request 29216: Removing client v1 code.

2015-01-05 Thread Zameer Manji

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review66715
---

Ship it!


These changes LTGM. I'm not a fan of the churn in the admin code but it seems 
unavoidable.

- Zameer Manji


On Dec. 22, 2014, 5:35 p.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Dec. 22, 2014, 5:35 p.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
 -
 
   examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
   src/main/python/apache/aurora/admin/BUILD 
 f874264bdf07a9cbb2f0990739be3c95f851b040 
   src/main/python/apache/aurora/client/BUILD 
 a2a29ff53ffb0088909a10e1673f9578425df5ce 
   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 
 258c1fa87645ec48854198529392c84ae08c93be 
   src/main/python/apache/aurora/client/bin/BUILD 
 671bd78148788d885e50a93bdc006361297a63a2 
   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 
 e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
   src/main/python/apache/aurora/client/cli/bridge.py 
 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
   src/main/python/apache/aurora/client/cli/context.py 
 f062afc8de1769f6796fc307107c61a59e9366c5 
   src/main/python/apache/aurora/client/cli/jobs.py 
 8bdb7f488f68b10b767ced627e55e57698e4f5f2 
   src/main/python/apache/aurora/client/cli/update.py 
 d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
   src/main/python/apache/aurora/client/commands/BUILD 
 78a2f57b4b42edf363f40e2988cf9a69c36ad003 
   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 
 cbaa67a78533fedf679eb993c9deb0f09b705616 
   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 
 2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
   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 
 c748212febf5867f5f7cc54e34bf91a8890d 
   src/test/python/apache/aurora/client/cli/util.py 
 6dba1855f8951d38882c0d5b01a5a609b774d00f 
   src/test/python/apache/aurora/client/commands/BUILD 
 86ab65e40536a28bddbf8efc9148111ba1518465 
   

Re: Review Request 29216: Removing client v1 code.

2014-12-22 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/
---

(Updated Dec. 23, 2014, 1:35 a.m.)


Review request for Aurora, Kevin Sweeney and Zameer Manji.


Changes
---

Minor target refactoring and rebasing.


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 (updated)
-

  examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
  src/main/python/apache/aurora/admin/BUILD 
f874264bdf07a9cbb2f0990739be3c95f851b040 
  src/main/python/apache/aurora/client/BUILD 
a2a29ff53ffb0088909a10e1673f9578425df5ce 
  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 
258c1fa87645ec48854198529392c84ae08c93be 
  src/main/python/apache/aurora/client/bin/BUILD 
671bd78148788d885e50a93bdc006361297a63a2 
  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 
e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
  src/main/python/apache/aurora/client/cli/bridge.py 
227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
  src/main/python/apache/aurora/client/cli/context.py 
f062afc8de1769f6796fc307107c61a59e9366c5 
  src/main/python/apache/aurora/client/cli/jobs.py 
8bdb7f488f68b10b767ced627e55e57698e4f5f2 
  src/main/python/apache/aurora/client/cli/update.py 
d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
  src/main/python/apache/aurora/client/commands/BUILD 
78a2f57b4b42edf363f40e2988cf9a69c36ad003 
  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 
cbaa67a78533fedf679eb993c9deb0f09b705616 
  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 
2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
  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 
c748212febf5867f5f7cc54e34bf91a8890d 
  src/test/python/apache/aurora/client/cli/util.py 
6dba1855f8951d38882c0d5b01a5a609b774d00f 
  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 

Re: Review Request 29216: Removing client v1 code.

2014-12-22 Thread Aurora ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review65854
---


Master (84ad650) 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 Dec. 23, 2014, 1:35 a.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Dec. 23, 2014, 1:35 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
 -
 
   examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
   src/main/python/apache/aurora/admin/BUILD 
 f874264bdf07a9cbb2f0990739be3c95f851b040 
   src/main/python/apache/aurora/client/BUILD 
 a2a29ff53ffb0088909a10e1673f9578425df5ce 
   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 
 258c1fa87645ec48854198529392c84ae08c93be 
   src/main/python/apache/aurora/client/bin/BUILD 
 671bd78148788d885e50a93bdc006361297a63a2 
   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 
 e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
   src/main/python/apache/aurora/client/cli/bridge.py 
 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
   src/main/python/apache/aurora/client/cli/context.py 
 f062afc8de1769f6796fc307107c61a59e9366c5 
   src/main/python/apache/aurora/client/cli/jobs.py 
 8bdb7f488f68b10b767ced627e55e57698e4f5f2 
   src/main/python/apache/aurora/client/cli/update.py 
 d1d0b2946c0513381345d90fabe22c1b7c6b8ca9 
   src/main/python/apache/aurora/client/commands/BUILD 
 78a2f57b4b42edf363f40e2988cf9a69c36ad003 
   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 
 cbaa67a78533fedf679eb993c9deb0f09b705616 
   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 
 2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
   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 
 c748212febf5867f5f7cc54e34bf91a8890d 
   src/test/python/apache/aurora/client/cli/util.py 
 6dba1855f8951d38882c0d5b01a5a609b774d00f 
   

Review Request 29216: Removing client v1 code.

2014-12-18 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/
---

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
-

  examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
  src/main/python/apache/aurora/admin/BUILD 
f874264bdf07a9cbb2f0990739be3c95f851b040 
  src/main/python/apache/aurora/client/BUILD 
2a66825426923fd7e5b7c70101747bc292208e19 
  src/main/python/apache/aurora/client/api/BUILD 
65e5a85e23c4c698356c8b45c45943e560c1bcd5 
  src/main/python/apache/aurora/client/api/disambiguator.py 
6a78ccd44533ef327f751a08c9e2e16555354d97 
  src/main/python/apache/aurora/client/base.py 
a897e879dfda7340d7561247d3fa7cda2dbb5bc6 
  src/main/python/apache/aurora/client/bin/BUILD 
671bd78148788d885e50a93bdc006361297a63a2 
  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 
e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
  src/main/python/apache/aurora/client/cli/bridge.py 
227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
  src/main/python/apache/aurora/client/commands/BUILD 
f2564839ea782b9b2492afb37756db3f0e7fa2ac 
  src/main/python/apache/aurora/client/commands/__init__.py  
  src/main/python/apache/aurora/client/commands/admin.py 
4b86c5a5851206d875f07c9afa9d68a8f57f5ab7 
  src/main/python/apache/aurora/client/commands/core.py 
7b970e8872c73a30947ce8f407c37da3d06f20df 
  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 
b0916edb0f90906e07a80302473ae7a7446f62a9 
  src/main/python/apache/aurora/client/options.py 
f43aced0131cea3adac018d8abcd938ad45af35f 
  src/test/python/apache/aurora/admin/BUILD 
3a216809d1e31247f7d01451fcc7fd877a4c1fb2 
  src/test/python/apache/aurora/client/BUILD 
e31e434723a8aa884bc528987b8aa98077ec0548 
  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 
c91e74e78a35eef7fc48dcb2a00d1bd3c6cb7291 
  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 
2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
  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 
c748212febf5867f5f7cc54e34bf91a8890d 
  src/test/python/apache/aurora/client/cli/util.py 
6dba1855f8951d38882c0d5b01a5a609b774d00f 
  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 
c7d970c805e7e56f9ecce159c93cd2dac14d1fca 
  src/test/python/apache/aurora/client/commands/test_admin_sla.py 
74fc79c23b4718fe0e1bc921c77533ac12c7d085 
  src/test/python/apache/aurora/client/commands/test_cancel_update.py 
e827b386323906ba827c8f1ebc7745d2303ddc23 
  src/test/python/apache/aurora/client/commands/test_create.py 
d7c32b368605d14c55e87eb6e6cc04d83a8ef406 
  src/test/python/apache/aurora/client/commands/test_diff.py 
b48b53d3282d25c74a8424efe15ad19493acb3a5 
  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 
  

Re: Review Request 29216: Removing client v1 code.

2014-12-18 Thread Aurora ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29216/#review65553
---

Ship it!


Master (9042c56) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing @ReviewBot 
retry

- Aurora ReviewBot


On Dec. 18, 2014, 10:32 p.m., Maxim Khutornenko wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29216/
 ---
 
 (Updated Dec. 18, 2014, 10:32 p.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
 -
 
   examples/vagrant/aurorabuild.sh 69983d0140b76c6869cd04e55d760f3e3a1e4262 
   src/main/python/apache/aurora/admin/BUILD 
 f874264bdf07a9cbb2f0990739be3c95f851b040 
   src/main/python/apache/aurora/client/BUILD 
 2a66825426923fd7e5b7c70101747bc292208e19 
   src/main/python/apache/aurora/client/api/BUILD 
 65e5a85e23c4c698356c8b45c45943e560c1bcd5 
   src/main/python/apache/aurora/client/api/disambiguator.py 
 6a78ccd44533ef327f751a08c9e2e16555354d97 
   src/main/python/apache/aurora/client/base.py 
 a897e879dfda7340d7561247d3fa7cda2dbb5bc6 
   src/main/python/apache/aurora/client/bin/BUILD 
 671bd78148788d885e50a93bdc006361297a63a2 
   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 
 e61cdfb5f3370ac1c5069632d4158f5ee641bc3a 
   src/main/python/apache/aurora/client/cli/bridge.py 
 227abf8e21186dd079bc6ce6ff0408a973ad3ab8 
   src/main/python/apache/aurora/client/commands/BUILD 
 f2564839ea782b9b2492afb37756db3f0e7fa2ac 
   src/main/python/apache/aurora/client/commands/__init__.py  
   src/main/python/apache/aurora/client/commands/admin.py 
 4b86c5a5851206d875f07c9afa9d68a8f57f5ab7 
   src/main/python/apache/aurora/client/commands/core.py 
 7b970e8872c73a30947ce8f407c37da3d06f20df 
   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 
 b0916edb0f90906e07a80302473ae7a7446f62a9 
   src/main/python/apache/aurora/client/options.py 
 f43aced0131cea3adac018d8abcd938ad45af35f 
   src/test/python/apache/aurora/admin/BUILD 
 3a216809d1e31247f7d01451fcc7fd877a4c1fb2 
   src/test/python/apache/aurora/client/BUILD 
 e31e434723a8aa884bc528987b8aa98077ec0548 
   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 
 c91e74e78a35eef7fc48dcb2a00d1bd3c6cb7291 
   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 
 2fc76d21ca63ae7f33b1e03ccb88f52fe82dc76c 
   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 
 c748212febf5867f5f7cc54e34bf91a8890d 
   src/test/python/apache/aurora/client/cli/util.py 
 6dba1855f8951d38882c0d5b01a5a609b774d00f 
   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 
 c7d970c805e7e56f9ecce159c93cd2dac14d1fca 
   src/test/python/apache/aurora/client/commands/test_admin_sla.py 
 74fc79c23b4718fe0e1bc921c77533ac12c7d085 
   src/test/python/apache/aurora/client/commands/test_cancel_update.py