)]}'
{"id":"openstack%2Fcinder~667331","triplet_id":"openstack%2Fcinder~master~I77b14eced31750936d7ac975c2f159e625c478b8","project":"openstack/cinder","branch":"master","topic":"bug/1834181","hashtags":[],"change_id":"I77b14eced31750936d7ac975c2f159e625c478b8","subject":"Fix when create qos set key QoS_Specs_Name has wrong","status":"ABANDONED","created":"2019-06-25 11:20:01.000000000","updated":"2019-06-25 14:12:56.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"26914b1611ec01eed4c72a1707af88b7bcad6c50","_number":667331,"virtual_id_number":667331,"owner":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"actions":{},"labels":{"Verified":{"all":[{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"date":"2019-06-25 14:12:56.000000000","_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"}],"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":{"disliked":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":-1,"date":"2019-06-25 12:02:42.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"}],"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-25 12:02:42.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":"2019-06-25 14:12:56.000000000","updated_by":{"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"},"reviewer":{"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"},"state":"REVIEWER"}],"messages":[{"id":"e0dc91bdc15fa4e0a9cb9f1ec7c86c74d46541f2","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 11:20:01.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"52beb7ea0e56a0e9105e4b24b45a143293edb1b2","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-06-25 12:02:42.000000000","message":"Patch Set 1: Code-Review-1\n\nThis bug is invalid. Setting properties is not a name-value pair. See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html","accounts_in_message":[],"_revision_number":1},{"id":"25d1bc1c880ea8e56b22b6cb8002562b66cd7cd8","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-06-25 12:05:10.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1},{"id":"4c25b093c034cbaf4592b4a7b3fdebc6458f9eb5","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 12:14:28.000000000","message":"Patch Set 1:\n\n\u003e This bug is invalid. Setting properties is not a name-value pair.\n \u003e See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html\n\nWhat if I use command？\nopenstack volume qos create qos --property QoS_Specs_Name\u003dqos-test","accounts_in_message":[],"_revision_number":1},{"id":"b686a2d3f8ee367ef490cd67539cffe4e29b99f9","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 12:16:56.000000000","message":"Patch Set 1:\n\n\u003e \u003e This bug is invalid. Setting properties is not a name-value pair.\n \u003e \u003e See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html\n \u003e \n \u003e What if I use command？\n \u003e openstack volume qos create qos --property QoS_Specs_Name\u003dqos-test\n\nNot everyone knows how to use it","accounts_in_message":[],"_revision_number":1},{"id":"6299819ee293d9cf2252b52538bab850bc293f8a","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-06-25 12:19:22.000000000","message":"Patch Set 1:\n\n\u003e \u003e This bug is invalid. Setting properties is not a name-value pair.\n \u003e \u003e See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html\n \u003e \n \u003e What if I use command？\n \u003e openstack volume qos create qos --property QoS_Specs_Name\u003dqos-test\n\nThen as you have shown, it will fail, because that is not the correct way to use the --property argument of that command.","accounts_in_message":[],"_revision_number":1},{"id":"1b5a713e4cdd56eb6e40997a82dc806ad03ce145","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 12:29:54.000000000","message":"Patch Set 1:\n\n\u003e \u003e \u003e This bug is invalid. Setting properties is not a name-value\n \u003e pair.\n \u003e \u003e \u003e See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html\n \u003e \u003e\n \u003e \u003e What if I use command？\n \u003e \u003e openstack volume qos create qos --property QoS_Specs_Name\u003dqos-test\n \u003e \n \u003e Then as you have shown, it will fail, because that is not the\n \u003e correct way to use the --property argument of that command.\n\nthx\nI found this error in Q version\nhttps://docs.openstack.org/python-openstackclient/queens/cli/command-objects/volume-qos.html\n\nCan I fix it in Q version？","accounts_in_message":[],"_revision_number":1},{"id":"81e2239a0394026d75d7ba1fa6929af687f4b806","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 12:30:48.000000000","message":"Patch Set 1:\n\n\u003e \u003e \u003e \u003e This bug is invalid. Setting properties is not a name-value\n \u003e \u003e pair.\n \u003e \u003e \u003e \u003e See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html\n \u003e \u003e \u003e\n \u003e \u003e \u003e What if I use command？\n \u003e \u003e \u003e openstack volume qos create qos --property QoS_Specs_Name\u003dqos-test\n \u003e \u003e\n \u003e \u003e Then as you have shown, it will fail, because that is not the\n \u003e \u003e correct way to use the --property argument of that command.\n \u003e \n \u003e thx\n \u003e I found this error in Q version\n \u003e https://docs.openstack.org/python-openstackclient/queens/cli/command-objects/volume-qos.html\n \u003e \n \u003e Can I fix it in Q version？\n\nin Q version use command\n\nopenstack volume qos create\n    [--consumer \u003cconsumer\u003e]\n    [--property \u003ckey\u003dvalue\u003e [...] ]\n    \u003cname\u003e","accounts_in_message":[],"_revision_number":1},{"id":"609117c02ae8cc8ce076861d695af7141490ada7","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 12:31:40.000000000","message":"Patch Set 1:\n\n\u003e \u003e \u003e \u003e \u003e This bug is invalid. Setting properties is not a name-value\n \u003e \u003e \u003e pair.\n \u003e \u003e \u003e \u003e \u003e See https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e What if I use command？\n \u003e \u003e \u003e \u003e openstack volume qos create qos --property QoS_Specs_Name\u003dqos-test\n \u003e \u003e \u003e\n \u003e \u003e \u003e Then as you have shown, it will fail, because that is not the\n \u003e \u003e \u003e correct way to use the --property argument of that command.\n \u003e \u003e\n \u003e \u003e thx\n \u003e \u003e I found this error in Q version\n \u003e \u003e https://docs.openstack.org/python-openstackclient/queens/cli/command-objects/volume-qos.html\n \u003e \u003e\n \u003e Can I fix it in Q version？\n \u003e \n \u003e in Q version use command\n \u003e \n \u003e openstack volume qos create\n \u003e [--consumer \u003cconsumer\u003e]\n \u003e [--property \u003ckey\u003dvalue\u003e [...] ]\n \u003e \u003cname\u003e","accounts_in_message":[],"_revision_number":1},{"id":"2eb743f7807dc20e2f48f82cb5e61ab8d596c681","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-06-25 12:33:08.000000000","message":"Patch Set 1:\n\nYeah, looks like the python-openstackclient docs are wrong.","accounts_in_message":[],"_revision_number":1},{"id":"725b9404a87f639b0be7e4a262085130af6b319a","author":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"date":"2019-06-25 12:43:55.000000000","message":"Patch Set 1:\n\n\u003e Yeah, looks like the python-openstackclient docs are wrong.\n\nUsing the this command in the Q version will still cause problems ：\nopenstack volume qos create --consumer \"front-end\" --property \"QoS_Specs_Name\u003dqos-test\" qos-test-str\n\n+--------------------------------------+--------------+-----------+---------------+-----------------------------------------------+\n| ID                                   | Name         | Consumer  | Associations  | Properties                                    |\n+--------------------------------------+--------------+-----------+---------------+-----------------------------------------------+\n| b3a71408-3ada-4b51-93b2-6d3cf0e2ad28 | qos-test     | back-end  |               | None                                          |\n| 9339a77f-531d-47fa-8371-704a3c66e779 | qos-test-str | front-end |               | QoS_Specs_Name\u003d\u0027qos-test\u0027                     |","accounts_in_message":[],"_revision_number":1},{"id":"1586950ed11e4f992e9464a366cadfc440c1bc25","author":{"_account_id":24921,"name":"Hedvig CI","email":"openstack@hedviginc.com","username":"hedvig_ci"},"date":"2019-06-25 14:12:56.000000000","message":"Patch Set 1:\n\n* Hedvig-CI http://os-logserver.hedviginc.com/refs/changes/31/667331/1 : SUCCESS","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"711eb15927380dbb1354de20516fc470033615ca","revisions":{"711eb15927380dbb1354de20516fc470033615ca":{"kind":"REWORK","_number":1,"created":"2019-06-25 11:20:01.000000000","uploader":{"_account_id":29344,"name":"白永君","email":"bai.yongjun@99cloud.net","username":"yj.bai"},"ref":"refs/changes/31/667331/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/31/667331/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/31/667331/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/31/667331/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/31/667331/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/31/667331/1"}}},"commit":{"parents":[{"commit":"852bf88c669fc8652e0e215f559d974762b3d2e9","subject":"Merge \"Add Active/Active HA to Support Matrix\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/852bf88c669fc8652e0e215f559d974762b3d2e9"}]}],"author":{"name":"yj.bai","email":"bai.yongjun@99cloud.net","date":"2019-06-25 11:18:34.000000000","tz":480},"committer":{"name":"yj.bai","email":"bai.yongjun@99cloud.net","date":"2019-06-25 11:20:00.000000000","tz":480},"subject":"Fix when create qos set key QoS_Specs_Name has wrong","message":"Fix when create qos set key QoS_Specs_Name has wrong\n\nChange-Id: I77b14eced31750936d7ac975c2f159e625c478b8\nCloses-Bug: #1834181\nSigned-off-by: yj.bai \u003cbai.yongjun@99cloud.net\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/711eb15927380dbb1354de20516fc470033615ca"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/711eb15927380dbb1354de20516fc470033615ca"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
