)]}'
{"id":"openstack%2Fdevstack~114268","triplet_id":"openstack%2Fdevstack~master~Iba39cedbde7736f2647778dfc8107ac4f2c36b64","project":"openstack/devstack","branch":"master","topic":"fix-tftpboot","hashtags":[],"change_id":"Iba39cedbde7736f2647778dfc8107ac4f2c36b64","subject":"Add pxebin location for SUSE","status":"ABANDONED","created":"2014-08-14 15:19:21.000000000","updated":"2014-12-05 14:31:35.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"60b6fe98070009b03b080e017e749e66216ce0bc","_number":114268,"virtual_id_number":114268,"owner":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2014-08-19 22:47:26.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":-1,"date":"2014-08-19 22:34:50.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2014-08-20 16:18:16.000000000","_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"date":"2014-08-20 10:01:38.000000000","_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"date":"2014-12-05 14:31:35.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2014-08-20 16:29:17.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":-1,"date":"2014-08-22 01:04:52.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-08-19 19:59:21.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":"2014-08-19 22:34:50.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2014-08-19 22:47:26.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2014-08-20 16:18:16.000000000","updated_by":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"reviewer":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"state":"REVIEWER"},{"updated":"2014-08-22 01:04:52.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2014-12-05 14:31:35.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"}],"messages":[{"id":"a65b3f74d234d8ebaac74c76391a80cacaadeded","author":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"date":"2014-08-14 15:19:21.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4f02a2b4d6ec3a4dca4081dab5f0158b265f3c1f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-08-14 15:19:25.000000000","message":"Patch Set 1:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"46dbd10d2334e6e454a4919244285a73f18c5826","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-08-14 16:12:44.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded.\n\n- check-tempest-dsvm-full http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-full/64d6060 : SUCCESS in 41m 15s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-postgres-full/aa70bd7 : SUCCESS in 49m 06s\n- check-tempest-dsvm-neutron http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-neutron/4368ccd : SUCCESS in 48m 35s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-neutron-full/9d93018 : SUCCESS in 52m 54s (non-voting)\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-neutron-heat-slow/fa7a5a1 : SUCCESS in 20m 05s\n- check-grenade-dsvm http://logs.openstack.org/68/114268/1/check/check-grenade-dsvm/c4e0f6f : SUCCESS in 36m 27s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/68/114268/1/check/gate-tempest-dsvm-large-ops/150f13f : SUCCESS in 25m 31s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/68/114268/1/check/gate-tempest-dsvm-neutron-large-ops/18ba2c5 : SUCCESS in 23m 19s\n- gate-devstack-bashate http://logs.openstack.org/68/114268/1/check/gate-devstack-bashate/b637808 : SUCCESS in 51s\n- check-devstack-dsvm-cells http://logs.openstack.org/68/114268/1/check/check-devstack-dsvm-cells/a186f3f : SUCCESS in 17m 33s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/68/114268/1/check/check-grenade-dsvm-partial-ncpu/5e5b578 : SUCCESS in 38m 17s\n- check-neutron-dsvm-functional http://logs.openstack.org/68/114268/1/check/check-neutron-dsvm-functional/375c2e4 : SUCCESS in 19m 22s\n- check-swift-dsvm-functional http://logs.openstack.org/68/114268/1/check/check-swift-dsvm-functional/c1b605b : SUCCESS in 16m 05s\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-ironic-nv/d6b992a : SUCCESS in 37m 12s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-ironic-postgres-nv/ae814af : SUCCESS in 34m 53s (non-voting)\n- check-tempest-dsvm-virtual-ironic-nv http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-virtual-ironic-nv/84cb726 : SUCCESS in 31m 49s (non-voting)\n- check-tempest-dsvm-f20 http://logs.openstack.org/68/114268/1/check/check-tempest-dsvm-f20/5ea2df1 : SUCCESS in 32m 25s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"deb06185e089d16ef18c44eb292fcaf4ce88e48c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-08-19 19:59:21.000000000","message":"Patch Set 1: -Verified\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"0e31e4d1428870054662b60a7326350482428dab","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-08-19 19:59:21.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"b67f0047765082095d837b025d90f24bc5f0b8a0","author":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"date":"2014-08-19 20:09:50.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f16eb89a6217bd7f73743f9ab3d5e84e0ff5cec4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-08-19 20:13:30.000000000","message":"Patch Set 2:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"3e332cdf243c062f3a40cd46e55f71d6e44a703b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-08-19 22:34:50.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- check-tempest-dsvm-full http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-full/1433834 : SUCCESS in 44m 06s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-postgres-full/67e300d : SUCCESS in 43m 42s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-neutron-full/dd52cfe : SUCCESS in 46m 17s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-neutron-heat-slow/6f9e2bf : SUCCESS in 23m 28s\n- check-grenade-dsvm http://logs.openstack.org/68/114268/2/check/check-grenade-dsvm/5fcfa96 : FAILURE in 43m 07s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/68/114268/2/check/gate-tempest-dsvm-large-ops/68fa608 : SUCCESS in 20m 51s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/68/114268/2/check/gate-tempest-dsvm-neutron-large-ops/c7875cc : SUCCESS in 23m 16s\n- gate-devstack-bashate http://logs.openstack.org/68/114268/2/check/gate-devstack-bashate/e7f3424 : SUCCESS in 23s\n- check-devstack-dsvm-cells http://logs.openstack.org/68/114268/2/check/check-devstack-dsvm-cells/c92c8f7 : SUCCESS in 16m 51s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/68/114268/2/check/check-grenade-dsvm-partial-ncpu/c622590 : SUCCESS in 40m 22s\n- check-neutron-dsvm-functional http://logs.openstack.org/68/114268/2/check/check-neutron-dsvm-functional/9205523 : SUCCESS in 19m 26s\n- check-swift-dsvm-functional http://logs.openstack.org/68/114268/2/check/check-swift-dsvm-functional/fe26245 : SUCCESS in 19m 08s\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-ironic-nv/8f66f76 : SUCCESS in 33m 37s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-ironic-postgres-nv/db6049b : SUCCESS in 33m 01s (non-voting)\n- check-tempest-dsvm-virtual-ironic-nv http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-virtual-ironic-nv/8dd368f : SUCCESS in 34m 35s (non-voting)\n- check-tempest-dsvm-f20 http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-f20/4f56ff0 : SUCCESS in 33m 21s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"8928ef65e628e7ca1fbc3ed7064beb81ca828387","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2014-08-19 22:47:26.000000000","message":"Patch Set 2: Verified+1\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/68/114268/2/22332/results.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":2},{"id":"26873fd0e1a1e7f40a9756c36f54e3adb63fc20c","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2014-08-19 23:32:31.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"9ca6bbb8dd57e2fad66a56ee4bcacc02304ed9a5","author":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"date":"2014-08-20 10:01:38.000000000","message":"Patch Set 2:\n\ncheck-grenade-dsvm failed with the following error in nova-compute logs: \u0027libvirtError: Unable to read from monitor: Connection reset by peer\u0027[0]\n\nrecheck no bug\n\n[0] http://logs.openstack.org/68/114268/2/check/check-grenade-dsvm/5fcfa96/logs/new/screen-n-cpu.txt.gz?#_2014-08-19_21_38_26_094","accounts_in_message":[],"_revision_number":2},{"id":"7ca0634bca3b376b522f8eea8abb16c7c0942aaa","author":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"date":"2014-08-20 11:05:56.000000000","message":"Patch Set 2:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":2},{"id":"54717838c16dd253e7bcf1da73e43b1312857287","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-08-20 12:08:12.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- check-tempest-dsvm-full http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-full/24e886c : SUCCESS in 40m 03s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-postgres-full/f3b366b : SUCCESS in 51m 07s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-neutron-full/2bb70d4 : SUCCESS in 54m 36s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-neutron-heat-slow/9aa2c1c : SUCCESS in 24m 53s\n- check-grenade-dsvm http://logs.openstack.org/68/114268/2/check/check-grenade-dsvm/4d694bd : FAILURE in 31m 50s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/68/114268/2/check/gate-tempest-dsvm-large-ops/7171ec8 : SUCCESS in 22m 10s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/68/114268/2/check/gate-tempest-dsvm-neutron-large-ops/df70a06 : SUCCESS in 20m 12s\n- gate-devstack-bashate http://logs.openstack.org/68/114268/2/check/gate-devstack-bashate/440e4a4 : SUCCESS in 13s\n- check-devstack-dsvm-cells http://logs.openstack.org/68/114268/2/check/check-devstack-dsvm-cells/e8559c4 : SUCCESS in 14m 39s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/68/114268/2/check/check-grenade-dsvm-partial-ncpu/22f01a6 : SUCCESS in 36m 11s\n- check-neutron-dsvm-functional http://logs.openstack.org/68/114268/2/check/check-neutron-dsvm-functional/03862b7 : SUCCESS in 20m 23s\n- check-swift-dsvm-functional http://logs.openstack.org/68/114268/2/check/check-swift-dsvm-functional/496af50 : SUCCESS in 17m 08s\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-ironic-nv/2b0d1b2 : SUCCESS in 34m 47s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-ironic-postgres-nv/d8d6b57 : SUCCESS in 34m 40s (non-voting)\n- check-tempest-dsvm-virtual-ironic-nv http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-virtual-ironic-nv/0d2c879 : SUCCESS in 34m 08s (non-voting)\n- check-tempest-dsvm-f20 http://logs.openstack.org/68/114268/2/check/check-tempest-dsvm-f20/7a7502e : SUCCESS in 1h 01m 53s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"6fab87984c0251174f6a6caa14ba6e5c756b210d","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2014-08-20 16:18:16.000000000","message":"Patch Set 2:\n\nHi Felipe,\n\nI\u0027ve a patch that touches exactly this part of the code[1] so I was thinking about rebasing on top of this patch, but I first need you to rebase it on top of master, can you do that please?\n\nThanks\n\n[1] https://review.openstack.org/#/c/99677","accounts_in_message":[],"_revision_number":2},{"id":"c5de22079a909615ba86a777a9d375f8cca0bead","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2014-08-20 16:29:17.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nOne more thing, this check is duplicated in the code :/ so you need to add \"|| is_suse\" on that conditional as well","accounts_in_message":[],"_revision_number":2},{"id":"256f77e8ecf36e16b0744b6f968bf7fb125aaafa","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2014-08-22 01:04:52.000000000","message":"Patch Set 2: Code-Review-1\n\nthis should now be covered by [1]\n\n[1] https://review.openstack.org/#/c/99677/","accounts_in_message":[],"_revision_number":2},{"id":"1945e642c5a60de90246fa47622ff99ecd06bbd3","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-12-05 14:31:35.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},{"id":"59a5b7fb1b082e289ce267c3f2b2784ef469dff7","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-12-05 14:31:35.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}],"current_revision_number":2,"current_revision":"2789a81a9c542d479b0c7e7732b94f92865598d8","revisions":{"1600f30c2153fbd4068f9ba8cc4f5968a85fab0e":{"kind":"REWORK","_number":1,"created":"2014-08-14 15:19:21.000000000","uploader":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"ref":"refs/changes/68/114268/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/68/114268/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/114268/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/114268/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/114268/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/68/114268/1"}}},"commit":{"parents":[{"commit":"6db41b9716c5a82a32818d87ad3af662ecc1382a","subject":"Merge \"Exact match rhel6\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6db41b9716c5a82a32818d87ad3af662ecc1382a"}]}],"author":{"name":"Felipe Reyes","email":"freyes@suse.com","date":"2014-08-14 15:17:17.000000000","tz":120},"committer":{"name":"Felipe Reyes","email":"freyes@suse.com","date":"2014-08-14 15:18:06.000000000","tz":120},"subject":"Add pxebin location for SUSE","message":"Add pxebin location for SUSE\n\nThis patch checks if the system running a SUSE based distribution and sets\nthe path where pxelinux.0 is.\n\nChange-Id: Iba39cedbde7736f2647778dfc8107ac4f2c36b64\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1600f30c2153fbd4068f9ba8cc4f5968a85fab0e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1600f30c2153fbd4068f9ba8cc4f5968a85fab0e"}]},"branch":"refs/heads/master"},"2789a81a9c542d479b0c7e7732b94f92865598d8":{"kind":"REWORK","_number":2,"created":"2014-08-19 20:09:50.000000000","uploader":{"_account_id":2424,"name":"Felipe Reyes","email":"felipe.reyes@canonical.com","username":"freyes"},"ref":"refs/changes/68/114268/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/68/114268/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/114268/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/114268/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/114268/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/68/114268/2"}}},"commit":{"parents":[{"commit":"6db41b9716c5a82a32818d87ad3af662ecc1382a","subject":"Merge \"Exact match rhel6\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6db41b9716c5a82a32818d87ad3af662ecc1382a"}]}],"author":{"name":"Felipe Reyes","email":"freyes@suse.com","date":"2014-08-14 15:17:17.000000000","tz":120},"committer":{"name":"Felipe Reyes","email":"freyes@suse.com","date":"2014-08-19 20:07:35.000000000","tz":120},"subject":"Add pxebin location for SUSE","message":"Add pxebin location for SUSE\n\nThis patch checks if the system running a SUSE based distribution and sets\nthe path where pxelinux.0 is.\n\nChange-Id: Iba39cedbde7736f2647778dfc8107ac4f2c36b64\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2789a81a9c542d479b0c7e7732b94f92865598d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2789a81a9c542d479b0c7e7732b94f92865598d8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}