)]}'
{"id":"openstack%2Fdevstack~53236","triplet_id":"openstack%2Fdevstack~master~Ibe19085545ecc848498506e8b8ee14e71825b273","project":"openstack/devstack","branch":"master","topic":"check-value","hashtags":[],"change_id":"Ibe19085545ecc848498506e8b8ee14e71825b273","subject":"add assertions for blind grep","status":"MERGED","created":"2013-10-22 22:07:37.000000000","updated":"2013-12-05 00:19:50.000000000","submitted":"2013-12-05 00:19:49.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2735e49a2a54bac138ed92f0d4d2dcff9c34ed2b","_number":53236,"virtual_id_number":53236,"owner":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":2,"date":"2013-12-05 00:19:49.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"all":[{"value":1,"date":"2013-11-04 17:04:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-11-20 12:47:18.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},{"value":1,"date":"2013-11-05 19:33:17.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":2,"date":"2013-12-04 17:40:02.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":2,"date":"2013-11-28 13:22:13.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"date":"2013-10-22 22:12:44.000000000","_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":1,"date":"2013-12-04 17:40:02.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-11-04 17:04:32.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"date":"2013-11-04 17:04:21.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-11-04 17:04:21.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-11-04 17:04:32.000000000","updated_by":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"reviewer":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"state":"REVIEWER"},{"updated":"2013-11-05 19:33:17.000000000","updated_by":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"reviewer":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"state":"REVIEWER"},{"updated":"2013-11-20 12:47:18.000000000","updated_by":{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},"reviewer":{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},"state":"REVIEWER"},{"updated":"2013-12-04 17:40:02.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-12-05 00:19:49.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"e2df4dd419ca309b265d3e2f7c8d631704cca3d3","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-22 22:12:44.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"837c737a5d23bf9e91fc55248fbc64459049e1a3","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-10-22 22:12:51.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"cc990503834c8d158c759d7cc7cb2c70f25a47b4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-22 22:50:43.000000000","message":"Patch Set 2: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/36/53236/2/check/gate-devstack-bash8/41d101d : SUCCESS in 4s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-full/57903f0 : SUCCESS in 30m 39s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-postgres-full/70d2ad4 : FAILURE in 37m 50s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-neutron/248f7d3 : SUCCESS in 20m 08s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/36/53236/2/check/gate-tempest-devstack-vm-large-ops/b58d5e0 : SUCCESS in 16m 20s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/36/53236/2/check/gate-tempest-devstack-vm-neutron-large-ops/8e71240 : SUCCESS in 16m 48s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/36/53236/2/check/check-grenade-devstack-vm/62bbfeb : SUCCESS in 24m 16s\n- check-swift-devstack-vm-functional http://logs.openstack.org/36/53236/2/check/check-swift-devstack-vm-functional/e34b33b : SUCCESS in 12m 27s\n","accounts_in_message":[],"_revision_number":2},{"id":"bc13ce6659ef3eff8a912937976598a747162200","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2013-10-22 22:51:36.000000000","message":"Patch Set 2:\n\nI noticed tempest failed, I think you hit bug(s): https://bugs.launchpad.net/bugs/1239856","accounts_in_message":[],"_revision_number":2},{"id":"8e4a9dbdd7ab1cb087dadde7e8fb69fb8e965f5c","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-23 00:53:46.000000000","message":"Patch Set 2:\n\nrecheck bug 1239856","accounts_in_message":[],"_revision_number":2},{"id":"0a4f17bf3d8ffc720a7cf01e5e39d5074e43ce4d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-23 01:41:25.000000000","message":"Patch Set 2:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/36/53236/2/check/gate-devstack-bash8/01153b9 : SUCCESS in 13s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-full/4fd543a : FAILURE in 43m 41s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-postgres-full/95fbe07 : SUCCESS in 46m 15s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-neutron/b4a3cd5 : SUCCESS in 41m 36s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/36/53236/2/check/gate-tempest-devstack-vm-large-ops/cbc03af : SUCCESS in 37m 36s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/36/53236/2/check/gate-tempest-devstack-vm-neutron-large-ops/4c49259 : SUCCESS in 38m 51s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/36/53236/2/check/check-grenade-devstack-vm/3c6e277 : SUCCESS in 46m 51s\n- check-swift-devstack-vm-functional http://logs.openstack.org/36/53236/2/check/check-swift-devstack-vm-functional/2f56165 : SUCCESS in 38m 37s\n","accounts_in_message":[],"_revision_number":2},{"id":"127d4c2b92f82a5c9d745a351e92351ce635da54","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-23 03:29:14.000000000","message":"Patch Set 2:\n\nrecheck bug 1239856","accounts_in_message":[],"_revision_number":2},{"id":"1cf274d30419c84ba01640372b0ee344baf6713b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-23 04:50:20.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/36/53236/2/check/gate-devstack-bash8/ac95d47 : SUCCESS in 10s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-full/39de485 : SUCCESS in 29m 12s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-postgres-full/f85e9f7 : SUCCESS in 41m 02s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/36/53236/2/check/check-tempest-devstack-vm-neutron/0bb899c : SUCCESS in 18m 12s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/36/53236/2/check/gate-tempest-devstack-vm-large-ops/19b5424 : SUCCESS in 14m 20s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/36/53236/2/check/gate-tempest-devstack-vm-neutron-large-ops/b0f08ed : SUCCESS in 16m 45s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/36/53236/2/check/check-grenade-devstack-vm/fc006ab : SUCCESS in 23m 56s\n- check-swift-devstack-vm-functional http://logs.openstack.org/36/53236/2/check/check-swift-devstack-vm-functional/83a346a : SUCCESS in 12m 26s\n","accounts_in_message":[],"_revision_number":2},{"id":"153fa733a15108046a792d717b3047b9224d7e7a","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-11-04 17:04:06.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nAdd +1 for myself, since current assertion help to detect many failures.","accounts_in_message":[],"_revision_number":2},{"id":"28a6ddc60a092004c01efb33a7622bcb2ff6564a","author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"date":"2013-11-05 19:33:17.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n(1 inline comment)\n\nSee my comment. I think when we have something like:\n\nsome_command | grep \"$VAR\"\n\nwe should first test that \"some_command\" did not fail, then try to grep. This way, the user can know whether \"some_command\" failed or if it just did not output what was expected.\n\nStill, I +1 this patch because it seems helpful as is.","accounts_in_message":[],"_revision_number":2},{"id":"c0bed094659e4eceaa803ff9a834bad61d20a265","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-11-05 19:55:16.000000000","message":"Patch Set 2:\n\nCyril, I\u0027m totally agree with you. And it\u0027s also my first intent, when I try to solve the problem.\n\nAfter a second thought, this would make each change need several extra lines: check $?, echo message, then grep. If we provide a common function for this, this would decrease the code readability.\n\nOf a trade-off, I submit current patch with assertions.\n\nAny better suggestions are warmly welcome.","accounts_in_message":[],"_revision_number":2},{"id":"e55ff6fee00e39a2c3030bf27569521d2c2eae4a","author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"date":"2013-11-06 12:34:50.000000000","message":"Patch Set 2:\n\nSure, it\u0027s ok to keep the patch as is. As I said, it\u0027s already quite helpful.","accounts_in_message":[],"_revision_number":2},{"id":"fc020d64464eda973bce51d7085880a447298ed8","author":{"_account_id":9126,"name":"Chris Buccella","email":"chris@buccella.org","username":"hdd"},"date":"2013-11-20 12:47:18.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"e6a6158b35246fb3caf91d4c477cb7d1ec0a3ba5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-11-28 13:22:13.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nnice add! I like the idea of actually knowing when we fail :)","accounts_in_message":[],"_revision_number":2},{"id":"ec95b8325b4b9422a78c855f67e29fe4a8887fac","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-04 17:40:02.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"fad87bba21692cdf042305ab0dce256982712bdb","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-12-04 17:40:02.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"a872a4f5e7d9dfdee821892e16c1e7c91505697a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-04 19:35:55.000000000","message":"Patch Set 2: Fails\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/36/53236/2/gate/gate-devstack-bash8/a9af94a : SUCCESS in 7s\n- gate-tempest-dsvm-full http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-full/eb8185c : FAILURE in 43m 40s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-postgres-full/2a17075 : FAILURE in 37m 46s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-neutron/3c33b8e : FAILURE in 27m 30s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-large-ops/aeb8999 : SUCCESS in 14m 26s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-neutron-large-ops/a64946a : SUCCESS in 17m 30s\n- gate-grenade-dsvm http://logs.openstack.org/36/53236/2/gate/gate-grenade-dsvm/e616e16 : SUCCESS in 32m 49s\n- gate-swift-dsvm-functional http://logs.openstack.org/36/53236/2/gate/gate-swift-dsvm-functional/3f823d8 : SUCCESS in 12m 37s\n","accounts_in_message":[],"_revision_number":2},{"id":"5f732ca56f7481231f2b1abdf7dcb4129d19981d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-04 20:29:35.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"60837912e3999e1b1fe1eb0a3b2ee28b1bb27e93","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-12-04 20:29:35.000000000","message":"Patch Set 2:\n\nreverify no bug","accounts_in_message":[],"_revision_number":2},{"id":"01a492053f774ff236bb309f380d21ec6bb3a42e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-04 20:45:39.000000000","message":"Patch Set 2:\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"9600f63d033a62f1344ce732dd78c8bec620e38b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-05 00:19:49.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/36/53236/2/gate/gate-devstack-bash8/3b2a26b : SUCCESS in 4s\n- gate-tempest-dsvm-full http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-full/8f51742 : SUCCESS in 37m 45s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-postgres-full/c34b540 : SUCCESS in 42m 12s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-neutron/4602b7a : SUCCESS in 25m 39s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-large-ops/0434108 : SUCCESS in 19m 42s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/36/53236/2/gate/gate-tempest-dsvm-neutron-large-ops/be46e03 : SUCCESS in 18m 28s\n- gate-grenade-dsvm http://logs.openstack.org/36/53236/2/gate/gate-grenade-dsvm/4ddf68e : SUCCESS in 33m 54s\n- gate-swift-dsvm-functional http://logs.openstack.org/36/53236/2/gate/gate-swift-dsvm-functional/472b657 : SUCCESS in 15m 26s\n","accounts_in_message":[],"_revision_number":2},{"id":"fcd4d1cb8c86a33ec1c2bb822e7be5f27f0a44f8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-05 00:19:50.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"23178a997a3b0abd1922f356e572e2933f454dc1","revisions":{"672ebf8dbdedbdcad9343cea45f2841d23f258b0":{"kind":"REWORK","_number":1,"created":"2013-10-22 22:07:37.000000000","uploader":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"ref":"refs/changes/36/53236/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/36/53236/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/53236/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/53236/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/53236/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/36/53236/1"}}},"commit":{"parents":[{"commit":"80c3ea52ed1caec16e5c6c6beecd434225afbd97","subject":"Merge \"exit cleanup in functions\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/80c3ea52ed1caec16e5c6c6beecd434225afbd97"}]}],"author":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-22 22:07:32.000000000","tz":-300},"committer":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-22 22:07:38.000000000","tz":-300},"subject":"add assertion for unexpected grep command","message":"add assertion for unexpected grep command\n\nblind grep is error-prone.\nAdd assertions for errors we can not handle\n\nChange-Id: Ibe19085545ecc848498506e8b8ee14e71825b273\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/672ebf8dbdedbdcad9343cea45f2841d23f258b0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/672ebf8dbdedbdcad9343cea45f2841d23f258b0"}]},"branch":"refs/heads/master"},"23178a997a3b0abd1922f356e572e2933f454dc1":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-10-22 22:12:44.000000000","uploader":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"ref":"refs/changes/36/53236/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/36/53236/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/53236/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/53236/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/53236/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/36/53236/2"}}},"commit":{"parents":[{"commit":"80c3ea52ed1caec16e5c6c6beecd434225afbd97","subject":"Merge \"exit cleanup in functions\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/80c3ea52ed1caec16e5c6c6beecd434225afbd97"}]}],"author":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-22 22:07:32.000000000","tz":-300},"committer":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-22 22:12:18.000000000","tz":-300},"subject":"add assertions for blind grep","message":"add assertions for blind grep\n\nblind grep is error-prone.\nAdd assertions for errors we can not handle\n\nChange-Id: Ibe19085545ecc848498506e8b8ee14e71825b273\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/23178a997a3b0abd1922f356e572e2933f454dc1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/23178a997a3b0abd1922f356e572e2933f454dc1"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}