----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65423 -----------------------------------------------------------
Ship it! lgtm, minor comments below src/main/python/apache/aurora/client/api/__init__.py <https://reviews.apache.org/r/29165/#comment108538> s/,// src/main/python/apache/aurora/client/api/scheduler_client.py <https://reviews.apache.org/r/29165/#comment108550> Do you still need kwargs? src/main/python/apache/aurora/client/base.py <https://reviews.apache.org/r/29165/#comment108544> comments? src/main/python/apache/aurora/client/base.py <https://reviews.apache.org/r/29165/#comment108540> make 'Unknown Version' a constant? src/main/python/apache/aurora/client/base.py <https://reviews.apache.org/r/29165/#comment108547> Adding space after ';' should make it more readable. src/main/python/apache/aurora/client/commands/admin.py <https://reviews.apache.org/r/29165/#comment108554> fits to one line now src/test/python/apache/aurora/common/test_transport.py <https://reviews.apache.org/r/29165/#comment108562> why x? - Maxim Khutornenko On Dec. 18, 2014, 12:59 a.m., Joshua Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29165/ > ----------------------------------------------------------- > > (Updated Dec. 18, 2014, 12:59 a.m.) > > > Review request for Aurora, Kevin Sweeney, Maxim Khutornenko, and Brian > Wickman. > > > Bugs: AURORA-934 > https://issues.apache.org/jira/browse/AURORA-934 > > > Repository: aurora > > > Description > ------- > > Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients. > > > Diffs > ----- > > src/main/python/apache/aurora/client/BUILD > 2a66825426923fd7e5b7c70101747bc292208e19 > src/main/python/apache/aurora/client/api/__init__.py > 9977b419b58bbe2c7f860b1c6ffde6d1a791a05d > src/main/python/apache/aurora/client/api/command_runner.py > 4d8f0107ed6766603cb44a6b231de6cc44c3df4b > src/main/python/apache/aurora/client/api/scheduler_client.py > 4ae18f01380893d842674077d98caca135f7d233 > src/main/python/apache/aurora/client/base.py > a897e879dfda7340d7561247d3fa7cda2dbb5bc6 > src/main/python/apache/aurora/client/cli/context.py > b1ffc05ebd520e91be58704615c5a12c62e75f03 > src/main/python/apache/aurora/client/commands/BUILD > f2564839ea782b9b2492afb37756db3f0e7fa2ac > 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/ssh.py > b0916edb0f90906e07a80302473ae7a7446f62a9 > src/main/python/apache/aurora/client/factory.py > 4b31e8503938a3f2af2c2d2a9a7c812cd489fa65 > src/main/python/apache/aurora/common/transport.py > 76e079ac4c0e7a678af7b1c38ea1398b24ce5e53 > src/test/python/apache/aurora/client/api/test_scheduler_client.py > b3232e9b00075c8429230fdf4858d039d3df4e36 > src/test/python/apache/aurora/client/cli/test_diff.py > 95c7c92942c7cbde344a093789b3e04b63200cc5 > 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_create.py > d7c32b368605d14c55e87eb6e6cc04d83a8ef406 > src/test/python/apache/aurora/client/commands/test_diff.py > b48b53d3282d25c74a8424efe15ad19493acb3a5 > src/test/python/apache/aurora/common/test_transport.py > 2045f64eac638fbdccf59e7971d23b8854972654 > > Diff: https://reviews.apache.org/r/29165/diff/ > > > Testing > ------- > > ./pants build src/test/python/apache/aurora:: > > Also ran e2e tests for v1 and v2 and verified the user agent in the scheduler > logs. > > > Thanks, > > Joshua Cohen > >