)]}'
{"id":"openstack%2Fpython-neutronclient~350432","triplet_id":"openstack%2Fpython-neutronclient~master~I50c88bc256de1a7a747e8b7f37d32ab803bdb54e","project":"openstack/python-neutronclient","branch":"master","topic":"translations","hashtags":[],"change_id":"I50c88bc256de1a7a747e8b7f37d32ab803bdb54e","subject":":Add in missing translation hint for log error","status":"ABANDONED","created":"2016-08-03 08:17:57.000000000","updated":"2016-10-24 17:14:37.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6f4f0be8cd17916eedf60ada10bd243bf6e13160","_number":350432,"virtual_id_number":350432,"owner":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},{"value":-1,"date":"2016-08-09 21:29:54.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2016-08-04 08:21:01.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"date":"2016-08-04 07:01:54.000000000","_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},{"date":"2016-08-04 18:57:17.000000000","_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"}],"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":{"disliked":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},{"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},{"value":-1,"date":"2016-08-10 04:44:05.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"value":-1,"date":"2016-08-04 06:49:22.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"}],"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":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-08-04 05:20:41.000000000","updated_by":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"reviewer":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"state":"REVIEWER"},{"updated":"2016-08-04 06:49:22.000000000","updated_by":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"reviewer":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"state":"REVIEWER"},{"updated":"2016-08-04 07:01:54.000000000","updated_by":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"reviewer":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"state":"REVIEWER"},{"updated":"2016-08-04 18:57:17.000000000","updated_by":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"reviewer":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"state":"REVIEWER"},{"updated":"2016-08-09 21:29:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-08-10 04:44:05.000000000","updated_by":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"reviewer":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"state":"REVIEWER"}],"messages":[{"id":"d99cc47dbe53e3f1f40b275b12344b11a12b8e21","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2016-08-03 08:17:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3f213251bc710515ce6bdfe5fa48a3d41c8358e4","author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"date":"2016-08-03 08:37:46.000000000","message":"Patch Set 1:\n\nI think we should adopt the hacking check that would catch.","accounts_in_message":[],"_revision_number":1},{"id":"219bd9367512f1f069c50a308197ba7be8b8954f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-03 09:51:33.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-neutronclient-docs-ubuntu-xenial http://docs-draft.openstack.org/32/350432/1/check/gate-python-neutronclient-docs-ubuntu-xenial/a541754//doc/build/html/ : SUCCESS in 5m 17s\n- gate-python-neutronclient-pep8-ubuntu-xenial http://logs.openstack.org/32/350432/1/check/gate-python-neutronclient-pep8-ubuntu-xenial/5eea4a6/ : FAILURE in 3m 59s\n- gate-python-neutronclient-python27-ubuntu-xenial http://logs.openstack.org/32/350432/1/check/gate-python-neutronclient-python27-ubuntu-xenial/a4e13c3/ : SUCCESS in 7m 00s\n- gate-python-neutronclient-python34 http://logs.openstack.org/32/350432/1/check/gate-python-neutronclient-python34/ed32c36/ : SUCCESS in 4m 44s\n- gate-python-neutronclient-python35 http://logs.openstack.org/32/350432/1/check/gate-python-neutronclient-python35/47d832f/ : SUCCESS in 9m 02s\n- gate-python-neutronclient-pypy http://logs.openstack.org/32/350432/1/check/gate-python-neutronclient-pypy/04f36f8/ : SUCCESS in 10m 24s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-neutronclient http://logs.openstack.org/32/350432/1/check/gate-tempest-dsvm-neutron-src-python-neutronclient/cf6875b/ : SUCCESS in 1h 11m 25s\n- check-osc-plugins http://logs.openstack.org/32/350432/1/check/check-osc-plugins/33129d7/ : SUCCESS in 8m 03s\n- gate-neutronclient-test-dsvm-functional http://logs.openstack.org/32/350432/1/check/gate-neutronclient-test-dsvm-functional/13fd0be/ : SUCCESS in 26m 08s\n- gate-neutronclient-test-dsvm-functional-adv-svcs http://logs.openstack.org/32/350432/1/check/gate-neutronclient-test-dsvm-functional-adv-svcs/48df7bb/ : SUCCESS in 39m 41s","accounts_in_message":[],"_revision_number":1},{"id":"ac49b71369182db4e273e6db8e6ad319c440c34f","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-03 15:51:14.000000000","message":"Patch Set 1:\n\n\u003e I think we should adopt the hacking check that would catch.\n\nYeah, I thought we had one, like on the server side that\u0027s why I never bother looking at these things!","accounts_in_message":[],"_revision_number":1},{"id":"650f40cdb679e19f8c8de3b2dcb95e4640e9aab5","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2016-08-04 04:58:10.000000000","message":"Patch Set 1:\n\nneutron-lib has the hacking checks. do we want the client to have a dependency on this? if so we can do it.\nanother option is to add this directly.\nwhat do you guys suggest? i am in favor of the latter as one less dependency is better","accounts_in_message":[],"_revision_number":1},{"id":"eaa0cf28904d8d2256a13199abed95f50b4eb31f","author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"date":"2016-08-04 05:20:41.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"8ea046a87ef89e9cd5806de2f55b05003273ca9e","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2016-08-04 05:46:39.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"61a24752ac06add9519e4b89b08e88a7543c1995","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-04 06:49:22.000000000","message":"Patch Set 2: Code-Review-1\n\n\u003e neutron-lib has the hacking checks. do we want the client to have a\n \u003e dependency on this? if so we can do it.\n \u003e another option is to add this directly.\n \u003e what do you guys suggest? i am in favor of the latter as one less\n \u003e dependency is better\n\nNo, let\u0027s not make neutronclient depend on neutron-lib.","accounts_in_message":[],"_revision_number":2},{"id":"db43eac6ea0bfb373352e64f5aeced05c0c32126","author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"date":"2016-08-04 07:01:54.000000000","message":"Patch Set 2:\n\nArmando, do you think duplicating hacking effort is viable? neutron-lib would be a test only dep, what\u0027s the issue with that?","accounts_in_message":[],"_revision_number":2},{"id":"1bfdb6521fcf3e8ebc4193994be3164a9aec1109","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-04 07:17:47.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-neutronclient-docs-ubuntu-xenial http://docs-draft.openstack.org/32/350432/2/check/gate-python-neutronclient-docs-ubuntu-xenial/8324090//doc/build/html/ : SUCCESS in 2m 14s\n- gate-python-neutronclient-pep8-ubuntu-xenial http://logs.openstack.org/32/350432/2/check/gate-python-neutronclient-pep8-ubuntu-xenial/7547895/ : SUCCESS in 2m 12s\n- gate-python-neutronclient-python27-ubuntu-xenial http://logs.openstack.org/32/350432/2/check/gate-python-neutronclient-python27-ubuntu-xenial/a22368a/ : SUCCESS in 2m 20s\n- gate-python-neutronclient-python34 http://logs.openstack.org/32/350432/2/check/gate-python-neutronclient-python34/109f79a/ : SUCCESS in 5m 13s\n- gate-python-neutronclient-python35 http://logs.openstack.org/32/350432/2/check/gate-python-neutronclient-python35/22d7e70/ : SUCCESS in 4m 18s\n- gate-python-neutronclient-pypy http://logs.openstack.org/32/350432/2/check/gate-python-neutronclient-pypy/b3a09ce/ : SUCCESS in 4m 29s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-neutronclient http://logs.openstack.org/32/350432/2/check/gate-tempest-dsvm-neutron-src-python-neutronclient/4e0697f/ : SUCCESS in 1h 25m 55s\n- check-osc-plugins http://logs.openstack.org/32/350432/2/check/check-osc-plugins/930d399/ : SUCCESS in 6m 30s\n- gate-neutronclient-test-dsvm-functional http://logs.openstack.org/32/350432/2/check/gate-neutronclient-test-dsvm-functional/ddaed7e/ : SUCCESS in 34m 52s\n- gate-neutronclient-test-dsvm-functional-adv-svcs http://logs.openstack.org/32/350432/2/check/gate-neutronclient-test-dsvm-functional-adv-svcs/b0429df/ : SUCCESS in 37m 45s","accounts_in_message":[],"_revision_number":2},{"id":"b6c07131d95d410ffb648bf131b616b29ec5f12a","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2016-08-04 08:21:01.000000000","message":"Patch Set 2:\n\ni think that we should duplicate the checks. i agree with armando. will add this","accounts_in_message":[],"_revision_number":2},{"id":"0d9ff7958913eba6bbaee99d995d6b02ac112e2a","author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"date":"2016-08-04 18:57:17.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"8775c0da625c85761aa974969b443c8ed6d86fd1","author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"date":"2016-08-05 07:29:45.000000000","message":"Patch Set 2:\n\nAnyone can explain what\u0027s the issue with adding the test only (!) dependency?","accounts_in_message":[],"_revision_number":2},{"id":"4f87fd3328e8bee333b2d2a72199a61ecbb536b7","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-05 22:57:34.000000000","message":"Patch Set 2:\n\n\u003e Armando, do you think duplicating hacking effort is viable?\n \u003e neutron-lib would be a test only dep, what\u0027s the issue with that?\n\nneutron-lib is not a container of random library functions, IMO, neutron-lib is a server side library. Making it a requirement for the client makes no sense to me.\n\nDealing with the translation hints is something you do once and forget about it, if someone really cared to avoid the duplication. it should be into an hypothetical oslo.\u003caddyourname\u003e.\n\nI\u0027d rather not have have lib potentially break yet another project for no good reason, and translation hints is definitely NOT a good reason.","accounts_in_message":[],"_revision_number":2},{"id":"02eb573897f9cf24d6a16ba8a78547cf3ce97300","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-05 22:57:56.000000000","message":"Patch Set 2:\n\n\u003e Anyone can explain what\u0027s the issue with adding the test only (!)\n \u003e dependency?\n\nSorry, it took me a while to notice this, be patient.","accounts_in_message":[],"_revision_number":2},{"id":"c9057198758554998a424d802fec9dff52bf74c0","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-05 23:00:57.000000000","message":"Patch Set 2:\n\nMy apologies for not having justified why I thought it was not a good idea have neutron-lib be a dependency for the client. Hopefully I was able to present a reasonable argument.","accounts_in_message":[],"_revision_number":2},{"id":"27383b02285cdb6f0a315d1def694f88e83c3393","author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"date":"2016-08-06 07:09:42.000000000","message":"Patch Set 2:\n\nI am not convinced. If potential breakage of neutronclient pep8 job by a new release of neutron-lib hacking rules a problem to solve by duplicating code, then the same argument should work for other projects (server or not), and we should just stop advertising those hacking rules for consumption.\n\nThe breakage surface is really isolated to that single check, since lately neutron-lib allows to pick and choose checks to enforce: https://review.openstack.org/#/c/333500/\n\nIf we really want to avoid using the library for non-server projects, then we should at least update setup.cfg to give the hint to potential consumers that it\u0027s discouraged, since currently it just says the library is \"Neutron shared routines and utilities\". Though I don\u0027t think that\u0027s the path we should take, at least that would be a tad more consistent.","accounts_in_message":[],"_revision_number":2},{"id":"d7e103a2dd34eecd9a0e2f75001288e74af00083","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-06 15:35:35.000000000","message":"Patch Set 2:\n\n\u003e I am not convinced. If potential breakage of neutronclient pep8 job\n \u003e by a new release of neutron-lib hacking rules a problem to solve by\n \u003e duplicating code, then the same argument should work for other\n \u003e projects (server or not), and we should just stop advertising those\n \u003e hacking rules for consumption.\n \u003e \n \u003e The breakage surface is really isolated to that single check, since\n \u003e lately neutron-lib allows to pick and choose checks to enforce:\n \u003e https://review.openstack.org/#/c/333500/\n \u003e \n \u003e If we really want to avoid using the library for non-server\n \u003e projects, then we should at least update setup.cfg to give the hint\n \u003e to potential consumers that it\u0027s discouraged, since currently it\n \u003e just says the library is \"Neutron shared routines and utilities\".\n \u003e Though I don\u0027t think that\u0027s the path we should take, at least that\n \u003e would be a tad more consistent.\n\nI expressed my opinion, we don\u0027t have to agree :)\n\nLet\u0027s see what others think and we can decide on a path forward. I remain of my opinion that making the client depend on neutron-lib makes no sense whatsoever.","accounts_in_message":[],"_revision_number":2},{"id":"7115378dc8bfcf045bd8771551180adcc73a5e91","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-09 21:29:54.000000000","message":"Patch Set 2: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":2},{"id":"d60801f37a0634f1eb4a1a34d4fc55562eabc393","author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"date":"2016-08-10 04:44:05.000000000","message":"Patch Set 2: Code-Review-1\n\nHonestly I don\u0027t think we need to use _LE in the CLI code. Both of exception messages and error messages are displayed to CLI users (on terminal), so there is no reason to distinguish these two. This is the reason we don\u0027t use _LE/_LW/_LI family in neutronclient code.\n\nFrom the above reason, the infra and I18n team set up separate PO/POT files only for server projects. We use a single PO/POT file for client or library projects.","accounts_in_message":[],"_revision_number":2},{"id":"eb0885543b0527e7467bab5e45e72d65b245fa4c","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-08-10 18:04:58.000000000","message":"Patch Set 2:\n\n\u003e Honestly I don\u0027t think we need to use _LE in the CLI code. Both of\n \u003e exception messages and error messages are displayed to CLI users\n \u003e (on terminal), so there is no reason to distinguish these two. This\n \u003e is the reason we don\u0027t use _LE/_LW/_LI family in neutronclient\n \u003e code.\n \u003e \n \u003e From the above reason, the infra and I18n team set up separate\n \u003e PO/POT files only for server projects. We use a single PO/POT file\n \u003e for client or library projects.\n\nUnderstood, my suggestion is to focus on higher priority items.","accounts_in_message":[],"_revision_number":2},{"id":"994c62075acbe518118d5b7d973b7fc6bdd945f1","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-10-24 17:14:37.000000000","message":"Patch Set 2:\n\nThis review is \u003e 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":2},{"id":"5f43990aa9f22c02ed563183dbfef17efb57ce4c","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2016-10-24 17:14:37.000000000","message":"Abandoned\n\nThis review is \u003e 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"ae365f5fcdbb1ab757ee29a85b42d69c5b35e68c","revisions":{"7547fea5778d177d5a916ada1fb7179e65cce1d1":{"kind":"REWORK","_number":1,"created":"2016-08-03 08:17:57.000000000","uploader":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"ref":"refs/changes/32/350432/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-neutronclient","ref":"refs/changes/32/350432/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/1"}}},"commit":{"parents":[{"commit":"5da767d36e8b2343fabab674dbacb75181efb774","subject":"Merge \"Add trunk commands to openstackclient\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-neutronclient/commit/5da767d36e8b2343fabab674dbacb75181efb774"}]}],"author":{"name":"Gary Kotton","email":"gkotton@vmware.com","date":"2016-08-03 08:16:59.000000000","tz":-420},"committer":{"name":"Gary Kotton","email":"gkotton@vmware.com","date":"2016-08-03 08:16:59.000000000","tz":-420},"subject":"Add in missing translation hint for log error","message":"Add in missing translation hint for log error\n\nA log error should have the hint _LE\n\nTrivialFix\n\nChange-Id: I50c88bc256de1a7a747e8b7f37d32ab803bdb54e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-neutronclient/commit/7547fea5778d177d5a916ada1fb7179e65cce1d1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-neutronclient/commit/7547fea5778d177d5a916ada1fb7179e65cce1d1"}]},"branch":"refs/heads/master"},"ae365f5fcdbb1ab757ee29a85b42d69c5b35e68c":{"kind":"REWORK","_number":2,"created":"2016-08-04 05:46:39.000000000","uploader":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"ref":"refs/changes/32/350432/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-neutronclient","ref":"refs/changes/32/350432/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-neutronclient refs/changes/32/350432/2"}}},"commit":{"parents":[{"commit":"5da767d36e8b2343fabab674dbacb75181efb774","subject":"Merge \"Add trunk commands to openstackclient\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-neutronclient/commit/5da767d36e8b2343fabab674dbacb75181efb774"}]}],"author":{"name":"Gary Kotton","email":"gkotton@vmware.com","date":"2016-08-03 08:16:59.000000000","tz":-420},"committer":{"name":"Gary Kotton","email":"gkotton@vmware.com","date":"2016-08-04 05:46:25.000000000","tz":-420},"subject":":Add in missing translation hint for log error","message":":Add in missing translation hint for log error\n\nA log error should have the hint _LE\n\nTrivialFix\n\nChange-Id: I50c88bc256de1a7a747e8b7f37d32ab803bdb54e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-neutronclient/commit/ae365f5fcdbb1ab757ee29a85b42d69c5b35e68c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-neutronclient/commit/ae365f5fcdbb1ab757ee29a85b42d69c5b35e68c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}