)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"b681d5d190c1bc2b52a4f06433ca04490fa8d5ba","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nolwenn Cauchois \u003cnolwenn.cauchois@cloudwatt.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-12 17:28:35 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug nova does not clean-up the second ephemeral disk when this one isn\u0027t any more used by instances"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, the backing file for the second ephemeral disk is never deleted on the hypervisor."},{"line_number":10,"context_line":"After some instance create/delete period, the hypervisor local storage capacities are reduced."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_4980c246","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":103},"updated":"2016-07-20 03:52:35.000000000","message":"Here are a few conventions the OpenStack community tries to follow for commit messages:\n\nCommit messages should start with a short 50 character or less summary on a single line (without a period). Subsequent lines should be wrapped at 72 characters.\n\n- http://docs.openstack.org/infra/manual/developers.html#committing-a-change\n- https://wiki.openstack.org/wiki/GitCommitMessages\n\nHappy coding, Nolwenn!","commit_id":"8b0372c2b845166083073a80e0e91a92b9e5e1d9"},{"author":{"_account_id":6802,"name":"Joel Coffman","email":"jmc7tp@gmail.com","username":"joel-coffman"},"change_message_id":"d3226b9311432d930c22cd0a7b828ca6ed650f01","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nolwenn Cauchois \u003cnolwenn.cauchois@cloudwatt.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-20 10:44:35 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug nova doesn\u0027t clean-up the second ephemeral disk"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, the backing file for the second ephemeral disk"},{"line_number":10,"context_line":"is never deleted on the hypervisor, even when this one isn\u0027t"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dada55a8_0ab7b3f5","line":7,"updated":"2016-07-20 17:14:19.000000000","message":"I think \"Clean up second ephemeral disk\" is a clearer description of this change.","commit_id":"227ff8ad1f7fda4f3ac1be63800f7281d51c342f"}],"nova/tests/unit/virt/libvirt/test_imagecache.py":[{"author":{"_account_id":6802,"name":"Joel Coffman","email":"jmc7tp@gmail.com","username":"joel-coffman"},"change_message_id":"d3226b9311432d930c22cd0a7b828ca6ed650f01","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import contextlib"},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":"import time"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_ca8e3b2d","side":"PARENT","line":16,"updated":"2016-07-20 17:14:19.000000000","message":"Unrelated whitespace change.","commit_id":"d3c1ce263552acc1902173a6e8e5a4ad5114f19a"}],"nova/virt/libvirt/imagecache.py":[{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"74ddb8a339d23eddff9a194c8e1529fbb6323d71","unresolved":false,"context_lines":[{"line_number":169,"context_line":"                disk_path \u003d os.path.join(CONF.instances_path, ent, \u0027disk\u0027)"},{"line_number":170,"context_line":"                eph_disk_path \u003d os.path.join(CONF.instances_path,"},{"line_number":171,"context_line":"                    ent, \u0027disk.eph0\u0027)"},{"line_number":172,"context_line":"                if os.path.exists(disk_path):"},{"line_number":173,"context_line":"                    LOG.debug(\u0027%s has a disk file\u0027, ent)"},{"line_number":174,"context_line":"                    try:"},{"line_number":175,"context_line":"                        backing_file \u003d libvirt_utils.get_disk_backing_file("}],"source_content_type":"text/x-python","patch_set":8,"id":"7f287b81_cc5a0e29","line":172,"updated":"2017-09-04 12:16:49.000000000","message":"You are adding ephemeral image only if disk_path exists, right?\nWhy don\u0027t do it in independent way, I mean to check disk_path and eph_disk_path existence separately? As I understand it can help to delete previously skipped ephemeral disks, no?","commit_id":"bdf3a03ad8b12e3a7e801ee20f9c33600663fca4"}]}