)]}'
{"ironic/drivers/modules/ipmitool.py":[{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"adb6df85b0ee9c63ee04626154ed4c9b1e0c6e4f","unresolved":false,"context_lines":[{"line_number":895,"context_line":""},{"line_number":896,"context_line":"        :param task: a task from TaskManager."},{"line_number":897,"context_line":"        :param kwargs: info for action."},{"line_number":898,"context_line":"        :raises: InvalidParameterValue when an invalid parameter value is"},{"line_number":899,"context_line":"                 specified."},{"line_number":900,"context_line":"        :raises: MissingParameterValue if a required parameter is missing."},{"line_number":901,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_33d8477f","line":898,"updated":"2014-11-28 00:32:18.000000000","message":"Should be MissingParameterValue ??","commit_id":"593ef380294525cc41f644e60b8331980f1e555f"},{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"0286a90dbaaa357f36628d335305ee703c992464","unresolved":false,"context_lines":[{"line_number":895,"context_line":""},{"line_number":896,"context_line":"        :param task: a task from TaskManager."},{"line_number":897,"context_line":"        :param kwargs: info for action."},{"line_number":898,"context_line":"        :raises: InvalidParameterValue when an invalid parameter value is"},{"line_number":899,"context_line":"                 specified."},{"line_number":900,"context_line":"        :raises: MissingParameterValue if a required parameter is missing."},{"line_number":901,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_33f4c7ce","line":898,"in_reply_to":"5a890539_33d8477f","updated":"2014-11-28 00:42:48.000000000","message":"I mean we just docstring the raise the exception which is raised in current method, make sense?","commit_id":"593ef380294525cc41f644e60b8331980f1e555f"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"344f9068d4d9c1b9ac91fd6f3f92da52bc28785b","unresolved":false,"context_lines":[{"line_number":895,"context_line":""},{"line_number":896,"context_line":"        :param task: a task from TaskManager."},{"line_number":897,"context_line":"        :param kwargs: info for action."},{"line_number":898,"context_line":"        :raises: InvalidParameterValue when an invalid parameter value is"},{"line_number":899,"context_line":"                 specified."},{"line_number":900,"context_line":"        :raises: MissingParameterValue if a required parameter is missing."},{"line_number":901,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_ff5e7d78","line":898,"in_reply_to":"5a890539_33f4c7ce","updated":"2014-11-28 08:50:25.000000000","message":"Actually we do document exceptions raised by other methods that this one depends on. In this case _parse_driver_info() raises InvalidParameterValue","commit_id":"593ef380294525cc41f644e60b8331980f1e555f"}],"ironic/drivers/modules/iscsi_deploy.py":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"01cdf96d97df12d6202b19cee38a71983586b5bb","unresolved":false,"context_lines":[{"line_number":406,"context_line":"    except (exception.KeystoneFailure,"},{"line_number":407,"context_line":"            exception.CatalogNotFound,"},{"line_number":408,"context_line":"            exception.KeystoneUnauthorized):"},{"line_number":409,"context_line":"        raise exception.MissingParameterValue(_("},{"line_number":410,"context_line":"            \"Couldn\u0027t get the URL of the Ironic API service from the \""},{"line_number":411,"context_line":"            \"configuration file or keystone catalog.\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_90acbaf0","line":409,"updated":"2014-11-26 16:02:26.000000000","message":"this one I\u0027m not sure about. If the exception was KeystoneUnauthorized then it could be due to invalid password/tenant (instead of missing information).","commit_id":"873e295f8fdfaf2bd1d6112e15da112079da18d4"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"bec6bf6789debe2645b1e9d688d622d022ef404d","unresolved":false,"context_lines":[{"line_number":406,"context_line":"    except (exception.KeystoneFailure,"},{"line_number":407,"context_line":"            exception.CatalogNotFound,"},{"line_number":408,"context_line":"            exception.KeystoneUnauthorized):"},{"line_number":409,"context_line":"        raise exception.MissingParameterValue(_("},{"line_number":410,"context_line":"            \"Couldn\u0027t get the URL of the Ironic API service from the \""},{"line_number":411,"context_line":"            \"configuration file or keystone catalog.\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_a7d3235e","line":409,"in_reply_to":"5a890539_90acbaf0","updated":"2014-11-27 09:59:49.000000000","message":"Done","commit_id":"873e295f8fdfaf2bd1d6112e15da112079da18d4"}]}