)]}'
{"id":"openstack%2Fpython-cinderclient~573622","triplet_id":"openstack%2Fpython-cinderclient~master~I37744bfd0b0bc59682c3e680c1200f608ad3991b","project":"openstack/python-cinderclient","branch":"master","topic":"fix_unecessary_params","hashtags":[],"change_id":"I37744bfd0b0bc59682c3e680c1200f608ad3991b","subject":"Remove unnecessary parameters from volume create APIs","status":"MERGED","created":"2018-06-08 08:40:44.000000000","updated":"2018-07-31 09:40:55.000000000","submitted":"2018-07-26 21:27:19.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"573622-1532640439929-33245e15","meta_rev_id":"0232db2d86dbbaac75b1483c7891e82930badaf5","_number":573622,"virtual_id_number":573622,"owner":{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},{"value":0,"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":2,"date":"2018-07-26 21:27:19.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"},{"value":0,"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},{"value":0,"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"},{"value":2,"date":"2018-07-26 19:33:52.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":2,"date":"2018-07-26 18:14:28.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},{"value":0,"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2018-07-26 16:44:17.000000000","_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"},{"value":0,"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":1,"date":"2018-07-26 20:03:25.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},{"value":0,"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"},{"value":0,"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},{"_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-06-08 12:08:08.000000000","updated_by":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"reviewer":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"state":"REVIEWER"},{"updated":"2018-06-12 01:31:17.000000000","updated_by":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"reviewer":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"state":"REVIEWER"},{"updated":"2018-07-25 16:15:11.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2018-07-26 16:44:17.000000000","updated_by":{"_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"},"reviewer":{"_account_id":26541,"name":"Neha Alhat","email":"neha.alhat@nttdata.com","username":"nehaalhat"},"state":"REVIEWER"},{"updated":"2018-07-26 19:33:52.000000000","updated_by":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"reviewer":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"state":"REVIEWER"},{"updated":"2018-07-26 20:03:25.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"},{"updated":"2018-07-26 21:27:19.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"22137336f5987947a413fea2eb6e246f208d71e2","author":{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},"date":"2018-06-08 08:40:44.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c0669a5e99411642c771de1338808701728dd817","author":{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},"date":"2018-06-08 08:41:24.000000000","message":"Topic changed from bp/s to fix_unecessary_params","accounts_in_message":[],"_revision_number":1},{"id":"d95ba0de85aac328863c0739f32b09f727d602a6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-08 11:54:45.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 http://logs.openstack.org/22/573622/1/check/openstack-tox-pep8/eeeec28/ : SUCCESS in 4m 34s\n- openstack-tox-py27 http://logs.openstack.org/22/573622/1/check/openstack-tox-py27/fe22dc5/ : SUCCESS in 4m 49s\n- openstack-tox-py35 http://logs.openstack.org/22/573622/1/check/openstack-tox-py35/e5ea553/ : SUCCESS in 4m 04s\n- build-openstack-sphinx-docs http://logs.openstack.org/22/573622/1/check/build-openstack-sphinx-docs/f330c72/html/ : SUCCESS in 3m 47s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/22/573622/1/check/legacy-tempest-dsvm-neutron-src/e57565e/ : FAILURE in 1h 00m 22s\n- openstack-tox-pylint http://logs.openstack.org/22/573622/1/check/openstack-tox-pylint/b716259/ : SUCCESS in 3m 50s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/22/573622/1/check/cinderclient-dsvm-functional/e0402e1/ : FAILURE in 31m 17s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/22/573622/1/check/cinderclient-dsvm-functional-identity-v3-only/caa9ff8/ : FAILURE in 29m 24s (non-voting)\n- openstack-tox-lower-constraints finger://ze09.openstack.org/061f2e6d91bb4c94a8c9c5a8a81018f2 : POST_FAILURE in 4m 08s","accounts_in_message":[],"_revision_number":1},{"id":"7f350747d7a6b36b16cd2ed142e1743a536b70fd","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2018-06-08 12:08:08.000000000","message":"Patch Set 1: Code-Review-1\n\nThe problem in the client is that those parameters are always being sent, the client should only send them (even set to None) when the right microversion is being used.","accounts_in_message":[],"_revision_number":1},{"id":"58661f8c4040519428462fe038c3d9707fccc62a","author":{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},"date":"2018-06-11 12:12:11.000000000","message":"Patch Set 1:\n\n\u003e The problem in the client is that those parameters are always being\n \u003e sent, the client should only send them (even set to None) when the\n \u003e right microversion is being used.\n\nThanks gorka for review.\n\n* Below parameters are not required in the request body as per api-ref [1][2], but these parameter are still passed in cinderclient from the beginning. Now in rocky release, we are implementing blueprint [3] which adds schema validation support using jsonschema for each api. Since these below parameters are not part of request body, Cinder service will return 400 error as additionalProperties are not allowed by APIs anymore.\n\nstatus\nattach_status\nuser_id\nproject_id\n\n* source_replica parameter is deprecated from v2 version onwards, if user passes this parameter in the request body in v2/v3, it is returning 400 error on the master branch. Hence removed all references of \"source_replica\" in this patch.\n\n* backup_id is added from microversion 3.47 onwards, so backup_id will be passed in the request body only if api_version is \u003e\u003d 3.47.\n\n[1] : https://developer.openstack.org/api-ref/block-storage/v3/index.html#volumes-volumes\n\n[2] : https://developer.openstack.org/api-ref/block-storage/v2/index.html#volumes-volumes\n\n[3] : https://blueprints.launchpad.net/cinder/+spec/json-schema-validation","accounts_in_message":[],"_revision_number":1},{"id":"b7970d3a6ea27d187778d4603085ee157a1de34d","author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"date":"2018-06-12 01:31:17.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"9698e82b69d836fc206400f708e9a7e75b039959","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-12 02:52:23.000000000","message":"Patch Set 1:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 http://logs.openstack.org/22/573622/1/check/openstack-tox-pep8/6a81474/ : SUCCESS in 2m 45s\n- openstack-tox-py27 http://logs.openstack.org/22/573622/1/check/openstack-tox-py27/97bd33c/ : SUCCESS in 3m 02s\n- openstack-tox-py35 http://logs.openstack.org/22/573622/1/check/openstack-tox-py35/c8e0d2f/ : SUCCESS in 3m 43s\n- build-openstack-sphinx-docs http://logs.openstack.org/22/573622/1/check/build-openstack-sphinx-docs/ecaa17d/html/ : SUCCESS in 3m 04s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/22/573622/1/check/legacy-tempest-dsvm-neutron-src/feed05e/ : FAILURE in 1h 19m 17s\n- openstack-tox-pylint http://logs.openstack.org/22/573622/1/check/openstack-tox-pylint/5895b59/ : SUCCESS in 2m 56s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/22/573622/1/check/cinderclient-dsvm-functional/3bb4b0f/ : FAILURE in 29m 33s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/22/573622/1/check/cinderclient-dsvm-functional-identity-v3-only/6cf9ebd/ : FAILURE in 31m 12s (non-voting)\n- openstack-tox-lower-constraints http://logs.openstack.org/22/573622/1/check/openstack-tox-lower-constraints/ec03949/ : SUCCESS in 3m 04s","accounts_in_message":[],"_revision_number":1},{"id":"3f34a1cd5485d5772279c765aec41b21891fc4c0","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2018-07-23 17:02:44.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"5f7e3598a2e61af1c94df170b5a97630acd0589a","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2018-07-25 16:15:11.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"38b498b8ed4278256669ece83acc7734c3ed6380","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-07-25 19:30:32.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/22/573622/1/check/openstack-tox-pep8/d497f8e/ : SUCCESS in 3m 34s\n- openstack-tox-py27 http://logs.openstack.org/22/573622/1/check/openstack-tox-py27/26b98de/ : SUCCESS in 6m 11s\n- openstack-tox-py35 http://logs.openstack.org/22/573622/1/check/openstack-tox-py35/ab0ae8f/ : SUCCESS in 4m 51s\n- build-openstack-sphinx-docs http://logs.openstack.org/22/573622/1/check/build-openstack-sphinx-docs/2419f67/html/ : SUCCESS in 4m 58s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/22/573622/1/check/legacy-tempest-dsvm-neutron-src/c709287/ : SUCCESS in 1h 31m 50s\n- openstack-tox-pylint http://logs.openstack.org/22/573622/1/check/openstack-tox-pylint/f9dd350/ : SUCCESS in 6m 59s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/22/573622/1/check/cinderclient-dsvm-functional/6c75bce/ : FAILURE in 29m 53s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/22/573622/1/check/cinderclient-dsvm-functional-identity-v3-only/210fd8f/ : FAILURE in 40m 53s (non-voting)\n- openstack-tox-lower-constraints http://logs.openstack.org/22/573622/1/check/openstack-tox-lower-constraints/ea40ecc/ : SUCCESS in 4m 52s","accounts_in_message":[],"_revision_number":1},{"id":"1815fa310351edb6aecc3d5388d89d7d5e9b131a","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2018-07-26 15:54:08.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"9eb9d92833ea3e8b2d4c85a6c513d46ee302fe56","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2018-07-26 16:44:17.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e98f81abed5b28cedb268f14e35d33cdcc712e23","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2018-07-26 18:14:28.000000000","message":"Patch Set 2: Code-Review+2\n\nI am good with this solution.","accounts_in_message":[],"_revision_number":2},{"id":"a9f0112f3f8868d1658c180e65251b2e8755e1f1","author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"date":"2018-07-26 19:33:52.000000000","message":"Patch Set 2: Code-Review+2\n\nwaiting on zuul","accounts_in_message":[],"_revision_number":2},{"id":"8f1fbb23666fa371e617f7952cbbe5c450cc450e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-07-26 19:40:27.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/22/573622/2/check/openstack-tox-pep8/cd3a161/ : SUCCESS in 3m 54s\n- openstack-tox-py27 http://logs.openstack.org/22/573622/2/check/openstack-tox-py27/1cc550f/ : SUCCESS in 4m 14s\n- openstack-tox-py35 http://logs.openstack.org/22/573622/2/check/openstack-tox-py35/de9b797/ : SUCCESS in 4m 58s\n- build-openstack-sphinx-docs http://logs.openstack.org/22/573622/2/check/build-openstack-sphinx-docs/cdeb13f/html/ : SUCCESS in 5m 13s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/22/573622/2/check/legacy-tempest-dsvm-neutron-src/89188ff/ : SUCCESS in 1h 37m 29s\n- openstack-tox-pylint http://logs.openstack.org/22/573622/2/check/openstack-tox-pylint/3c8887a/ : SUCCESS in 6m 26s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/22/573622/2/check/cinderclient-dsvm-functional/3a20d9b/ : FAILURE in 31m 26s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/22/573622/2/check/cinderclient-dsvm-functional-identity-v3-only/1bfa596/ : FAILURE in 28m 06s (non-voting)\n- openstack-tox-lower-constraints http://logs.openstack.org/22/573622/2/check/openstack-tox-lower-constraints/90d8917/ : SUCCESS in 5m 46s","accounts_in_message":[],"_revision_number":2},{"id":"6465c4a54fa411f93679fb89ead26550dbd5bf2b","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2018-07-26 20:03:25.000000000","message":"Patch Set 2: Workflow+1\n\nBombs away!","accounts_in_message":[],"_revision_number":2},{"id":"1cc8c585ff4b195bc7eef078bb12f8b844879d94","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-07-26 20:03:38.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"8df9e0869086182949ff50ac0b9bcbc213439c62","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-07-26 21:27:19.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/22/573622/2/gate/openstack-tox-pep8/d2f800b/ : SUCCESS in 5m 01s\n- openstack-tox-py27 http://logs.openstack.org/22/573622/2/gate/openstack-tox-py27/86c2bd4/ : SUCCESS in 6m 34s\n- openstack-tox-py35 http://logs.openstack.org/22/573622/2/gate/openstack-tox-py35/2e9e454/ : SUCCESS in 4m 18s\n- build-openstack-sphinx-docs http://logs.openstack.org/22/573622/2/gate/build-openstack-sphinx-docs/5f9e3f2/html/ : SUCCESS in 3m 55s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/22/573622/2/gate/legacy-tempest-dsvm-neutron-src/cc1a281/ : SUCCESS in 1h 19m 04s\n- openstack-tox-lower-constraints http://logs.openstack.org/22/573622/2/gate/openstack-tox-lower-constraints/eef9f67/ : SUCCESS in 4m 50s","accounts_in_message":[],"_revision_number":2},{"id":"5d6222048f985f3be4312a6a0d67a84733a33dde","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-07-26 21:27:19.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"0232db2d86dbbaac75b1483c7891e82930badaf5","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2018-07-31 09:40:55.000000000","message":"Created a revert of this change as I46dce6df6876db1adc2020025c01430f63717018","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"8d566689001a442c2312e366acc167af88888fd3","revisions":{"e76b464684bd2a3402f72c55f2f2a25f50d4e3cb":{"kind":"REWORK","_number":1,"created":"2018-06-08 08:40:44.000000000","uploader":{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},"ref":"refs/changes/22/573622/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/22/573622/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/1"}}},"commit":{"parents":[{"commit":"35b99baf6e0da5b13fa72b50d7a20ff1944776de","subject":"Merge \"update the value of OS_AUTH_URL and OS_VOLUME_API_VERSION\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/35b99baf6e0da5b13fa72b50d7a20ff1944776de"}]}],"author":{"name":"Neha Alhat","email":"neha.alhat@nttdata.com","date":"2018-06-07 12:52:16.000000000","tz":330},"committer":{"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","date":"2018-06-08 08:29:48.000000000","tz":330},"subject":"Remove unnecessary parameters from volume create APIs","message":"Remove unnecessary parameters from volume create APIs\n\nAs per Cinder code, following parameters are not required to be\npassed in the request body of create volume API.\n\n    * status\n    * user_id\n    * attach_status\n    * project_id\n    * source_replica\n\nIf you pass these parameters, previously it were ignored but in the\nschema validation changes[1] we don\u0027t allow additionalProperties to be\npassed in the request body. If user passes additional parameters which\nare not as per API specs[2], then it will be rejected with 400 error.\nAnd source replica was part of the replication v1 implementation which\nis no longer available.\nOn patch[3], tempest tests: test_volume_snapshot_create_get_list_delete,\ntest_volume_create_get_delete\" are failing because of these unnecessary\nparameters.\n\nThis patch removes these unnecessary parameters passed to the create\nVolume API.\n\n[1]https://blueprints.launchpad.net/cinder/+spec/json-schema-validation\n[2]https://review.openstack.org/#/c/507386/\n[3]https://review.openstack.org/#/c/573093/\n\nChange-Id: I37744bfd0b0bc59682c3e680c1200f608ad3991b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e76b464684bd2a3402f72c55f2f2a25f50d4e3cb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e76b464684bd2a3402f72c55f2f2a25f50d4e3cb"}]},"branch":"refs/heads/master"},"8d566689001a442c2312e366acc167af88888fd3":{"kind":"REWORK","_number":2,"created":"2018-07-26 16:44:17.000000000","uploader":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"ref":"refs/changes/22/573622/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/22/573622/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/22/573622/2"}}},"commit":{"parents":[{"commit":"0f56085c0286a022c31258689d9a416f77979d90","subject":"Merge \"Remove deprecated CLI options\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0f56085c0286a022c31258689d9a416f77979d90"}]}],"author":{"name":"Neha Alhat","email":"neha.alhat@nttdata.com","date":"2018-06-07 12:52:16.000000000","tz":330},"committer":{"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","date":"2018-07-26 16:29:08.000000000","tz":-300},"subject":"Remove unnecessary parameters from volume create APIs","message":"Remove unnecessary parameters from volume create APIs\n\nAs per Cinder code, following parameters are not required to be\npassed in the request body of create volume API.\n\n    * status\n    * user_id\n    * attach_status\n    * project_id\n    * source_replica\n\nIf you pass these parameters, previously it was ignored but in the\nschema validation changes[1] we don\u0027t allow additionalProperties to be\npassed in the request body. If user passes additional parameters which\nare not as per API specs[2], then it will be rejected with 400 error.\nOn patch[3], tempest tests: test_volume_snapshot_create_get_list_delete,\ntest_volume_create_get_delete\" are failing because of these unnecessary\nparameters.\n\nThis patch removes these unnecessary parameters passed to the create\nVolume API.\n\n[1]https://blueprints.launchpad.net/cinder/+spec/json-schema-validation\n[2]https://review.openstack.org/#/c/507386/\n[3]https://review.openstack.org/#/c/573093/\n\nChange-Id: I37744bfd0b0bc59682c3e680c1200f608ad3991b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8d566689001a442c2312e366acc167af88888fd3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8d566689001a442c2312e366acc167af88888fd3"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}