)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1851adf5474acf716ebbfa17614e8f5587610de6","unresolved":false,"context_lines":[{"line_number":13,"context_line":"On a side note, it\u0027s a bit odd that this is a required"},{"line_number":14,"context_line":"parameter in the response (Tempest requires it in the"},{"line_number":15,"context_line":"response schema) even though it\u0027s only set if the"},{"line_number":16,"context_line":"action failed."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ic84031119e2ffb6b1fb0b0a193967b2823464651"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_a747ef49","line":16,"range":{"start_line":16,"start_character":13,"end_line":16,"end_character":14},"updated":"2017-02-13 04:50:41.000000000","message":"Yea it\u0027s always none if no fail.","commit_id":"a72968a9e06ce2d9710e2c80064f10c203f993d6"}],"api-ref/source/parameters.yaml":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6f9c77640cbc81cb15e5ee187a7ea0ab449d4fdc","unresolved":false,"context_lines":[{"line_number":2952,"context_line":"  type: integer"},{"line_number":2953,"context_line":"message:"},{"line_number":2954,"context_line":"  description: |"},{"line_number":2955,"context_line":"    The related error message for when an action fails."},{"line_number":2956,"context_line":"  in: body"},{"line_number":2957,"context_line":"  required: true"},{"line_number":2958,"context_line":"  type: string"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fa31d9ce_f56423e6","line":2955,"updated":"2017-02-13 12:16:42.000000000","message":"Maybe add:\n\n  This is null when an action succeeds\n\nfor clarity?","commit_id":"a72968a9e06ce2d9710e2c80064f10c203f993d6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"3d3cc2b0c5c726e5c1a334b4785ff12a278fdfb3","unresolved":false,"context_lines":[{"line_number":2952,"context_line":"  type: integer"},{"line_number":2953,"context_line":"message:"},{"line_number":2954,"context_line":"  description: |"},{"line_number":2955,"context_line":"    The related error message for when an action fails."},{"line_number":2956,"context_line":"  in: body"},{"line_number":2957,"context_line":"  required: true"},{"line_number":2958,"context_line":"  type: string"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fa31d9ce_b57b00bd","line":2955,"in_reply_to":"fa31d9ce_f56423e6","updated":"2017-02-16 00:47:37.000000000","message":"The API samples show it as an empty string, but it looks like that\u0027s maybe just because of the sample template:\n\nhttps://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/api_samples/os-instance-actions/instance-actions-list-resp.json.tpl\n\nI think those would actually be None by looking at the code.\n\nI\u0027m not sure if \"This is null when an action succeeds\" is more clear to me. Maybe the description should just be something like:\n\n\"If the action fails, the related error message is provided here.\"","commit_id":"a72968a9e06ce2d9710e2c80064f10c203f993d6"}]}