)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"91c9ab77397f4dff6ed15659456bd56034b6aee6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactoring config options for cache_utils opts"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Refactoring neutron cache_utils config opts to be in neutron/conf so that all"},{"line_number":10,"context_line":"the configuration options for cache_utils reside in a centralized location."},{"line_number":11,"context_line":"This simplifies the process of looking up the cache_utils config opts and"},{"line_number":12,"context_line":"provides an easy way to import."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa7ab95a_672b4b34","line":9,"range":{"start_line":9,"start_character":72,"end_line":9,"end_character":77},"updated":"2016-08-31 17:38:46.000000000","message":"nit: exceeds the limit of characters in the description","commit_id":"5f1e5c86483e4f92197f806508f2a0e82208969a"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"91c9ab77397f4dff6ed15659456bd56034b6aee6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Refactoring config options for cache_utils opts"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Refactoring neutron cache_utils config opts to be in neutron/conf so that all"},{"line_number":10,"context_line":"the configuration options for cache_utils reside in a centralized location."},{"line_number":11,"context_line":"This simplifies the process of looking up the cache_utils config opts and"},{"line_number":12,"context_line":"provides an easy way to import."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa7ab95a_c72a572c","line":10,"range":{"start_line":10,"start_character":72,"end_line":10,"end_character":75},"updated":"2016-08-31 17:38:46.000000000","message":"ditto: line 9","commit_id":"5f1e5c86483e4f92197f806508f2a0e82208969a"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"91c9ab77397f4dff6ed15659456bd56034b6aee6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Refactoring neutron cache_utils config opts to be in neutron/conf so that all"},{"line_number":10,"context_line":"the configuration options for cache_utils reside in a centralized location."},{"line_number":11,"context_line":"This simplifies the process of looking up the cache_utils config opts and"},{"line_number":12,"context_line":"provides an easy way to import."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I722a2fa7d72c62e6d878d0137f899d09a9ad64ae"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa7ab95a_67d26b36","line":11,"range":{"start_line":11,"start_character":72,"end_line":11,"end_character":73},"updated":"2016-08-31 17:38:46.000000000","message":"ditto: line 9","commit_id":"5f1e5c86483e4f92197f806508f2a0e82208969a"}],"neutron/conf/cache_utils.py":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"4a8a1c0787238cd9ebbf82bc33150b7cdd57e4f9","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"cache_opts \u003d ["},{"line_number":20,"context_line":"    cfg.StrOpt(\u0027cache_url\u0027, default\u003d\u0027\u0027,"},{"line_number":21,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":22,"context_line":"               help\u003d_(\u0027URL to connect to the cache back end. \u0027"},{"line_number":23,"context_line":"                      \u0027This option is deprecated in the Newton release and \u0027"},{"line_number":24,"context_line":"                      \u0027will be removed. Please add a [cache] group for \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_16d39460","line":21,"range":{"start_line":21,"start_character":15,"end_line":21,"end_character":43},"updated":"2016-10-18 21:49:54.000000000","message":"I may be missing something, but does it make sense to refactor an option that is already deprecated and will soon be removed? Why not just let it die at the end of this cycle?","commit_id":"d7e2d33046d5d169a3b86611b7128b2c3b5e717b"},{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"ff9ea33ecc1a74d0236abf97a84221a12abc2aa7","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"cache_opts \u003d ["},{"line_number":20,"context_line":"    cfg.StrOpt(\u0027cache_url\u0027, default\u003d\u0027\u0027,"},{"line_number":21,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":22,"context_line":"               help\u003d_(\u0027URL to connect to the cache back end. \u0027"},{"line_number":23,"context_line":"                      \u0027This option is deprecated in the Newton release and \u0027"},{"line_number":24,"context_line":"                      \u0027will be removed. Please add a [cache] group for \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_e068e423","line":21,"range":{"start_line":21,"start_character":15,"end_line":21,"end_character":43},"in_reply_to":"fa6399be_16d39460","updated":"2016-10-19 19:39:36.000000000","message":"Since this just got deprecated in the Newton release, this probably has to stick around for at least one more cycle.  I could be wrong though. If that is the case then it should be moved with the rest of them.  If it can be deleted now, it probably can just be removed instead.","commit_id":"d7e2d33046d5d169a3b86611b7128b2c3b5e717b"},{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"39e7f70206aae9a3ba19a3fc15cf4616179ee2e0","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"cache_opts \u003d ["},{"line_number":20,"context_line":"    cfg.StrOpt(\u0027cache_url\u0027, default\u003d\u0027\u0027,"},{"line_number":21,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":22,"context_line":"               help\u003d_(\u0027URL to connect to the cache back end. \u0027"},{"line_number":23,"context_line":"                      \u0027This option is deprecated in the Newton release and \u0027"},{"line_number":24,"context_line":"                      \u0027will be removed. Please add a [cache] group for \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"da6895a0_52e291d6","line":21,"range":{"start_line":21,"start_character":15,"end_line":21,"end_character":43},"in_reply_to":"fa6399be_b3c8fa56","updated":"2016-10-21 19:18:37.000000000","message":"fine by me","commit_id":"d7e2d33046d5d169a3b86611b7128b2c3b5e717b"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"0eac3d404a7e0e007e8b5835b010f251e7dcf897","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"cache_opts \u003d ["},{"line_number":20,"context_line":"    cfg.StrOpt(\u0027cache_url\u0027, default\u003d\u0027\u0027,"},{"line_number":21,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":22,"context_line":"               help\u003d_(\u0027URL to connect to the cache back end. \u0027"},{"line_number":23,"context_line":"                      \u0027This option is deprecated in the Newton release and \u0027"},{"line_number":24,"context_line":"                      \u0027will be removed. Please add a [cache] group for \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_b3c8fa56","line":21,"range":{"start_line":21,"start_character":15,"end_line":21,"end_character":43},"in_reply_to":"fa6399be_e068e423","updated":"2016-10-20 07:42:31.000000000","message":"i think that we can and should do that in a separate patch","commit_id":"d7e2d33046d5d169a3b86611b7128b2c3b5e717b"}]}