)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"2409308b8799b177987831bf8bf9c25f50450230","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add volume type filter to Get-Pools"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add feature that administrators can get back-end storage pools filtered by"},{"line_number":10,"context_line":"volume-type, cinder will return the pools filtered by volume-type\u0027s extra-spec."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8ed01bbee0c1f54f1761c812987ac70b4c4bbbcf"},{"line_number":13,"context_line":"blueprint: add-volume-type-filter-to-get-pool"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"ffe62b97_893fc185","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":79},"updated":"2017-03-29 15:26:10.000000000","message":"Subsequent lines should be wrapped at 72 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"2aadde7a4dcd1b7b866ab954981b85cd893d4163"}],"specs/ocata/add-volume-type-filter-to-get-pools.rst":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"7b9319433da03987ff156f744193cdcfcf1747c2","unresolved":false,"context_lines":[{"line_number":25,"context_line":"based capability statistics is required,the administrators also can get all"},{"line_number":26,"context_line":"pools and filter them on their own,but it\u0027s more complicated and inefficient."},{"line_number":27,"context_line":"This change intends to cover this situation and bring more convenience to"},{"line_number":28,"context_line":"administrators"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Use Cases"},{"line_number":31,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_e8d40c21","line":28,"range":{"start_line":28,"start_character":13,"end_line":28,"end_character":14},"updated":"2016-09-26 17:22:30.000000000","message":".","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"4a6a835a6e1e27c465dd05dd2230d27d38e90281","unresolved":false,"context_lines":[{"line_number":25,"context_line":"based capability statistics is required,the administrators also can get all"},{"line_number":26,"context_line":"pools and filter them on their own,but it\u0027s more complicated and inefficient."},{"line_number":27,"context_line":"This change intends to cover this situation and bring more convenience to"},{"line_number":28,"context_line":"administrators"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Use Cases"},{"line_number":31,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_56dcac45","line":28,"range":{"start_line":28,"start_character":13,"end_line":28,"end_character":14},"in_reply_to":"5a8bc5a2_e8d40c21","updated":"2016-09-27 02:52:01.000000000","message":"Done","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"603ecbbe3d6f38478aa309a3ecd8b48d634c7fc0","unresolved":false,"context_lines":[{"line_number":30,"context_line":"Use Cases"},{"line_number":31,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"In production environment, administrators often need to have an overall"},{"line_number":34,"context_line":"pool avaliable statistics grouped by volume type,this will help them to make"},{"line_number":35,"context_line":"an adjustment before resources run out."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa7ab95a_767303b5","line":35,"range":{"start_line":33,"start_character":0,"end_line":35,"end_character":39},"updated":"2016-08-31 17:02:01.000000000","message":"Is this about administrator would like to know how many (more) type X volumes can be created on certain pool(s)?  If it is, the get-pool API doesn\u0027t seem to be the right place for this use case.\n\nget-pool was added in order to provide a clear view how many pools are there managed by Cinder, it does that by getting information from c-vol status update and check c-vol DB heartbeat.  It doesn\u0027t go through c-sch.  But your requirement would either have to go through c-sch or end up implement a c-sch.","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"418351fdf894f03c1bfb588e3f095119a9ff472d","unresolved":false,"context_lines":[{"line_number":30,"context_line":"Use Cases"},{"line_number":31,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"In production environment, administrators often need to have an overall"},{"line_number":34,"context_line":"pool avaliable statistics grouped by volume type,this will help them to make"},{"line_number":35,"context_line":"an adjustment before resources run out."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa7ab95a_2fa8e376","line":35,"range":{"start_line":33,"start_character":0,"end_line":35,"end_character":39},"in_reply_to":"fa7ab95a_767303b5","updated":"2016-09-02 01:12:32.000000000","message":"All we wanna do is a host_filter to get_pools, If we go through C-scheduler we have to introduce a new API，which will do almost thing the get-pools have done before","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"7b9319433da03987ff156f744193cdcfcf1747c2","unresolved":false,"context_lines":[{"line_number":40,"context_line":"Proposed change"},{"line_number":41,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Introduce volume filter when retrieving pools. It accepts a volume-type name"},{"line_number":44,"context_line":"  or ID and return the filtered pools which\u0027s capabilities meets volume-type\u0027s"},{"line_number":45,"context_line":"  extra-specs"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_cb010a08","line":43,"range":{"start_line":43,"start_character":12,"end_line":43,"end_character":25},"updated":"2016-09-26 17:22:30.000000000","message":"You mean a new command argument? The way you are saying seems that is something related with the scheduler filters.","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"4a6a835a6e1e27c465dd05dd2230d27d38e90281","unresolved":false,"context_lines":[{"line_number":40,"context_line":"Proposed change"},{"line_number":41,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Introduce volume filter when retrieving pools. It accepts a volume-type name"},{"line_number":44,"context_line":"  or ID and return the filtered pools which\u0027s capabilities meets volume-type\u0027s"},{"line_number":45,"context_line":"  extra-specs"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_f6705842","line":43,"range":{"start_line":43,"start_character":12,"end_line":43,"end_character":25},"in_reply_to":"5a8bc5a2_cb010a08","updated":"2016-09-27 02:52:01.000000000","message":"sorry,what I propose here is to introduce a volume type filter, and it\u0027s indeed to filter pools by schduler filters with volume type filters","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"7b9319433da03987ff156f744193cdcfcf1747c2","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Introduce volume filter when retrieving pools. It accepts a volume-type name"},{"line_number":44,"context_line":"  or ID and return the filtered pools which\u0027s capabilities meets volume-type\u0027s"},{"line_number":45,"context_line":"  extra-specs"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_cbf74a57","line":45,"range":{"start_line":45,"start_character":2,"end_line":45,"end_character":13},"updated":"2016-09-26 17:22:30.000000000","message":"Can you add a proposed CLI call and desired output? How do you expect this to be in the user point of view?","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"4a6a835a6e1e27c465dd05dd2230d27d38e90281","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Introduce volume filter when retrieving pools. It accepts a volume-type name"},{"line_number":44,"context_line":"  or ID and return the filtered pools which\u0027s capabilities meets volume-type\u0027s"},{"line_number":45,"context_line":"  extra-specs"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_998ca5db","line":45,"range":{"start_line":45,"start_character":2,"end_line":45,"end_character":13},"in_reply_to":"5a8bc5a2_cbf74a57","updated":"2016-09-27 02:52:01.000000000","message":"I have modified the \u0027Proposed change\u0027 with detail changes,please take a look at the next patch","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"7b9319433da03987ff156f744193cdcfcf1747c2","unresolved":false,"context_lines":[{"line_number":49,"context_line":"Alternatives"},{"line_number":50,"context_line":"------------"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Administrators also can retrieve and filter by themselves, but it\u0027s more"},{"line_number":53,"context_line":"complicated and inefficient. This change can reduce the request amount and"},{"line_number":54,"context_line":"filter unnecessary data transmitted from server to client."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_7e127658","line":52,"range":{"start_line":52,"start_character":20,"end_line":52,"end_character":56},"updated":"2016-09-26 17:22:30.000000000","message":"how?","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"4a6a835a6e1e27c465dd05dd2230d27d38e90281","unresolved":false,"context_lines":[{"line_number":49,"context_line":"Alternatives"},{"line_number":50,"context_line":"------------"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Administrators also can retrieve and filter by themselves, but it\u0027s more"},{"line_number":53,"context_line":"complicated and inefficient. This change can reduce the request amount and"},{"line_number":54,"context_line":"filter unnecessary data transmitted from server to client."},{"line_number":55,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a8bc5a2_9642b4ac","line":52,"range":{"start_line":52,"start_character":20,"end_line":52,"end_character":56},"in_reply_to":"5a8bc5a2_7e127658","updated":"2016-09-27 02:52:01.000000000","message":"what I mean here is that they can retrive all the pools and filter the pools on their own,they can achieve this by compare the properties one by one(just for demonstation)","commit_id":"894c90024fdf37c42b066a19e2c77fa62ea8813a"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"f966191b1cc73081d3a408cfa2010bac4821f8a1","unresolved":false,"context_lines":[{"line_number":88,"context_line":"Other end user impact"},{"line_number":89,"context_line":"---------------------"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Python client need to add new parameter \u0027--volume-type\u0027 when get_pools"},{"line_number":92,"context_line":"to support this."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a98d1f4_b2bbdac3","line":91,"updated":"2016-09-29 06:47:31.000000000","message":"Since we will have other filters support, it\u0027d be better to incorporate this type of filter with others, i.e. not use \u0027--volume-type\u0027 optional argument but let it be part of \u0027--filters\u0027.","commit_id":"5644a76fe565cfeb01e5472ebd54d047c53ae7eb"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"bc5e2a5610d917731586d98f66fc9e3692dc8856","unresolved":false,"context_lines":[{"line_number":91,"context_line":"As the API will have other filter arguments such as \u0027--name\u0027, the py-"},{"line_number":92,"context_line":"client will have a new argument called \u0027--filters\u0027 which accepts"},{"line_number":93,"context_line":"different filter key to do different filter logic. multiple keys are"},{"line_number":94,"context_line":"combined with back space. The use of filter get-pools API by volume-type"},{"line_number":95,"context_line":"and name will be something like::"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"--filters volume-type\u003d\u0027name or id\u0027 name\u003d\u0027pool name\u0027"}],"source_content_type":"text/x-rst","patch_set":8,"id":"1a95cdbc_de90e645","line":94,"updated":"2016-10-08 01:22:21.000000000","message":"back space or space?","commit_id":"5e47a4e0aecf632640acf6e099bcce39ee79815d"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"9c5008ec13c3608ab898692eb448cd8d995333cd","unresolved":false,"context_lines":[{"line_number":91,"context_line":"As the API will have other filter arguments such as \u0027--name\u0027, the py-"},{"line_number":92,"context_line":"client will have a new argument called \u0027--filters\u0027 which accepts"},{"line_number":93,"context_line":"different filter key to do different filter logic. multiple keys are"},{"line_number":94,"context_line":"combined with back space. The use of filter get-pools API by volume-type"},{"line_number":95,"context_line":"and name will be something like::"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"--filters volume-type\u003d\u0027name or id\u0027 name\u003d\u0027pool name\u0027"}],"source_content_type":"text/x-rst","patch_set":8,"id":"1a95cdbc_395ac42a","line":94,"in_reply_to":"1a95cdbc_de90e645","updated":"2016-10-08 02:05:48.000000000","message":"Done","commit_id":"5e47a4e0aecf632640acf6e099bcce39ee79815d"}],"specs/pike/add-volume-type-filter-to-get-pools.rst":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"f8e98981a277adefb01335f02b0dafe994f453c8","unresolved":false,"context_lines":[{"line_number":19,"context_line":"Problem description"},{"line_number":20,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Now get-pools\u0027s API only support filter pools by name,this is not enough when"},{"line_number":23,"context_line":"administrators wants to know the pools state which also can meet the specified"},{"line_number":24,"context_line":"volume type\u0027s requirements,the statistics is useful when kind of volume-type"},{"line_number":25,"context_line":"based capability statistics is required,the administrators also can get all"}],"source_content_type":"text/x-rst","patch_set":10,"id":"7a77a97e_76d35ad5","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":53},"updated":"2016-11-18 15:30:55.000000000","message":"Correct if I\u0027m wrong, but it seems that get-pools method doesn\u0027t support filters now.\nhttp://paste.openstack.org/show/589725/\n\nhttps://github.com/openstack/cinder/blob/master/cinder/scheduler/host_manager.py#L521\n\nhttps://github.com/openstack/cinder/blob/master/cinder/scheduler/filter_scheduler.py#L193","commit_id":"5d789c1bc5951191ffe2830ba1012a3255bb1a4c"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"39daa6da8bf44455a1d2a878c59b72db8d46df31","unresolved":false,"context_lines":[{"line_number":19,"context_line":"Problem description"},{"line_number":20,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Now get-pools\u0027s API only support filter pools by name,this is not enough when"},{"line_number":23,"context_line":"administrators wants to know the pools state which also can meet the specified"},{"line_number":24,"context_line":"volume type\u0027s requirements,the statistics is useful when kind of volume-type"},{"line_number":25,"context_line":"based capability statistics is required,the administrators also can get all"}],"source_content_type":"text/x-rst","patch_set":10,"id":"5a74a57a_8696934a","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":53},"in_reply_to":"7a77a97e_76d35ad5","updated":"2016-11-24 08:17:50.000000000","message":"We have a patch which already focus on this right now [1], But I\u0027d like to remove this part as this spec does not rely on this patch actually.\n\n[1] https://review.openstack.org/#/c/321492/","commit_id":"5d789c1bc5951191ffe2830ba1012a3255bb1a4c"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"b7a2845810c49d98fedec1bb76d51768e520f3e4","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Problem description"},{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Now cinder\u0027s ```get-pools``` API doesn\u0027t support filter pools, and this is"},{"line_number":22,"context_line":"inconvenient when administrators want to know the specified pool\u0027s state"},{"line_number":23,"context_line":"which also can meet volume type\u0027s requirements, the administrators also can"},{"line_number":24,"context_line":"get all pools and filter them on their own, but it\u0027s more complicated and"}],"source_content_type":"text/x-rst","patch_set":13,"id":"bff0334d_a87c111f","line":21,"range":{"start_line":21,"start_character":40,"end_line":21,"end_character":61},"updated":"2017-04-10 10:54:32.000000000","message":"v3.28 support filter pools already.","commit_id":"55684fed6892c39f057fa6162fc0aab4634f9049"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"b7a2845810c49d98fedec1bb76d51768e520f3e4","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  ``host.get_filtered_hosts``."},{"line_number":48,"context_line":"* the filter properties of ``get_filtered_hosts`` only consists of volume-type"},{"line_number":49,"context_line":"  properties."},{"line_number":50,"context_line":"* the cinder-client will also add a argument ``--volume-type`` to support this."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Alternatives"},{"line_number":53,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":13,"id":"bff0334d_0b5537fa","line":50,"range":{"start_line":50,"start_character":0,"end_line":50,"end_character":79},"updated":"2017-04-10 10:54:32.000000000","message":"I think this should base on John\u0027s generalized resource filtering feature, then we don\u0027t need any new argument.","commit_id":"55684fed6892c39f057fa6162fc0aab4634f9049"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fb66bd4ca75db205098d302b4a1f89d9316dc67e","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Problem description"},{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Now cinder\u0027s ``get-pools`` API doesn\u0027t support filter pools by volume type,"},{"line_number":22,"context_line":"and this is inconvenient when administrators want to know the specified pool\u0027s"},{"line_number":23,"context_line":"state which also can meet volume type\u0027s requirements, the administrators also"},{"line_number":24,"context_line":"can get all pools and filter them on their own, but it\u0027s more complicated and"}],"source_content_type":"text/x-rst","patch_set":15,"id":"5ff73747_867aeb85","line":21,"range":{"start_line":21,"start_character":47,"end_line":21,"end_character":53},"updated":"2017-04-22 16:16:12.000000000","message":"filtering","commit_id":"e56a8e40e3ac58a2d010e93d3f50cdacf78ad4a8"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fb66bd4ca75db205098d302b4a1f89d9316dc67e","unresolved":false,"context_lines":[{"line_number":39,"context_line":"outside cinder, the only thing we should do is to advertise that we can"},{"line_number":40,"context_line":"support this filter now:"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"From the view of inside cinder. We will mark ``volume-type`` recognisable, and"},{"line_number":43,"context_line":"support for this filter in logic:"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* once the volume-type (name or id) is detected, the volume type object will be"}],"source_content_type":"text/x-rst","patch_set":15,"id":"5ff73747_8651cb11","line":42,"range":{"start_line":42,"start_character":61,"end_line":42,"end_character":73},"updated":"2017-04-22 16:16:12.000000000","message":"recognizable","commit_id":"e56a8e40e3ac58a2d010e93d3f50cdacf78ad4a8"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fb66bd4ca75db205098d302b4a1f89d9316dc67e","unresolved":false,"context_lines":[{"line_number":43,"context_line":"support for this filter in logic:"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* once the volume-type (name or id) is detected, the volume type object will be"},{"line_number":46,"context_line":"  retrieved before scheduler api called, and will be passed as a filter item."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* the ``scheduler.get_pools`` called, which will call the"},{"line_number":49,"context_line":"  ``host_manager.get_pools`` in result."}],"source_content_type":"text/x-rst","patch_set":15,"id":"5ff73747_6656bf09","line":46,"range":{"start_line":46,"start_character":29,"end_line":46,"end_character":39},"updated":"2017-04-22 16:16:12.000000000","message":"api is called","commit_id":"e56a8e40e3ac58a2d010e93d3f50cdacf78ad4a8"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fb66bd4ca75db205098d302b4a1f89d9316dc67e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  ``host_manager.get_pools`` in result."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"* the ``host_manager.get_pools`` will collect the pools information as normal,"},{"line_number":52,"context_line":"  and before it returns,the result will be filtered by"},{"line_number":53,"context_line":"  ``host.get_filtered_hosts``."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"* the filter properties of ``get_filtered_hosts`` only consists of volume-type"}],"source_content_type":"text/x-rst","patch_set":15,"id":"5ff73747_2640b735","line":52,"range":{"start_line":52,"start_character":24,"end_line":52,"end_character":27},"updated":"2017-04-22 16:16:12.000000000","message":"missing space before \u0027the\u0027","commit_id":"e56a8e40e3ac58a2d010e93d3f50cdacf78ad4a8"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fb66bd4ca75db205098d302b4a1f89d9316dc67e","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  properties."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"* As already proposed by generalized resource filtering `[1]`, the changes on"},{"line_number":59,"context_line":"  cinder-client for this feature is not needed."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":15,"id":"5ff73747_e649af60","line":59,"range":{"start_line":59,"start_character":33,"end_line":59,"end_character":34},"updated":"2017-04-22 16:16:12.000000000","message":"are","commit_id":"e56a8e40e3ac58a2d010e93d3f50cdacf78ad4a8"}]}