)]}'
{"doc/source/devref/driver_requirements.rst":[{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"57c82b200941f9c7e29d36c61ebffade9e2e4f9f","unresolved":false,"context_lines":[{"line_number":29,"context_line":"networks by making use of Manila Share Networks, or if it is up to the"},{"line_number":30,"context_line":"administrator to manually configure all networks (DHSS* \u003d false) and be"},{"line_number":31,"context_line":"responsible for segregation, if that is desired. At least one driver mode must"},{"line_number":32,"context_line":"be supported. In DHSS \u003d true mode, Share Server entities are used, so the"},{"line_number":33,"context_line":"driver must implement functions that setup and teardown such servers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* DHSS: driver_handles_share_servers"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_3dbc45db","line":32,"updated":"2015-10-16 13:49:34.000000000","message":"Need refer to dictionary.","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"f052a34707696e6a45949dd3d143733cf808ab6f","unresolved":false,"context_lines":[{"line_number":29,"context_line":"networks by making use of Manila Share Networks, or if it is up to the"},{"line_number":30,"context_line":"administrator to manually configure all networks (DHSS* \u003d false) and be"},{"line_number":31,"context_line":"responsible for segregation, if that is desired. At least one driver mode must"},{"line_number":32,"context_line":"be supported. In DHSS \u003d true mode, Share Server entities are used, so the"},{"line_number":33,"context_line":"driver must implement functions that setup and teardown such servers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* DHSS: driver_handles_share_servers"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_20717aa8","line":32,"in_reply_to":"5a2ca52d_3dbc45db","updated":"2015-10-16 15:27:46.000000000","message":"Done","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"57c82b200941f9c7e29d36c61ebffade9e2e4f9f","unresolved":false,"context_lines":[{"line_number":32,"context_line":"be supported. In DHSS \u003d true mode, Share Server entities are used, so the"},{"line_number":33,"context_line":"driver must implement functions that setup and teardown such servers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* DHSS: driver_handles_share_servers"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"At least one file system sharing protocol"},{"line_number":38,"context_line":"-----------------------------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_fdb1ed01","line":35,"updated":"2015-10-16 13:49:34.000000000","message":"same here.","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"f052a34707696e6a45949dd3d143733cf808ab6f","unresolved":false,"context_lines":[{"line_number":32,"context_line":"be supported. In DHSS \u003d true mode, Share Server entities are used, so the"},{"line_number":33,"context_line":"driver must implement functions that setup and teardown such servers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* DHSS: driver_handles_share_servers"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"At least one file system sharing protocol"},{"line_number":38,"context_line":"-----------------------------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_006c7ece","line":35,"in_reply_to":"5a2ca52d_fdb1ed01","updated":"2015-10-16 15:27:46.000000000","message":"Done","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14863,"name":"Diego Zamboni Latance","email":"dzambonil@gmail.com","username":"DiegoZamboni"},"change_message_id":"b9f83e977db8d8c22a50d5410623710a69f5d96c","unresolved":false,"context_lines":[{"line_number":47,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":48,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":49,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":50,"context_line":"supported access levels should be read-write and read-only."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Shares"},{"line_number":53,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_0b3edfd9","line":50,"updated":"2015-10-16 14:26:37.000000000","message":"I guess he means that each access rule should be able to be RW or RO, not at the same time, but in the way that both RW and RO should be supported.","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"57c82b200941f9c7e29d36c61ebffade9e2e4f9f","unresolved":false,"context_lines":[{"line_number":47,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":48,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":49,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":50,"context_line":"supported access levels should be read-write and read-only."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Shares"},{"line_number":53,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_3d8a051c","line":50,"updated":"2015-10-16 13:49:34.000000000","message":"It is unclear, You propose both - RW and RO be required access levels?","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"f052a34707696e6a45949dd3d143733cf808ab6f","unresolved":false,"context_lines":[{"line_number":47,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":48,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":49,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":50,"context_line":"supported access levels should be read-write and read-only."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Shares"},{"line_number":53,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_607bf28c","line":50,"in_reply_to":"5a2ca52d_3d8a051c","updated":"2015-10-16 15:27:46.000000000","message":"yes, we discussed that in the etherpad. RW and RO must be supported in the following releases.","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14863,"name":"Diego Zamboni Latance","email":"dzambonil@gmail.com","username":"DiegoZamboni"},"change_message_id":"b9f83e977db8d8c22a50d5410623710a69f5d96c","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Capabilities"},{"line_number":69,"context_line":"------------"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"In order to Manila to function accordingly to the driver being used, the driver"},{"line_number":72,"context_line":"must provide a set of information to Manila, known as capabilities, as follows:"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"- share_backend_name: a name for the backend;"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_7da87d20","line":71,"updated":"2015-10-16 14:26:37.000000000","message":"s/to Manila/for Manila","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"f052a34707696e6a45949dd3d143733cf808ab6f","unresolved":false,"context_lines":[{"line_number":68,"context_line":"Capabilities"},{"line_number":69,"context_line":"------------"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"In order to Manila to function accordingly to the driver being used, the driver"},{"line_number":72,"context_line":"must provide a set of information to Manila, known as capabilities, as follows:"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"- share_backend_name: a name for the backend;"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_319d4ebc","line":71,"in_reply_to":"5a2ca52d_7da87d20","updated":"2015-10-16 15:27:46.000000000","message":"Done","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"57c82b200941f9c7e29d36c61ebffade9e2e4f9f","unresolved":false,"context_lines":[{"line_number":99,"context_line":"Continuous Integration systems"},{"line_number":100,"context_line":"------------------------------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":103,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":104,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":105,"context_line":"patch submitted affects an existing drivers. The CI system should run all"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_1db389ea","line":102,"updated":"2015-10-16 13:49:34.000000000","message":"need add link for more details on it.","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"f052a34707696e6a45949dd3d143733cf808ab6f","unresolved":false,"context_lines":[{"line_number":99,"context_line":"Continuous Integration systems"},{"line_number":100,"context_line":"------------------------------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":103,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":104,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":105,"context_line":"patch submitted affects an existing drivers. The CI system should run all"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_ec96193d","line":102,"in_reply_to":"5a2ca52d_1db389ea","updated":"2015-10-16 15:27:46.000000000","message":"Done","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14863,"name":"Diego Zamboni Latance","email":"dzambonil@gmail.com","username":"DiegoZamboni"},"change_message_id":"b9f83e977db8d8c22a50d5410623710a69f5d96c","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":103,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":104,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":105,"context_line":"patch submitted affects an existing drivers. The CI system should run all"},{"line_number":106,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":107,"context_line":"test results."}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_fd380df0","line":105,"updated":"2015-10-16 14:26:37.000000000","message":"s/drivers/driver or remove \u0027an\u0027","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"f052a34707696e6a45949dd3d143733cf808ab6f","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":103,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":104,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":105,"context_line":"patch submitted affects an existing drivers. The CI system should run all"},{"line_number":106,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":107,"context_line":"test results."}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a2ca52d_7160a6cf","line":105,"in_reply_to":"5a2ca52d_fd380df0","updated":"2015-10-16 15:27:46.000000000","message":"Done","commit_id":"dc59e73f2011713b80dc45db11e8225a7d1d6676"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"In order for a driver to be accepted into Manila code base, there are certain"},{"line_number":21,"context_line":"minimum requirements and features that should be met, in order to ensure"},{"line_number":22,"context_line":"common and standardized Manila functionality among cloud providers."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"At least one driver mode (:term:`DHSS` true/false)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_6a018405","line":21,"updated":"2015-10-16 17:47:17.000000000","message":"s/should/must/\n\nThis document is about required features, so we shouldn\u0027t say \u0027should\u0027.  You\u0027ve done a good job using \u0027must\u0027 in most places.\n\nIn fact, you could consider using CAPS for such words as often seen in RFCs.\n\nhttps://www.ietf.org/rfc/rfc2119.txt","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"In order for a driver to be accepted into Manila code base, there are certain"},{"line_number":21,"context_line":"minimum requirements and features that should be met, in order to ensure"},{"line_number":22,"context_line":"common and standardized Manila functionality among cloud providers."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"At least one driver mode (:term:`DHSS` true/false)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_d305f413","line":21,"in_reply_to":"5a2ca52d_6a018405","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e79307a5db15f413e3be16394a4b0dd09b45856d","unresolved":false,"context_lines":[{"line_number":36,"context_line":"-----------------------------------------"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"The driver must support at least one file system sharing protocol in order to"},{"line_number":39,"context_line":"serve shares as a shared file system service."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Access rules"},{"line_number":42,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_56047b16","line":39,"updated":"2015-10-16 16:48:42.000000000","message":"Can you list the supported protocols here?","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":36,"context_line":"-----------------------------------------"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"The driver must support at least one file system sharing protocol in order to"},{"line_number":39,"context_line":"serve shares as a shared file system service."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Access rules"},{"line_number":42,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_cd764d3e","line":39,"in_reply_to":"5a2ca52d_56047b16","updated":"2015-10-19 13:01:08.000000000","message":"Sure, let\u0027s compile a list, I couldn\u0027t find a list anywhere so I will add the ones I know and please let me know if there are protocols missing.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":41,"context_line":"Access rules"},{"line_number":42,"context_line":"------------"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Access rules control how shares are accessible, by who, and what the level of"},{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_6a9ae46f","line":44,"updated":"2015-10-16 17:47:17.000000000","message":"\u0027by whom\u0027","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":41,"context_line":"Access rules"},{"line_number":42,"context_line":"------------"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Access rules control how shares are accessible, by who, and what the level of"},{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_f329109c","line":44,"in_reply_to":"5a2ca52d_6a9ae46f","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Access rules control how shares are accessible, by who, and what the level of"},{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_79563a07","line":46,"updated":"2015-10-16 22:48:24.000000000","message":"I\u0027d delete \"For each file system sharing protocol, \".","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Access rules control how shares are accessible, by who, and what the level of"},{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_ad30b157","line":46,"in_reply_to":"5a2ca52d_79563a07","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":44,"context_line":"Access rules control how shares are accessible, by who, and what the level of"},{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_9959d637","line":47,"updated":"2015-10-16 22:48:24.000000000","message":"s#or#and/or#","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":44,"context_line":"Access rules control how shares are accessible, by who, and what the level of"},{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_4d0c7d94","line":47,"in_reply_to":"5a2ca52d_9959d637","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e79307a5db15f413e3be16394a4b0dd09b45856d","unresolved":false,"context_lines":[{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"},{"line_number":51,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_f95a58f1","line":48,"updated":"2015-10-16 16:48:42.000000000","message":"So you mean driver must support one of the 3 authentication types: IP, User, or Certificate and driver must support both read-write and read-only for each access rule?","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e2ea01b185a1a13d2e257c5ffde52060be97b39f","unresolved":false,"context_lines":[{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"},{"line_number":51,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_30245ec3","line":48,"in_reply_to":"5a2ca52d_8ad3c003","updated":"2015-10-16 19:14:03.000000000","message":"That\u0027s why I suggested that we discuss about this at the summit so that everyone is on the same page.  So we should probably discuss about this one at the next IRC meeting or at the summit.\n\nAny way, I think we should have a documented minimum required features for Mitaka shortly after the summit so driver maintainers can start allocating resources to work on them.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"},{"line_number":51,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_566d52ca","line":48,"in_reply_to":"5a2ca52d_8ad3c003","updated":"2015-10-19 13:01:08.000000000","message":"Yes Xing.\n\nClinton, I remember we discussed this in a weekly meeting where read only support would be good for consistency, for drivers that do not support they would need to sort it out in Mitaka, unless the protocol makes that impossible. For instance, I mentioned that Generic driver does not implement read-only in CIFS, and Valeriy said it can be implemented.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":45,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"},{"line_number":51,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_8ad3c003","line":48,"in_reply_to":"5a2ca52d_f95a58f1","updated":"2015-10-16 17:47:17.000000000","message":"Not all drivers support both read-write and read-only, and we haven\u0027t discussed making that a requirement.  I\u0027m not sure that\u0027s even practical.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"},{"line_number":51,"context_line":"------"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_59dc9e72","line":49,"updated":"2015-10-16 22:48:24.000000000","message":"In this section there are new minimum reqs to propose and consider.  Could you make this document clarify what is being proposed for M.  In this case, proposed for liberty:  1. shares require access rule before allowing access  2. driver must support at least one access type for a share.  3. We might need to break-down the rw/ro part into alternatives. 3a. drivers must support ro/rw? 3b. ro/rw is optional?","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":46,"context_line":"For each file system sharing protocol, the authentication type"},{"line_number":47,"context_line":"should be based on IP, User or Certificate. For each access rule, the"},{"line_number":48,"context_line":"supported access levels should be read-write and read-only."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"Shares"},{"line_number":51,"context_line":"------"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_3fcfe4b1","line":49,"in_reply_to":"5a2ca52d_59dc9e72","updated":"2015-10-19 13:01:08.000000000","message":"I think this document should always reflect what is the minimum for the current release, in this case, Mitaka. There is no development for Liberty anymore.\n\n1) not sure if I understood correctly, but that is covered in \"Access\" section.\n2) same as (1)\n3) Why?\n3a) yes, we decided that in a meeting, please see my reply to Xing\u0027s comment.\n3b) no, as far as I know.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":54,"context_line":"a driver must be able to create and delete shares. In some situations"},{"line_number":55,"context_line":"Manila attempts to ensure created shares, so the driver must implement a"},{"line_number":56,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":57,"context_line":"re-exporting."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Resizing"},{"line_number":60,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_b9b932ce","line":57,"updated":"2015-10-16 22:48:24.000000000","message":"I think a lot of drivers \"pass\" on ensure_share.  So this would be another \"Proposed for Mitaka\" requirement and I think it needs more specific clarification.  Is re-exporting / re-creating a share? a requirement?  This should be optional until clarified, but could be proposed here and in Tokyo.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":54,"context_line":"a driver must be able to create and delete shares. In some situations"},{"line_number":55,"context_line":"Manila attempts to ensure created shares, so the driver must implement a"},{"line_number":56,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":57,"context_line":"re-exporting."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Resizing"},{"line_number":60,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_ff1dec0f","line":57,"in_reply_to":"5a2ca52d_b9b932ce","updated":"2015-10-19 13:01:08.000000000","message":"I see this as an important function that should be implemented. I thought I was clear enough by saying that this is a function that performs maintenance, please share how you rewrite it to make it more clear for the user. \nSince this function is already in the code, and there is no way to avoid calling this function, I already considered it as a requirement, for all versions.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"},{"line_number":63,"context_line":"drivers must implement a share extend function that allows shares\u0027 size to be"},{"line_number":64,"context_line":"increased."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Capabilities"},{"line_number":67,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_19d626f1","line":64,"updated":"2015-10-16 22:48:24.000000000","message":"Proposed for Mitaka","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"},{"line_number":63,"context_line":"drivers must implement a share extend function that allows shares\u0027 size to be"},{"line_number":64,"context_line":"increased."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Capabilities"},{"line_number":67,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_5fd3c038","line":64,"in_reply_to":"5a2ca52d_19d626f1","updated":"2015-10-19 13:01:08.000000000","message":"If we decide this document reflects the current release, then it is already correct.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":62,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"},{"line_number":63,"context_line":"drivers must implement a share extend function that allows shares\u0027 size to be"},{"line_number":64,"context_line":"increased."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Capabilities"},{"line_number":67,"context_line":"------------"},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_39fc426f","line":65,"updated":"2015-10-16 22:48:24.000000000","message":"I think it\u0027s worth mentioning that shrink is optional (but perhaps the authors judgement on that is better than mine).","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"b6f46f069debc519aacf72be3e6f5ba2363bde1d","unresolved":false,"context_lines":[{"line_number":62,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"},{"line_number":63,"context_line":"drivers must implement a share extend function that allows shares\u0027 size to be"},{"line_number":64,"context_line":"increased."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Capabilities"},{"line_number":67,"context_line":"------------"},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_69e9261a","line":65,"in_reply_to":"3a29b11f_5fdd0013","updated":"2015-10-19 14:28:45.000000000","message":"+1  A separate section for optional features sounds good.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":62,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"},{"line_number":63,"context_line":"drivers must implement a share extend function that allows shares\u0027 size to be"},{"line_number":64,"context_line":"increased."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Capabilities"},{"line_number":67,"context_line":"------------"},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_5fdd0013","line":65,"in_reply_to":"5a2ca52d_39fc426f","updated":"2015-10-19 13:01:08.000000000","message":"I thought about it when I was writing the document. We could have another section in the document about optional requirements, because I was thinking as an user, and it feels like features that are not listed here may not exist, although they are listed in the API. This may look confusing for the user. I am up for suggestions, I personally think they should be in the document, but in another section. What do you think?","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e79307a5db15f413e3be16394a4b0dd09b45856d","unresolved":false,"context_lines":[{"line_number":78,"context_line":"- storage_protocol: list of shared file system protocols supported by this"},{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_b9a2a097","line":81,"updated":"2015-10-16 16:48:42.000000000","message":"Please add the following:\n\nprovisioned_capacity_gb, thin_provisioning, dedupe, compression, max_over_subscription_ratio.\n\nhttps://github.com/openstack/manila/blob/master/doc/source/devref/capabilities_and_extra_specs.rst","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":78,"context_line":"- storage_protocol: list of shared file system protocols supported by this"},{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_d380948d","line":81,"in_reply_to":"5a2ca52d_2af5cc55","updated":"2015-10-19 13:01:08.000000000","message":"I also think they do not need to be in this list because they are not mandatory. If the default values do not impact behavior and the driver vendor may choose to not provide them, then it I don\u0027t see it as a minimum requirement. This is another interesting topic to debate.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e2ea01b185a1a13d2e257c5ffde52060be97b39f","unresolved":false,"context_lines":[{"line_number":78,"context_line":"- storage_protocol: list of shared file system protocols supported by this"},{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_d3f17496","line":81,"in_reply_to":"5a2ca52d_2af5cc55","updated":"2015-10-16 19:14:03.000000000","message":"You are right that some have default values, but it is better for driver to report them based its own knowledge.  So we can say it is recommended.\n\nIf compression can\u0027t be reported as True/False, then it seems pre-mature to have that added in the doc.  Maybe it should be removed until we know exactly what it should be or at least add note saying it will be changed.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":78,"context_line":"- storage_protocol: list of shared file system protocols supported by this"},{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_2af5cc55","line":81,"in_reply_to":"5a2ca52d_b9a2a097","updated":"2015-10-16 17:47:17.000000000","message":"Some of these have default values, so does it make sense to make them required?\n\nBesides, I\u0027m not planning to support them until the scheduler can accept a list (i.e. [True, False]), since (for example) a given pool may be able to handle both compressed and uncompressed shares.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"b332dbe59e280ac1c71be781b00742bb744783bd","unresolved":false,"context_lines":[{"line_number":78,"context_line":"- storage_protocol: list of shared file system protocols supported by this"},{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_2bea2d10","line":81,"in_reply_to":"5a2ca52d_d380948d","updated":"2015-10-19 21:28:46.000000000","message":"The default values do impact behavior.  thin_provisioning is either True or False and the scheduler is using it when making share placement decisions.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":78,"context_line":"- storage_protocol: list of shared file system protocols supported by this"},{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_996f96fb","line":81,"in_reply_to":"5a2ca52d_d3f17496","updated":"2015-10-16 22:48:24.000000000","message":"We need to decide if reporting these wrong (eg default is not accurate) is just a bug or a reason to remove a driver.  I like having these documented, but I\u0027m not sure which are minimum reqs that would cause us to remove a driver.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_d91e6e6f","line":82,"updated":"2015-10-16 22:48:24.000000000","message":"For example, this is probably an easy one to enter a bug for drivers that don\u0027t do it.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":79,"context_line":"  driver instance;"},{"line_number":80,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_df6d9033","line":82,"in_reply_to":"5a2ca52d_d91e6e6f","updated":"2015-10-19 13:01:08.000000000","message":"The default is 0, if drivers do not override this value, why is it a bug?","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_6ca43607","line":84,"updated":"2015-10-16 22:48:24.000000000","message":"I don\u0027t think this is defined yet.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":81,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_ff864cb8","line":84,"in_reply_to":"5a2ca52d_6ca43607","updated":"2015-10-19 13:01:08.000000000","message":"Then I think it should be removed. I have not checked any core code to see if this is used, but it does not make much sense for the driver to override a value that nobody uses if not itself.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_ac856e5a","line":85,"updated":"2015-10-16 22:48:24.000000000","message":"pools is not required yet.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"b332dbe59e280ac1c71be781b00742bb744783bd","unresolved":false,"context_lines":[{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_cb3db9bc","line":85,"in_reply_to":"3a29b11f_49b0eaf4","updated":"2015-10-19 21:28:46.000000000","message":"pools defaults to None if not reported.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"b6f46f069debc519aacf72be3e6f5ba2363bde1d","unresolved":false,"context_lines":[{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_49b0eaf4","line":85,"in_reply_to":"5a2ca52d_ac856e5a","updated":"2015-10-19 14:28:45.000000000","message":"+1  I hadn\u0027t considered this, but requiring pools would let us clean up the mess in HostManager.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":82,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_bfa7744e","line":85,"in_reply_to":"5a2ca52d_ac856e5a","updated":"2015-10-19 13:01:08.000000000","message":"same as thin_provisioning? If we are going to include thin provisioning then this also should be included.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"b332dbe59e280ac1c71be781b00742bb744783bd","unresolved":false,"context_lines":[{"line_number":83,"context_line":"  used;"},{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"},{"line_number":89,"context_line":"--------------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_eb09952d","line":86,"updated":"2015-10-19 21:28:46.000000000","message":"Whether snapshot_support is True or False is figured out by driver.py automatically.  None of the drivers need to report this.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"},{"line_number":89,"context_line":"--------------------------"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_8c5af2f0","line":87,"updated":"2015-10-16 22:48:24.000000000","message":"So the \"Capabilities\" section here is a nice helpful description for driver developers, but maybe \"minimum reqs\" should be streamlined a bit.  I\u0027m not sure.  Maybe these could be reqs in M.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":84,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":85,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":86,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Managing/Unmanaging shares"},{"line_number":89,"context_line":"--------------------------"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_3f17e452","line":87,"in_reply_to":"5a2ca52d_8c5af2f0","updated":"2015-10-19 13:01:08.000000000","message":"by \"streamlined\" do you mean we should include less detail? I tried to cover most relevant items without getting in too much detail so both users and driver developers can understand. Driver developers can handle more detail, users may not, are we focusing on users by streamlining?","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":93,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":94,"context_line":"completely controlled by Manila and should not be handled externally anymore."},{"line_number":95,"context_line":"Additionally, a function that de-registers such shares from Manila but do"},{"line_number":96,"context_line":"not delete from backend must also be supported."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Continuous Integration systems"},{"line_number":99,"context_line":"------------------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_4cf85ac3","line":96,"updated":"2015-10-16 22:48:24.000000000","message":"Need so say propsed for Mitaka and then assuming we agree it would say Required since Mitaka.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":93,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":94,"context_line":"completely controlled by Manila and should not be handled externally anymore."},{"line_number":95,"context_line":"Additionally, a function that de-registers such shares from Manila but do"},{"line_number":96,"context_line":"not delete from backend must also be supported."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Continuous Integration systems"},{"line_number":99,"context_line":"------------------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_e213135b","line":96,"in_reply_to":"5a2ca52d_4cf85ac3","updated":"2015-10-19 13:01:08.000000000","message":"or current release.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":98,"context_line":"Continuous Integration systems"},{"line_number":99,"context_line":"------------------------------"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":102,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":103,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":104,"context_line":"patch submitted affects an existing driver. The CI system should run all"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_aada3ce6","line":101,"updated":"2015-10-16 17:47:17.000000000","message":"s/should/must/","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":98,"context_line":"Continuous Integration systems"},{"line_number":99,"context_line":"------------------------------"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":102,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":103,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":104,"context_line":"patch submitted affects an existing driver. The CI system should run all"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_5311e45b","line":101,"in_reply_to":"5a2ca52d_aada3ce6","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":101,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":102,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":103,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":104,"context_line":"patch submitted affects an existing driver. The CI system should run all"},{"line_number":105,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":106,"context_line":"test results."},{"line_number":107,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_6a2ae491","line":104,"updated":"2015-10-16 17:47:17.000000000","message":"s/should/must/","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":101,"context_line":"Every driver vendor should supply a CI system that tests its drivers"},{"line_number":102,"context_line":"continuously for each patch submitted to OpenStack gerrit. This allows for"},{"line_number":103,"context_line":"better QA and quicker response and notification for driver vendors when a"},{"line_number":104,"context_line":"patch submitted affects an existing driver. The CI system should run all"},{"line_number":105,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":106,"context_line":"test results."},{"line_number":107,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_b31e386e","line":104,"in_reply_to":"5a2ca52d_6a2ae491","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"32b25c264290eccf32b45ee3709553bf28bf34f8","unresolved":false,"context_lines":[{"line_number":105,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":106,"context_line":"test results."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":".. note:: for more information please see http://docs.openstack.org/infra/system-config/third_party.html"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_a5aeefe3","line":108,"updated":"2015-10-16 17:47:17.000000000","message":"I suggest adding a section describing our expectations for unit test coverage.  The tests must be located in-tree using a directory and filename structure that mirrors the functional code, they must utilize the mock framework (i.e. not mox), and code coverage should exceed 90%.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"b6f46f069debc519aacf72be3e6f5ba2363bde1d","unresolved":false,"context_lines":[{"line_number":105,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":106,"context_line":"test results."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":".. note:: for more information please see http://docs.openstack.org/infra/system-config/third_party.html"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_e9d1b6fc","line":108,"in_reply_to":"3a29b11f_82525f12","updated":"2015-10-19 14:28:45.000000000","message":"The new section looks good, Rodrigo, thanks.","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"1fddd5d7d49fb67dedc4ae628f0f32c96b6f3708","unresolved":false,"context_lines":[{"line_number":105,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":106,"context_line":"test results."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":".. note:: for more information please see http://docs.openstack.org/infra/system-config/third_party.html"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a29b11f_82525f12","line":108,"in_reply_to":"5a2ca52d_a5aeefe3","updated":"2015-10-19 13:01:08.000000000","message":"Done","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"7b28e83bde69eb010f01ca41b0360afb6f27270e","unresolved":false,"context_lines":[{"line_number":105,"context_line":"applicable tempest tests, test all patches Jenkins has posted +1 and post its"},{"line_number":106,"context_line":"test results."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":".. note:: for more information please see http://docs.openstack.org/infra/system-config/third_party.html"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_cca78abb","line":108,"in_reply_to":"5a2ca52d_a5aeefe3","updated":"2015-10-16 22:48:24.000000000","message":"Hmmm.  Clinton said \"should\".  I think he meant it though.  We can probably start with the \"must\" be in-tree part because that is non-controversial.  We could then propose a \"must\" coverage number but we should get agreement. For now it could be a guideline(?).","commit_id":"fdbbcd1a124976c2dacf04c8cace63c68ea103b6"},{"author":{"_account_id":11865,"name":"Clinton Knight","email":"cknight@netapp.com","username":"cknight"},"change_message_id":"4679f9bcfaeb4c8c3279951172d062b418718b0e","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"The driver must support at least one file system sharing protocol in order to"},{"line_number":39,"context_line":"serve shares as a shared file system service. The current list of supported"},{"line_number":40,"context_line":"protocols are as follows:"},{"line_number":41,"context_line":"- NFS"},{"line_number":42,"context_line":"- CIFS"},{"line_number":43,"context_line":"- ZFS"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a29b11f_2e125463","line":40,"updated":"2015-10-19 14:28:37.000000000","message":"s/are/is/","commit_id":"48e399ee3f765a6b89e1a2391ba0532b5a797a50"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"7e15ab271c801d4907dabb6c0fcb6010e26082de","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"The driver must support at least one file system sharing protocol in order to"},{"line_number":39,"context_line":"serve shares as a shared file system service. The current list of supported"},{"line_number":40,"context_line":"protocols are as follows:"},{"line_number":41,"context_line":"- NFS"},{"line_number":42,"context_line":"- CIFS"},{"line_number":43,"context_line":"- ZFS"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a29b11f_362377bf","line":40,"in_reply_to":"3a29b11f_2e125463","updated":"2015-10-19 19:14:53.000000000","message":"Done","commit_id":"48e399ee3f765a6b89e1a2391ba0532b5a797a50"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a14a5abd28dc9457b00ebbe8dd141b58033a3f93","unresolved":false,"context_lines":[{"line_number":14,"context_line":"      License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"      under the License."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Manila minimum requirements and features"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"In order for a driver to be accepted into Manila code base, there are certain"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_908ccc80","line":17,"updated":"2015-10-19 21:32:52.000000000","message":"Can you add \"since Mitaka\" as suggested by Mark?","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"068b8841e8f49a7a6fddc14607711e35092cf4b7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"      License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"      under the License."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Manila minimum requirements and features"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"In order for a driver to be accepted into Manila code base, there are certain"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_a32ac82a","line":17,"in_reply_to":"3a29b11f_908ccc80","updated":"2015-10-20 13:33:07.000000000","message":"Done","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"a6379617ee2088e8ec735d801e8cca9c4ae764be","unresolved":false,"context_lines":[{"line_number":30,"context_line":"administrator to manually configure all networks (:term:`DHSS` \u003d false) and be"},{"line_number":31,"context_line":"responsible for segregation, if that is desired. At least one driver mode must"},{"line_number":32,"context_line":"be supported. In :term:`DHSS` \u003d true mode, Share Server entities are used, so"},{"line_number":33,"context_line":"the driver must implement functions that setup and teardown such servers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"At least one file system sharing protocol"},{"line_number":36,"context_line":"-----------------------------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_11be74d4","line":33,"updated":"2015-10-20 04:08:21.000000000","message":"If \u0027term:`DHSS` \u003d true\u0027, the driver must implement function \u0027Get Network Allocations Number\u0027.","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"068b8841e8f49a7a6fddc14607711e35092cf4b7","unresolved":false,"context_lines":[{"line_number":30,"context_line":"administrator to manually configure all networks (:term:`DHSS` \u003d false) and be"},{"line_number":31,"context_line":"responsible for segregation, if that is desired. At least one driver mode must"},{"line_number":32,"context_line":"be supported. In :term:`DHSS` \u003d true mode, Share Server entities are used, so"},{"line_number":33,"context_line":"the driver must implement functions that setup and teardown such servers."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"At least one file system sharing protocol"},{"line_number":36,"context_line":"-----------------------------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_75cbfe72","line":33,"in_reply_to":"3a29b11f_11be74d4","updated":"2015-10-20 13:33:07.000000000","message":"Only if not using VM approach. I am not sure this is the document to get into such detail. If the driver vendor decides to support DHSS \u003d true then he is going to learn about this. Also, I mentioned that in DHSS \u003d true mode the driver is responsible for managing network resources, which is related to that.","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a14a5abd28dc9457b00ebbe8dd141b58033a3f93","unresolved":false,"context_lines":[{"line_number":44,"context_line":"- GPFS"},{"line_number":45,"context_line":"- GlusterFS"},{"line_number":46,"context_line":"- HDFS"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Access rules"},{"line_number":49,"context_line":"------------"},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_706d7097","line":47,"updated":"2015-10-19 21:32:52.000000000","message":"We should mention that a new driver can support a new file system sharing protocol not listed above.","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"068b8841e8f49a7a6fddc14607711e35092cf4b7","unresolved":false,"context_lines":[{"line_number":44,"context_line":"- GPFS"},{"line_number":45,"context_line":"- GlusterFS"},{"line_number":46,"context_line":"- HDFS"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Access rules"},{"line_number":49,"context_line":"------------"},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_43cc146a","line":47,"in_reply_to":"3a29b11f_706d7097","updated":"2015-10-20 13:33:07.000000000","message":"Done","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a14a5abd28dc9457b00ebbe8dd141b58033a3f93","unresolved":false,"context_lines":[{"line_number":87,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":88,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":89,"context_line":"  used;"},{"line_number":90,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":91,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":92,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_cbf5b9cb","line":90,"updated":"2015-10-19 21:32:52.000000000","message":"This is actually not used yet as we don\u0027t support QoS yet.  If QoS support is added in Mitaka, then we may start requiring this.","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"068b8841e8f49a7a6fddc14607711e35092cf4b7","unresolved":false,"context_lines":[{"line_number":87,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":88,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":89,"context_line":"  used;"},{"line_number":90,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":91,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":92,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":93,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_faf29d6e","line":90,"in_reply_to":"3a29b11f_cbf5b9cb","updated":"2015-10-20 13:33:07.000000000","message":"Done","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a14a5abd28dc9457b00ebbe8dd141b58033a3f93","unresolved":false,"context_lines":[{"line_number":89,"context_line":"  used;"},{"line_number":90,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":91,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":92,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Managing/Unmanaging shares"},{"line_number":95,"context_line":"--------------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_2b06adce","line":92,"updated":"2015-10-19 21:32:52.000000000","message":"Whether snapshot_support is True or False is figured out by driver.py automatically. None of the drivers need to report this.","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"068b8841e8f49a7a6fddc14607711e35092cf4b7","unresolved":false,"context_lines":[{"line_number":89,"context_line":"  used;"},{"line_number":90,"context_line":"- QoS_support: whether this driver instance has QoS support enabled;"},{"line_number":91,"context_line":"- pools: list of storage pools managed by this driver instance;"},{"line_number":92,"context_line":"- snapshot_support: whether this driver instance supports snapshots."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Managing/Unmanaging shares"},{"line_number":95,"context_line":"--------------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a29b11f_9ae92983","line":92,"in_reply_to":"3a29b11f_2b06adce","updated":"2015-10-20 13:33:07.000000000","message":"Done","commit_id":"b44b7f169817cd8efee89ad74128cee2fa746cfe"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"fc4de3ba04b1dd80cd416b85f4528bfcb6d4fa35","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Manila optional requirements and features since Mitaka"},{"line_number":134,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Additional to the minimum required features supported by Manila, other optional"},{"line_number":137,"context_line":"features can be supported by drivers as they are already supported in Manila"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a29b11f_0a038810","line":134,"updated":"2015-10-20 14:24:27.000000000","message":"We have another optional features in Manila, such as Consistency Group, Share Migration.\n\nhttps://wiki.openstack.org/wiki/Manila/design/manila-liberty-consistency-groups\n\nhttps://github.com/openstack/manila/commit/f5add61a8d62c5aacb8c3a6df3b4455ab17ea23f","commit_id":"eeed697f6a1aa38d82d656b866105943dd525d52"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"4b7373f6281e816a9d60e414b6f5cabae35f2a54","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Manila optional requirements and features since Mitaka"},{"line_number":134,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Additional to the minimum required features supported by Manila, other optional"},{"line_number":137,"context_line":"features can be supported by drivers as they are already supported in Manila"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a29b11f_9bba886d","line":134,"in_reply_to":"3a29b11f_0a038810","updated":"2015-10-20 15:00:12.000000000","message":"Both are still experimental. I am not sure if experimental features should be included here. Since this is the document that will be updated during Mitaka, this may change along this cycle.","commit_id":"eeed697f6a1aa38d82d656b866105943dd525d52"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"ac88f5f9e530ca126d43c18d8adc84472644a317","unresolved":false,"context_lines":[{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Manila optional requirements and features since Mitaka"},{"line_number":134,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"Additional to the minimum required features supported by Manila, other optional"},{"line_number":137,"context_line":"features can be supported by drivers as they are already supported in Manila"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a29b11f_b62759b5","line":134,"in_reply_to":"3a29b11f_9bba886d","updated":"2015-10-20 15:02:09.000000000","message":"There could be another section in this document for experimental features though. Not sure if everybody agrees with this.","commit_id":"eeed697f6a1aa38d82d656b866105943dd525d52"},{"author":{"_account_id":2417,"name":"Ben Swartzlander","email":"ben@swartzlander.org","username":"bswartz"},"change_message_id":"50f5950b38b533e88dc3fd48eaf9e33e9b2734fb","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":".. note:: for more information please see http://docs.openstack.org/developer/manila/devref/capabilities_and_extra_specs.html"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Managing/Unmanaging shares"},{"line_number":103,"context_line":"--------------------------"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"If :term:`DHSS` \u003d false mode is used, then drivers must implement a function"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a29b11f_1bcb3c20","line":102,"updated":"2015-10-23 17:06:48.000000000","message":"After discussing this feature during the meeting, I think we should move this section to optional to avoid confusion.\n\nWe\u0027ve always had the stance that drivers aren\u0027t required to implement anything for manage/unmanage. The contact for this feature says the manage is never required to succeed, so a no-op implementation technically satisfies this contract. However drivers SHOULD implement a way to do manage/unmanage that will succeed sometimes.","commit_id":"dff0d431d789e31e324fe6e6222361f125972c49"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"95b9c79a715daf0bc29cf6f8e1489ec52a2d11e5","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":".. note:: for more information please see http://docs.openstack.org/developer/manila/devref/capabilities_and_extra_specs.html"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Managing/Unmanaging shares"},{"line_number":103,"context_line":"--------------------------"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"If :term:`DHSS` \u003d false mode is used, then drivers must implement a function"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3a29b11f_60c929ac","line":102,"in_reply_to":"3a29b11f_1bcb3c20","updated":"2015-10-27 01:24:38.000000000","message":"Done","commit_id":"dff0d431d789e31e324fe6e6222361f125972c49"},{"author":{"_account_id":13484,"name":"jason bishop","email":"jason.bishop@gmail.com","username":"jason.bishop"},"change_message_id":"8f7272950b7c8f4bb3c27f7755a8d76fcc33b468","unresolved":false,"context_lines":[{"line_number":41,"context_line":"protocols is as follows:"},{"line_number":42,"context_line":"- NFS"},{"line_number":43,"context_line":"- CIFS"},{"line_number":44,"context_line":"- ZFS"},{"line_number":45,"context_line":"- GPFS"},{"line_number":46,"context_line":"- GlusterFS"},{"line_number":47,"context_line":"- HDFS"}],"source_content_type":"text/x-rst","patch_set":7,"id":"1a26ad4f_a5ce4eb2","line":44,"updated":"2015-10-30 01:20:59.000000000","message":"(ben points out in friday meetup) ZFS is not wire level protocol.  should be removed","commit_id":"8d5a07b0466ed6aacdc9778c8f77e361359afc64"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"859515724aa8cdeb71fb8ec31b0404a160b8d0b1","unresolved":false,"context_lines":[{"line_number":61,"context_line":"a driver must be able to create and delete shares. In some situations"},{"line_number":62,"context_line":"Manila attempts to ensure created shares, so the driver must implement a"},{"line_number":63,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":64,"context_line":"re-exporting."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Resizing"},{"line_number":67,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"1a26ad4f_fb638818","line":64,"updated":"2015-10-29 23:59:51.000000000","message":"This is still too vague for a \"must\".  We need a better definition of what ensure_share is expected to do if it is a must.","commit_id":"8d5a07b0466ed6aacdc9778c8f77e361359afc64"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"859515724aa8cdeb71fb8ec31b0404a160b8d0b1","unresolved":false,"context_lines":[{"line_number":87,"context_line":"- total_capacity_gb: total amount of storage space provided, in GB;"},{"line_number":88,"context_line":"- free_capacity_gb: amount of storage space available for use, in GB;"},{"line_number":89,"context_line":"- reserved_percentage: percentage of total storage space to be kept from being"},{"line_number":90,"context_line":"  used."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"Certain features, if supported by drivers, need to be reported in order to"},{"line_number":93,"context_line":"function correctly in Manila, such as:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"1a26ad4f_fb4aa882","line":90,"updated":"2015-10-29 23:59:51.000000000","message":"I think we need to verify that total, free and reserved are \"must\".  Ideally they would be, but we need to confirm.","commit_id":"8d5a07b0466ed6aacdc9778c8f77e361359afc64"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"c38538181e5213170270533206e695a74f84035b","unresolved":false,"context_lines":[{"line_number":140,"context_line":"Snapshots"},{"line_number":141,"context_line":"---------"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Share Snapshots allow for data respective to a particular point in time to be"},{"line_number":144,"context_line":"saved in order to be used later. In Manila API, share snapshots taken can only"},{"line_number":145,"context_line":"be restored by creating new shares from them, thus the original share remains"},{"line_number":146,"context_line":"unaffected."}],"source_content_type":"text/x-rst","patch_set":7,"id":"1a26ad4f_2548fe5a","line":143,"updated":"2015-10-30 01:17:03.000000000","message":"We should add here that Snapshots must be crash-consistent.","commit_id":"8d5a07b0466ed6aacdc9778c8f77e361359afc64"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"33f2eeb8ec158febcd35507e9e9ccc088240d50b","unresolved":false,"context_lines":[{"line_number":53,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":54,"context_line":"The authentication type should be based on IP, User and/or Certificate. For each"},{"line_number":55,"context_line":"access rule, the supported access levels must be read-write and read-only as"},{"line_number":56,"context_line":"allowed per protocol."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Shares"},{"line_number":59,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1a26ad4f_39aa6371","line":56,"updated":"2015-10-30 03:50:57.000000000","message":"From the summit, this needs to be documented better.","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"d1d5d319444f3babb62f9fb6767442997df39066","unresolved":false,"context_lines":[{"line_number":53,"context_line":"access is. Access rule operations include allowing access and denying access."},{"line_number":54,"context_line":"The authentication type should be based on IP, User and/or Certificate. For each"},{"line_number":55,"context_line":"access rule, the supported access levels must be read-write and read-only as"},{"line_number":56,"context_line":"allowed per protocol."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Shares"},{"line_number":59,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa80f949_91c9e17a","line":56,"in_reply_to":"1a26ad4f_39aa6371","updated":"2015-11-05 19:48:55.000000000","message":"Done, according to weekly meeting decision.","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"33f2eeb8ec158febcd35507e9e9ccc088240d50b","unresolved":false,"context_lines":[{"line_number":62,"context_line":"a driver must be able to create and delete shares. In some situations"},{"line_number":63,"context_line":"Manila attempts to ensure created shares, so the driver can implement a"},{"line_number":64,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":65,"context_line":"re-exporting."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Resizing"},{"line_number":68,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1a26ad4f_f9baabbf","line":65,"updated":"2015-10-30 03:50:57.000000000","message":"The \"ensure\" part should go to the optional section. Then it would be nice to explain the intention better there, but not as critical.","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"d1d5d319444f3babb62f9fb6767442997df39066","unresolved":false,"context_lines":[{"line_number":62,"context_line":"a driver must be able to create and delete shares. In some situations"},{"line_number":63,"context_line":"Manila attempts to ensure created shares, so the driver can implement a"},{"line_number":64,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":65,"context_line":"re-exporting."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Resizing"},{"line_number":68,"context_line":"--------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa80f949_8fba316d","line":65,"in_reply_to":"1a26ad4f_f9baabbf","updated":"2015-11-05 19:48:55.000000000","message":"Done","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"b980e07aaf40fc5f483dba4a7a3f3d768c7b9f40","unresolved":false,"context_lines":[{"line_number":64,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":65,"context_line":"re-exporting."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Resizing"},{"line_number":68,"context_line":"--------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa80f949_bb850d08","line":67,"updated":"2015-11-04 23:24:34.000000000","message":"Can you change this title to \"Extend share\"?  \"Resizing\" can go both ways.","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"d1d5d319444f3babb62f9fb6767442997df39066","unresolved":false,"context_lines":[{"line_number":64,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":65,"context_line":"re-exporting."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Resizing"},{"line_number":68,"context_line":"--------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa80f949_ef4addc4","line":67,"in_reply_to":"fa80f949_9392659e","updated":"2015-11-05 19:48:55.000000000","message":"Done","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":13915,"name":"Silvan Kaiser","email":"silvan@quobyte.com","username":"kaisers"},"change_message_id":"d2dfd1ef76ee8e03ea1396264431db7603a17f22","unresolved":false,"context_lines":[{"line_number":64,"context_line":"function that checks shares\u0027 status and perform maintenance if needed, such as"},{"line_number":65,"context_line":"re-exporting."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"Resizing"},{"line_number":68,"context_line":"--------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"In order to best satisfy cloud service requirements, shares must be elastic, so"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa80f949_9392659e","line":67,"in_reply_to":"fa80f949_bb850d08","updated":"2015-11-05 08:20:42.000000000","message":"+1\nThere\u0027s a explicit shrink share section down below, so it seems good to rename this to extend share...","commit_id":"bf38c1f72798e1a516f3f284cd0594afc83bb4f1"},{"author":{"_account_id":14863,"name":"Diego Zamboni Latance","email":"dzambonil@gmail.com","username":"DiegoZamboni"},"change_message_id":"2911aa63c5070765d6331b62088c347c0a882d87","unresolved":false,"context_lines":[{"line_number":54,"context_line":"to a given share. The authentication type should be based on IP, User and/or"},{"line_number":55,"context_line":"Certificate. Drivers must support read-write and read-only access levels for each"},{"line_number":56,"context_line":"supported protocol, either through individual access rules or separate export"},{"line_number":57,"context_line":"locations. "},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Shares"},{"line_number":60,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa80f949_9ea010a1","line":57,"updated":"2015-11-05 19:00:30.000000000","message":"Trailing space","commit_id":"517f9edffcbf055f55a7629d21dfdf915f705d35"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"0dcf5a49776f13713330e2886e2a1e34f00548bd","unresolved":false,"context_lines":[{"line_number":54,"context_line":"to a given share. The authentication type should be based on IP, User and/or"},{"line_number":55,"context_line":"Certificate. Drivers must support read-write and read-only access levels for each"},{"line_number":56,"context_line":"supported protocol, either through individual access rules or separate export"},{"line_number":57,"context_line":"locations. "},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Shares"},{"line_number":60,"context_line":"------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa80f949_f46e1b95","line":57,"in_reply_to":"fa80f949_9ea010a1","updated":"2015-11-06 01:00:14.000000000","message":"Done","commit_id":"517f9edffcbf055f55a7629d21dfdf915f705d35"},{"author":{"_account_id":14863,"name":"Diego Zamboni Latance","email":"dzambonil@gmail.com","username":"DiegoZamboni"},"change_message_id":"2911aa63c5070765d6331b62088c347c0a882d87","unresolved":false,"context_lines":[{"line_number":166,"context_line":"--------------"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"In some situations, such as when the driver is restarted, Manila attempts to"},{"line_number":169,"context_line":"perform maintenance on created shares, on the purpose to ensure previously"},{"line_number":170,"context_line":"created shares are available and being serviced correctly. The driver can"},{"line_number":171,"context_line":"implement this function by checking shares\u0027 status and perform maintenance"},{"line_number":172,"context_line":"operations if needed, such as re-exporting."}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa80f949_fe9fdc4d","line":169,"updated":"2015-11-05 19:00:30.000000000","message":"s/to ensure/of ensuring","commit_id":"517f9edffcbf055f55a7629d21dfdf915f705d35"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"0dcf5a49776f13713330e2886e2a1e34f00548bd","unresolved":false,"context_lines":[{"line_number":166,"context_line":"--------------"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"In some situations, such as when the driver is restarted, Manila attempts to"},{"line_number":169,"context_line":"perform maintenance on created shares, on the purpose to ensure previously"},{"line_number":170,"context_line":"created shares are available and being serviced correctly. The driver can"},{"line_number":171,"context_line":"implement this function by checking shares\u0027 status and perform maintenance"},{"line_number":172,"context_line":"operations if needed, such as re-exporting."}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa80f949_b173c514","line":169,"in_reply_to":"fa80f949_fe9fdc4d","updated":"2015-11-06 01:00:14.000000000","message":"Done","commit_id":"517f9edffcbf055f55a7629d21dfdf915f705d35"},{"author":{"_account_id":14863,"name":"Diego Zamboni Latance","email":"dzambonil@gmail.com","username":"DiegoZamboni"},"change_message_id":"2911aa63c5070765d6331b62088c347c0a882d87","unresolved":false,"context_lines":[{"line_number":168,"context_line":"In some situations, such as when the driver is restarted, Manila attempts to"},{"line_number":169,"context_line":"perform maintenance on created shares, on the purpose to ensure previously"},{"line_number":170,"context_line":"created shares are available and being serviced correctly. The driver can"},{"line_number":171,"context_line":"implement this function by checking shares\u0027 status and perform maintenance"},{"line_number":172,"context_line":"operations if needed, such as re-exporting."},{"line_number":173,"context_line":""},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa80f949_bec0742b","line":171,"updated":"2015-11-05 19:00:30.000000000","message":"s/perform/performing","commit_id":"517f9edffcbf055f55a7629d21dfdf915f705d35"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"0dcf5a49776f13713330e2886e2a1e34f00548bd","unresolved":false,"context_lines":[{"line_number":168,"context_line":"In some situations, such as when the driver is restarted, Manila attempts to"},{"line_number":169,"context_line":"perform maintenance on created shares, on the purpose to ensure previously"},{"line_number":170,"context_line":"created shares are available and being serviced correctly. The driver can"},{"line_number":171,"context_line":"implement this function by checking shares\u0027 status and perform maintenance"},{"line_number":172,"context_line":"operations if needed, such as re-exporting."},{"line_number":173,"context_line":""},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa80f949_f6e938e7","line":171,"in_reply_to":"fa80f949_bec0742b","updated":"2015-11-06 01:00:14.000000000","message":"Done","commit_id":"517f9edffcbf055f55a7629d21dfdf915f705d35"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"6f750f7706daf0b16a0d932acc24a4e1b3eed482","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"- NFS"},{"line_number":44,"context_line":"- CIFS"},{"line_number":45,"context_line":"- GPFS"},{"line_number":46,"context_line":"- GlusterFS"},{"line_number":47,"context_line":"- HDFS"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_4411155c","line":45,"updated":"2015-11-12 10:15:34.000000000","message":"We do not have possibility to use \"GPFS\" native protocol in Manila API.","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"379e7da148f1a73504ce19e1bd0d07f8340a747c","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"- NFS"},{"line_number":44,"context_line":"- CIFS"},{"line_number":45,"context_line":"- GPFS"},{"line_number":46,"context_line":"- GlusterFS"},{"line_number":47,"context_line":"- HDFS"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_158819c4","line":45,"in_reply_to":"da85f559_4411155c","updated":"2015-11-12 16:16:49.000000000","message":"Done","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"6f750f7706daf0b16a0d932acc24a4e1b3eed482","unresolved":false,"context_lines":[{"line_number":116,"context_line":"All drivers submitted must be contemplated with unit tests covering at least"},{"line_number":117,"context_line":"90% of the code, preferably 100% if possible. Unit tests must use mock"},{"line_number":118,"context_line":"framework and be located in-tree using a structure that mirrors the functional"},{"line_number":119,"context_line":"code, such as directory names and filenames."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"Documentation"},{"line_number":122,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_3f409c06","line":119,"updated":"2015-11-12 10:15:34.000000000","message":"Would be great to add template as folllowing:\n\nmanila/[tests/]path/to/brand/new/[test_]driver.py","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"379e7da148f1a73504ce19e1bd0d07f8340a747c","unresolved":false,"context_lines":[{"line_number":116,"context_line":"All drivers submitted must be contemplated with unit tests covering at least"},{"line_number":117,"context_line":"90% of the code, preferably 100% if possible. Unit tests must use mock"},{"line_number":118,"context_line":"framework and be located in-tree using a structure that mirrors the functional"},{"line_number":119,"context_line":"code, such as directory names and filenames."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"Documentation"},{"line_number":122,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_72b3a498","line":119,"in_reply_to":"da85f559_3f409c06","updated":"2015-11-12 16:16:49.000000000","message":"Done","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"9e541e0d776d90dbae8134da1b77e2d6970c8f41","unresolved":false,"context_lines":[{"line_number":148,"context_line":"Managing/Unmanaging shares"},{"line_number":149,"context_line":"--------------------------"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"If :term:`DHSS` \u003d false mode is used, then drivers should implement a function"},{"line_number":152,"context_line":"that supports reading existing shares in the backend that were not created by"},{"line_number":153,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":154,"context_line":"completely controlled by Manila and should not be handled externally anymore."}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_fac67815","line":151,"updated":"2015-11-07 04:14:58.000000000","message":"If you upload another patch, can you change the word \"should\" to \"can\"?  \"should\" sounds like it is required.  This is under the optional requirements section, so I won\u0027t block it because of this.","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"6f750f7706daf0b16a0d932acc24a4e1b3eed482","unresolved":false,"context_lines":[{"line_number":148,"context_line":"Managing/Unmanaging shares"},{"line_number":149,"context_line":"--------------------------"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"If :term:`DHSS` \u003d false mode is used, then drivers should implement a function"},{"line_number":152,"context_line":"that supports reading existing shares in the backend that were not created by"},{"line_number":153,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":154,"context_line":"completely controlled by Manila and should not be handled externally anymore."}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_ff98d479","line":151,"in_reply_to":"da85f559_fac67815","updated":"2015-11-12 10:15:34.000000000","message":"I would say \"\u0027manage\u0027 is required for driver that implements DHSS\u003dFalse mode\".","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"379e7da148f1a73504ce19e1bd0d07f8340a747c","unresolved":false,"context_lines":[{"line_number":148,"context_line":"Managing/Unmanaging shares"},{"line_number":149,"context_line":"--------------------------"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"If :term:`DHSS` \u003d false mode is used, then drivers should implement a function"},{"line_number":152,"context_line":"that supports reading existing shares in the backend that were not created by"},{"line_number":153,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":154,"context_line":"completely controlled by Manila and should not be handled externally anymore."}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_daecfebc","line":151,"in_reply_to":"da85f559_ff98d479","updated":"2015-11-12 16:16:49.000000000","message":"We decided in a meeting that manage implementation is not required even for DHSS\u003dFalse.","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"9e541e0d776d90dbae8134da1b77e2d6970c8f41","unresolved":false,"context_lines":[{"line_number":153,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":154,"context_line":"completely controlled by Manila and should not be handled externally anymore."},{"line_number":155,"context_line":"Additionally, a function that de-registers such shares from Manila but do"},{"line_number":156,"context_line":"not delete from backend must also be supported."},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"Share shrinking"},{"line_number":159,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_7adb68ee","line":156,"updated":"2015-11-07 04:14:58.000000000","message":"Can you change \"must\" to \"can\" or change the sentence to \"…must also be supported if manage share is supported\"?","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"379e7da148f1a73504ce19e1bd0d07f8340a747c","unresolved":false,"context_lines":[{"line_number":153,"context_line":"Manila. After the previously existing share is registered in Manila, it is"},{"line_number":154,"context_line":"completely controlled by Manila and should not be handled externally anymore."},{"line_number":155,"context_line":"Additionally, a function that de-registers such shares from Manila but do"},{"line_number":156,"context_line":"not delete from backend must also be supported."},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"Share shrinking"},{"line_number":159,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":11,"id":"da85f559_32f0bcc8","line":156,"in_reply_to":"da85f559_7adb68ee","updated":"2015-11-12 16:16:49.000000000","message":"Done","commit_id":"84a1b1bd9dc562ee4671bad81a29807460251279"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"c82aeee76fd203824647b514d18d850f4e9d914a","unresolved":false,"context_lines":[{"line_number":117,"context_line":"framework and be located in-tree using a structure that mirrors the functional"},{"line_number":118,"context_line":"code, such as directory names and filenames. See template below:"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":".. note:: manila/[tests/]path/to/brand/new/[test_]driver.py"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"Documentation"},{"line_number":123,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":12,"id":"da85f559_226a781b","line":120,"updated":"2015-11-13 13:30:09.000000000","message":"This looks very strange in compiled variant. You use \"note\" for wrong case. You want just to hightlight template, but \"note\" is not correct thing to use for it.\n\nShould be something like following:\n\n::\n\n    manila/[tests/]path/to/brand/new/[test_]driver.py","commit_id":"998dec784c33102389b7eeade26649d44f249e8f"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"925e85a7e3773cf7a9c7b7a526407b0046acfc39","unresolved":false,"context_lines":[{"line_number":117,"context_line":"framework and be located in-tree using a structure that mirrors the functional"},{"line_number":118,"context_line":"code, such as directory names and filenames. See template below:"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":".. note:: manila/[tests/]path/to/brand/new/[test_]driver.py"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"Documentation"},{"line_number":123,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":12,"id":"ba8a016a_c61f1b5a","line":120,"in_reply_to":"da85f559_226a781b","updated":"2015-11-16 13:28:19.000000000","message":"Done","commit_id":"998dec784c33102389b7eeade26649d44f249e8f"}]}