)]}'
{"nova/volume/cinder.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5ecfa1382baf9906a72ed2d438f2517fb43ae573","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":93,"context_line":"        LOG.info(_LI(\"The support for the Cinder API v2 is deprecated, please \""},{"line_number":94,"context_line":"                     \"consider to upgrade to Cinder API v3.\"))"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if version \u003d\u003d \u00273\u0027:"},{"line_number":97,"context_line":"        # TODO(ildikov): Add microversion support for picking up the new"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a30ddce_2e58dc89","line":94,"range":{"start_line":94,"start_character":22,"end_line":94,"end_character":30},"updated":"2017-03-10 13:25:27.000000000","message":"I would just remove \"consider to\" and just state \"please upgrade to\".","commit_id":"c9cd98010351fb184613a6fe00053a7be0eb10d6"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d1aa8b7f1bda7ec2c4f6ca29f08683eadc5e7d40","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    if version \u003d\u003d \u00271\u0027:"},{"line_number":90,"context_line":"        raise exception.UnsupportedCinderAPIVersion(version\u003dversion)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":93,"context_line":"        LOG.info(_LI(\"The support for the Cinder API v2 is deprecated, please \""},{"line_number":94,"context_line":"                     \"upgrade to Cinder API v3.\"))"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if version \u003d\u003d \u00273\u0027:"},{"line_number":97,"context_line":"        # TODO(ildikov): Add microversion support for picking up the new"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a30ddce_577aea88","line":94,"range":{"start_line":92,"start_character":0,"end_line":94,"end_character":50},"updated":"2017-03-11 01:45:09.000000000","message":"Doesn\u0027t cinderclient shows this kind of warning?\nMy concern is that the clouds could show a lot of same logs on Cinder, cincerclient and Nova with this patch to operators.","commit_id":"95658fc68f918b97bc9457e1228470b193c13768"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0e6c5227d682074389dc1f7ca82016f7cff30044","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    if version \u003d\u003d \u00271\u0027:"},{"line_number":90,"context_line":"        raise exception.UnsupportedCinderAPIVersion(version\u003dversion)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":93,"context_line":"        LOG.info(_LI(\"The support for the Cinder API v2 is deprecated, please \""},{"line_number":94,"context_line":"                     \"upgrade to Cinder API v3.\"))"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if version \u003d\u003d \u00273\u0027:"},{"line_number":97,"context_line":"        # TODO(ildikov): Add microversion support for picking up the new"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a30ddce_f7611ed4","line":94,"range":{"start_line":92,"start_character":0,"end_line":94,"end_character":50},"in_reply_to":"9a30ddce_577aea88","updated":"2017-03-11 01:46:10.000000000","message":"one more thing is that why don\u0027t we set this as warning instead of info? A warning could get attention from operators more easily.","commit_id":"95658fc68f918b97bc9457e1228470b193c13768"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3cd060ba52732ab32452897e9cacf903e7ee40dc","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    if version \u003d\u003d \u00271\u0027:"},{"line_number":90,"context_line":"        raise exception.UnsupportedCinderAPIVersion(version\u003dversion)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":93,"context_line":"        LOG.info(_LI(\"The support for the Cinder API v2 is deprecated, please \""},{"line_number":94,"context_line":"                     \"upgrade to Cinder API v3.\"))"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if version \u003d\u003d \u00273\u0027:"},{"line_number":97,"context_line":"        # TODO(ildikov): Add microversion support for picking up the new"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a1ff146_b292f955","line":94,"range":{"start_line":92,"start_character":0,"end_line":94,"end_character":50},"in_reply_to":"9a30ddce_f7611ed4","updated":"2017-03-16 11:18:37.000000000","message":"We had a similar LOG entry for v1 before it got fully unsupported, wanted to mimic that behavior here.\n\nI also changed it to warning, I don\u0027t really remember at this point why I chose info. Thanks for the heads up.","commit_id":"95658fc68f918b97bc9457e1228470b193c13768"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"b33c266b3a6ca116fbffe8b4d2f7ae831446854d","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        raise exception.UnsupportedCinderAPIVersion(version\u003dversion)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":92,"context_line":"        LOG.warning(_LW(\"The support for the Cinder API v2 is deprecated, \""},{"line_number":93,"context_line":"                        \"please upgrade to Cinder API v3.\"))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    if version \u003d\u003d \u00273\u0027:"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_849e2d61","line":92,"range":{"start_line":92,"start_character":20,"end_line":92,"end_character":23},"updated":"2017-04-06 16:49:51.000000000","message":"We don\u0027t need to put _LW() anymore based on https://review.openstack.org/#/c/447528/","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"28e265bdad08047dfb65c48e7d2b22ce4de2bb75","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        raise exception.UnsupportedCinderAPIVersion(version\u003dversion)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":92,"context_line":"        LOG.warning(_LW(\"The support for the Cinder API v2 is deprecated, \""},{"line_number":93,"context_line":"                        \"please upgrade to Cinder API v3.\"))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    if version \u003d\u003d \u00273\u0027:"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_29685cf3","line":92,"updated":"2017-04-06 16:39:34.000000000","message":"We don\u0027t really test this because we don\u0027t define cinder.catalog_info in nova.conf in the grenade jobs, so it just takes the defaults, which is v2 in ocata and v3 in pike, so we don\u0027t get the warning. That\u0027s probably fine, I\u0027m just pointing it out.","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"4a665979a71cd7377a320ff3bf30d06b07373449","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        raise exception.UnsupportedCinderAPIVersion(version\u003dversion)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    if version \u003d\u003d \u00272\u0027:"},{"line_number":92,"context_line":"        LOG.warning(_LW(\"The support for the Cinder API v2 is deprecated, \""},{"line_number":93,"context_line":"                        \"please upgrade to Cinder API v3.\"))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    if version \u003d\u003d \u00273\u0027:"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_641e89bb","line":92,"range":{"start_line":92,"start_character":20,"end_line":92,"end_character":23},"in_reply_to":"bff0334d_849e2d61","updated":"2017-04-06 16:56:34.000000000","message":"Done","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"}],"releasenotes/notes/deprecate-the-cinder-v2-support-0cebc90580a3e80f.yaml":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5ecfa1382baf9906a72ed2d438f2517fb43ae573","unresolved":false,"context_lines":[{"line_number":1,"context_line":"deprecations:"},{"line_number":2,"context_line":"- |"},{"line_number":3,"context_line":"  The support for the v2 version of the Cinder API is now deprecated and will"},{"line_number":4,"context_line":"  removed in a future release."},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  You can configure to use a different version of the Cinder API by changing"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9a30ddce_8e53886a","line":3,"range":{"start_line":3,"start_character":73,"end_line":3,"end_character":77},"updated":"2017-03-10 13:25:27.000000000","message":"/will/will be/","commit_id":"c9cd98010351fb184613a6fe00053a7be0eb10d6"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d1aa8b7f1bda7ec2c4f6ca29f08683eadc5e7d40","unresolved":false,"context_lines":[{"line_number":1,"context_line":"deprecations:"},{"line_number":2,"context_line":"- |"},{"line_number":3,"context_line":"  The support for the v2 version of the Cinder API is now deprecated and will"},{"line_number":4,"context_line":"  be removed in a future release."},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  You can configure to use a different version of the Cinder API by changing"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9a30ddce_94a1c048","line":3,"range":{"start_line":3,"start_character":18,"end_line":3,"end_character":68},"updated":"2017-03-11 01:45:09.000000000","message":"2 weeks ago, I saw the api doc and v2 was Supported instead of deprecated. So I guess v2 has been deprecated in this cycle, right? It is better to describe which version v2 has been deprecated since for operators because there are clouds which consist of different versions of components.","commit_id":"95658fc68f918b97bc9457e1228470b193c13768"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3cd060ba52732ab32452897e9cacf903e7ee40dc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"deprecations:"},{"line_number":2,"context_line":"- |"},{"line_number":3,"context_line":"  The support for the v2 version of the Cinder API is now deprecated and will"},{"line_number":4,"context_line":"  be removed in a future release."},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  You can configure to use a different version of the Cinder API by changing"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a1ff146_128f4d24","line":3,"range":{"start_line":3,"start_character":18,"end_line":3,"end_character":68},"in_reply_to":"9a30ddce_94a1c048","updated":"2017-03-16 11:18:37.000000000","message":"Done","commit_id":"95658fc68f918b97bc9457e1228470b193c13768"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"28e265bdad08047dfb65c48e7d2b22ce4de2bb75","unresolved":false,"context_lines":[{"line_number":1,"context_line":"deprecations:"},{"line_number":2,"context_line":"- |"},{"line_number":3,"context_line":"  The support for the v2 version of the Cinder API is now deprecated and will"},{"line_number":4,"context_line":"  be removed in a future release as the v2 version of the Cinder API got"},{"line_number":5,"context_line":"  deprecated in Pike."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"  You can configure to use a different version of the Cinder API by changing"},{"line_number":8,"context_line":"  the ``catalog_info`` configuration option in the ``cinder`` section in the"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bff0334d_491a807d","line":5,"range":{"start_line":3,"start_character":2,"end_line":5,"end_character":21},"updated":"2017-04-06 16:39:34.000000000","message":"This reads a bit weird. The first part is about nova\u0027s support as a client to use cinder v2. The second part is about cinder deprecating the v2 API itself.\n\nI don\u0027t think we need to mention the cinder v2 API deprecation itself. We can just say:\n\n\"Nova support for using the Block Storage (Cinder) v2 API is now deprecated and will be removed in the 17.0.0 Queens release. The v3 API is now the default and is backward compatible with the v2 API.\"","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"4a665979a71cd7377a320ff3bf30d06b07373449","unresolved":false,"context_lines":[{"line_number":1,"context_line":"deprecations:"},{"line_number":2,"context_line":"- |"},{"line_number":3,"context_line":"  The support for the v2 version of the Cinder API is now deprecated and will"},{"line_number":4,"context_line":"  be removed in a future release as the v2 version of the Cinder API got"},{"line_number":5,"context_line":"  deprecated in Pike."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"  You can configure to use a different version of the Cinder API by changing"},{"line_number":8,"context_line":"  the ``catalog_info`` configuration option in the ``cinder`` section in the"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bff0334d_e404d97d","line":5,"range":{"start_line":3,"start_character":2,"end_line":5,"end_character":21},"in_reply_to":"bff0334d_491a807d","updated":"2017-04-06 16:56:34.000000000","message":"Done","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"b33c266b3a6ca116fbffe8b4d2f7ae831446854d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"deprecations:"},{"line_number":2,"context_line":"- |"},{"line_number":3,"context_line":"  The support for the v2 version of the Cinder API is now deprecated and will"},{"line_number":4,"context_line":"  be removed in a future release as the v2 version of the Cinder API got"},{"line_number":5,"context_line":"  deprecated in Pike."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"  You can configure to use a different version of the Cinder API by changing"},{"line_number":8,"context_line":"  the ``catalog_info`` configuration option in the ``cinder`` section in the"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bff0334d_a49ff14e","line":5,"range":{"start_line":3,"start_character":2,"end_line":5,"end_character":21},"in_reply_to":"bff0334d_491a807d","updated":"2017-04-06 16:49:51.000000000","message":"Nice description, +1","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"28e265bdad08047dfb65c48e7d2b22ce4de2bb75","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  be removed in a future release as the v2 version of the Cinder API got"},{"line_number":5,"context_line":"  deprecated in Pike."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"  You can configure to use a different version of the Cinder API by changing"},{"line_number":8,"context_line":"  the ``catalog_info`` configuration option in the ``cinder`` section in the"},{"line_number":9,"context_line":"  ``nova.conf`` file. If you don\u0027t specify a value there it will default to the"},{"line_number":10,"context_line":"  v3 version."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bff0334d_e42619b8","line":10,"range":{"start_line":7,"start_character":2,"end_line":10,"end_character":13},"updated":"2017-04-06 16:39:34.000000000","message":"Based on my suggested rewording, I think we could remove this.","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"4a665979a71cd7377a320ff3bf30d06b07373449","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  be removed in a future release as the v2 version of the Cinder API got"},{"line_number":5,"context_line":"  deprecated in Pike."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"  You can configure to use a different version of the Cinder API by changing"},{"line_number":8,"context_line":"  the ``catalog_info`` configuration option in the ``cinder`` section in the"},{"line_number":9,"context_line":"  ``nova.conf`` file. If you don\u0027t specify a value there it will default to the"},{"line_number":10,"context_line":"  v3 version."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bff0334d_c4f9d580","line":10,"range":{"start_line":7,"start_character":2,"end_line":10,"end_character":13},"in_reply_to":"bff0334d_e42619b8","updated":"2017-04-06 16:56:34.000000000","message":"Done","commit_id":"7511747040c09ff9b8ad76065b88483fac18fa49"}]}