)]}'
{"id":"openstack%2Fpython-cinderclient~766882","triplet_id":"openstack%2Fpython-cinderclient~master~Ic97bd74ebd61b0ad65730ab8752a88d4ae283115","project":"openstack/python-cinderclient","branch":"master","hashtags":[],"change_id":"Ic97bd74ebd61b0ad65730ab8752a88d4ae283115","subject":"Change downgrade log message to debug.","status":"ABANDONED","created":"2020-12-14 05:46:15.000000000","updated":"2025-08-26 13:47:47.000000000","total_comment_count":3,"unresolved_comment_count":3,"has_review_started":true,"meta_rev_id":"9fd05fe2d85e47179da19beb698bc3192fc75393","_number":766882,"virtual_id_number":766882,"owner":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-01-14 01:29:57.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2021-03-10 16:48:12.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":{"CC":[{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}],"REVIEWER":[{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-12-14 07:03:37.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"},{"updated":"2021-03-10 16:48:12.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":"2021-03-19 15:44:08.000000000","updated_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"reviewer":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"state":"CC"}],"messages":[{"id":"ea1b2b2cef5041306fa31999c021a92bd96d70f2","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"date":"2020-12-14 05:46:15.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"50091542ffdbaf949514f3d912f35558a7c66c1a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-14 07:03:37.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/ea799897249a41a2b70b00e949d68047 : SUCCESS in 1h 15m 57s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/5ce5bce1c5c44cf494cd1c6844928775 : SUCCESS in 5m 13s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/d13d9c74537f467796424c4073760350 : FAILURE in 6m 03s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c8a0030c9bd2461a82e91e1ecafa8e69 : SUCCESS in 4m 07s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/57129a1d7a28425fbb9791c571a8e9fb : SUCCESS in 4m 41s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/43bcfefe18ee4e32918bafc1d10b481e : SUCCESS in 4m 48s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/23e474653a7e4927bc46e1c73c3a7d23 : SUCCESS in 4m 30s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3b3493562e8d42dcb5f8755421118d7c : SUCCESS in 4m 48s\n- python-cinderclient-functional-py36 https://zuul.opendev.org/t/openstack/build/e65b86c04cfe46e68cb494f91a631a36 : SUCCESS in 26m 07s\n- python-cinderclient-functional-py38 https://zuul.opendev.org/t/openstack/build/47baa3967ee14f3d8736286f3ef7cf7d : SUCCESS in 27m 50s\n- openstack-tox-pylint https://zuul.opendev.org/t/openstack/build/99d2db13a51247dc8c348c1f78fafc72 : FAILURE in 2m 52s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6f2d839f1ea8d8f7e9fa3abcf15bdabff05cd19a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"date":"2021-01-13 23:54:44.000000000","message":"Patch Set 2: Patch Set 1 was rebased","accounts_in_message":[],"_revision_number":2},{"id":"200aa96498f9437099b0939507cdc81e6427e4da","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-01-14 01:29:57.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/289411d555bb4e09bd815ffad18d3e70 : SUCCESS in 1h 24m 57s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/7d5335e6e68545029c28c4a43a53a62e : SUCCESS in 4m 16s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4bb8de9e6a994c2ebcb9c9cfbc4af60c : SUCCESS in 5m 42s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b45afb73a03942ada0d1af68403038d1 : SUCCESS in 4m 51s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cf899b943f0543a8a872da7be7e5b1ec : SUCCESS in 4m 01s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d3223e03d6d44b879eea1297b9e99997 : SUCCESS in 4m 12s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/56c833689ca740d8a40c0b43beb10f5e : SUCCESS in 4m 34s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/50ae7852f18f43948cfb18ecb1fa1e5c : SUCCESS in 6m 12s\n- python-cinderclient-functional-py36 https://zuul.opendev.org/t/openstack/build/ef00e76371b942cdb387bbf1bc86b146 : SUCCESS in 35m 07s\n- python-cinderclient-functional-py38 https://zuul.opendev.org/t/openstack/build/7d8129ce639246238e88bb34c6d0072f : SUCCESS in 50m 42s\n- openstack-tox-pylint https://zuul.opendev.org/t/openstack/build/07303c2d25a4474384d71744ca34889c : FAILURE in 3m 49s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"0696ca4d587cb0a1fb297f2a8b664255082be21f","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2021-03-10 16:48:12.000000000","message":"Patch Set 2: Code-Review-1\n\nI am concerned about this change.  Making it log.DEBUG will mean that we will, likely, be silently changing things in the background on users.  If users don\u0027t like the error they can avoid it by setting the os-volume-api-version to the max spported by that server as reported in the message.","accounts_in_message":[],"_revision_number":2},{"id":"3145c607c64ee2f24067d11600628af92c3c734a","author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"date":"2021-03-15 03:58:33.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2: Code-Review-1\n\u003e \n\u003e I am concerned about this change.  Making it log.DEBUG will mean that we will, likely, be silently changing things in the background on users.  If users don\u0027t like the error they can avoid it by setting the os-volume-api-version to the max spported by that server as reported in the message.\n\nYeah ok, our helpdesk have had a few support issues where our users thought this warning was causing them some issues and was hoping to avoid the confusion. If you look at this from a user of the CLI tool it doesn\u0027t seem to serve any useful purpose but that is just our opinion.","accounts_in_message":[],"_revision_number":2},{"id":"bee276a208245f0937c1d64c6387d1bca78a4a02","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-03-19 15:44:08.000000000","message":"Patch Set 2:\n\n(3 comments)\n\nThis could use some more looking into.  I think we\u0027re not distinguishing two use cases in the code that would call for different messages.  See what you think.\n\nI agree with Jay that we don\u0027t want to be completely silent about switching versions on the user, though.","accounts_in_message":[],"_revision_number":2},{"id":"9fd05fe2d85e47179da19beb698bc3192fc75393","tag":"autogenerated:gerrit:abandon","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2025-08-26 13:47:47.000000000","message":"Abandoned\n\nQuite old and no response to Brian\u0027s comments.  This patch could be updated, please update if I\u0027m wrong.  Abandoning for now to keep the review noise down.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"046c034fafe177124febf7ab600899c47678b0a4","revisions":{"3e488ce4d7f8af399335931f7b314a48c3fcb937":{"kind":"REWORK","_number":1,"created":"2020-12-14 05:46:15.000000000","uploader":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"ref":"refs/changes/82/766882/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/82/766882/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/1"}}},"commit":{"parents":[{"commit":"d92f15a09e59942bb60b7ef3a81c1e21dc16b578","subject":"Fix undesirable raw Python error","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/d92f15a09e59942bb60b7ef3a81c1e21dc16b578"}]}],"author":{"name":"Sam Morrison","email":"sorrison@gmail.com","date":"2020-12-14 05:44:15.000000000","tz":660},"committer":{"name":"Sam Morrison","email":"sorrison@gmail.com","date":"2020-12-14 05:46:12.000000000","tz":660},"subject":"Change downgrade log message to debug.","message":"Change downgrade log message to debug.\n\nThis message that is printed to the user is confusing and is a\nwarning they cannot do anything about.\n\nMy guess is that it is more common to have a client that is newer\nthan the server and so this message is printed everytime to the user.\n\nChange-Id: Ic97bd74ebd61b0ad65730ab8752a88d4ae283115\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/3e488ce4d7f8af399335931f7b314a48c3fcb937"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/3e488ce4d7f8af399335931f7b314a48c3fcb937"}]},"branch":"refs/heads/master"},"046c034fafe177124febf7ab600899c47678b0a4":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2021-01-13 23:54:44.000000000","uploader":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"ref":"refs/changes/82/766882/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/82/766882/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/82/766882/2"}}},"commit":{"parents":[{"commit":"1abc1b5d404c523a696f7186bc4c4b6fc7407cad","subject":"Update requirements and lower-constraints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/1abc1b5d404c523a696f7186bc4c4b6fc7407cad"}]}],"author":{"name":"Sam Morrison","email":"sorrison@gmail.com","date":"2020-12-14 05:44:15.000000000","tz":660},"committer":{"name":"Sam Morrison","email":"sorrison@gmail.com","date":"2021-01-13 23:54:44.000000000","tz":0},"subject":"Change downgrade log message to debug.","message":"Change downgrade log message to debug.\n\nThis message that is printed to the user is confusing and is a\nwarning they cannot do anything about.\n\nMy guess is that it is more common to have a client that is newer\nthan the server and so this message is printed everytime to the user.\n\nChange-Id: Ic97bd74ebd61b0ad65730ab8752a88d4ae283115\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/046c034fafe177124febf7ab600899c47678b0a4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/046c034fafe177124febf7ab600899c47678b0a4"}]},"branch":"refs/heads/master","description":"Rebase"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review priority","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"-label:Review-Priority\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":[],"failing_atoms":["label:Review-Priority\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}