)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":16839,"name":"Cale Rath","email":"ctrath@us.ibm.com","username":"ctrath"},"change_message_id":"6339536f27d0dce430668340572cbe3d909a10e1","unresolved":false,"context_lines":[{"line_number":144,"context_line":"                    \u0027allocation on failures\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Cinder since the liberty release can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"},{"line_number":148,"context_line":"                    \u0027image_volume_cache_enabled configuration option.\u0027)"},{"line_number":149,"context_line":"    ]"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_8523c557","line":147,"updated":"2015-09-29 17:28:12.000000000","message":"2 minor issues.  \"Cinder, since the Liberty release, ...\" notice the commas and capitalization of Liberty.","commit_id":"f728d47939205562722c0796b18e8ece5e04b1d7"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"d3266e6fd9819e8eb31939d14b367a4022b0e053","unresolved":false,"context_lines":[{"line_number":144,"context_line":"                    \u0027allocation on failures\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Cinder since the liberty release can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"},{"line_number":148,"context_line":"                    \u0027image_volume_cache_enabled configuration option.\u0027)"},{"line_number":149,"context_line":"    ]"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_25f7595a","line":147,"in_reply_to":"9a1a9d01_8523c557","updated":"2015-09-29 17:35:52.000000000","message":"Same for the commit message.","commit_id":"f728d47939205562722c0796b18e8ece5e04b1d7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"92ef348145396b1bd9fde6d3e32a657603470313","unresolved":false,"context_lines":[{"line_number":142,"context_line":"               default\u003d60,"},{"line_number":143,"context_line":"               help\u003d\u0027Number of times to retry block device \u0027"},{"line_number":144,"context_line":"                    \u0027allocation on failures\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Cinder, since the Liberty release, can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"},{"line_number":148,"context_line":"                    \u0027image_volume_cache_enabled configuration option.\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a29b11f_5eff62aa","line":145,"updated":"2015-10-23 17:28:32.000000000","message":"if we are going to change this, let\u0027s make it less stilted. Add a period on the first sentence.\n\nStart the second one with \"Starting with Liberty, Cinder can use ... \"","commit_id":"3b5f852fa0b35586766de7ca0ace59040f6cecf0"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"66723889fa3f0713886a569a9cfa8028e49385dd","unresolved":false,"context_lines":[{"line_number":141,"context_line":"    cfg.IntOpt(\u0027block_device_allocate_retries\u0027,"},{"line_number":142,"context_line":"               default\u003d60,"},{"line_number":143,"context_line":"               help\u003d\u0027Number of times to retry block device \u0027"},{"line_number":144,"context_line":"                    \u0027allocation on failures.\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Starting with Liberty, Cinder can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a26ad4f_5f79fac6","line":144,"updated":"2015-11-02 19:42:41.000000000","message":"why do we need the \\n here. Do we need that?","commit_id":"5508e11cf873384a28dc7416168d34e85f2c06cf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"290f57a0a55d1ccf13d3df9eac0f2fc70591c679","unresolved":false,"context_lines":[{"line_number":141,"context_line":"    cfg.IntOpt(\u0027block_device_allocate_retries\u0027,"},{"line_number":142,"context_line":"               default\u003d60,"},{"line_number":143,"context_line":"               help\u003d\u0027Number of times to retry block device \u0027"},{"line_number":144,"context_line":"                    \u0027allocation on failures.\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Starting with Liberty, Cinder can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a26ad4f_15e363a5","line":144,"in_reply_to":"1a26ad4f_1a9dd065","updated":"2015-11-02 20:52:04.000000000","message":"It does render OK:\n\nhttp://paste.openstack.org/show/477815/","commit_id":"5508e11cf873384a28dc7416168d34e85f2c06cf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"193dae8972890684b9f4c565a87da88d9a302d02","unresolved":false,"context_lines":[{"line_number":141,"context_line":"    cfg.IntOpt(\u0027block_device_allocate_retries\u0027,"},{"line_number":142,"context_line":"               default\u003d60,"},{"line_number":143,"context_line":"               help\u003d\u0027Number of times to retry block device \u0027"},{"line_number":144,"context_line":"                    \u0027allocation on failures.\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Starting with Liberty, Cinder can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a26ad4f_1a9dd065","line":144,"in_reply_to":"1a26ad4f_5f79fac6","updated":"2015-11-02 20:03:40.000000000","message":"I\u0027m not sure if this will be rendered properly in the config sample, it might literally show up as \\n, I\u0027m waiting for the docs job to complete to tell.","commit_id":"5508e11cf873384a28dc7416168d34e85f2c06cf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"193dae8972890684b9f4c565a87da88d9a302d02","unresolved":false,"context_lines":[{"line_number":142,"context_line":"               default\u003d60,"},{"line_number":143,"context_line":"               help\u003d\u0027Number of times to retry block device \u0027"},{"line_number":144,"context_line":"                    \u0027allocation on failures.\\n\u0027"},{"line_number":145,"context_line":"                    \u0027Starting with Liberty, Cinder can use image volume \u0027"},{"line_number":146,"context_line":"                    \u0027cache. This may help with block device allocation \u0027"},{"line_number":147,"context_line":"                    \u0027performance. Look at the cinder \u0027"},{"line_number":148,"context_line":"                    \u0027image_volume_cache_enabled configuration option.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a26ad4f_7acde483","line":145,"updated":"2015-11-02 20:03:40.000000000","message":"Given the cinder service has a version for liberty (7.0.0), it might be better to point that out rather than the release code name, or say something like:\n\n\"Starting with Cinder 7.0.0 (Liberty), Cinder has image volume cache support. This may help with block device allocation performance. Look at the Cinder image_volume_cache_enabled configuration option for details.\"","commit_id":"5508e11cf873384a28dc7416168d34e85f2c06cf"}]}