)]}'
{"nova/conf/vmware.py":[{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"32f09104b8496c3e39c4ccd6111ac6748700f552","unresolved":false,"context_lines":[{"line_number":193,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":194,"context_line":"               deprecated_reason\u003d\u0027Only vCenter versions earlier than 5.1 \u0027"},{"line_number":195,"context_line":"                                 \u0027require this option and the current \u0027"},{"line_number":196,"context_line":"                                 \u0027minimum version is 5.1.\u0027,"},{"line_number":197,"context_line":"               help\u003d\"\"\""},{"line_number":198,"context_line":"This option specifies VIM Service WSDL Location"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_9aeb0bb0","line":196,"updated":"2016-09-06 17:44:33.000000000","message":"Add \"deprecated_since\" parameter. That helps when removing the option. Refer to https://review.openstack.org/#/c/353631","commit_id":"f9e14b8b330186eabc66012dc14f44a27fac2385"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"5c3d860a40879148576716af4f0e35332321bdb1","unresolved":false,"context_lines":[{"line_number":193,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":194,"context_line":"               deprecated_reason\u003d\u0027Only vCenter versions earlier than 5.1 \u0027"},{"line_number":195,"context_line":"                                 \u0027require this option and the current \u0027"},{"line_number":196,"context_line":"                                 \u0027minimum version is 5.1.\u0027,"},{"line_number":197,"context_line":"               help\u003d\"\"\""},{"line_number":198,"context_line":"This option specifies VIM Service WSDL Location"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_55db0c4f","line":196,"in_reply_to":"9a89bdaa_9aeb0bb0","updated":"2016-09-06 22:25:19.000000000","message":"Done","commit_id":"f9e14b8b330186eabc66012dc14f44a27fac2385"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6aac62cb55ecf1382f30b7405e19c4d589e0c9a9","unresolved":false,"context_lines":[{"line_number":195,"context_line":"               deprecated_reason\u003d\u0027Only vCenter versions earlier than 5.1 \u0027"},{"line_number":196,"context_line":"                                 \u0027require this option and the current \u0027"},{"line_number":197,"context_line":"                                 \u0027minimum version is 5.1.\u0027,"},{"line_number":198,"context_line":"               deprecated_since\u003dversionutils.deprecated.NEWTON,"},{"line_number":199,"context_line":"               help\u003d\"\"\""},{"line_number":200,"context_line":"This option specifies VIM Service WSDL Location"},{"line_number":201,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"5a8bc5a2_82d5a144","line":198,"range":{"start_line":198,"start_character":56,"end_line":198,"end_character":62},"updated":"2016-09-23 01:36:30.000000000","message":"Actually this would be Ocata now, or just 15.0.0.","commit_id":"031a7d378942de6795e2fd40d9bccc936e517295"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7da926c4234d52c98e23ee3e26f023847ac9f453","unresolved":false,"context_lines":[{"line_number":195,"context_line":"               deprecated_reason\u003d\u0027Only vCenter versions earlier than 5.1 \u0027"},{"line_number":196,"context_line":"                                 \u0027require this option and the current \u0027"},{"line_number":197,"context_line":"                                 \u0027minimum version is 5.1.\u0027,"},{"line_number":198,"context_line":"               deprecated_since\u003dversionutils.deprecated.NEWTON,"},{"line_number":199,"context_line":"               help\u003d\"\"\""},{"line_number":200,"context_line":"This option specifies VIM Service WSDL Location"},{"line_number":201,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"5a8bc5a2_625ed5ea","line":198,"range":{"start_line":198,"start_character":56,"end_line":198,"end_character":62},"in_reply_to":"5a8bc5a2_82d5a144","updated":"2016-09-23 01:39:12.000000000","message":"This shows up as:\n\n\"# This option is deprecated for removal since N.\"\n\nLet\u0027s just say \"15.0.0\" since that\u0027s the version for Ocata.","commit_id":"031a7d378942de6795e2fd40d9bccc936e517295"}],"nova/virt/vmwareapi/driver.py":[{"author":{"_account_id":7400,"name":"Tracy Jones","email":"tjones@vmware.com","username":"tracyajones"},"change_message_id":"85f353f4bd90b3631347a17b7f56a0ffa7d92296","unresolved":false,"context_lines":[{"line_number":515,"context_line":"                task_poll_interval\u003dCONF.vmware.task_poll_interval,"},{"line_number":516,"context_line":"                scheme\u003dscheme,"},{"line_number":517,"context_line":"                create_session\u003dTrue,"},{"line_number":518,"context_line":"                wsdl_loc\u003dCONF.vmware.wsdl_location,"},{"line_number":519,"context_line":"                cacert\u003dcacert,"},{"line_number":520,"context_line":"                insecure\u003dinsecure)"},{"line_number":521,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_f956e9d0","side":"PARENT","line":518,"updated":"2016-09-06 16:43:28.000000000","message":"shouldn\u0027t this be removed in the next cycle since you are deprecating this time?","commit_id":"8f35bb321d26bd7d296c57f4188ec12fcde897c3"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"5c3d860a40879148576716af4f0e35332321bdb1","unresolved":false,"context_lines":[{"line_number":515,"context_line":"                task_poll_interval\u003dCONF.vmware.task_poll_interval,"},{"line_number":516,"context_line":"                scheme\u003dscheme,"},{"line_number":517,"context_line":"                create_session\u003dTrue,"},{"line_number":518,"context_line":"                wsdl_loc\u003dCONF.vmware.wsdl_location,"},{"line_number":519,"context_line":"                cacert\u003dcacert,"},{"line_number":520,"context_line":"                insecure\u003dinsecure)"},{"line_number":521,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_75b730f5","side":"PARENT","line":518,"in_reply_to":"9a89bdaa_5a2fc3f5","updated":"2016-09-06 22:25:19.000000000","message":"This only works for VC versions 4.x and earlier.  We already don\u0027t support versions prior to 5.1. So its already unsupported.","commit_id":"8f35bb321d26bd7d296c57f4188ec12fcde897c3"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"32f09104b8496c3e39c4ccd6111ac6748700f552","unresolved":false,"context_lines":[{"line_number":515,"context_line":"                task_poll_interval\u003dCONF.vmware.task_poll_interval,"},{"line_number":516,"context_line":"                scheme\u003dscheme,"},{"line_number":517,"context_line":"                create_session\u003dTrue,"},{"line_number":518,"context_line":"                wsdl_loc\u003dCONF.vmware.wsdl_location,"},{"line_number":519,"context_line":"                cacert\u003dcacert,"},{"line_number":520,"context_line":"                insecure\u003dinsecure)"},{"line_number":521,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_5a2fc3f5","side":"PARENT","line":518,"in_reply_to":"9a89bdaa_f956e9d0","updated":"2016-09-06 17:44:33.000000000","message":"+1, you need to deprecate option for atleast 1 cycle before removal.","commit_id":"8f35bb321d26bd7d296c57f4188ec12fcde897c3"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"cdb5d052a7e211f118cf239c727bd2e0a8bf181b","unresolved":false,"context_lines":[{"line_number":515,"context_line":"                task_poll_interval\u003dCONF.vmware.task_poll_interval,"},{"line_number":516,"context_line":"                scheme\u003dscheme,"},{"line_number":517,"context_line":"                create_session\u003dTrue,"},{"line_number":518,"context_line":"                wsdl_loc\u003dCONF.vmware.wsdl_location,"},{"line_number":519,"context_line":"                cacert\u003dcacert,"},{"line_number":520,"context_line":"                insecure\u003dinsecure)"},{"line_number":521,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_fd7a19f0","side":"PARENT","line":518,"updated":"2016-09-07 15:20:40.000000000","message":"So my understanding is that options need to be deprecated for atleast 1 cycle because otherwise it will break operators who scripted for this option. Now we can deprecate it and omit warning in the code saying this is not used. We can wait for cores to respond on this.","commit_id":"1bfd7d679c0802509f9dc2c1d032c3c9bd409539"}]}
