)]}'
{"id":"openstack%2Fdevstack-gate~21249","triplet_id":"openstack%2Fdevstack-gate~master~If54f851f9c1ac3221056fd1557af32b669424d63","project":"openstack/devstack-gate","branch":"master","topic":"bug/1116542","hashtags":[],"change_id":"If54f851f9c1ac3221056fd1557af32b669424d63","subject":"Add statsd to pip-requires.","status":"MERGED","created":"2013-02-05 21:14:50.000000000","updated":"2013-02-22 00:27:25.000000000","submitted":"2013-02-22 00:27:25.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f989b7c52ccb4bc715248fb868048b8a74e8d83d","_number":21249,"virtual_id_number":21249,"owner":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"date":"2013-02-20 20:32:47.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":2,"date":"2013-02-22 00:27:25.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":2,"date":"2013-02-20 20:32:47.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":2,"date":"2013-02-21 23:47:05.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":2,"date":"2013-02-17 08:03:42.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"all":[{"value":0,"date":"2013-02-20 20:32:47.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":1,"date":"2013-02-21 23:47:05.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"date":"2013-02-17 08:03:42.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-02-17 08:03:42.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2013-02-20 20:32:47.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"},{"updated":"2013-02-21 23:47:05.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2013-02-22 00:27:25.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"84b3cbb912ac4e8e9b59da1aa277430a0ea19848","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-05 22:46:33.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21249/1/check/gate-tempest-devstack-vm-full/1051 : SUCCESS\n- http://logs.openstack.org/21249/1/check/gate-tempest-devstack-vm-postgres-full/779 : SUCCESS (non-voting)\n- http://logs.openstack.org/21249/1/check/gate-tempest-devstack-vm-quantum/4985 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"b7cbbd767b34584f4d18dfef6bb9b37def6ed991","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-02-13 18:20:29.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nI think I would prefer keeping the import at the head of the file wrapped in the env var check. This way we aren\u0027t importing the module each time we want to do a statsd operation (though python does handle that properly). Should be easy to document why we are doing this check in one place in each file that way as well.","accounts_in_message":[],"_revision_number":1},{"id":"c42cdc93a817c44b5ab31c6c2466962ab8a6bb5b","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2013-02-14 23:17:12.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f67810089af452c9a351f9ba5e1aeb2fb40028a8","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-02-14 23:20:49.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"06ecb8aa0a1d535af96fdbe8e2382bb15a665579","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-15 00:39:53.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21249/2/check/gate-tempest-devstack-vm-full/2913 : SUCCESS\n- http://logs.openstack.org/21249/2/check/gate-tempest-devstack-vm-postgres-full/2639 : SUCCESS\n- http://logs.openstack.org/21249/2/check/gate-tempest-devstack-vm-quantum/7172 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"cd7967f771c8815bb2cbd8532515676967140da5","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2013-02-15 19:02:22.000000000","message":"Patch Set 2: Work In Progress\n\nPer IRC discussion I\u0027ll just add it to pip-requires instead, with an appropriate version constraint.","accounts_in_message":[],"_revision_number":2},{"id":"2c93f38e8dec594219c8e8bfd89264359202dd3c","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2013-02-15 19:03:31.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nI\u0027d rather add statsd to pip-requires; it will gracefully degrade if there is no configuration and keep the code much simpler.","accounts_in_message":[],"_revision_number":2},{"id":"8702dc8d590bdbc6e2a62826c0cbaafef3643896","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2013-02-15 20:55:24.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"311d74ddb465133b2c1f8965821be82401b85677","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-15 23:15:17.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/21249/3/check/gate-tempest-devstack-vm-full/3154 : SUCCESS\n- http://logs.openstack.org/21249/3/check/gate-tempest-devstack-vm-postgres-full/2878 : SUCCESS\n- http://logs.openstack.org/21249/3/check/gate-tempest-devstack-vm-quantum/7448 : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"01e8a45f9412845c31fe6557b0acdb4cdc984c30","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2013-02-17 08:03:42.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"9105f43edfe12119084bc5bf433c38e013b6a42e","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2013-02-20 20:32:47.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"a6dff41b580ab7d45a7f80900ca23d3220ff0f5c","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-02-21 23:47:05.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"8a813a70b5053768cfbd595f770771fa490cc32f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-21 23:47:07.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"908caa28fbb563bdd3df2c7c60eb4049c90f68d0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-22 00:27:25.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/21249/3/gate/gate-tempest-devstack-vm-full/4981 : SUCCESS\n- http://logs.openstack.org/21249/3/gate/gate-tempest-devstack-vm-postgres-full/4700 : SUCCESS\n- http://logs.openstack.org/21249/3/gate/gate-tempest-devstack-vm-quantum/9546 : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"37b8d54fbf470576d8b512fbe219f6a0874aa441","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-22 00:27:25.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"bdcec2851620a9c96f3d53f8352e6fe9457dbfb9","revisions":{"6a7eca4c76a20784f8fc646cb78a5f9f9373183e":{"kind":"REWORK","_number":1,"created":"2013-02-05 21:14:50.000000000","uploader":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"ref":"refs/changes/49/21249/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/49/21249/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/1"}}},"commit":{"parents":[{"commit":"7314480f992db0b2bb31442b4fc6d2350efa1599","subject":"Merge \"Compress logs before Jenkins can copy them.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/7314480f992db0b2bb31442b4fc6d2350efa1599"}]}],"author":{"name":"Jeremy Stanley","email":"fungi@yuggoth.org","date":"2013-02-05 21:12:30.000000000","tz":0},"committer":{"name":"Jeremy Stanley","email":"fungi@yuggoth.org","date":"2013-02-05 21:12:30.000000000","tz":0},"subject":"Only use statsd if STATSD_HOST is set.","message":"Only use statsd if STATSD_HOST is set.\n\nWrap any references to statsd including the module import itself in\na check for the STATSD_HOST environment variable. Fixes bug 1116542.\n\nChange-Id: If54f851f9c1ac3221056fd1557af32b669424d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/6a7eca4c76a20784f8fc646cb78a5f9f9373183e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/6a7eca4c76a20784f8fc646cb78a5f9f9373183e"}]},"branch":"refs/heads/master"},"3f71808d22734962017239d72243b3f073cfc1f7":{"kind":"REWORK","_number":2,"created":"2013-02-14 23:17:12.000000000","uploader":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"ref":"refs/changes/49/21249/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/49/21249/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/2"}}},"commit":{"parents":[{"commit":"6a5db298441a410e54d4cd81024213fb268e47e0","subject":"Support SCREEN_DEV\u003dFalse","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/6a5db298441a410e54d4cd81024213fb268e47e0"}]}],"author":{"name":"Jeremy Stanley","email":"fungi@yuggoth.org","date":"2013-02-05 21:12:30.000000000","tz":0},"committer":{"name":"Jeremy Stanley","email":"fungi@yuggoth.org","date":"2013-02-14 23:17:05.000000000","tz":0},"subject":"Only use statsd if STATSD_HOST is set.","message":"Only use statsd if STATSD_HOST is set.\n\nWrap any references to statsd including the module import itself in\na check for the STATSD_HOST environment variable. Fixes bug 1116542.\n\nChange-Id: If54f851f9c1ac3221056fd1557af32b669424d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/3f71808d22734962017239d72243b3f073cfc1f7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/3f71808d22734962017239d72243b3f073cfc1f7"}]},"branch":"refs/heads/master"},"bdcec2851620a9c96f3d53f8352e6fe9457dbfb9":{"kind":"REWORK","_number":3,"created":"2013-02-15 20:55:24.000000000","uploader":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"ref":"refs/changes/49/21249/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/49/21249/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/49/21249/3"}}},"commit":{"parents":[{"commit":"6a5db298441a410e54d4cd81024213fb268e47e0","subject":"Support SCREEN_DEV\u003dFalse","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/6a5db298441a410e54d4cd81024213fb268e47e0"}]}],"author":{"name":"Jeremy Stanley","email":"fungi@yuggoth.org","date":"2013-02-15 20:54:34.000000000","tz":0},"committer":{"name":"Jeremy Stanley","email":"fungi@yuggoth.org","date":"2013-02-15 20:54:34.000000000","tz":0},"subject":"Add statsd to pip-requires.","message":"Add statsd to pip-requires.\n\n* tools/pip-requires: Added statsd. Fixes bug 1116542.\n\nChange-Id: If54f851f9c1ac3221056fd1557af32b669424d63\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/bdcec2851620a9c96f3d53f8352e6fe9457dbfb9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/bdcec2851620a9c96f3d53f8352e6fe9457dbfb9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}