)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"1aca2f56aa8b7a64d821309414ff4a8edddb8dcb","unresolved":false,"context_lines":[{"line_number":44,"context_line":" 4. If Tempest if configured to tests v2 APIs, configure tempest to skip microversion tests"},{"line_number":45,"context_line":"    as no microversion supported in v2 APIs."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Project-Config: TODO"},{"line_number":48,"context_line":" 5. Add new job for v2 APIs testing and set ``[volume].catalog_type`` to ``volumev2``"},{"line_number":49,"context_line":"    to run all tests against v2 APIs."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bf659307_c6d52e45","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":20},"updated":"2018-03-27 03:09:04.000000000","message":"No longer true due to .zuul.yaml","commit_id":"a33e0ca51befafe1d9540abf8eea75676f214431"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0cea9e12e9df68206bdb70269849e8afcbf3ee9f","unresolved":false,"context_lines":[{"line_number":44,"context_line":" 4. If Tempest if configured to tests v2 APIs, configure tempest to skip microversion tests"},{"line_number":45,"context_line":"    as no microversion supported in v2 APIs."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Project-Config: TODO"},{"line_number":48,"context_line":" 5. Add new job for v2 APIs testing and set ``[volume].catalog_type`` to ``volumev2``"},{"line_number":49,"context_line":"    to run all tests against v2 APIs."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9f6a8fd7_177c501d","line":47,"range":{"start_line":47,"start_character":0,"end_line":47,"end_character":20},"in_reply_to":"bf659307_c6d52e45","updated":"2018-04-22 11:28:20.000000000","message":"yea, we can add those job in tree now.","commit_id":"a33e0ca51befafe1d9540abf8eea75676f214431"}],"releasenotes/notes/volume-service-testing-default-to-v3-endpoints-20b86895a590925d.yaml":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"1f8ed827b5e2c15e3b0de0be6018481f022043f9","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    ``volumev3`` which is v3 API endpoint configured in devstack."},{"line_number":6,"context_line":"    With this change Tempest will be testing v3 API as default."},{"line_number":7,"context_line":"    User who want to test v2 API can still test by configuring the"},{"line_number":8,"context_line":"    ``catalog_type`` to v2 endppoint."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f6a8fd7_98a72dbe","line":8,"range":{"start_line":8,"start_character":27,"end_line":8,"end_character":36},"updated":"2018-04-23 00:57:52.000000000","message":"endpoint","commit_id":"567203e1548b5854676a5e6013094a243b8f01a6"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b9be195e4838fbe12104354eb13c8ab79d352855","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    ``volumev3`` which is v3 API endpoint configured in devstack."},{"line_number":6,"context_line":"    With this change Tempest will be testing v3 API as default."},{"line_number":7,"context_line":"    User who want to test v2 API can still test by configuring the"},{"line_number":8,"context_line":"    ``catalog_type`` to v2 endppoint."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f6a8fd7_2c24a6ae","line":8,"range":{"start_line":8,"start_character":27,"end_line":8,"end_character":36},"in_reply_to":"9f6a8fd7_98a72dbe","updated":"2018-04-23 02:26:21.000000000","message":"thanks, done","commit_id":"567203e1548b5854676a5e6013094a243b8f01a6"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"ae28f7fed2ebb7b3bafe865902621538647c9118","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    ``volumev3`` which is v3 API endpoint configured in devstack."},{"line_number":6,"context_line":"    With this change Tempest will be testing v3 API as default."},{"line_number":7,"context_line":"    User who want to test v2 API can still test by configuring the"},{"line_number":8,"context_line":"    ``catalog_type`` to v2 endpoint."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_57d5951a","line":8,"updated":"2018-05-16 16:13:37.000000000","message":"I see deprecation notes are here: https://review.openstack.org/#/c/536121/12/releasenotes/notes/volume-v3-service-clients-a863a6336af56cca.yaml@7\n\nSo LGTM","commit_id":"89c213fb355f0ec672ae8002c516cf11f568960b"}],"tempest/config.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"54fc376287f093ad244b3753e1a97b76fc997cb7","unresolved":false,"context_lines":[{"line_number":745,"context_line":"                    \u0027available.\u0027),"},{"line_number":746,"context_line":"    cfg.StrOpt(\u0027catalog_type\u0027,"},{"line_number":747,"context_line":"               default\u003d\u0027volumev3\u0027,"},{"line_number":748,"context_line":"               help\u003d\"Catalog type of the Volume Service\"),"},{"line_number":749,"context_line":"    cfg.StrOpt(\u0027region\u0027,"},{"line_number":750,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":751,"context_line":"               help\u003d\"The volume region name to use. If empty, the value \""}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_ebee148a","line":748,"range":{"start_line":748,"start_character":0,"end_line":748,"end_character":58},"updated":"2017-12-14 09:33:55.000000000","message":"changing this default value need upgrade reno. ll add if direction is ok for all.","commit_id":"116c088b82d189eb6eac37372577018d8c329518"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"0f2b958774e1a3b577d6e220b6358767c7e6cdc6","unresolved":false,"context_lines":[{"line_number":830,"context_line":"                deprecated_reason\u003d\"The v1 volume API has been deprecated \""},{"line_number":831,"context_line":"                                  \"since Juno release, and the API will be \""},{"line_number":832,"context_line":"                                  \"removed.\"),"},{"line_number":833,"context_line":"    cfg.BoolOpt(\u0027api_v2\u0027,"},{"line_number":834,"context_line":"                default\u003dTrue,"},{"line_number":835,"context_line":"                help\u003d\"Is the v2 volume API enabled\"),"},{"line_number":836,"context_line":"    cfg.BoolOpt(\u0027api_v3\u0027,"},{"line_number":837,"context_line":"                default\u003dTrue,"},{"line_number":838,"context_line":"                help\u003d\"Is the v3 volume API enabled\"),"},{"line_number":839,"context_line":"    cfg.BoolOpt(\u0027extend_attached_volume\u0027,"},{"line_number":840,"context_line":"                default\u003dFalse,"},{"line_number":841,"context_line":"                help\u003d\u0027Does the cloud support extending the size of a volume \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf659307_c3e0c0cf","line":838,"range":{"start_line":833,"start_character":0,"end_line":838,"end_character":53},"updated":"2018-03-27 02:28:12.000000000","message":"These two config items and the relative code may also need modification:)\ne.g., if we set api_v3\u003dFalse and catalog_type\u003dvolumev3, what should we expect?","commit_id":"a33e0ca51befafe1d9540abf8eea75676f214431"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0cea9e12e9df68206bdb70269849e8afcbf3ee9f","unresolved":false,"context_lines":[{"line_number":830,"context_line":"                deprecated_reason\u003d\"The v1 volume API has been deprecated \""},{"line_number":831,"context_line":"                                  \"since Juno release, and the API will be \""},{"line_number":832,"context_line":"                                  \"removed.\"),"},{"line_number":833,"context_line":"    cfg.BoolOpt(\u0027api_v2\u0027,"},{"line_number":834,"context_line":"                default\u003dTrue,"},{"line_number":835,"context_line":"                help\u003d\"Is the v2 volume API enabled\"),"},{"line_number":836,"context_line":"    cfg.BoolOpt(\u0027api_v3\u0027,"},{"line_number":837,"context_line":"                default\u003dTrue,"},{"line_number":838,"context_line":"                help\u003d\"Is the v3 volume API enabled\"),"},{"line_number":839,"context_line":"    cfg.BoolOpt(\u0027extend_attached_volume\u0027,"},{"line_number":840,"context_line":"                default\u003dFalse,"},{"line_number":841,"context_line":"                help\u003d\u0027Does the cloud support extending the size of a volume \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_f77e7c17","line":838,"range":{"start_line":833,"start_character":0,"end_line":838,"end_character":53},"in_reply_to":"bf659307_c3e0c0cf","updated":"2018-04-22 11:28:20.000000000","message":"that case is invalid configuration.  we can document it but cannot throw error as catalog_type name can be anything depends on cloud endpoints.","commit_id":"a33e0ca51befafe1d9540abf8eea75676f214431"}],"tempest/lib/services/volume/v2/availability_zone_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"983ce49a30f6d68810bade9e3c3d31d989adc8cd","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class AvailabilityZoneClient(rest_client.RestClient):"},{"line_number":22,"context_line":"    api_version \u003d \"v2\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    def list_availability_zones(self):"},{"line_number":25,"context_line":"        resp, body \u003d self.get(\u0027os-availability-zone\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_eb1294b4","side":"PARENT","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":22},"updated":"2017-12-14 09:26:22.000000000","message":"this seems backward incompatible change from lib perspective. \nwhere current class AvailabilityZoneClient make v2 APIs request and after change it can make v3 depends on configuration. \n\nWe can keep the old behavior by keeping the class with deprecated cycle.","commit_id":"1d8dfec13ddc3ca04ab940df0a3d1297b4035f2f"}]}