)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"0ba891033df4e1b7182cbf574999d7322ed092f4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Turn block_device_mapping.volume_size to int"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It was a string before, while it really should be an integer."},{"line_number":10,"context_line":"This change is backward compatible, as it still accepts string values."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I599ea2bf77a187cc53f864a9e1a42755142211ff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2FRqA%3D","line":9,"updated":"2014-01-15 15:41:38.000000000","message":"Please explain why it should be an integer.","commit_id":"5e60dbdb28be83f803e8a25a367f2b6ec4ac6854"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"5abb8ce4f8e8239ec2fc4d738f9c45ea73ff8b1e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Turn block_device_mapping.volume_size to int"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It was a string before, while it by its meaning should be an integer."},{"line_number":10,"context_line":"This change is backward compatible, as it still accepts string values."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I599ea2bf77a187cc53f864a9e1a42755142211ff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAUH%2F%2FMjo%3D","line":9,"updated":"2014-01-16 15:00:45.000000000","message":"This isn\u0027t a good reason.  It\u0027s like saying the change is obvious.  Is the point of this change to allow for better constraints?","commit_id":"25a814eec53ac59b2181eb11d3e1ee78e0833ed8"},{"author":{"_account_id":7135,"name":"Liang Chen","email":"liangchen.linux@gmail.com","username":"Liang"},"change_message_id":"80ef01f3f1598778ff5700ef0caa30aa01f799a5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Turn block_device_mapping.volume_size to int"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It was a string before, while it by its meaning should be an integer."},{"line_number":10,"context_line":"This change is backward compatible, as it still accepts string values."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I599ea2bf77a187cc53f864a9e1a42755142211ff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAUH%2F%2FICo%3D","line":9,"in_reply_to":"AAAAUH%2F%2FMjo%3D","updated":"2014-01-17 07:05:49.000000000","message":"Feel the same way. Might be better to say Nova actually expects an integer for the size entry.","commit_id":"25a814eec53ac59b2181eb11d3e1ee78e0833ed8"}],"etc/heat/heat.conf.sample":[{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"0ba891033df4e1b7182cbf574999d7322ed092f4","unresolved":false,"context_lines":[{"line_number":1001,"context_line":"# catalog on token validation and will not set the X-Service-"},{"line_number":1002,"context_line":"# Catalog header. (boolean value)"},{"line_number":1003,"context_line":"#include_service_catalog\u003dtrue"},{"line_number":1004,"context_line":""},{"line_number":1005,"context_line":""},{"line_number":1006,"context_line":"[matchmaker_redis]"},{"line_number":1007,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F%2FRrM%3D","side":"PARENT","line":1004,"updated":"2014-01-15 15:41:38.000000000","message":"This shouldn\u0027t have been removed.  It was just added in https://review.openstack.org/#/c/66428/\n\nYou may need to update your local python-keystoneclient.","commit_id":"e3f6a7f184e674614d7c24bc7f24f3e380944d4e"},{"author":{"_account_id":6577,"name":"Sergey Kraynev","email":"sergejyit@gmail.com","username":"skraynev"},"change_message_id":"bf0c53e2c1281774462cc356eec1276338b4f04f","unresolved":false,"context_lines":[{"line_number":1001,"context_line":"# catalog on token validation and will not set the X-Service-"},{"line_number":1002,"context_line":"# Catalog header. (boolean value)"},{"line_number":1003,"context_line":"#include_service_catalog\u003dtrue"},{"line_number":1004,"context_line":""},{"line_number":1005,"context_line":""},{"line_number":1006,"context_line":"[matchmaker_redis]"},{"line_number":1007,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAUH%2F%2FRgM%3D","side":"PARENT","line":1004,"in_reply_to":"AAAAUH%2F%2FRrM%3D","updated":"2014-01-15 16:06:17.000000000","message":"I agree with Jason. I have met current problem. solution -  rebase.","commit_id":"e3f6a7f184e674614d7c24bc7f24f3e380944d4e"}],"heat/tests/test_server.py":[{"author":{"_account_id":7135,"name":"Liang Chen","email":"liangchen.linux@gmail.com","username":"Liang"},"change_message_id":"6aa37d537a8024c9212f577983f56bf7a5c7fd00","unresolved":false,"context_lines":[{"line_number":1149,"context_line":"    def test_validate_insufficient_block_device_mapping_props(self):"},{"line_number":1150,"context_line":"        stack_name \u003d \u0027val_blkdev2\u0027"},{"line_number":1151,"context_line":"        (t, stack) \u003d self._setup_test_stack(stack_name)"},{"line_number":1152,"context_line":""},{"line_number":1153,"context_line":"        bdm \u003d [{\u0027device_name\u0027: \u0027vdb\u0027, \u0027volume_size\u0027: \u00271\u0027,"},{"line_number":1154,"context_line":"                \u0027delete_on_termination\u0027: True}]"},{"line_number":1155,"context_line":"        t[\u0027Resources\u0027][\u0027WebServer\u0027][\u0027Properties\u0027][\u0027block_device_mapping\u0027] \u003d bdm"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2FOP0%3D","line":1152,"updated":"2014-01-16 07:59:34.000000000","message":"Forgot to update this one ;)","commit_id":"a6e8e5d2ebef7d075166c810e5db26a51f9516c3"},{"author":{"_account_id":7135,"name":"Liang Chen","email":"liangchen.linux@gmail.com","username":"Liang"},"change_message_id":"352783fe58158778f41b618a1d2d29cec854c07e","unresolved":false,"context_lines":[{"line_number":1149,"context_line":"    def test_validate_insufficient_block_device_mapping_props(self):"},{"line_number":1150,"context_line":"        stack_name \u003d \u0027val_blkdev2\u0027"},{"line_number":1151,"context_line":"        (t, stack) \u003d self._setup_test_stack(stack_name)"},{"line_number":1152,"context_line":""},{"line_number":1153,"context_line":"        bdm \u003d [{\u0027device_name\u0027: \u0027vdb\u0027, \u0027volume_size\u0027: \u00271\u0027,"},{"line_number":1154,"context_line":"                \u0027delete_on_termination\u0027: True}]"},{"line_number":1155,"context_line":"        t[\u0027Resources\u0027][\u0027WebServer\u0027][\u0027Properties\u0027][\u0027block_device_mapping\u0027] \u003d bdm"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2FIFI%3D","line":1152,"in_reply_to":"AAAAUH%2F%2FOOk%3D","updated":"2014-01-17 07:04:56.000000000","message":"Yeah. Though it didn\u0027t seem really harmful. It\u0027s better to use the right type to avoid possible confusions when someone later makes a change to this test.","commit_id":"a6e8e5d2ebef7d075166c810e5db26a51f9516c3"},{"author":{"_account_id":6577,"name":"Sergey Kraynev","email":"sergejyit@gmail.com","username":"skraynev"},"change_message_id":"88d624de0332fbc5f1db3c2cc87fb7aff4e00d84","unresolved":false,"context_lines":[{"line_number":1149,"context_line":"    def test_validate_insufficient_block_device_mapping_props(self):"},{"line_number":1150,"context_line":"        stack_name \u003d \u0027val_blkdev2\u0027"},{"line_number":1151,"context_line":"        (t, stack) \u003d self._setup_test_stack(stack_name)"},{"line_number":1152,"context_line":""},{"line_number":1153,"context_line":"        bdm \u003d [{\u0027device_name\u0027: \u0027vdb\u0027, \u0027volume_size\u0027: \u00271\u0027,"},{"line_number":1154,"context_line":"                \u0027delete_on_termination\u0027: True}]"},{"line_number":1155,"context_line":"        t[\u0027Resources\u0027][\u0027WebServer\u0027][\u0027Properties\u0027][\u0027block_device_mapping\u0027] \u003d bdm"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2FOOk%3D","line":1152,"in_reply_to":"AAAAUH%2F%2FOP0%3D","updated":"2014-01-16 08:05:07.000000000","message":"Do you mean: using correct type or both variant?","commit_id":"a6e8e5d2ebef7d075166c810e5db26a51f9516c3"}]}