)]}'
{"id":"openstack%2Fdiskimage-builder~268241","triplet_id":"openstack%2Fdiskimage-builder~master~Ie143a1e94f0be6bf464e20bd9d978aed9868b629","project":"openstack/diskimage-builder","branch":"master","topic":"really-prune-stale-branches","hashtags":[],"change_id":"Ie143a1e94f0be6bf464e20bd9d978aed9868b629","subject":"Actually prune stale branches in src-repos","status":"ABANDONED","created":"2016-01-15 17:28:57.000000000","updated":"2016-02-12 02:34:16.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"93dd368cde30e5377c1eecac9fca99827d1ff4d7","_number":268241,"virtual_id_number":268241,"owner":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2016-01-29 03:59:41.000000000","_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"date":"2016-01-19 05:39:03.000000000","_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":1,"date":"2016-01-18 21:39:04.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"}],"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":{"recommended":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"value":1,"date":"2016-01-25 17:40:52.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"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":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"}],"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"}],"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":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-01-15 17:51:02.000000000","updated_by":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"reviewer":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"state":"REVIEWER"},{"updated":"2016-01-16 13:49:44.000000000","updated_by":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"reviewer":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"state":"REVIEWER"},{"updated":"2016-01-18 21:39:04.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-01-19 05:39:03.000000000","updated_by":{"_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},"reviewer":{"_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},"state":"REVIEWER"},{"updated":"2016-01-25 17:40:52.000000000","updated_by":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"reviewer":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"state":"REVIEWER"},{"updated":"2016-01-29 03:59:41.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"}],"messages":[{"id":"f60143d06576a33270072fb5b5cb518398751e89","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-01-15 17:28:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"676d81a2d8eae7af9300cded789e66cae8e4cd8d","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2016-01-15 17:31:11.000000000","message":"Patch Set 1: Code-Review+1\n\nVery much in favor of keeping DIB\u0027s Git repo caches in better (and more correct) shape.","accounts_in_message":[],"_revision_number":1},{"id":"c29af0295bed01c1f7aa89728504702a1418a0a8","author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"date":"2016-01-15 17:45:20.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"eb556bd4ba38fb5b9c9b8e707353aafe28fea0a8","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-01-15 17:48:05.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"86ded5360b388aa2d737e0c2f0dd202ce934bcc9","author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"date":"2016-01-15 17:48:52.000000000","message":"Patch Set 2: Code-Review-2","accounts_in_message":[],"_revision_number":2},{"id":"3bfe86b86d525cac0b82d525bd68c0751d56d8bb","author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"date":"2016-01-15 17:51:02.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"ec60ccf3c864534de047d75ae232f1c34c716dab","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-15 18:39:59.000000000","message":"Patch Set 2: 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-diskimage-builder-pep8 http://logs.openstack.org/41/268241/2/check/gate-diskimage-builder-pep8/d742f0f/ : SUCCESS in 1m 00s\n- gate-diskimage-builder-docs http://docs-draft.openstack.org/41/268241/2/check/gate-diskimage-builder-docs/18bab14//doc/build/html/ : SUCCESS in 1m 01s\n- gate-diskimage-builder-python27 http://logs.openstack.org/41/268241/2/check/gate-diskimage-builder-python27/4425cdd/ : SUCCESS in 50s\n- gate-diskimage-builder-python34 http://logs.openstack.org/41/268241/2/check/gate-diskimage-builder-python34/a54e74a/ : SUCCESS in 55s\n- gate-dib-dsvm-functests-devstack-trusty http://logs.openstack.org/41/268241/2/check/gate-dib-dsvm-functests-devstack-trusty/fe6d73f/ : SUCCESS in 35m 26s\n- gate-dib-dsvm-functests-devstack-centos7 http://logs.openstack.org/41/268241/2/check/gate-dib-dsvm-functests-devstack-centos7/a250e34/ : FAILURE in 21m 37s\n- gate-tempest-dsvm-ironic-pxe_ssh-dib http://logs.openstack.org/41/268241/2/check/gate-tempest-dsvm-ironic-pxe_ssh-dib/bf8ab9b/ : SUCCESS in 44m 39s","accounts_in_message":[],"_revision_number":2},{"id":"77c16c25bb669e27996a53349d9e3f465ac4cc00","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-15 19:35:44.000000000","message":"Patch Set 2:\n\nBuild succeeded (check-tripleo pipeline).\n\n- gate-tripleo-ci-f22-nonha http://logs.openstack.org/41/268241/2/check-tripleo/gate-tripleo-ci-f22-nonha/8468d7f/ : SUCCESS in 1h 47m 24s","accounts_in_message":[],"_revision_number":2},{"id":"23c5d03a7280c76b09863ac6aefcb4e49cb643fc","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-01-15 20:17:29.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"373f7c2555271e62fd503b6189711e1d2e2be02f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-15 21:01:35.000000000","message":"Patch Set 2:\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-diskimage-builder-pep8 http://logs.openstack.org/41/268241/2/check/gate-diskimage-builder-pep8/6a717eb/ : SUCCESS in 55s\n- gate-diskimage-builder-docs http://docs-draft.openstack.org/41/268241/2/check/gate-diskimage-builder-docs/e3360ef//doc/build/html/ : SUCCESS in 57s\n- gate-diskimage-builder-python27 http://logs.openstack.org/41/268241/2/check/gate-diskimage-builder-python27/28a5d47/ : SUCCESS in 43s\n- gate-diskimage-builder-python34 http://logs.openstack.org/41/268241/2/check/gate-diskimage-builder-python34/394471c/ : SUCCESS in 42s\n- gate-dib-dsvm-functests-devstack-trusty http://logs.openstack.org/41/268241/2/check/gate-dib-dsvm-functests-devstack-trusty/a9115a7/ : SUCCESS in 30m 23s\n- gate-dib-dsvm-functests-devstack-centos7 http://logs.openstack.org/41/268241/2/check/gate-dib-dsvm-functests-devstack-centos7/10886fc/ : FAILURE in 22m 26s\n- gate-tempest-dsvm-ironic-pxe_ssh-dib http://logs.openstack.org/41/268241/2/check/gate-tempest-dsvm-ironic-pxe_ssh-dib/25d944c/ : SUCCESS in 34m 59s","accounts_in_message":[],"_revision_number":2},{"id":"13ab5b32b79350b4b58d11e7ee4de86b73c4c9da","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-01-15 21:31:58.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"9eb19d58e18c2f6e643010d9fe21f7f4b8f2342c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-15 22:17:23.000000000","message":"Patch Set 3: 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-diskimage-builder-pep8 http://logs.openstack.org/41/268241/3/check/gate-diskimage-builder-pep8/796a17e/ : SUCCESS in 54s\n- gate-diskimage-builder-docs http://docs-draft.openstack.org/41/268241/3/check/gate-diskimage-builder-docs/c1cae40//doc/build/html/ : SUCCESS in 1m 02s\n- gate-diskimage-builder-python27 http://logs.openstack.org/41/268241/3/check/gate-diskimage-builder-python27/ac22b4a/ : SUCCESS in 56s\n- gate-diskimage-builder-python34 http://logs.openstack.org/41/268241/3/check/gate-diskimage-builder-python34/ddffff2/ : SUCCESS in 47s\n- gate-dib-dsvm-functests-devstack-trusty http://logs.openstack.org/41/268241/3/check/gate-dib-dsvm-functests-devstack-trusty/c0a35c1/ : FAILURE in 22m 23s\n- gate-dib-dsvm-functests-devstack-centos7 http://logs.openstack.org/41/268241/3/check/gate-dib-dsvm-functests-devstack-centos7/3037633/ : FAILURE in 25m 37s\n- gate-tempest-dsvm-ironic-pxe_ssh-dib http://logs.openstack.org/41/268241/3/check/gate-tempest-dsvm-ironic-pxe_ssh-dib/e5919e3/ : SUCCESS in 42m 55s","accounts_in_message":[],"_revision_number":3},{"id":"ffc5d983c7b1add2b3066c533e9024893763b5e7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-16 00:18:42.000000000","message":"Patch Set 3:\n\nBuild failed (check-tripleo pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- gate-tripleo-ci-f22-nonha http://logs.openstack.org/41/268241/3/check-tripleo/gate-tripleo-ci-f22-nonha/f1cc70f/ : FAILURE in 2h 46m 35s","accounts_in_message":[],"_revision_number":3},{"id":"68e02b5c141eedbff5c3de020af931a0bde4147a","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2016-01-16 13:49:44.000000000","message":"Patch Set 3: Code-Review+1\n\nWhat a hassle... :/","accounts_in_message":[],"_revision_number":3},{"id":"f49cd462a9e4eb4f5889ca07a0a07b72616c2211","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-01-17 23:41:38.000000000","message":"Patch Set 3:\n\nrecheck\n\nfailures are unrelated","accounts_in_message":[],"_revision_number":3},{"id":"4cabb34af939467447ed631059749d0083528202","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-01-18 00:04:24.000000000","message":"Patch Set 3: Code-Review-1\n\nI want to see this fixed as much as the next person, but I really don\u0027t understand how we are getting *local* tracking branches appearing in what is supposed to be a git mirror that has been fetched into\n\nin my local dib .cache I don\u0027t see anything other than master in the source repos cache\n\n$ for d in *; do if [ -d ./$d/.git ]; then  git --git-dir\u003d./$d/.git branch; fi; done | grep -v master\n$ \n\nwhat checked out the branches?  was it a bug potentially months ago that is since resolved, and we just need to clear ~/.cache on the dib builder?  \n\nit does not seem like it is expected, and my concern is that if we\u0027re things like local branches in the cache that we don\u0027t expect, then is there a chance we\u0027re not always testing what we think we\u0027re testing?  so this might solve the problem, but i\u0027d really like us to know the root cause...","accounts_in_message":[],"_revision_number":3},{"id":"605ffa831ec1b0613d43a0ff55f91e481a184d7e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-18 00:18:54.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-diskimage-builder-pep8 http://logs.openstack.org/41/268241/3/check/gate-diskimage-builder-pep8/8f31c2e/ : SUCCESS in 1m 04s\n- gate-diskimage-builder-docs http://docs-draft.openstack.org/41/268241/3/check/gate-diskimage-builder-docs/2211f6d//doc/build/html/ : SUCCESS in 57s\n- gate-diskimage-builder-python27 http://logs.openstack.org/41/268241/3/check/gate-diskimage-builder-python27/5ca1edf/ : SUCCESS in 34s\n- gate-diskimage-builder-python34 http://logs.openstack.org/41/268241/3/check/gate-diskimage-builder-python34/298c375/ : SUCCESS in 47s\n- gate-dib-dsvm-functests-devstack-trusty http://logs.openstack.org/41/268241/3/check/gate-dib-dsvm-functests-devstack-trusty/7f83937/ : SUCCESS in 33m 07s\n- gate-dib-dsvm-functests-devstack-centos7 http://logs.openstack.org/41/268241/3/check/gate-dib-dsvm-functests-devstack-centos7/43a09a4/ : SUCCESS in 31m 04s\n- gate-tempest-dsvm-ironic-pxe_ssh-dib http://logs.openstack.org/41/268241/3/check/gate-tempest-dsvm-ironic-pxe_ssh-dib/831fecf/ : SUCCESS in 36m 30s","accounts_in_message":[],"_revision_number":3},{"id":"8582974c7363ed8388bfc8a04f1caefef75fa40d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-18 00:37:01.000000000","message":"Patch Set 3:\n\nBuild failed (check-tripleo pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- gate-tripleo-ci-f22-nonha http://logs.openstack.org/41/268241/3/check-tripleo/gate-tripleo-ci-f22-nonha/eb84feb/ : FAILURE in 55m 15s","accounts_in_message":[],"_revision_number":3},{"id":"6a9caab09b82dac2f2f251adb8fce45b78ed8da6","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-01-18 02:39:59.000000000","message":"Patch Set 3:\n\nafter digging a bit more, I believe the correct fix is in project-config -- I41e81d6bac98875eecde2376e0865784626e11a8","accounts_in_message":[],"_revision_number":3},{"id":"60656b858810868562bffffc79c8d207e5b88f23","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-01-18 20:21:25.000000000","message":"Uploaded patch set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"60b0b8444b2ea7c66c4a03e29a4b1ecd1b5f60af","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-18 21:39:04.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-diskimage-builder-pep8 http://logs.openstack.org/41/268241/4/check/gate-diskimage-builder-pep8/439fe0f/ : SUCCESS in 37s\n- gate-diskimage-builder-docs http://docs-draft.openstack.org/41/268241/4/check/gate-diskimage-builder-docs/6d3ae31//doc/build/html/ : SUCCESS in 51s\n- gate-diskimage-builder-python27 http://logs.openstack.org/41/268241/4/check/gate-diskimage-builder-python27/47557a1/ : SUCCESS in 29s\n- gate-diskimage-builder-python34 http://logs.openstack.org/41/268241/4/check/gate-diskimage-builder-python34/1f79ad5/ : SUCCESS in 41s\n- gate-dib-dsvm-functests-devstack-trusty http://logs.openstack.org/41/268241/4/check/gate-dib-dsvm-functests-devstack-trusty/fa17158/ : SUCCESS in 40m 18s\n- gate-dib-dsvm-functests-devstack-centos7 http://logs.openstack.org/41/268241/4/check/gate-dib-dsvm-functests-devstack-centos7/4dec265/ : SUCCESS in 34m 40s\n- gate-tempest-dsvm-ironic-pxe_ssh-dib http://logs.openstack.org/41/268241/4/check/gate-tempest-dsvm-ironic-pxe_ssh-dib/8fc9220/ : SUCCESS in 44m 00s","accounts_in_message":[],"_revision_number":4},{"id":"e9f3a9e743df5a3693133a1683002cbb483ecb80","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-18 21:50:07.000000000","message":"Patch Set 4:\n\nBuild succeeded (check-tripleo pipeline).\n\n- gate-tripleo-ci-f22-nonha http://logs.openstack.org/41/268241/4/check-tripleo/gate-tripleo-ci-f22-nonha/33483bc/ : SUCCESS in 1h 28m 32s","accounts_in_message":[],"_revision_number":4},{"id":"ce8cb3ee3ca72d1deea29ece6258adcc0434f638","author":{"_account_id":9369,"name":"Steve Kowalik","email":"steven@wedontsleep.org","username":"stevenk"},"date":"2016-01-19 05:39:03.000000000","message":"Patch Set 4:\n\nThis strikes me as slightly inelegant. A much shorter (and sadly untested) change is:\n\nlocal\u003d$(git for-each-ref --format\u003d\u0027%(refname:short)\u0027 refs/heads)\nremotes\u003d$(git for-each-ref --format\u003d\u0027%(refname:short)\u0027 refs/remotes | cut -d/ -f2)\nfor branch in $(echo $local | grep -vf \u003c($echo $remotes)); do\n    git branch -D $branch\ndone","accounts_in_message":[],"_revision_number":4},{"id":"badd3ff7a8bb526288681478a8f08d71d23845ff","author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"date":"2016-01-25 17:40:52.000000000","message":"Patch Set 4: Code-Review+1\n\nI don\u0027t see anything wrong with this patch. But based on Ian\u0027s research, I\u0027d like to see Clark re-assert that it is still needed. I\u0027m not sure it is at this point.","accounts_in_message":[],"_revision_number":4},{"id":"b63ea228c6b99666225d759bb1d3360de2e3da82","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-01-29 03:59:41.000000000","message":"Patch Set 4:\n\nI think i\u0027d prefer we didn\u0027t do this.  extensive digging ended up showing us we were not fetching the changes in the way we thought we were fetching them in the infra elements.  This file needs a lot less logic ... not more\n\n@clarkb ok to abandon?","accounts_in_message":[],"_revision_number":4},{"id":"590d6c5e679f4253880f834329eb9caf1aff9537","author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"date":"2016-02-12 02:34:16.000000000","message":"Abandoned\n\nclarkb says this can be abandoned","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"d943bebf8d2553a62bd8206232a2d0d0144edabf","revisions":{"16149111a392233d01647117e669938b58a4aaac":{"kind":"REWORK","_number":1,"created":"2016-01-15 17:28:57.000000000","uploader":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"ref":"refs/changes/41/268241/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/diskimage-builder","ref":"refs/changes/41/268241/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/1"}}},"commit":{"parents":[{"commit":"cd9fdf05e90d65922e0a48bcfcb471834037149f","subject":"Fill out bootloader pkg-map","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/cd9fdf05e90d65922e0a48bcfcb471834037149f"}]}],"author":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 17:26:52.000000000","tz":-480},"committer":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 17:26:52.000000000","tz":-480},"subject":"Actually prune stale branches in src-repos","message":"Actually prune stale branches in src-repos\n\nThe source-repositories element attempted to prune stale branches from\nthe local cache repos but failed because it was fetching specific refs\ninstead of updating remote tracking branches. Address this by performing\nan explicit `git remote update --prune` prior to other git operations.\n\nThis has the side effect of fetching refs early short circuiting much of\nthe fetch the ref we need code in this element. That code is left in\nplace as there may be corner cases where a git remote update will not\nfetch the ref that we want requiring us to be more explicit.\n\nChange-Id: Ie143a1e94f0be6bf464e20bd9d978aed9868b629\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/16149111a392233d01647117e669938b58a4aaac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/16149111a392233d01647117e669938b58a4aaac"}]},"branch":"refs/heads/master"},"95a23ca248dd71062db21ef735ffac99ccae7f41":{"kind":"REWORK","_number":2,"created":"2016-01-15 17:48:05.000000000","uploader":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"ref":"refs/changes/41/268241/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/diskimage-builder","ref":"refs/changes/41/268241/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/2"}}},"commit":{"parents":[{"commit":"cd9fdf05e90d65922e0a48bcfcb471834037149f","subject":"Fill out bootloader pkg-map","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/cd9fdf05e90d65922e0a48bcfcb471834037149f"}]}],"author":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 17:26:52.000000000","tz":-480},"committer":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 17:47:58.000000000","tz":-480},"subject":"Actually prune stale branches in src-repos","message":"Actually prune stale branches in src-repos\n\nThe source-repositories element attempted to prune stale branches from\nthe local cache repos but failed because it was fetching specific refs\ninstead of updating remote tracking branches. Address this by performing\nan explicit `git remote update --prune` prior to other git operations.\n\nThis has the side effect of fetching refs early short circuiting much of\nthe fetch the ref we need code in this element. That code is left in\nplace as there may be corner cases where a git remote update will not\nfetch the ref that we want requiring us to be more explicit.\n\nChange-Id: Ie143a1e94f0be6bf464e20bd9d978aed9868b629\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/95a23ca248dd71062db21ef735ffac99ccae7f41"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/95a23ca248dd71062db21ef735ffac99ccae7f41"}]},"branch":"refs/heads/master"},"ec439d4282cab91149656c00cbbd70efd00edc15":{"kind":"REWORK","_number":3,"created":"2016-01-15 21:31:58.000000000","uploader":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"ref":"refs/changes/41/268241/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/diskimage-builder","ref":"refs/changes/41/268241/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/3"}}},"commit":{"parents":[{"commit":"cd9fdf05e90d65922e0a48bcfcb471834037149f","subject":"Fill out bootloader pkg-map","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/cd9fdf05e90d65922e0a48bcfcb471834037149f"}]}],"author":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 17:26:52.000000000","tz":-480},"committer":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 21:30:17.000000000","tz":-480},"subject":"Actually prune stale branches in src-repos","message":"Actually prune stale branches in src-repos\n\nThe source-repositories element attempted to prune stale branches from\nthe local cache repos but failed because it was fetching specific refs\ninstead of updating remote tracking branches. Address this by performing\nan explicit `git remote update --prune` prior to other git operations,\nthen comparing the list of local branches to the list of remote\nbranches. Any branch that is present locally but not on any remote will\nbe deleted.\n\nThis has the side effect of fetching refs early short circuiting much of\nthe fetch the ref we need code in this element. That code is left in\nplace as there may be corner cases where a git remote update will not\nfetch the ref that we want requiring us to be more explicit.\n\nChange-Id: Ie143a1e94f0be6bf464e20bd9d978aed9868b629\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/ec439d4282cab91149656c00cbbd70efd00edc15"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/ec439d4282cab91149656c00cbbd70efd00edc15"}]},"branch":"refs/heads/master"},"d943bebf8d2553a62bd8206232a2d0d0144edabf":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2016-01-18 20:21:25.000000000","uploader":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"ref":"refs/changes/41/268241/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/diskimage-builder","ref":"refs/changes/41/268241/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/diskimage-builder refs/changes/41/268241/4"}}},"commit":{"parents":[{"commit":"cd9fdf05e90d65922e0a48bcfcb471834037149f","subject":"Fill out bootloader pkg-map","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/cd9fdf05e90d65922e0a48bcfcb471834037149f"}]}],"author":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-15 17:26:52.000000000","tz":-480},"committer":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-01-18 20:20:52.000000000","tz":-480},"subject":"Actually prune stale branches in src-repos","message":"Actually prune stale branches in src-repos\n\nThe source-repositories element attempted to prune stale branches from\nthe local cache repos but failed because it was fetching specific refs\ninstead of updating remote tracking branches. Address this by performing\nan explicit `git remote update --prune` prior to other git operations,\nthen comparing the list of local branches to the list of remote\nbranches. Any branch that is present locally but not on any remote will\nbe deleted.\n\nThis has the side effect of fetching refs early, short circuiting much\nof the \"fetch the ref we need\" code in this element. That code is left\nin place as there may be corner cases where a git remote update will not\nfetch the ref that we want requiring us to be more explicit.\n\nChange-Id: Ie143a1e94f0be6bf464e20bd9d978aed9868b629\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/d943bebf8d2553a62bd8206232a2d0d0144edabf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/diskimage-builder/commit/d943bebf8d2553a62bd8206232a2d0d0144edabf"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
