)]}'
{"id":"openstack%2Fpython-openstackclient~797059","triplet_id":"openstack%2Fpython-openstackclient~master~Ifda58214ac442112bf1801ab2f15496d3be96fcb","project":"openstack/python-openstackclient","branch":"master","topic":"hyphenated-commands","hashtags":[],"change_id":"Ifda58214ac442112bf1801ab2f15496d3be96fcb","subject":"Add hyphenated commands","status":"ABANDONED","created":"2021-06-18 11:16:50.000000000","updated":"2021-10-21 16:22:31.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b45456382bd9b675d28ffa82b0d7869c727bc1ba","_number":797059,"virtual_id_number":797059,"owner":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-06-18 13:45:58.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"}],"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":{"approved":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"disliked":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"all":[{"value":-1,"date":"2021-07-01 15:44:17.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2021-07-01 15:36:04.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":2,"date":"2021-06-18 14:03:35.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"}],"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":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-06-18 13:45:58.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-06-18 14:03:35.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2021-06-18 17:37:21.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"CC"},{"updated":"2021-07-01 15:36:04.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"REVIEWER"},{"updated":"2021-07-01 15:44:17.000000000","updated_by":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"reviewer":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"state":"REVIEWER"}],"messages":[{"id":"417cf18c7954044e30e388a362c085d5121e3bf8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-06-18 11:16:50.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b03543253f98af49915d36c9ebc8ceaa4f0896da","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-18 13:45:58.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/e06f3bc3bec94cc7bebb7b8f5ca185c2 : SUCCESS in 3m 01s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/3f4265e25fab45ca9025802dcbe10d67 : SUCCESS in 7m 59s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/c06299d308fa4e19ba608f5eda936efc : SUCCESS in 6m 45s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b1c81a886c7c42dc822fc24fa5198596 : SUCCESS in 6m 25s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/5da9a6e58b0f443da037583d4e64fe90 : SUCCESS in 5m 46s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/32f16c05635c439b969cdde1abe3b0f0 : SUCCESS in 4m 48s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/44029ea795864a4ebe79b9aba781c69d : SUCCESS in 4m 37s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/7398ee60e5f346eda034c4f773b21578 : SUCCESS in 4m 57s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/8982fa6e2e8d4437985a61919fc33891 : SUCCESS in 4m 44s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/cddaf418928e4e6381535a0ea107c08c : SUCCESS in 7m 49s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/cb32d795653c41ef82f3ed7bd75ff138 : SUCCESS in 5m 35s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/f41ad391613b4cec84350bd1af01de0c : SUCCESS in 7m 38s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/f8d423ded54042679e6a313051fc1798 : SUCCESS in 1h 06m 46s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/9fd3d979133840deb3176857d5124326 : SUCCESS in 1h 00m 42s","accounts_in_message":[],"_revision_number":1},{"id":"98985b7811d1038a4bf10de9c13cc2d6af4f788c","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2021-06-18 14:03:35.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"f20db88113d6c56ea034a7dd8da98a3008eadd35","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2021-06-18 17:37:21.000000000","message":"Patch Set 1:\n\nSo I have two comments.\n\nAfter years of work to move everything under a unified client, removing all the hyphens, we want to put them all back?\n\n  old: neutron floatingip-create\n  new: openstack floating ip create\n  newer: openstack floating-ip-create\n\nPersonally I like the current syntax: openstack \u003cobject\u003e \u003coperation\u003e \u003carguments\u003e\nSwitching this could create confusion, and will keep the doc writers busy :)\nJust my opinion. I could live with both but don\u0027t think we should deprecate the current syntax.\n\nWhat about things that aren\u0027t in-tree?  For example, like the Octavia client operations?","accounts_in_message":[],"_revision_number":1},{"id":"3d85b26ae6cf83c274022bb8df994ed10811e420","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2021-07-01 15:36:04.000000000","message":"Patch Set 1: Code-Review-1\n\nUpdating to -1 since after thinking about it I don\u0027t think this is a good idea.","accounts_in_message":[],"_revision_number":1},{"id":"de39c9d10f238d6040d66c6923372d361d103025","author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"date":"2021-07-01 15:44:17.000000000","message":"Patch Set 1: Code-Review-1\n\nI don\u0027t think this is a good idea. We put a lot of work into moving away from the monolithic commands of the old clients.\n\nSpecifically, this adds yet more confusion by having multiple commands for the same thing, some client plugins will probably lag the change, and all of the documentation referencing the CLI will need to be re-written again. Not to mention the stackoverflow articles that would be different.\n\nAs a user, you have the option of creating these aliases locally for your use.\nWhy can\u0027t you just add them to a local aliases list?","accounts_in_message":[],"_revision_number":1},{"id":"b45456382bd9b675d28ffa82b0d7869c727bc1ba","tag":"autogenerated:gerrit:abandon","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-10-21 16:22:31.000000000","message":"Abandoned\n\nYeah, as much as this design pains me, it\u0027s not really worth inflicting real pain on users. Oh well","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d70004b04c06ab0cfc42d41bca0110ead5b4cbe3","revisions":{"d70004b04c06ab0cfc42d41bca0110ead5b4cbe3":{"kind":"REWORK","_number":1,"created":"2021-06-18 11:16:50.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/59/797059/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/59/797059/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/59/797059/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/59/797059/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/59/797059/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/59/797059/1"}}},"commit":{"parents":[{"commit":"2be1bc023b257364e0a6bac5f3b0a15cff6698d4","subject":"setup.cfg: Drop underscores","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/2be1bc023b257364e0a6bac5f3b0a15cff6698d4"}]}],"author":{"name":"Stephen Finucane","email":"sfinucan@redhat.com","date":"2021-01-08 15:25:19.000000000","tz":0},"committer":{"name":"Stephen Finucane","email":"sfinucan@redhat.com","date":"2021-06-18 11:16:07.000000000","tz":60},"subject":"Add hyphenated commands","message":"Add hyphenated commands\n\nInstead of using e.g. \u0027openstack server add floating ip\u0027, start using\n\u0027openstack server add-floating-ip\u0027. This is easier to autocomplete and\navoids conflicts where part of the command name conflicts with an\narbitrary argument. Old aliases are preserved but can be removed in the\nfuture. This is likely to be contentious but it resolves a years-long\nbugbear of mine and in theory should make the UI far more pleasant to\nuse over the long-term.\n\nChange-Id: Ifda58214ac442112bf1801ab2f15496d3be96fcb\nSigned-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d70004b04c06ab0cfc42d41bca0110ead5b4cbe3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d70004b04c06ab0cfc42d41bca0110ead5b4cbe3"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
