Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65644 --- src/main/python/apache/aurora/common/transport.py https://reviews.apache.org/r/29165/#comment108858 does it make sense to have both session_factory and user_agent as arguments to TRequestsTransport if user_agent just gets passed directly to the session_factory anyway? seems like it would be better to facilitate the population of a user-agent by passing a custom session_factory. - Brian Wickman On Dec. 18, 2014, 1:45 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, 1:45 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
On Dec. 19, 2014, 10:25 a.m., Brian Wickman wrote: src/main/python/apache/aurora/common/transport.py, lines 44-45 https://reviews.apache.org/r/29165/diff/3/?file=795507#file795507line44 does it make sense to have both session_factory and user_agent as arguments to TRequestsTransport if user_agent just gets passed directly to the session_factory anyway? seems like it would be better to facilitate the population of a user-agent by passing a custom session_factory. +1 - Kevin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65644 --- On Dec. 17, 2014, 5:45 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 17, 2014, 5:45 p.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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
On Dec. 19, 2014, 6:25 p.m., Brian Wickman wrote: src/main/python/apache/aurora/common/transport.py, lines 44-45 https://reviews.apache.org/r/29165/diff/3/?file=795507#file795507line44 does it make sense to have both session_factory and user_agent as arguments to TRequestsTransport if user_agent just gets passed directly to the session_factory anyway? seems like it would be better to facilitate the population of a user-agent by passing a custom session_factory. Kevin Sweeney wrote: +1 Joshua Cohen wrote: I went back and forth on this. My thinking was the default session factory might do work that we wouldn't want to replicate everywhere that we want to pass in a custom user agent. How would you feel if instead of passing the user agent to the session factory, we explicitly set the user agent on the returned session if the header is not already present? Brian Wickman wrote: This sounds fine to me. Ok, done! - Joshua --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65644 --- On Dec. 19, 2014, 7:39 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 19, 2014, 7:39 p.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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65664 --- 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. 19, 2014, 7:39 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 19, 2014, 7:39 p.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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65684 --- Ship it! Ship It! - Brian Wickman On Dec. 19, 2014, 7:39 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 19, 2014, 7:39 p.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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65531 --- Ship it! Ship It! - Kevin Sweeney On Dec. 17, 2014, 5:45 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 17, 2014, 5:45 p.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
Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- 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/BUILD 65e5a85e23c4c698356c8b45c45943e560c1bcd5 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65367 --- src/main/python/apache/aurora/client/factory.py https://reviews.apache.org/r/29165/#comment108473 This is effecively a runtime constant, why the factory indirection? (as opposed to resolving the string in the entry point). - Kevin Sweeney On Dec. 17, 2014, 11:52 a.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 17, 2014, 11:52 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/BUILD 65e5a85e23c4c698356c8b45c45943e560c1bcd5 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65368 --- Ship it! Master (ab18bd0) 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. 17, 2014, 7:52 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 17, 2014, 7:52 p.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/BUILD 65e5a85e23c4c698356c8b45c45943e560c1bcd5 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
On Dec. 17, 2014, 8:07 p.m., Kevin Sweeney wrote: src/main/python/apache/aurora/client/factory.py, lines 27-31 https://reviews.apache.org/r/29165/diff/1/?file=795016#file795016line27 This is effecively a runtime constant, why the factory indirection? (as opposed to resolving the string in the entry point). Before I was able to get this review posted to RB, I pushed my branch to github and Maxim had commented there that it might be better to make the user agent a factory to maintain consistency with the session key factory. Looking at the finished product, I agree that plumbing the raw user agent is a bit cleaner though. Maxim, what are your thoughts? - Joshua --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65367 --- On Dec. 17, 2014, 7:52 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 17, 2014, 7:52 p.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/BUILD 65e5a85e23c4c698356c8b45c45943e560c1bcd5 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
On Dec. 17, 2014, 8:07 p.m., Kevin Sweeney wrote: src/main/python/apache/aurora/client/factory.py, lines 27-31 https://reviews.apache.org/r/29165/diff/1/?file=795016#file795016line27 This is effecively a runtime constant, why the factory indirection? (as opposed to resolving the string in the entry point). Joshua Cohen wrote: Before I was able to get this review posted to RB, I pushed my branch to github and Maxim had commented there that it might be better to make the user agent a factory to maintain consistency with the session key factory. Looking at the finished product, I agree that plumbing the raw user agent is a bit cleaner though. Maxim, what are your thoughts? Sorry for not making it clear. What I wanted to see is a meaningful default value for the the AuroraClientAPI or making agent_name a required parameter to make it impossible to use AuroraClientAPI without specifying the agent name. - Maxim --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65367 --- On Dec. 17, 2014, 7:52 p.m., Joshua Cohen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 17, 2014, 7:52 p.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/BUILD 65e5a85e23c4c698356c8b45c45943e560c1bcd5 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- 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. Changes --- Update to pass around a raw user agent. 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 (updated) - 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/api/scheduler_client.py, line 121 https://reviews.apache.org/r/29165/diff/2/?file=795321#file795321line121 Do you still need kwargs? Yeah, this is still used to pass user_agent to the super class (otherwise we'd have to change the interface for these SchedulerClient sub-classes which seemed like it would balloon this change further). On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/api/__init__.py, line 56 https://reviews.apache.org/r/29165/diff/2/?file=795319#file795319line56 s/,// Fixed. On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/base.py, line 255 https://reviews.apache.org/r/29165/diff/2/?file=795322#file795322line255 make 'Unknown Version' a constant? Fixed. On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/base.py, line 257 https://reviews.apache.org/r/29165/diff/2/?file=795322#file795322line257 Adding space after ';' should make it more readable. I was going for parseability, rather than human readability. Right now we'll get something like: Aurora v2;c168976a819ad93d847ba9d44a0fb68dd36ce3ae-Tuesday Dec 16, 2014 00:29:49 Do you think the space will help enough to offset the (admittedly trivial) need to trim the split values when parsing the UA? On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/test/python/apache/aurora/common/test_transport.py, line 67 https://reviews.apache.org/r/29165/diff/2/?file=795336#file795336line67 why x? Fixed. On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/commands/admin.py, lines 260-261 https://reviews.apache.org/r/29165/diff/2/?file=795325#file795325line260 fits to one line now Fixed. On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/base.py, line 251 https://reviews.apache.org/r/29165/diff/2/?file=795322#file795322line251 comments? Added docstring. - Joshua --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65423 --- 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/
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/ --- (Updated Dec. 18, 2014, 1:45 a.m.) Review request for Aurora, Kevin Sweeney, Maxim Khutornenko, and Brian Wickman. Changes --- Maxim's review feedback. 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 (updated) - 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
On Dec. 18, 2014, 1:19 a.m., Maxim Khutornenko wrote: src/main/python/apache/aurora/client/base.py, line 257 https://reviews.apache.org/r/29165/diff/2/?file=795322#file795322line257 Adding space after ';' should make it more readable. Joshua Cohen wrote: I was going for parseability, rather than human readability. Right now we'll get something like: Aurora v2;c168976a819ad93d847ba9d44a0fb68dd36ce3ae-Tuesday Dec 16, 2014 00:29:49 Do you think the space will help enough to offset the (admittedly trivial) need to trim the split values when parsing the UA? Good point. Let's keep it this way. - Maxim --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65423 --- On Dec. 18, 2014, 1:45 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, 1:45 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
Re: Review Request 29165: Add custom user agent for Aurora v1, Aurora v2 and Aurora Admin clients.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29165/#review65440 --- 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, 1:45 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, 1:45 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