)]}'
{"id":"openstack%2Fnova~22777","triplet_id":"openstack%2Fnova~master~I56bba9fa6596601104498e262c2e657f0eae2fa0","project":"openstack/nova","branch":"master","hashtags":[],"change_id":"I56bba9fa6596601104498e262c2e657f0eae2fa0","subject":"Clean unused kernels and ramdisks from image cache","status":"MERGED","created":"2013-02-23 16:04:12.000000000","updated":"2013-02-25 16:09:33.000000000","submitted":"2013-02-25 16:09:31.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"8ec882e6bd12cdbc4da6a50efc560a5ad5af2e6f","_number":22777,"virtual_id_number":22777,"owner":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"recommended":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},{"value":0,"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},{"value":1,"date":"2013-02-23 16:39:01.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":2,"date":"2013-02-25 16:09:31.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":{"approved":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"all":[{"value":2,"date":"2013-02-25 15:33:30.000000000","_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"date":"2013-02-23 18:27:59.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":2,"date":"2013-02-24 18:35:10.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":1,"date":"2013-02-25 09:06:40.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},{"value":0,"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},{"value":1,"date":"2013-02-24 16:30:49.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"date":"2013-02-24 15:47:33.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"all":[{"value":1,"date":"2013-02-25 15:33:30.000000000","_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"date":"2013-02-23 18:27:59.000000000","_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"date":"2013-02-24 18:35:10.000000000","_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},{"value":0,"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},{"value":0,"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},{"value":0,"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},{"value":0,"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-02-23 16:04:41.000000000","updated_by":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"reviewer":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"state":"REVIEWER"},{"updated":"2013-02-23 16:39:01.000000000","updated_by":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"reviewer":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2013-02-24 15:47:33.000000000","updated_by":{"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},"reviewer":{"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},"state":"REVIEWER"},{"updated":"2013-02-24 16:30:49.000000000","updated_by":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"reviewer":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"state":"REVIEWER"},{"updated":"2013-02-24 18:35:10.000000000","updated_by":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"reviewer":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"state":"REVIEWER"},{"updated":"2013-02-25 09:06:40.000000000","updated_by":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"reviewer":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"state":"REVIEWER"},{"updated":"2013-02-25 15:33:30.000000000","updated_by":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"reviewer":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"state":"REVIEWER"},{"updated":"2013-02-25 16:09:31.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"7badae63cc9467e4499103d06cf7afbc1a0f895b","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-23 16:06:03.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"f5f444badbc716b4d2798a141eb7efc217006f9e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-23 16:12:16.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/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/22777/2/check/gate-nova-pep8/21446 : FAILURE\n- gate-nova-docs : SKIPPED\n- gate-nova-python26 : SKIPPED\n- gate-nova-python27 : SKIPPED\n- gate-tempest-devstack-vm-full : SKIPPED\n- gate-tempest-devstack-vm-postgres-full : SKIPPED\n- gate-tempest-devstack-vm-quantum : SKIPPED\n- gate-nova-pylint : SKIPPED (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"ff3be8ae5f4e31a16911bfe3bf03f8b9a5ee469f","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-23 16:19:10.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6373b1d0a05d8747611a22e61d067ac70a2c98f8","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-23 16:39:01.000000000","message":"Patch Set 3: Works for me\n\nSmokeStack Results (patch set 3):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/74296\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/74297\n","accounts_in_message":[],"_revision_number":3},{"id":"7fb1021114325f658bc9d9426a1a27aa9e7e0db9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-23 17:02:30.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/22777/3/check/gate-nova-pep8/21447 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-nova-docs/15993 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-nova-python26/17423 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-nova-python27/17491 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-tempest-devstack-vm-full/5208 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-tempest-devstack-vm-postgres-full/4927 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-tempest-devstack-vm-quantum/9815 : SUCCESS\n- http://logs.openstack.org/22777/3/check/gate-nova-pylint/8769 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"8a2ad516682fa7c007725e7729543dfa3983d11e","author":{"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},"date":"2013-02-23 18:20:33.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\nGiven that this introduces a new config option and we\u0027re in feature freeze, I suspect we\u0027ll need to wait for master to open for Havana commits before we merge.","accounts_in_message":[],"_revision_number":3},{"id":"76017a33f307a1b3bcfe92f92cc91d24e67c6e68","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-02-23 18:27:59.000000000","message":"Patch Set 3:\n\nIt\u0027s not an option freeze, it\u0027s a feature freeze.\n\nThis is a bugfix for an existing feature. The option is only there because it\u0027s unsafe to enable the bugfix by default until Havana.","accounts_in_message":[],"_revision_number":3},{"id":"78741a1766ed9063fa6e448c697fea9a98989937","author":{"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},"date":"2013-02-24 15:47:33.000000000","message":"Patch Set 3: No score\n\n","accounts_in_message":[],"_revision_number":3},{"id":"6b436e0743fe425d3f911bd882271a3c78af979c","author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"date":"2013-02-24 16:30:49.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"6e4006237abae36eb00af69c8146ecfac3732804","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-02-24 18:35:10.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nnice work, looks good.  attention to the upgrade complication much appreciated","accounts_in_message":[],"_revision_number":3},{"id":"8d320134caec5e448d03b4211b9b9fb9fb19dc2d","author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"date":"2013-02-25 09:06:40.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"28b2d97a5857b76a9e474c50d6d5b93c89e4b7c4","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-02-25 15:33:30.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\nThis is cool. Thanks for doing this.","accounts_in_message":[],"_revision_number":3},{"id":"b0cd2e4fa9fac4359a9c1ef938220d36083da4da","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-25 15:33:32.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"60f9697beedde5595df6318532590f924e91217d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-25 16:09:31.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/22777/3/gate/gate-nova-docs/16007 : SUCCESS\n- http://logs.openstack.org/22777/3/gate/gate-nova-pep8/21466 : SUCCESS\n- http://logs.openstack.org/22777/3/gate/gate-nova-python26/17437 : SUCCESS\n- http://logs.openstack.org/22777/3/gate/gate-nova-python27/17505 : SUCCESS\n- http://logs.openstack.org/22777/3/gate/gate-tempest-devstack-vm-full/5296 : SUCCESS\n- http://logs.openstack.org/22777/3/gate/gate-tempest-devstack-vm-postgres-full/5015 : SUCCESS\n- http://logs.openstack.org/22777/3/gate/gate-tempest-devstack-vm-quantum/9940 : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"8ec882e6bd12cdbc4da6a50efc560a5ad5af2e6f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-25 16:09:33.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"38997fc916904e47dcb847e1737eb26cbd38f6e1","revisions":{"e836372beeb8f78252ff5ff2f56fc6037095278c":{"kind":"REWORK","_number":1,"created":"2013-02-23 16:04:12.000000000","uploader":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"ref":"refs/changes/77/22777/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/77/22777/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/77/22777/1"}}},"commit":{"parents":[{"commit":"a42845e455c74f41852babbbd09a3514021ea71d","subject":"Merge \"Fix launching libvirt instances with swap\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a42845e455c74f41852babbbd09a3514021ea71d"}]}],"author":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-02-23 15:24:20.000000000","tz":0},"committer":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-02-23 16:03:35.000000000","tz":0},"subject":"Clean unused kernels and ramdisks from image cache","message":"Clean unused kernels and ramdisks from image cache\n\nFixes bug #1132138\n\nOnly unused disk images are currently cleaned up by the image cache\nmanager but it seems logical to clean up unused kernels and ramdisks\ntoo.\n\nAchieve that by writing kernels and ramdisks to disk using the sha1\nsum of their ID as the filename. This is the same scheme as used for\ndisk image filenames and causes the image cache manager to consider\nthem for cleanup. We also make the cache manager take note of in use\nkernels and ramdisks when iterating over the list of instances.\n\nA nasty upgrade concern is that if we immediately switch to writing\nkernels to disk using this scheme then, where shared storage is used,\nwe can have older image cache managers on remote compute nodes cleaning\nup kernels because they appear unused. To mitigate that, turn off this\nbehaviour by default and allow it to be enabled using a new config\noption. This option will be removed in future and the behaviour enabled\nby default.\n\nDocImpact: new remove_unused_kernels option\n\nChange-Id: I56bba9fa6596601104498e262c2e657f0eae2fa0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e836372beeb8f78252ff5ff2f56fc6037095278c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e836372beeb8f78252ff5ff2f56fc6037095278c"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"a42845e455c74f41852babbbd09a3514021ea71d","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"a247c9a5ad3ec172cceb1f2ffad62ceca75c3bb2":{"kind":"REWORK","_number":2,"created":"2013-02-23 16:06:03.000000000","uploader":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"ref":"refs/changes/77/22777/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/77/22777/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/77/22777/2"}}},"commit":{"parents":[{"commit":"a42845e455c74f41852babbbd09a3514021ea71d","subject":"Merge \"Fix launching libvirt instances with swap\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a42845e455c74f41852babbbd09a3514021ea71d"}]}],"author":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-02-23 15:24:20.000000000","tz":0},"committer":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-02-23 16:05:46.000000000","tz":0},"subject":"Clean unused kernels and ramdisks from image cache","message":"Clean unused kernels and ramdisks from image cache\n\nFixes bug #1132138\n\nOnly unused disk images are currently cleaned up by the image cache\nmanager but it seems logical to clean up unused kernels and ramdisks\ntoo.\n\nAchieve that by writing kernels and ramdisks to disk using the sha1\nsum of their ID as the filename. This is the same scheme as used for\ndisk image filenames and causes the image cache manager to consider\nthem for cleanup. We also make the cache manager take note of in use\nkernels and ramdisks when iterating over the list of instances.\n\nA nasty upgrade concern is that if we immediately switch to writing\nkernels to disk using this scheme then, where shared storage is used,\nwe can have older image cache managers on remote compute nodes cleaning\nup kernels because they appear unused. To mitigate that, turn off this\nbehaviour by default and allow it to be enabled using a new config\noption. This option will be removed in future and the behaviour enabled\nby default.\n\nDocImpact: new remove_unused_kernels option\n\nChange-Id: I56bba9fa6596601104498e262c2e657f0eae2fa0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a247c9a5ad3ec172cceb1f2ffad62ceca75c3bb2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a247c9a5ad3ec172cceb1f2ffad62ceca75c3bb2"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"a42845e455c74f41852babbbd09a3514021ea71d","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"38997fc916904e47dcb847e1737eb26cbd38f6e1":{"kind":"REWORK","_number":3,"created":"2013-02-23 16:19:10.000000000","uploader":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"ref":"refs/changes/77/22777/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/77/22777/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/77/22777/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/77/22777/3"}}},"commit":{"parents":[{"commit":"a42845e455c74f41852babbbd09a3514021ea71d","subject":"Merge \"Fix launching libvirt instances with swap\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a42845e455c74f41852babbbd09a3514021ea71d"}]}],"author":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-02-23 15:24:20.000000000","tz":0},"committer":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-02-23 16:18:47.000000000","tz":0},"subject":"Clean unused kernels and ramdisks from image cache","message":"Clean unused kernels and ramdisks from image cache\n\nFixes bug #1132138\n\nOnly unused disk images are currently cleaned up by the image cache\nmanager but it seems logical to clean up unused kernels and ramdisks\ntoo.\n\nAchieve that by writing kernels and ramdisks to disk using the sha1\nsum of their ID as the filename. This is the same scheme as used for\ndisk image filenames and causes the image cache manager to consider\nthem for cleanup. We also make the cache manager take note of in use\nkernels and ramdisks when iterating over the list of instances.\n\nA nasty upgrade concern is that if we immediately switch to writing\nkernels to disk using this scheme then, where shared storage is used,\nwe can have older image cache managers on remote compute nodes cleaning\nup kernels because they appear unused. To mitigate that, turn off this\nbehaviour by default and allow it to be enabled using a new config\noption. This option will be removed in future and the behaviour enabled\nby default.\n\nDocImpact: new remove_unused_kernels option\n\nChange-Id: I56bba9fa6596601104498e262c2e657f0eae2fa0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/38997fc916904e47dcb847e1737eb26cbd38f6e1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/38997fc916904e47dcb847e1737eb26cbd38f6e1"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"a42845e455c74f41852babbbd09a3514021ea71d","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}