)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"e4fed1f86fbb7b625556c5049dadece995547d7d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"nothing denoting this, causing errors and confusion for code that"},{"line_number":15,"context_line":"thought it could get a host from the list, then get detailed info"},{"line_number":16,"context_line":"about that host. The details return volume and snapshot counts,"},{"line_number":17,"context_line":"so it really only makes sense for cinder-volume, so the code ends"},{"line_number":18,"context_line":"up returning a 404 for anything else."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This API design seems a little disjointed, but since this is how"},{"line_number":21,"context_line":"it appears to have always been, just make sure the api docs have"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_716b40f0","line":18,"range":{"start_line":17,"start_character":0,"end_line":18,"end_character":37},"updated":"2017-08-31 07:33:49.000000000","message":"Some additions: If the cinder-volume service on the host is disabled, the os-host api will also return a 404.","commit_id":"1fd23280b7a0db9bcf5a7d60b3e48b189652c493"}],"api-ref/source/v2/hosts.inc":[{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"41e0a8a96407247d3386257a2277432c33642d3c","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Shows volume and snapshot details for a cinder-volume host."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"*Note:* This API is meant specifically for disabled cinder-volume hosts only."},{"line_number":55,"context_line":"It is not valid against other Cinder service hosts or hosts where the"},{"line_number":56,"context_line":"cinder-volume service has been disabled."},{"line_number":57,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":2,"id":"7f287b81_b90066ac","line":54,"range":{"start_line":54,"start_character":43,"end_line":54,"end_character":51},"updated":"2017-08-31 14:54:09.000000000","message":"I think this description is not right， the api is specifically for cinder-volume host which is enabled. Or the \u0027disabled\u0027 may be removed.","commit_id":"1801e195c4aec90b288a0bdf00ea682d51b62653"}],"api-ref/source/v3/hosts.inc":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"b3e5a753e440622e1c57801afde132876501d174","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Shows volume and snapshot details for a cinder-volume host."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"*Note:* This API is meant specifically for disabled cinder-volume hosts only."},{"line_number":55,"context_line":"It is not valid against other Cinder service hosts or hosts where the"},{"line_number":56,"context_line":"cinder-volume service has been disabled."},{"line_number":57,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":2,"id":"7f287b81_ee229401","line":54,"range":{"start_line":54,"start_character":43,"end_line":54,"end_character":51},"updated":"2017-09-01 00:50:44.000000000","message":"Oops, not sure how that ended up in there. Fixed.","commit_id":"1801e195c4aec90b288a0bdf00ea682d51b62653"}]}
