)]}'
{"nova/virt/imagecache.py":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"d6544345d459f0502dbd115e29eb173cda74d7ed","unresolved":false,"context_lines":[{"line_number":26,"context_line":"CONF.import_opt(\u0027image_cache_subdirectory_name\u0027, \u0027nova.virt.config\u0027)"},{"line_number":27,"context_line":"CONF.import_opt(\u0027remove_unused_base_images\u0027, \u0027nova.virt.config\u0027)"},{"line_number":28,"context_line":"CONF.import_opt(\u0027remove_unused_original_minimum_age_seconds\u0027,"},{"line_number":29,"context_line":"                \u0027nova.virt.config\u0027)"},{"line_number":30,"context_line":"CONF.import_opt(\u0027host\u0027, \u0027nova.netconf\u0027)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_61997bc3","line":29,"updated":"2015-07-23 16:52:20.000000000","message":"I\u0027m not really seeing a benefit in moving the config options out of this file, into nova/virt/config.py  - I think it is good that we have the image cache config options defined in the imagecache.py file - the nova/virt/config.py file is just going to end up as a massive dumping ground of unrelated stuff, that will cause endless merge conflicts as multiple competing patches try to add to the same file - same pain we have with the horrible exceptions.py class.","commit_id":"87167bec7e0c014e9c2c00b638e704efd5df019c"}]}