)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"d423271a6154346b2b9d951035b9580b9a3baf99","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement scheduler hints"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Implement scheduler hints for grizzly release"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"bp scheduler-hints"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAPH%2F%2F8uk%3D","line":9,"updated":"2013-05-17 23:44:08.000000000","message":"havana","commit_id":"cf38f928e037c5ddde922d3440cfca0ba3379f3a"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"fa9b88c968739967554291bb036fe1d17fb34f96","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Implement scheduler hints for grizzly release"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"bp scheduler-hints"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7e858e98110a30c5f8d5ff05d03ce2d61f04f92"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAOn%2F%2FVoQ%3D","line":11,"updated":"2013-05-14 08:15:28.000000000","message":"Make sure it follows https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"cf38f928e037c5ddde922d3440cfca0ba3379f3a"}],"cinderclient/v2/shell.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"abaaa5b1ca9021bbf1dd3732d806be73a1a4eb54","unresolved":false,"context_lines":[{"line_number":245,"context_line":""},{"line_number":246,"context_line":"    #NOTE(N.S.): take this piece from novaclient"},{"line_number":247,"context_line":"    hints \u003d {}"},{"line_number":248,"context_line":"    if args.scheduler_hints:"},{"line_number":249,"context_line":"        for hint in args.scheduler_hints:"},{"line_number":250,"context_line":"            key, _sep, value \u003d hint.partition(\u0027\u003d\u0027)"},{"line_number":251,"context_line":"            # NOTE(vish): multiple copies of the same hint will"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2Fgvo%3D","line":248,"updated":"2013-05-31 14:38:29.000000000","message":"The above checks (236, 239, 243) all do \"if x is not None\", would prefer that for consistency.","commit_id":"a2c58b9e6e23f603cd8b4ad9f00c14ada82b110e"},{"author":{"_account_id":1531,"name":"Ilya Kharin","email":"akscram@gmail.com","username":"akscram"},"change_message_id":"34bc033f63913c41e4168214809932531e9231aa","unresolved":false,"context_lines":[{"line_number":245,"context_line":""},{"line_number":246,"context_line":"    #NOTE(N.S.): take this piece from novaclient"},{"line_number":247,"context_line":"    hints \u003d {}"},{"line_number":248,"context_line":"    if args.scheduler_hints:"},{"line_number":249,"context_line":"        for hint in args.scheduler_hints:"},{"line_number":250,"context_line":"            key, _sep, value \u003d hint.partition(\u0027\u003d\u0027)"},{"line_number":251,"context_line":"            # NOTE(vish): multiple copies of the same hint will"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FgnU%3D","line":248,"in_reply_to":"AAAAPH%2F%2Fgvo%3D","updated":"2013-05-31 15:24:56.000000000","message":"These checks have to correspond to a default value of an argument. The \"if x is not None\" check is correct for arguments with \"None\" defaults, e.g. display_name, display_description and metadata, but is not suitable in the case with the scheduler_hints argument.","commit_id":"a2c58b9e6e23f603cd8b4ad9f00c14ada82b110e"}]}