)]}'
{"specs/juno/vmware-spbm-support.rst":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"78f24d8a99ba1f8513e081e98b72e604db2d384e","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Enable administrators and tenants to take advantage of backend storage"},{"line_number":20,"context_line":"policies."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F9rjI%3D","line":19,"updated":"2014-04-02 20:55:17.000000000","message":"Can you explain what a backend sotrage policy is.","commit_id":"3e524f55814cba53d69c99172b6e9f48ceac9758"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a37b9f372e6aa915aa7a1d972609b18983f54843","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Enable administrators and tenants to take advantage of backend storage"},{"line_number":20,"context_line":"policies."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F9WBk%3D","line":19,"in_reply_to":"AAAAWH%2F9rjI%3D","updated":"2014-04-06 11:07:34.000000000","message":"Done","commit_id":"3e524f55814cba53d69c99172b6e9f48ceac9758"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"78f24d8a99ba1f8513e081e98b72e604db2d384e","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Proposed change"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"In order for Nova to provide SPBM we will need to address the following:"},{"line_number":26,"context_line":"1. Enabling the tenant to make use of storage policies. The goal here"},{"line_number":27,"context_line":"will be to provide the administrator with the necessary tools to"},{"line_number":28,"context_line":"provide differential storage services to the tenant. More specifically"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F9riw%3D","line":25,"updated":"2014-04-02 20:55:17.000000000","message":"I assume SPBM is Storage Policy Based Management. You should clearly explain that. also don\u0027t know what they are.","commit_id":"3e524f55814cba53d69c99172b6e9f48ceac9758"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a37b9f372e6aa915aa7a1d972609b18983f54843","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Proposed change"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"In order for Nova to provide SPBM we will need to address the following:"},{"line_number":26,"context_line":"1. Enabling the tenant to make use of storage policies. The goal here"},{"line_number":27,"context_line":"will be to provide the administrator with the necessary tools to"},{"line_number":28,"context_line":"provide differential storage services to the tenant. More specifically"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F9WA8%3D","line":25,"in_reply_to":"AAAAWH%2F9riw%3D","updated":"2014-04-06 11:07:34.000000000","message":"Done","commit_id":"3e524f55814cba53d69c99172b6e9f48ceac9758"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"78f24d8a99ba1f8513e081e98b72e604db2d384e","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"In order for Nova to provide SPBM we will need to address the following:"},{"line_number":26,"context_line":"1. Enabling the tenant to make use of storage policies. The goal here"},{"line_number":27,"context_line":"will be to provide the administrator with the necessary tools to"},{"line_number":28,"context_line":"provide differential storage services to the tenant. More specifically"},{"line_number":29,"context_line":"the administrator will be able to leverage capabilities provided by the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F9riU%3D","line":26,"updated":"2014-04-02 20:55:17.000000000","message":"The formatting here is not right:\n\nhttp://docs-draft.openstack.org/52/84652/1/check/gate-nova-specs-docs/d047a75/doc/build/html/specs/juno/vmware-spbm-support.html","commit_id":"3e524f55814cba53d69c99172b6e9f48ceac9758"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a37b9f372e6aa915aa7a1d972609b18983f54843","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"In order for Nova to provide SPBM we will need to address the following:"},{"line_number":26,"context_line":"1. Enabling the tenant to make use of storage policies. The goal here"},{"line_number":27,"context_line":"will be to provide the administrator with the necessary tools to"},{"line_number":28,"context_line":"provide differential storage services to the tenant. More specifically"},{"line_number":29,"context_line":"the administrator will be able to leverage capabilities provided by the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"AAAAWH%2F9WAU%3D","line":26,"in_reply_to":"AAAAWH%2F9riU%3D","updated":"2014-04-06 11:07:34.000000000","message":"Done","commit_id":"3e524f55814cba53d69c99172b6e9f48ceac9758"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c6b15103d053440931d66786af0bef3dcac4c08","unresolved":false,"context_lines":[{"line_number":26,"context_line":"The disk(s) of the virtual machine will be placed on the storage that"},{"line_number":27,"context_line":"matches the storage policy. This can for example provide preferential"},{"line_number":28,"context_line":"services to the user."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Proposed change"},{"line_number":31,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"AAAAW3%2F%2F4t4%3D","line":29,"updated":"2014-04-09 17:33:13.000000000","message":"How does this affect the user? Is it just a different flavor? Is this stuff like fast vs faster, super persistent vs cheap, etc?","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"de547900c082102e47812760e7dbed858b9f7907","unresolved":false,"context_lines":[{"line_number":26,"context_line":"The disk(s) of the virtual machine will be placed on the storage that"},{"line_number":27,"context_line":"matches the storage policy. This can for example provide preferential"},{"line_number":28,"context_line":"services to the user."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Proposed change"},{"line_number":31,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bad4c104_d17ba327","line":29,"in_reply_to":"AAAAW3%2F%2F4t4%3D","updated":"2014-04-29 13:38:21.000000000","message":"Done","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c6b15103d053440931d66786af0bef3dcac4c08","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Driver support for the storage policies."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"  * This entails using the information passed by the tenant to the driver."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Alternatives"},{"line_number":49,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"AAAAW3%2F%2F4tw%3D","line":46,"updated":"2014-04-09 17:33:13.000000000","message":"I feel we need to cover more of the how here. you mention extra specs below, maybe walk through some examples?","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"de547900c082102e47812760e7dbed858b9f7907","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* Driver support for the storage policies."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"  * This entails using the information passed by the tenant to the driver."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Alternatives"},{"line_number":49,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bad4c104_f1698726","line":46,"in_reply_to":"AAAAW3%2F%2F4tw%3D","updated":"2014-04-29 13:38:21.000000000","message":"This is discussed below in more detail","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c6b15103d053440931d66786af0bef3dcac4c08","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"At the moment there is no way that a administrator can provide differenttial"},{"line_number":52,"context_line":"storage services to a tenant."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Data model impact"},{"line_number":55,"context_line":"-----------------"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"AAAAW3%2F%2F4tY%3D","line":53,"updated":"2014-04-09 17:33:13.000000000","message":"right, but if you don\u0027t use flavors, you could do image properties, but then billing is mental, etc...","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"de547900c082102e47812760e7dbed858b9f7907","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"At the moment there is no way that a administrator can provide differenttial"},{"line_number":52,"context_line":"storage services to a tenant."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Data model impact"},{"line_number":55,"context_line":"-----------------"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bad4c104_51eeb38c","line":53,"in_reply_to":"AAAAW3%2F%2F4tY%3D","updated":"2014-04-29 13:38:21.000000000","message":"i do not think that image metadata is correct here - it will mean loading the same image n times each with different metadata. that will be very costly with storage for the glance images and waist of space","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c6b15103d053440931d66786af0bef3dcac4c08","unresolved":false,"context_lines":[{"line_number":103,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"Detailed design can be found at:"},{"line_number":106,"context_line":"https://docs.google.com/document/d/14Fr76WsFxBPfQJHRdy389IxlxZHXq-Kr83PeCXgDP1M/edit"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Assignee(s)"},{"line_number":109,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"AAAAW3%2F%2F4s0%3D","line":106,"updated":"2014-04-09 17:33:13.000000000","message":"This spec should stand on its own, but feel free to reference external \"discussion\" in the reference.","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"de547900c082102e47812760e7dbed858b9f7907","unresolved":false,"context_lines":[{"line_number":103,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"Detailed design can be found at:"},{"line_number":106,"context_line":"https://docs.google.com/document/d/14Fr76WsFxBPfQJHRdy389IxlxZHXq-Kr83PeCXgDP1M/edit"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Assignee(s)"},{"line_number":109,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bad4c104_0c4b345a","line":106,"in_reply_to":"AAAAW3%2F%2F4s0%3D","updated":"2014-04-29 13:38:21.000000000","message":"Done","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c6b15103d053440931d66786af0bef3dcac4c08","unresolved":false,"context_lines":[{"line_number":116,"context_line":"----------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"Code was posted in the Icehouse cycle:"},{"line_number":119,"context_line":"* SPBM support: https://review.openstack.org/#/c/66666"},{"line_number":120,"context_line":"* Add support for default pbm policy: https://review.openstack.org/#/c/68131"},{"line_number":121,"context_line":"* Get storage policy from flavor: https://review.openstack.org/#/c/68421"},{"line_number":122,"context_line":"* Use storage policy in datastore selection: https://review.openstack.org/#/c/68539"}],"source_content_type":"text/x-rst","patch_set":3,"id":"AAAAW3%2F%2F4so%3D","line":119,"updated":"2014-04-09 17:33:13.000000000","message":"Please no review links in specs, these may change as you split patches.\n\nA general overview seems fine here.","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"de547900c082102e47812760e7dbed858b9f7907","unresolved":false,"context_lines":[{"line_number":116,"context_line":"----------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"Code was posted in the Icehouse cycle:"},{"line_number":119,"context_line":"* SPBM support: https://review.openstack.org/#/c/66666"},{"line_number":120,"context_line":"* Add support for default pbm policy: https://review.openstack.org/#/c/68131"},{"line_number":121,"context_line":"* Get storage policy from flavor: https://review.openstack.org/#/c/68421"},{"line_number":122,"context_line":"* Use storage policy in datastore selection: https://review.openstack.org/#/c/68539"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bad4c104_2c48f85a","line":119,"in_reply_to":"AAAAW3%2F%2F4so%3D","updated":"2014-04-29 13:38:21.000000000","message":"Done","commit_id":"08f71670c9373f1b1e3deae5eae0176d3253ba68"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"4383da744d42a0c06e28f1e46fada5e355b899a6","unresolved":false,"context_lines":[{"line_number":26,"context_line":"The disk(s) of the virtual machine will be placed on the storage that"},{"line_number":27,"context_line":"matches the storage policy. This can for example provide preferential"},{"line_number":28,"context_line":"services to the user. For example the user will have an option of"},{"line_number":29,"context_line":"selecting \u0027gold\u0027, \u0027silver\u0027 or \u0027bronze\u0027 stoarge. \u0027gold\u0027 can be for"},{"line_number":30,"context_line":"applications that require fast and reliable results. \u0027bronze\u0027 can be"},{"line_number":31,"context_line":"for a background VM running in the evening doing maintenance."},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"7adec928_fd156349","line":29,"updated":"2014-05-15 12:05:43.000000000","message":"typo: storage","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"176b2864c595da40d906b21afff2beafa777d489","unresolved":false,"context_lines":[{"line_number":26,"context_line":"The disk(s) of the virtual machine will be placed on the storage that"},{"line_number":27,"context_line":"matches the storage policy. This can for example provide preferential"},{"line_number":28,"context_line":"services to the user. For example the user will have an option of"},{"line_number":29,"context_line":"selecting \u0027gold\u0027, \u0027silver\u0027 or \u0027bronze\u0027 stoarge. \u0027gold\u0027 can be for"},{"line_number":30,"context_line":"applications that require fast and reliable results. \u0027bronze\u0027 can be"},{"line_number":31,"context_line":"for a background VM running in the evening doing maintenance."},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_bfc3b856","line":29,"in_reply_to":"1ae5cdf2_e681464e","updated":"2014-06-26 16:59:36.000000000","message":"Done","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"291ccadb0714296a4e685fa116685fdc66fddb3c","unresolved":false,"context_lines":[{"line_number":26,"context_line":"The disk(s) of the virtual machine will be placed on the storage that"},{"line_number":27,"context_line":"matches the storage policy. This can for example provide preferential"},{"line_number":28,"context_line":"services to the user. For example the user will have an option of"},{"line_number":29,"context_line":"selecting \u0027gold\u0027, \u0027silver\u0027 or \u0027bronze\u0027 stoarge. \u0027gold\u0027 can be for"},{"line_number":30,"context_line":"applications that require fast and reliable results. \u0027bronze\u0027 can be"},{"line_number":31,"context_line":"for a background VM running in the evening doing maintenance."},{"line_number":32,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_e681464e","line":29,"in_reply_to":"7adec928_fd156349","updated":"2014-06-26 09:45:17.000000000","message":"+1","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"4383da744d42a0c06e28f1e46fada5e355b899a6","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"The spawn method currently selects the ‘best’ datastore to use. The"},{"line_number":34,"context_line":"administrator is able to select one or more datastores for selection"},{"line_number":35,"context_line":"by configuring a datastore regular expression. This logic will not"},{"line_number":36,"context_line":"be required if the instance flavor contain extra spec information"},{"line_number":37,"context_line":"that is relevant to the SPBM. That is, the SPBM information will be"},{"line_number":38,"context_line":"used for the datastore selection."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7adec928_284dc73e","line":35,"updated":"2014-05-15 12:05:43.000000000","message":"are we going to silently ignore the regex if spbm policy is defined or we will throw an error in that case? you can elaborate on this in the implementation section","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"176b2864c595da40d906b21afff2beafa777d489","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"The spawn method currently selects the ‘best’ datastore to use. The"},{"line_number":34,"context_line":"administrator is able to select one or more datastores for selection"},{"line_number":35,"context_line":"by configuring a datastore regular expression. This logic will not"},{"line_number":36,"context_line":"be required if the instance flavor contain extra spec information"},{"line_number":37,"context_line":"that is relevant to the SPBM. That is, the SPBM information will be"},{"line_number":38,"context_line":"used for the datastore selection."}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_3a0ed63d","line":35,"in_reply_to":"7adec928_284dc73e","updated":"2014-06-26 16:59:36.000000000","message":"if the regex is set then we ignore it.","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"291ccadb0714296a4e685fa116685fdc66fddb3c","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"The driver will need to make use of a different endpoint to access the storage"},{"line_number":61,"context_line":"policies on the VC. This will require a new configuration variable, that is,"},{"line_number":62,"context_line":"the PBM WSDL location will need to be defined."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_66ae16e9","line":62,"updated":"2014-06-26 09:45:17.000000000","message":"Please talk about the use cases here, how does the end user request the faster storage?\n\nWhy is this not just using flavor extra-specs. If it does use flavor, what keys and values do you plan to use to describe the storage?\n\nHow do you switch between them, would \"resize\" do that?","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"176b2864c595da40d906b21afff2beafa777d489","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"The driver will need to make use of a different endpoint to access the storage"},{"line_number":61,"context_line":"policies on the VC. This will require a new configuration variable, that is,"},{"line_number":62,"context_line":"the PBM WSDL location will need to be defined."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_3836d9b4","line":62,"in_reply_to":"1ae5cdf2_66ae16e9","updated":"2014-06-26 16:59:36.000000000","message":"The admin will assign this to the tenant. That is, they will provide a tenant with a gold, silver and bronze flavor. The tenant will be billed accordingly.\n\nNot sure I understand the part about resize. If they do a resize it will be on the same storage.","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"4383da744d42a0c06e28f1e46fada5e355b899a6","unresolved":false,"context_lines":[{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"At the moment there is no way that a administrator can provide differenttial"},{"line_number":68,"context_line":"storage services to a tenant."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Data model impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7adec928_48121358","line":67,"updated":"2014-05-15 12:05:43.000000000","message":"typo: differential","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"176b2864c595da40d906b21afff2beafa777d489","unresolved":false,"context_lines":[{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"At the moment there is no way that a administrator can provide differenttial"},{"line_number":68,"context_line":"storage services to a tenant."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Data model impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_95cdfd5f","line":67,"in_reply_to":"7adec928_48121358","updated":"2014-06-26 16:59:36.000000000","message":"Done","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"291ccadb0714296a4e685fa116685fdc66fddb3c","unresolved":false,"context_lines":[{"line_number":92,"context_line":"Other end user impact"},{"line_number":93,"context_line":"---------------------"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"None"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Performance Impact"},{"line_number":98,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_c6b1aa45","line":95,"updated":"2014-06-26 09:45:17.000000000","message":"then how does the user chose this feature?","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"176b2864c595da40d906b21afff2beafa777d489","unresolved":false,"context_lines":[{"line_number":92,"context_line":"Other end user impact"},{"line_number":93,"context_line":"---------------------"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"None"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Performance Impact"},{"line_number":98,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_f84f1133","line":95,"in_reply_to":"1ae5cdf2_c6b1aa45","updated":"2014-06-26 16:59:36.000000000","message":"As mentioned above. I added a few lines of this to the spec","commit_id":"d5b605e637e986064daff5b9be374d5144f48dda"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"f5e5087fbdb6e6658b47abc282c8a597a44c5926","unresolved":false,"context_lines":[{"line_number":92,"context_line":"Other end user impact"},{"line_number":93,"context_line":"---------------------"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"None"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"The cloud provider will provide a flavor to the tenant that will enable them"},{"line_number":98,"context_line":"to have preferential storage capabilities."}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_ea700475","line":95,"updated":"2014-06-27 07:08:01.000000000","message":"delete this line","commit_id":"3a72605dad2a61df578528ba171919f70b74dd85"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"f5e5087fbdb6e6658b47abc282c8a597a44c5926","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"There are 2 new configuration vaiables (both in the vmware section):"},{"line_number":109,"context_line":"* pbm_wsdl_location - PBM service WSDL file location URL. e.g."},{"line_number":110,"context_line":"file:///opt/SDK/spbm/wsdl/pbmService.wsdl Not setting this"},{"line_number":111,"context_line":"will disable storage policy based placement of instances."},{"line_number":112,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_wsdl_location"},{"line_number":113,"context_line":"is set and there is no defined storage policy for the specific request"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_eae5640a","line":110,"updated":"2014-06-27 07:08:01.000000000","message":"There is a proposal to include the PBM WSDL file in oslo.vmware:\n\nhttps://review.openstack.org/#/c/102409\n\nwhich make sense to me because users won\u0027t be supposed to download this file from the SDK every time. I suggest to have three config properties for Nova:\n\n  pbm_enabled\n  pbm_wsdl_location\n  pbm_default_policy\n\npbm_wsdl_location will be optional and if set, it will override the WSDL file from oslo.vmware. What do you think?","commit_id":"3a72605dad2a61df578528ba171919f70b74dd85"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8fc5533ac6e1d3851342dfcd74b698092c42bde5","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"There are 2 new configuration vaiables (both in the vmware section):"},{"line_number":109,"context_line":"* pbm_wsdl_location - PBM service WSDL file location URL. e.g."},{"line_number":110,"context_line":"file:///opt/SDK/spbm/wsdl/pbmService.wsdl Not setting this"},{"line_number":111,"context_line":"will disable storage policy based placement of instances."},{"line_number":112,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_wsdl_location"},{"line_number":113,"context_line":"is set and there is no defined storage policy for the specific request"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_ef0e4e31","line":110,"in_reply_to":"1ae5cdf2_eae5640a","updated":"2014-06-28 07:43:06.000000000","message":"Done","commit_id":"3a72605dad2a61df578528ba171919f70b74dd85"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"c0b97ad10d59bfb82c28d04dd848921e0d208230","unresolved":false,"context_lines":[{"line_number":103,"context_line":"Other deployer impact"},{"line_number":104,"context_line":"---------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"There are 3 new configuration vaiables (both in the vmware section):"},{"line_number":107,"context_line":"* pbm_wsdl_location - PBM service WSDL file location URL. e.g."},{"line_number":108,"context_line":"file:///opt/SDK/spbm/wsdl/pbmService.wsdl. This will be optional."},{"line_number":109,"context_line":"* pbm_enabled - status of storage policy based placement of instances."}],"source_content_type":"text/x-rst","patch_set":7,"id":"1ae5cdf2_af352646","line":106,"updated":"2014-06-27 11:19:49.000000000","message":"typo: variables","commit_id":"36e79211c8aa49f4220c45329476ee795f3db13c"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"4c0215edce9f510bad8f517982ee176734f406fe","unresolved":false,"context_lines":[{"line_number":103,"context_line":"Other deployer impact"},{"line_number":104,"context_line":"---------------------"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"There are 3 new configuration vaiables (both in the vmware section):"},{"line_number":107,"context_line":"* pbm_wsdl_location - PBM service WSDL file location URL. e.g."},{"line_number":108,"context_line":"file:///opt/SDK/spbm/wsdl/pbmService.wsdl. This will be optional."},{"line_number":109,"context_line":"* pbm_enabled - status of storage policy based placement of instances."}],"source_content_type":"text/x-rst","patch_set":7,"id":"1ae5cdf2_f231b759","line":106,"in_reply_to":"1ae5cdf2_af352646","updated":"2014-06-28 07:42:59.000000000","message":"Done","commit_id":"36e79211c8aa49f4220c45329476ee795f3db13c"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"c0b97ad10d59bfb82c28d04dd848921e0d208230","unresolved":false,"context_lines":[{"line_number":107,"context_line":"* pbm_wsdl_location - PBM service WSDL file location URL. e.g."},{"line_number":108,"context_line":"file:///opt/SDK/spbm/wsdl/pbmService.wsdl. This will be optional."},{"line_number":109,"context_line":"* pbm_enabled - status of storage policy based placement of instances."},{"line_number":110,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_wsdl_location"},{"line_number":111,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":112,"context_line":"then this policy will be used."},{"line_number":113,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"1ae5cdf2_2f1836cc","line":110,"updated":"2014-06-27 11:19:49.000000000","message":"\"If pbm_enabled is set...\"","commit_id":"36e79211c8aa49f4220c45329476ee795f3db13c"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"4c0215edce9f510bad8f517982ee176734f406fe","unresolved":false,"context_lines":[{"line_number":107,"context_line":"* pbm_wsdl_location - PBM service WSDL file location URL. e.g."},{"line_number":108,"context_line":"file:///opt/SDK/spbm/wsdl/pbmService.wsdl. This will be optional."},{"line_number":109,"context_line":"* pbm_enabled - status of storage policy based placement of instances."},{"line_number":110,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_wsdl_location"},{"line_number":111,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":112,"context_line":"then this policy will be used."},{"line_number":113,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"1ae5cdf2_323f9f26","line":110,"in_reply_to":"1ae5cdf2_2f1836cc","updated":"2014-06-28 07:42:59.000000000","message":"Done","commit_id":"36e79211c8aa49f4220c45329476ee795f3db13c"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"2e99d24659629eb0304126651f2c8fdc0b2b17e4","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"The driver will need to make use of a different endpoint to access the storage"},{"line_number":61,"context_line":"policies on the VC. This will require a new configuration variable, that is,"},{"line_number":62,"context_line":"the PBM WSDL location will need to be defined."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_f5d06455","line":62,"updated":"2014-07-01 17:10:17.000000000","message":"Sorry, I find this confusing. Really the details you have in implementation should be here.\n\nI think the key things you need to say here are:\n* extra specs in flavors, defined by admin, used by end user\n* vmware driver changes to match extra specs to vc storage policies\n\nThen need to talk about the config to map what is in extra specs and exposed to the user, to what you have in the vmware driver.\n\nThe other thing I guess, is that all the storage is assumed to be available on all compute nodes, so there is no need to worry about scheduling?","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"fd0403ef85416e6010bcb1ebc9c5c1c0f36872e1","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"The driver will need to make use of a different endpoint to access the storage"},{"line_number":61,"context_line":"policies on the VC. This will require a new configuration variable, that is,"},{"line_number":62,"context_line":"the PBM WSDL location will need to be defined."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Alternatives"},{"line_number":65,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_12db807c","line":62,"in_reply_to":"baada198_f5d06455","updated":"2014-07-07 16:36:06.000000000","message":"1. Done\n2. Correct all of the nodes will share the same storage","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"2e99d24659629eb0304126651f2c8fdc0b2b17e4","unresolved":false,"context_lines":[{"line_number":109,"context_line":"* pbm_enabled - status of storage policy based placement of instances."},{"line_number":110,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_enabled"},{"line_number":111,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":112,"context_line":"then this policy will be used."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"An admin user will create a new flavor either via the dashboard or the CLI."},{"line_number":115,"context_line":"The following interfaces will take place:"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_956f2892","line":112,"updated":"2014-07-01 17:10:17.000000000","message":"Please define the type of these configuration variables, and the defaults.\n\nWhat is the \"pbm_default_policy\"?","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"fd0403ef85416e6010bcb1ebc9c5c1c0f36872e1","unresolved":false,"context_lines":[{"line_number":109,"context_line":"* pbm_enabled - status of storage policy based placement of instances."},{"line_number":110,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_enabled"},{"line_number":111,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":112,"context_line":"then this policy will be used."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"An admin user will create a new flavor either via the dashboard or the CLI."},{"line_number":115,"context_line":"The following interfaces will take place:"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_f2fb6cbe","line":112,"in_reply_to":"baada198_956f2892","updated":"2014-07-07 16:36:06.000000000","message":"Done","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"2e99d24659629eb0304126651f2c8fdc0b2b17e4","unresolved":false,"context_lines":[{"line_number":149,"context_line":"          spawn process. This step is defined in more detail below."},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"* Step 8. The VMDK is created on the relevant datastore and the VM is spawned"},{"line_number":152,"context_line":"          with the relevant storage."},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"The flavor extra spec ‘vmware:storage_policy’ will drive the datastore"},{"line_number":155,"context_line":"selection. In the event that this flag is not present and the pbm_wsdl_location"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_2f9b6f19","line":152,"updated":"2014-07-01 17:10:17.000000000","message":"I think what you are trying to say here is, the \"VMware compute driver reads the flavor extra spec to decide where to create the VM\u0027s disks\"?\n\nNo need to talk about all the existing code and end to end stuff that can be assumed.","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"fd0403ef85416e6010bcb1ebc9c5c1c0f36872e1","unresolved":false,"context_lines":[{"line_number":149,"context_line":"          spawn process. This step is defined in more detail below."},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"* Step 8. The VMDK is created on the relevant datastore and the VM is spawned"},{"line_number":152,"context_line":"          with the relevant storage."},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"The flavor extra spec ‘vmware:storage_policy’ will drive the datastore"},{"line_number":155,"context_line":"selection. In the event that this flag is not present and the pbm_wsdl_location"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_15f60a8f","line":152,"in_reply_to":"baada198_2f9b6f19","updated":"2014-07-07 16:36:06.000000000","message":"Exactly.","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"2e99d24659629eb0304126651f2c8fdc0b2b17e4","unresolved":false,"context_lines":[{"line_number":174,"context_line":"before."},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"The list of datastores will be processed by the existing code to select the"},{"line_number":177,"context_line":"best fit."},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"Assignee(s)"},{"line_number":180,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_cf2033fa","line":177,"updated":"2014-07-01 17:10:17.000000000","message":"All these details need to be moved to the \"proposed change\" section. This information should not live here.","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"fd0403ef85416e6010bcb1ebc9c5c1c0f36872e1","unresolved":false,"context_lines":[{"line_number":174,"context_line":"before."},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"The list of datastores will be processed by the existing code to select the"},{"line_number":177,"context_line":"best fit."},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"Assignee(s)"},{"line_number":180,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"baada198_554322ae","line":177,"in_reply_to":"baada198_cf2033fa","updated":"2014-07-07 16:36:06.000000000","message":"But this is the implementation section. What did I miss?","commit_id":"76ad65230593ca43c9e648fc1dfc489d8376672f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c2b7eb7fc4c8b73281da6a9390b6a70e519d4157","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"NOTE: all of the nodes will share the same storage so there will not be any"},{"line_number":71,"context_line":"issues regarding rescheduling."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_6cb5a37d","line":72,"updated":"2014-07-08 12:45:36.000000000","message":"still no discussion of the impact this makes on disk caching. That seems relevant given recent issues in this area.","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"877c29da7a73815a1d372f29ec37ffd330549981","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"NOTE: all of the nodes will share the same storage so there will not be any"},{"line_number":71,"context_line":"issues regarding rescheduling."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Alternatives"},{"line_number":74,"context_line":"------------"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_a7527466","line":72,"in_reply_to":"baada198_6cb5a37d","updated":"2014-07-08 13:14:28.000000000","message":"This is not related to the disk caching. It is where the disk will be placed for the running VM. I will update so that this is more clear","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c2b7eb7fc4c8b73281da6a9390b6a70e519d4157","unresolved":false,"context_lines":[{"line_number":118,"context_line":"* pbm_enabled - status of storage policy based placement of instances."},{"line_number":119,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_enabled"},{"line_number":120,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":121,"context_line":"then this policy will be used."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"More information about storage policies can be found at the following"},{"line_number":124,"context_line":"link http://pubs.vmware.com/vsphere-55/index.jsp#com.vmware.sdk.doc/GUID-49E0D6F3-EC25-4833-BCC7-55635FE0B0BD.html"}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_ac6b3b1e","line":121,"updated":"2014-07-08 12:45:36.000000000","message":"Sorry, I still don\u0027t see any defaults here, or any types.\n\nI just can\u0027t see what the \"policy\" is? is it refereing to something that I have configured in VMware.","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"877c29da7a73815a1d372f29ec37ffd330549981","unresolved":false,"context_lines":[{"line_number":118,"context_line":"* pbm_enabled - status of storage policy based placement of instances."},{"line_number":119,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_enabled"},{"line_number":120,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":121,"context_line":"then this policy will be used."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"More information about storage policies can be found at the following"},{"line_number":124,"context_line":"link http://pubs.vmware.com/vsphere-55/index.jsp#com.vmware.sdk.doc/GUID-49E0D6F3-EC25-4833-BCC7-55635FE0B0BD.html"}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_87ec9091","line":121,"in_reply_to":"baada198_ac6b3b1e","updated":"2014-07-08 13:14:28.000000000","message":"Done","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c2b7eb7fc4c8b73281da6a9390b6a70e519d4157","unresolved":false,"context_lines":[{"line_number":129,"context_line":"* Step 1. The Command will be passed from the management interface to the"},{"line_number":130,"context_line":"          Nova API"},{"line_number":131,"context_line":"* Step 2. The Nove API will validate the new flavor and write it to the"},{"line_number":132,"context_line":"          database"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"Developer impact"},{"line_number":135,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_ec7533fb","line":132,"updated":"2014-07-08 12:45:36.000000000","message":"I want to know what to set, not how it gets set. Thats already in the documentation else where, so this is just repetition of that.\n\nWhats the key in extra specs, and what is the value you I want to set?\n\nIts hidden in the details below, but should really go here, I think it is \"vmware:storage_policy\"","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"877c29da7a73815a1d372f29ec37ffd330549981","unresolved":false,"context_lines":[{"line_number":129,"context_line":"* Step 1. The Command will be passed from the management interface to the"},{"line_number":130,"context_line":"          Nova API"},{"line_number":131,"context_line":"* Step 2. The Nove API will validate the new flavor and write it to the"},{"line_number":132,"context_line":"          database"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"Developer impact"},{"line_number":135,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_2794a4e7","line":132,"in_reply_to":"baada198_ec7533fb","updated":"2014-07-08 13:14:28.000000000","message":"Done","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c2b7eb7fc4c8b73281da6a9390b6a70e519d4157","unresolved":false,"context_lines":[{"line_number":161,"context_line":"          spawn process. This step is defined in more detail below."},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"* Step 8. The VMDK is created on the relevant datastore and the VM is spawned"},{"line_number":164,"context_line":"          with the relevant storage."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"The flavor extra spec ‘vmware:storage_policy’ will drive the datastore"},{"line_number":167,"context_line":"selection. In the event that this flag is not present and the pbm_wsdl_location"}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_0cf8a748","line":164,"updated":"2014-07-08 12:45:36.000000000","message":"this is just repeating how nova works.\n\nall you need to say here, is that when picking where to create a disk, you choose where based on a flavor extra spec, the rest is just implied, as its how nova works today.","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"877c29da7a73815a1d372f29ec37ffd330549981","unresolved":false,"context_lines":[{"line_number":161,"context_line":"          spawn process. This step is defined in more detail below."},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"* Step 8. The VMDK is created on the relevant datastore and the VM is spawned"},{"line_number":164,"context_line":"          with the relevant storage."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"The flavor extra spec ‘vmware:storage_policy’ will drive the datastore"},{"line_number":167,"context_line":"selection. In the event that this flag is not present and the pbm_wsdl_location"}],"source_content_type":"text/x-rst","patch_set":9,"id":"baada198_87a3b04e","line":164,"in_reply_to":"baada198_0cf8a748","updated":"2014-07-08 13:14:28.000000000","message":"Done","commit_id":"9e3216e4325ff813404caa4d1e243fe4f3bbe272"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"759a8621f9dd859d949d45695b0be3dc5cb5484e","unresolved":false,"context_lines":[{"line_number":21,"context_line":"on the storage vendor provided capabilities and/or tag based capabilities"},{"line_number":22,"context_line":"of the underlying storage infrastructure. Refer to"},{"line_number":23,"context_line":"http://pubs.vmware.com/vsphere-55/topic/com.vmware.vsphere.storage.doc/GUID-A8BA9141-31F1-4555-A554-4B5B04D75E54.html"},{"line_number":24,"context_line":"to learn more about storage profiles on VC."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"The disk(s) of the virtual machine will be placed on the storage that"},{"line_number":27,"context_line":"matches the storage policy. This can for example provide preferential"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_ed8e0d79","line":24,"updated":"2014-07-08 13:38:08.000000000","message":"OK, that makes more sense now, thanks.","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"759a8621f9dd859d949d45695b0be3dc5cb5484e","unresolved":false,"context_lines":[{"line_number":124,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_enabled"},{"line_number":125,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":126,"context_line":"then this policy will be used. This value is a string. The default policy"},{"line_number":127,"context_line":"is defined out of band by the administrator on the Virtual Center."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"More information about storage policies can be found at the following"},{"line_number":130,"context_line":"link http://pubs.vmware.com/vsphere-55/index.jsp#com.vmware.sdk.doc/GUID-49E0D6F3-EC25-4833-BCC7-55635FE0B0BD.html"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_adc25584","line":127,"updated":"2014-07-08 13:38:08.000000000","message":"Thank you that makes sense. I assume there is no default value for this  config param?","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"26cf323e60c3bc608563f0a64f79af79eb7f5ddf","unresolved":false,"context_lines":[{"line_number":124,"context_line":"* pbm_default_policy - The PBM default policy. If pbm_enabled"},{"line_number":125,"context_line":"is set and there is no defined storage policy for the specific request"},{"line_number":126,"context_line":"then this policy will be used. This value is a string. The default policy"},{"line_number":127,"context_line":"is defined out of band by the administrator on the Virtual Center."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"More information about storage policies can be found at the following"},{"line_number":130,"context_line":"link http://pubs.vmware.com/vsphere-55/index.jsp#com.vmware.sdk.doc/GUID-49E0D6F3-EC25-4833-BCC7-55635FE0B0BD.html"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_785716cb","line":127,"in_reply_to":"baada198_adc25584","updated":"2014-07-08 15:10:35.000000000","message":"Done","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"759a8621f9dd859d949d45695b0be3dc5cb5484e","unresolved":false,"context_lines":[{"line_number":127,"context_line":"is defined out of band by the administrator on the Virtual Center."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"More information about storage policies can be found at the following"},{"line_number":130,"context_line":"link http://pubs.vmware.com/vsphere-55/index.jsp#com.vmware.sdk.doc/GUID-49E0D6F3-EC25-4833-BCC7-55635FE0B0BD.html"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"An admin user will create a new flavor either via the dashboard or the CLI."},{"line_number":133,"context_line":"The flavor extra spec will have a key ‘vmware:storage_policy’. The admin"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_0dba610a","line":130,"updated":"2014-07-08 13:38:08.000000000","message":"OK, linking twice seems excessive, I probably mis-read this before, but just mentioning this above seems fine to me.","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"26cf323e60c3bc608563f0a64f79af79eb7f5ddf","unresolved":false,"context_lines":[{"line_number":127,"context_line":"is defined out of band by the administrator on the Virtual Center."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"More information about storage policies can be found at the following"},{"line_number":130,"context_line":"link http://pubs.vmware.com/vsphere-55/index.jsp#com.vmware.sdk.doc/GUID-49E0D6F3-EC25-4833-BCC7-55635FE0B0BD.html"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"An admin user will create a new flavor either via the dashboard or the CLI."},{"line_number":133,"context_line":"The flavor extra spec will have a key ‘vmware:storage_policy’. The admin"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_38d2fe34","line":130,"in_reply_to":"baada198_0dba610a","updated":"2014-07-08 15:10:35.000000000","message":"Done","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"3e4b5cd01ae198a270ae8439a972a17b8be2d29d","unresolved":false,"context_lines":[{"line_number":164,"context_line":"before."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"The list of datastores will be processed by the existing code to select the"},{"line_number":167,"context_line":"best fit."},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"Assignee(s)"},{"line_number":170,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_2d81e54d","line":167,"updated":"2014-07-08 13:33:47.000000000","message":"As mentioned before, this section should be empty, it should go into the proposed change section.","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"759a8621f9dd859d949d45695b0be3dc5cb5484e","unresolved":false,"context_lines":[{"line_number":164,"context_line":"before."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"The list of datastores will be processed by the existing code to select the"},{"line_number":167,"context_line":"best fit."},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"Assignee(s)"},{"line_number":170,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_cd6dc99b","line":167,"in_reply_to":"baada198_2d81e54d","updated":"2014-07-08 13:38:08.000000000","message":"thinking a bit more, I think deleting this section is fine, but if you want this extra detail, please add it into the proposed change section.","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"26cf323e60c3bc608563f0a64f79af79eb7f5ddf","unresolved":false,"context_lines":[{"line_number":164,"context_line":"before."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"The list of datastores will be processed by the existing code to select the"},{"line_number":167,"context_line":"best fit."},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"Assignee(s)"},{"line_number":170,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"baada198_b8cc4e8c","line":167,"in_reply_to":"baada198_cd6dc99b","updated":"2014-07-08 15:10:35.000000000","message":"Done","commit_id":"bb3d3d0a55d9a827c54596a0f9dee905d7263b8b"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"b2deddaaf5ff32996a447cc5082069fbe8a64594","unresolved":false,"context_lines":[{"line_number":73,"context_line":"The change will not affect the cached images. This is only where the disk"},{"line_number":74,"context_line":"for the VM will be placed."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"The flavor extra spec ‘vmware:storage_policy’ will drive the datastore"},{"line_number":77,"context_line":"selection. In the event that this flag is not present and the pbm_enabled"},{"line_number":78,"context_line":"is set in the configuration file then we will make use of a configured default"},{"line_number":79,"context_line":"policy. That is, if this is present then it will be used to get the list of"}],"source_content_type":"text/x-rst","patch_set":11,"id":"baada198_a9e1499c","line":76,"updated":"2014-07-09 08:52:55.000000000","message":"Instead of this vmware specific name, how about trying something more generic like\n\n    \u0027image:storage_backend\u0027 ?\n\nOther name suggestions welcome...","commit_id":"3f86503ef1aed83c3387f7c1e6b6581f71adf415"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"00fbcf01708ab1c40775412e3fbdde5fd39099a7","unresolved":false,"context_lines":[{"line_number":73,"context_line":"The change will not affect the cached images. This is only where the disk"},{"line_number":74,"context_line":"for the VM will be placed."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"The flavor extra spec ‘vmware:storage_policy’ will drive the datastore"},{"line_number":77,"context_line":"selection. In the event that this flag is not present and the pbm_enabled"},{"line_number":78,"context_line":"is set in the configuration file then we will make use of a configured default"},{"line_number":79,"context_line":"policy. That is, if this is present then it will be used to get the list of"}],"source_content_type":"text/x-rst","patch_set":11,"id":"baada198_d4aa8d84","line":76,"in_reply_to":"baada198_a9e1499c","updated":"2014-07-09 11:20:27.000000000","message":"Nice suggestion","commit_id":"3f86503ef1aed83c3387f7c1e6b6581f71adf415"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9e2ee1de1d57516d0e543a6009bf3a0668c972c4","unresolved":false,"context_lines":[{"line_number":184,"context_line":"* Add support for default pbm policy: https://review.openstack.org/#/c/68131"},{"line_number":185,"context_line":"* Get storage policy from flavor: https://review.openstack.org/#/c/68421"},{"line_number":186,"context_line":"* Use storage policy in datastore selection: https://review.openstack.org/#/c/68539"},{"line_number":187,"context_line":"* Associate instance with storage policy: https://review.openstack.org/#/c/68547"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"Dependencies"},{"line_number":190,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":12,"id":"baada198_acae89e5","line":187,"updated":"2014-07-10 09:49:53.000000000","message":"I don\u0027t like having these review links here, but I have blocked this enough, so I will ignore that.","commit_id":"58c13528b3aa39716d602c6f2f4f4bfefa4717f4"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"c9f25d053515808292c441d3dedaa86f60787fd8","unresolved":false,"context_lines":[{"line_number":184,"context_line":"* Add support for default pbm policy: https://review.openstack.org/#/c/68131"},{"line_number":185,"context_line":"* Get storage policy from flavor: https://review.openstack.org/#/c/68421"},{"line_number":186,"context_line":"* Use storage policy in datastore selection: https://review.openstack.org/#/c/68539"},{"line_number":187,"context_line":"* Associate instance with storage policy: https://review.openstack.org/#/c/68547"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"Dependencies"},{"line_number":190,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":12,"id":"baada198_cccbbd48","line":187,"in_reply_to":"baada198_acae89e5","updated":"2014-07-10 10:04:11.000000000","message":"Done","commit_id":"58c13528b3aa39716d602c6f2f4f4bfefa4717f4"}]}