)]}'
{"id":"openstack%2Fceilometer~134713","triplet_id":"openstack%2Fceilometer~master~Ia54c5805fe2d2c6bcd2092b435c5242da6ddad4d","project":"openstack/ceilometer","branch":"master","topic":"specify_python_for_pep8","hashtags":[],"change_id":"Ia54c5805fe2d2c6bcd2092b435c5242da6ddad4d","subject":"Specify version of python for pep8","status":"ABANDONED","created":"2014-11-15 15:32:57.000000000","updated":"2014-12-05 20:30:19.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"badaec5aab0cb8367c7f9cb37f3e18d9e7ee2bdf","_number":134713,"virtual_id_number":134713,"owner":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2014-11-15 15:36:03.000000000","_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},{"value":1,"date":"2014-11-18 06:33:59.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2014-12-04 10:37:12.000000000","_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},{"date":"2014-11-15 15:51:35.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"date":"2014-12-03 19:59:17.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-11-15 15:51:35.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2014-11-18 06:33:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2014-12-03 19:59:17.000000000","updated_by":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"reviewer":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"state":"REVIEWER"},{"updated":"2014-12-04 10:37:12.000000000","updated_by":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"reviewer":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"state":"REVIEWER"}],"messages":[{"id":"7c8e5ed2ab96d683d06ac6a23fab477ed5859683","author":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"date":"2014-11-15 15:32:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"53a925eebb6090c78ee433fdcf2ed70dd24a9e93","author":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"date":"2014-11-15 15:36:03.000000000","message":"Patch Set 1:\n\nI don\u0027t pretend to understand this fully.  However, on Debian Jessie, I get the execfile error (below) when I try to run \u0027tox -e pep8\u0027.  Oddly, \u0027python -v\u0027 is 2.7.8, so I\u0027m not sure how tox determines the default python.\n\nThis fixes it, though I have no idea if this is the right fix.  If someone knows the right fix, please feel free to just submit a new patch if that would be easier than changing this one!\n\n\nexecfile error is:\n  File \"${HOME}/openstack/ceilometer/.tox/pep8/build/happybase/setup.py\", line 5, in \u003cmodule\u003e\n\n    execfile(\u0027happybase/_version.py\u0027)\n\nNameError: name \u0027execfile\u0027 is not defined","accounts_in_message":[],"_revision_number":1},{"id":"9f90c14cb550c6ef7ef44604b47619da6636fafe","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-11-15 15:50:30.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-ceilometer-pep8 http://logs.openstack.org/13/134713/1/check/gate-ceilometer-pep8/4478e99 : SUCCESS in 2m 58s\n- gate-ceilometer-docs http://docs-draft.openstack.org/13/134713/1/check/gate-ceilometer-docs/e78fc43/doc/build/html/ : SUCCESS in 4m 25s\n- gate-ceilometer-python27 http://logs.openstack.org/13/134713/1/check/gate-ceilometer-python27/7521ba4 : SUCCESS in 4m 56s\n- gate-ceilometer-python33 http://logs.openstack.org/13/134713/1/check/gate-ceilometer-python33/e2f4886 : SUCCESS in 4m 36s\n- check-tempest-dsvm-full http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-full/94f06b8 : FAILURE in 13m 30s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-postgres-full/d6e25b2 : FAILURE in 14m 27s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-neutron-full/1aeb9f3 : FAILURE in 14m 16s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-neutron-heat-slow/08e821e : FAILURE in 13m 55s\n- check-grenade-dsvm http://logs.openstack.org/13/134713/1/check/check-grenade-dsvm/bac6550 : FAILURE in 17m 28s\n- check-devstack-dsvm-cells http://logs.openstack.org/13/134713/1/check/check-devstack-dsvm-cells/db49b48 : SUCCESS in 12m 38s\n- check-swift-dsvm-functional http://logs.openstack.org/13/134713/1/check/check-swift-dsvm-functional/2f76c29 : SUCCESS in 15m 41s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/13/134713/1/check/gate-tempest-dsvm-ceilometer-mongodb/c803595 : FAILURE in 13m 06s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/13/134713/1/check/gate-tempest-dsvm-ceilometer-mongodb-full/5c0694b : FAILURE in 13m 35s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"c0030057cfbee1b2d78fefc11c044e64b3953e75","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-11-15 15:51:35.000000000","message":"Patch Set 1:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- check-grenade-dsvm: https://bugs.launchpad.net/bugs/1393006\n- check-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1393006\n- check-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1393006\n- check-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1393006\n- check-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1393006\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":1},{"id":"240318af4b312f1f0fb39b4d27da6e2065d6166a","author":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"date":"2014-11-18 05:32:12.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"7f11d3d12ecfd12f5ec9398025629f92260f4400","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-11-18 06:33:59.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/13/134713/1/check/gate-ceilometer-pep8/5130ccb : SUCCESS in 3m 16s\n- gate-ceilometer-docs http://docs-draft.openstack.org/13/134713/1/check/gate-ceilometer-docs/d6edbf4/doc/build/html/ : SUCCESS in 3m 39s\n- gate-ceilometer-python27 http://logs.openstack.org/13/134713/1/check/gate-ceilometer-python27/8475932 : SUCCESS in 5m 26s\n- gate-ceilometer-python33 http://logs.openstack.org/13/134713/1/check/gate-ceilometer-python33/54c84ee : SUCCESS in 4m 39s\n- check-tempest-dsvm-full http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-full/317a68f : SUCCESS in 48m 25s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-postgres-full/29fc4a7 : SUCCESS in 43m 42s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-neutron-full/8974326 : SUCCESS in 1h 01m 45s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/13/134713/1/check/check-tempest-dsvm-neutron-heat-slow/6922251 : SUCCESS in 22m 11s\n- check-grenade-dsvm http://logs.openstack.org/13/134713/1/check/check-grenade-dsvm/0a14024 : SUCCESS in 47m 13s\n- check-devstack-dsvm-cells http://logs.openstack.org/13/134713/1/check/check-devstack-dsvm-cells/f879ff1 : SUCCESS in 14m 31s\n- check-swift-dsvm-functional http://logs.openstack.org/13/134713/1/check/check-swift-dsvm-functional/6e498d9 : SUCCESS in 16m 55s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/13/134713/1/check/gate-tempest-dsvm-ceilometer-mongodb/07b4064 : SUCCESS in 26m 34s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/13/134713/1/check/gate-tempest-dsvm-ceilometer-mongodb-full/b2e9549 : SUCCESS in 53m 44s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"7f7cfe8138a23c4d10a79795b01d5a11ef14c987","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2014-12-03 19:59:17.000000000","message":"Patch Set 1:\n\nis this because happybase is py3 incompatible?","accounts_in_message":[],"_revision_number":1},{"id":"599b6451936eeb49bf1472ce52787ffa295e6eab","author":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"date":"2014-12-04 05:21:07.000000000","message":"Patch Set 1:\n\nYes, I believe so.  It isn\u0027t included in requirements-py3, but it is in requirements.  Again, I don\u0027t understand this fully, but I think that tox assumes that the default version of Python is Python 2, and I think on Debian Jessie (last time I installed it) the default was Python 3.\n\nIs this plausible?","accounts_in_message":[],"_revision_number":1},{"id":"b8026cbc4b971a0b81ca60bfa5bb381677e939aa","author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"date":"2014-12-04 10:37:12.000000000","message":"Patch Set 1:\n\nGordon, it looks like happy base is ok now, and (what\u0027s even more exciting!) underlying lib thrift (that was always only for py2.x) now is supporting py33 as well - https://thriftpy.readthedocs.org/en/latest/","accounts_in_message":[],"_revision_number":1},{"id":"289be6902a74920e1dd3459ab219bd438e9d4140","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2014-12-04 19:23:57.000000000","message":"Patch Set 1:\n\nso i take it this isn\u0027t needed then Dina? i could go either way... why are you such a trailblazer and running py3 as your default justinb? :)","accounts_in_message":[],"_revision_number":1},{"id":"1b5fe8da9df0ef30146bd95e28b79fa4e272ee71","author":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"date":"2014-12-05 20:30:02.000000000","message":"Patch Set 1:\n\nGreat news if it isn\u0027t needed - I\u0027ll close (it\u0027s so trivial that we can always get it back if needed)!\n\nI was running py3 by default because Debian Jessie gave it to me by default, and I was testing it out.  Not deliberately trailblazing, but having blazing trails forced upon me :-)","accounts_in_message":[],"_revision_number":1},{"id":"9d892aaa4fd48323af96eaceafc54ee97823519a","author":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"date":"2014-12-05 20:30:19.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"50ca3866166a267f5a0d2cedce0bb03a3b40613d","revisions":{"50ca3866166a267f5a0d2cedce0bb03a3b40613d":{"kind":"REWORK","_number":1,"created":"2014-11-15 15:32:57.000000000","uploader":{"_account_id":144,"name":"justinsb","email":"justin@fathomdb.com","username":"justin-fathomdb"},"ref":"refs/changes/13/134713/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/13/134713/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/13/134713/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/13/134713/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/13/134713/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/13/134713/1"}}},"commit":{"parents":[{"commit":"9af6af5e52f8d66b0fd3f6674aae71607b14626d","subject":"Merge \"Adding quotas on alarms\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/9af6af5e52f8d66b0fd3f6674aae71607b14626d"}]}],"author":{"name":"Justin Santa Barbara","email":"justin@fathomdb.com","date":"2014-11-15 15:31:11.000000000","tz":-480},"committer":{"name":"Justin Santa Barbara","email":"justin@fathomdb.com","date":"2014-11-15 15:31:45.000000000","tz":-480},"subject":"Specify version of python for pep8","message":"Specify version of python for pep8\n\nWithout this, the version of python is unspecified, and happybase\nmight give the execfile installation error.\n\nChange-Id: Ia54c5805fe2d2c6bcd2092b435c5242da6ddad4d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/50ca3866166a267f5a0d2cedce0bb03a3b40613d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/50ca3866166a267f5a0d2cedce0bb03a3b40613d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}