)]}'
{"id":"openstack%2Fpython-openstackclient~751262","triplet_id":"openstack%2Fpython-openstackclient~master~Ibe247825148788c549c2c1e991aae92338cdf557","project":"openstack/python-openstackclient","branch":"master","topic":"osc-first","hashtags":[],"change_id":"Ibe247825148788c549c2c1e991aae92338cdf557","subject":"Switch console url show operations to SDK","status":"MERGED","created":"2020-09-11 12:34:05.000000000","updated":"2020-11-02 17:02:23.000000000","submitted":"2020-11-02 17:00:50.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"751262-1604336451075-bdd4afc4","meta_rev_id":"389cb473ff6856902a77b84b084e2400e406721a","_number":751262,"virtual_id_number":751262,"owner":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"date":"2020-10-24 08:20:33.000000000","_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-11-02 17:00:50.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"}],"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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"all":[{"value":0,"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-10-27 16:09:17.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"}],"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":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"all":[{"value":0,"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2020-11-02 12:12:31.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-11-02 12:12:31.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"},{"updated":"2020-11-02 17:00:50.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":"e9f13d888b7978df2e4bc1499be9224ddf587d3d","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-09-11 12:34:05.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"aa82457cf6012a2bda328bf13f45ecb2a3783e94","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-11 14:29:07.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- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/72ddecc1b0e74ed7aa0e653d88846a2b : SUCCESS in 8m 13s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/1968ff77b69340889952b585914594f0 : SUCCESS in 7m 08s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/99499ba25bb44502bb70ea0ca8b674ea : RETRY_LIMIT in 2m 32s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ce6b8f9ada7340fb8458ce0e7fd11d50 : RETRY_LIMIT in 3m 30s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9377517f1002441382bad4be301957c3 : RETRY_LIMIT in 3m 06s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/adc4171818e349f1a752682ea977771d : SUCCESS in 11m 32s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/90f3a4c7f6864e868e86a308ec0bc2fb : RETRY_LIMIT in 6m 25s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/94e99550562b49dc9a2b3c92789bab6f : RETRY_LIMIT in 3m 57s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/11354b5badf643108331cef2e4dfff07 : SUCCESS in 1h 13m 54s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/4f0e751bf91c4331b67a35610ce8cae6 : SUCCESS in 10m 45s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/9494b564baf9426690480f4d163fd639 : SUCCESS in 58m 10s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/d8fb71ca1b4b447992ce45ae085a6390 : SUCCESS in 56m 17s","accounts_in_message":[],"_revision_number":1},{"id":"1ddfe45d96943045338bb8f12a9ccf2650095c1e","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-09-12 06:44:37.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"6ae75bc697810c9275fc28e8917e45e1f831d1ac","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-12 08:11:00.000000000","message":"Patch Set 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- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/76e38c10709a42008bc4fa816ff35b90 : SUCCESS in 5m 40s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/d963b783f2b1460898d527d9cd361552 : SUCCESS in 6m 01s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/3e837306a4bc4d37b5188a1fdc2fc22a : SUCCESS in 6m 06s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/44bcd3e4b29648f9b7681544bb6396a4 : FAILURE in 6m 48s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/365046b4dcbe4afea695ab63ed4a8748 : SUCCESS in 4m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/978a93b22b2748e7bed1c44f351e19ba : SUCCESS in 4m 55s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/6dbfb442f7d94a57944698c5df6b8743 : SUCCESS in 4m 56s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9ac4564705e9419abc3c87eb3d37f731 : SUCCESS in 7m 07s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/d5aa3af4e4d2430ab8fdbe55fa3ea462 : SUCCESS in 1h 05m 24s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/ca83dd962bac4414958e78ae11df0870 : SUCCESS in 7m 51s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/c17e3426518345c7b69609308f539ece : SUCCESS in 53m 23s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/257153a95388422ba1275c38f89bcf1d : SUCCESS in 53m 06s","accounts_in_message":[],"_revision_number":1},{"id":"1556557871502f48bfebfc1c856ec456b0327d40","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-09-12 08:59:09.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"81a77273a00915878df7778cae3dfb5ef26edfd0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-12 10:12:01.000000000","message":"Patch Set 2: 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- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/d618c02f7fb9429699230cffa1f6950b : SUCCESS in 4m 59s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/64d57b1659e24f428781dc7ccd6e1e5e : SUCCESS in 4m 31s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/72ca84fe310d4b45a29eccac6ee79a80 : SUCCESS in 4m 47s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/7d9dd88ae45e496ab60f317136ef442b : SUCCESS in 4m 57s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e953a5ae865349718edac8736e6b75d6 : SUCCESS in 3m 35s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f54d74f6aae24660840fd657dbd8179c : SUCCESS in 3m 49s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/65f87b500ac94351b6b9ee27858d3907 : SUCCESS in 3m 47s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/dca7f4e4d7f54571b159fa730c64219c : SUCCESS in 5m 42s\n- requirements-check https://zuul.opendev.org/t/openstack/build/bab934d5c58847cf9333298a3d6b4382 : SUCCESS in 2m 33s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/c6f5188505444df5b0e10e3d0c0e8f97 : SUCCESS in 1h 11m 45s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/63b70df650574187acc1f9633d5ed85c : SUCCESS in 7m 12s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/24e01d4dd7094872a48737c6a35d886d : FAILURE in 49m 05s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/4c58691dd2544d65ad7a7f7e3ee48bbc : SUCCESS in 48m 31s","accounts_in_message":[],"_revision_number":2},{"id":"5fb1b1b91f272759470855f2076fc33945e399c7","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-09-13 08:30:17.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"6c6803ea780fafca0e62c023459eac63dfb6ca7a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-13 09:41:48.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/37bd421684ae49d1b2c40e02b8805a37 : SUCCESS in 5m 02s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/f725b6932832498b802600396de574a5 : SUCCESS in 5m 35s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/edaffd527eca4e0d95bcb98719815789 : SUCCESS in 4m 44s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/08032896f49e43cbada71b65e68e9c5b : SUCCESS in 5m 05s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5d76a503e9ad42ba98ff4410c8e78f37 : SUCCESS in 4m 28s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/4bf2e48e64354a499db07b6f8aa908ad : SUCCESS in 4m 00s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/453337252f3244b5ae9309a79cde3014 : SUCCESS in 4m 30s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5a3203c7798b4dc68e08ac2f32f85b8d : SUCCESS in 5m 29s\n- requirements-check https://zuul.opendev.org/t/openstack/build/787192c312a44f34aff95715cb930a30 : SUCCESS in 2m 54s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/40b2d13e8fc24ad1b445fd7c080a8f5c : SUCCESS in 1h 10m 41s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/841d1b653fcc43cba456568acfc7ea5c : SUCCESS in 7m 02s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/8d50b1691e5a42bc949952c02923e792 : SUCCESS in 53m 25s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/1eafab24f61d406db5bbe0c565e99084 : SUCCESS in 46m 47s","accounts_in_message":[],"_revision_number":2},{"id":"a11ca66ae843dc3389009546139f221e3475a31c","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-01 11:40:29.000000000","message":"Patch Set 3: Patch Set 2 was rebased","accounts_in_message":[],"_revision_number":3},{"id":"933158cc48869c099e7b9b951059b09f5f0ff4b6","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-01 11:50:32.000000000","message":"Patch Set 3:\n\nIt looks like we\u0027re using the newest available microversion here, whereas before we used the oldest one. Is that expected?\n\nBefore:\n\n    $ openstack console url show test-server\n    +-------+--------------------------------------------------------------------------------------------+\n    | Field | Value                                                                                      |\n    +-------+--------------------------------------------------------------------------------------------+\n    | type  | novnc                                                                                      |\n    | url   | http://172.20.4.62:6080/vnc_lite.html?path\u003d%3Ftoken%3Dedcb056a-3ede-4961-a328-789d2a1e3c15 |\n    +-------+--------------------------------------------------------------------------------------------+\n\n    $ openstack --os-compute-api-version 2.78 console url show test-server\n    +----------+--------------------------------------------------------------------------------------------+\n    | Field    | Value                                                                                      |\n    +----------+--------------------------------------------------------------------------------------------+\n    | protocol | vnc                                                                                        |\n    | type     | novnc                                                                                      |\n    | url      | http://172.20.4.62:6080/vnc_lite.html?path\u003d%3Ftoken%3D2cdcb2d0-9c7b-429a-a8fb-c42460517214 |\n    +----------+--------------------------------------------------------------------------------------------+\n\nAfter:\n\n    $ openstack console url show test-server\n    +----------+--------------------------------------------------------------------------------------------+\n    | Field    | Value                                                                                      |\n    +----------+--------------------------------------------------------------------------------------------+\n    | protocol | vnc                                                                                        |\n    | type     | novnc                                                                                      |\n    | url      | http://172.20.4.62:6080/vnc_lite.html?path\u003d%3Ftoken%3D186b7b0d-f127-41db-8dba-9960b8cf1205 |\n    +----------+--------------------------------------------------------------------------------------------+\n\nThis appears to be because novaclient was falling back to the old e.g. \u0027os-getVNCConsole\u0027 server actions instead of the \u0027/servers/{id}/remote-consoles\u0027 API introduced in API microversion 2.6.\n\nWould it make sense to check the requested API and do that fallback if necessary?","accounts_in_message":[],"_revision_number":3},{"id":"7aa3b29054ef0bf0f511f77d8e65bb3b874f1dc7","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-01 13:25:37.000000000","message":"Patch Set 3:\n\nUsing the highest available MV was really the goal, since using the oldest one is current reason of permanent complains.\nThe server tells us - I can do more and. Why should we stick to the minimal supported version when we are also able to do more?\nThat is the sense of the complete series of patches switching to SDK - let\u0027s talk the highest version supported by both sides instead of permanently forcing user to pass the version (where feature was added) like --os-compute-api-version\u003d\u00272.61\u0027. Client and server should be able to both agree what they can and what not without user intervention.","accounts_in_message":[],"_revision_number":3},{"id":"a6ab343952092f0fc8ddae32913a54adea0e2847","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-01 13:36:28.000000000","message":"Patch Set 3:\n\nJust to add: We will currently try to use maximum version supported by the server, and not simply highest known","accounts_in_message":[],"_revision_number":3},{"id":"5e15d1b233f051a1232c645b05be4763f18edb2f","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-01 14:03:01.000000000","message":"Patch Set 3: 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- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/82ad93aa95ec4bc281e5b33c381b9a07 : SUCCESS in 5m 23s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/980d5df60a9f43d98bf0f62f4d7b02a6 : SUCCESS in 4m 55s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/060ec5f90a054168986d8beea485d858 : SUCCESS in 5m 04s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/897616cbd89f456a863d109af9630363 : SUCCESS in 5m 13s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0274d4c9bca34ac6b8a73944e160f807 : SUCCESS in 4m 01s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5af3aad22e214c2cb12adaacdcf10f13 : SUCCESS in 3m 52s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/480b1400c55d4827ac573188650e8f22 : SUCCESS in 5m 20s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9f8e72844752459d9979a9e2a829ec7b : SUCCESS in 6m 41s\n- requirements-check https://zuul.opendev.org/t/openstack/build/15c63264d05c42b383b783f8e8772321 : SUCCESS in 2m 47s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/866e2a9890ec4d05a634d4ddd2fe48d1 : SUCCESS in 1h 12m 34s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/442d56bbe774468fa9bc7cfd36518730 : SUCCESS in 7m 45s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/c63e0429f3284b8c83c4d6ac16c20c82 : FAILURE in 55m 56s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/d6f63e4d2ccb4e5c9928694cf3dc3231 : SUCCESS in 52m 03s","accounts_in_message":[],"_revision_number":3},{"id":"76d0c514f760b96e4c0c0fe5c801dfbe739a2323","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-01 14:09:29.000000000","message":"Patch Set 3:\n\n\u003e Using the highest available MV was really the goal, since using the\n \u003e oldest one is current reason of permanent complains.\n \u003e The server tells us - I can do more and. Why should we stick to the\n \u003e minimal supported version when we are also able to do more?\n \u003e That is the sense of the complete series of patches switching to\n \u003e SDK - let\u0027s talk the highest version supported by both sides\n \u003e instead of permanently forcing user to pass the version (where\n \u003e feature was added) like --os-compute-api-version\u003d\u00272.61\u0027. Client and\n \u003e server should be able to both agree what they can and what not\n \u003e without user intervention.\n\nOkay, that\u0027s a noble goal. Aren\u0027t we effectively dropping support for older clouds here though? You won\u0027t be able to use this command with API microversion \u003c 2.6. Shouldn\u0027t we handle those users too?","accounts_in_message":[],"_revision_number":3},{"id":"f09e22de3787592948570af9ad44fd8538b7120e","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-01 14:13:52.000000000","message":"Patch Set 3:\n\n\u003e \u003e Using the highest available MV was really the goal, since using\n \u003e the\n \u003e \u003e oldest one is current reason of permanent complains.\n \u003e \u003e The server tells us - I can do more and. Why should we stick to\n \u003e the\n \u003e \u003e minimal supported version when we are also able to do more?\n \u003e \u003e That is the sense of the complete series of patches switching to\n \u003e \u003e SDK - let\u0027s talk the highest version supported by both sides\n \u003e \u003e instead of permanently forcing user to pass the version (where\n \u003e \u003e feature was added) like --os-compute-api-version\u003d\u00272.61\u0027. Client\n \u003e and\n \u003e \u003e server should be able to both agree what they can and what not\n \u003e \u003e without user intervention.\n \u003e \n \u003e Okay, that\u0027s a noble goal. Aren\u0027t we effectively dropping support\n \u003e for older clouds here though? You won\u0027t be able to use this command\n \u003e with API microversion \u003c 2.6. Shouldn\u0027t we handle those users too?\n\nNo no, if the server supports only \u00272.5\u0027 SDK will fall back to older os-getXXXConsole. This logic is now there (see dependent patch). We are using what both client and server support","accounts_in_message":[],"_revision_number":3},{"id":"638504562160628945b3e786287ad4d5e2df5d8e","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-01 15:40:53.000000000","message":"Patch Set 3: Code-Review-1\n\n\u003e \u003e \u003e Using the highest available MV was really the goal, since using\n \u003e \u003e the\n \u003e \u003e \u003e oldest one is current reason of permanent complains.\n \u003e \u003e \u003e The server tells us - I can do more and. Why should we stick to\n \u003e \u003e the\n \u003e \u003e \u003e minimal supported version when we are also able to do more?\n \u003e \u003e \u003e That is the sense of the complete series of patches switching\n \u003e to\n \u003e \u003e \u003e SDK - let\u0027s talk the highest version supported by both sides\n \u003e \u003e \u003e instead of permanently forcing user to pass the version (where\n \u003e \u003e \u003e feature was added) like --os-compute-api-version\u003d\u00272.61\u0027. Client\n \u003e \u003e and\n \u003e \u003e \u003e server should be able to both agree what they can and what not\n \u003e \u003e \u003e without user intervention.\n \u003e \u003e\n \u003e \u003e Okay, that\u0027s a noble goal. Aren\u0027t we effectively dropping support\n \u003e \u003e for older clouds here though? You won\u0027t be able to use this\n \u003e command\n \u003e \u003e with API microversion \u003c 2.6. Shouldn\u0027t we handle those users too?\n \u003e \n \u003e No no, if the server supports only \u00272.5\u0027 SDK will fall back to\n \u003e older os-getXXXConsole. This logic is now there (see dependent\n \u003e patch). We are using what both client and server support\n\nAh, so that seems to be broken.\n\n  $ openstack --os-compute-api-version 2.5 console url show test-server\n  ResourceNotFound: 404: Client Error for url: http://172.20.4.62/compute/v2.1/servers/de798bd1-2575-4b65-a114-f0bae20aca37/remote-consoles, The resource could not be found","accounts_in_message":[],"_revision_number":3},{"id":"75ea6f178b8f88d8109855096c3fcef1b32263c3","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-02 07:35:09.000000000","message":"Patch Set 3:\n\n\u003e \n \u003e Ah, so that seems to be broken.\n \u003e \n \u003e $ openstack --os-compute-api-version 2.5 console url show\n \u003e test-server\n \u003e ResourceNotFound: 404: Client Error for url: http://172.20.4.62/compute/v2.1/servers/de798bd1-2575-4b65-a114-f0bae20aca37/remote-consoles,\n \u003e The resource could not be found\n\nugh, I see. We actually have not 2 versions to agree on, but 4: minimal exposed by nova, maximum exposed by nova, the one selected by user and the one required by the feature. This is getting funny.\nWill need to rework some bits in SDK, since likely such problem would be wide-spread.","accounts_in_message":[],"_revision_number":3},{"id":"0e7be9c1e3f954f94a6fc4f127038b2a7f3fee2c","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-06 12:59:08.000000000","message":"Patch Set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"1bd372c06780424067ef622d3210f5c393231064","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-06 12:59:38.000000000","message":"Patch Set 4:\n\n\u003e \u003e\n \u003e \u003e Ah, so that seems to be broken.\n \u003e \u003e\n \u003e \u003e $ openstack --os-compute-api-version 2.5 console url show\n \u003e \u003e test-server\n \u003e \u003e ResourceNotFound: 404: Client Error for url: http://172.20.4.62/compute/v2.1/servers/de798bd1-2575-4b65-a114-f0bae20aca37/remote-consoles,\n \u003e \u003e The resource could not be found\n \u003e \n \u003e ugh, I see. We actually have not 2 versions to agree on, but 4:\n \u003e minimal exposed by nova, maximum exposed by nova, the one selected\n \u003e by user and the one required by the feature. This is getting funny.\n \u003e Will need to rework some bits in SDK, since likely such problem\n \u003e would be wide-spread.\n\nAddressed in https://review.opendev.org/756286","accounts_in_message":[],"_revision_number":4},{"id":"bcc02a28c5b75f2fd5206afd90836a40daa65f75","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-06 20:07:10.000000000","message":"Patch Set 4: Verified-1\n\nThis change depends on a change that failed to merge.","accounts_in_message":[],"_revision_number":4},{"id":"9042a64887243bf1d8b9af3bf8daf261609ec6d1","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-08 16:10:07.000000000","message":"Patch Set 5: Patch Set 4 was rebased","accounts_in_message":[],"_revision_number":5},{"id":"45ea343dcdfbe76dec2402a1acfd53e871f1ee28","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-08 16:14:20.000000000","message":"Patch Set 5: Code-Review+2\n\nWith the associated patch, this works as expected now. Thanks! :)","accounts_in_message":[],"_revision_number":5},{"id":"fa7266047aed47e7684847af3c618b1bb20b11e4","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-09 03:56:37.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/f807d20bf33143968b7e8a78f052ffb7 : SUCCESS in 8m 33s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/e845a6f90cc9464abf9664863e73e984 : SUCCESS in 4m 44s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/a4ffdab7149446ab8235d1156baa350e : SUCCESS in 4m 42s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/01adc3b0be7543d79722b80ef2948638 : SUCCESS in 5m 15s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/53d95c2868294c20bb55a2e122a46d7d : SUCCESS in 4m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d1ba70a5ee204a189157daeb29a3563a : SUCCESS in 4m 39s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/90130771c785406caa438c009f832f35 : SUCCESS in 4m 30s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f8f880e3217e42a79c6ca7871d1cd7e2 : SUCCESS in 5m 52s\n- requirements-check https://zuul.opendev.org/t/openstack/build/cf530ca7f25b4675bd56746474cd2065 : SUCCESS in 3m 53s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/5699a30c26ab42b79d970a0dd2143359 : SUCCESS in 1h 26m 14s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/d3840e9b6522431dad16263fa5609eeb : SUCCESS in 9m 30s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/69ba0a20e78f4d7fa5a95189408d4811 : SUCCESS in 57m 19s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/02b2c0531d3043448bf8db657b398b11 : SUCCESS in 52m 36s","accounts_in_message":[],"_revision_number":5},{"id":"720afca667e70545c3fa2825ba843a46f311be2f","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-24 08:20:09.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"d82591c668871e3c7b2f9acc7c9e9358f8643bc1","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-24 08:20:33.000000000","message":"Patch Set 6:\n\nrebased on master","accounts_in_message":[],"_revision_number":6},{"id":"133d2ae9e92aad27073a15a27456738d42f73949","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-24 09:37:54.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/ad3b9c7a66154886ba8e2cc8f0076f64 : SUCCESS in 8m 23s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/e61cf47d109946b18f80024fadbe61f9 : SUCCESS in 4m 57s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/62b4e0c6738a4d908f59cc13b3494cf2 : SUCCESS in 7m 45s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/57a29cac25ee42e7b769300b57ee5957 : SUCCESS in 4m 52s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/57fd26e0e7584dc08d4e1782b93183a8 : SUCCESS in 4m 14s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c14424083dd54f72ae03034b3551f1e6 : SUCCESS in 6m 32s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/70159809fd2d4a54a51e8c48e826da71 : SUCCESS in 3m 44s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a8442ea235564f4f9946227e00efbb3c : SUCCESS in 5m 59s\n- requirements-check https://zuul.opendev.org/t/openstack/build/2b7dda4eeaf547be9ba15b5d8e276ef8 : SUCCESS in 4m 29s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/47655a1f236349f3a8cdacd7506bbe61 : SUCCESS in 14m 05s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/7de3146d439048e4ac09dcb3cfdde411 : SUCCESS in 58m 28s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/cec08e3c54fd461d92de2afdaf0b2245 : SUCCESS in 1h 11m 27s","accounts_in_message":[],"_revision_number":6},{"id":"453ce5b9bfb7391cd111b44ea867c5827e6fb127","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-10-27 16:09:17.000000000","message":"Patch Set 6: Code-Review+2","accounts_in_message":[],"_revision_number":6},{"id":"cadd35c11687e8150dc20b079bba3f802f342e58","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-02 12:12:31.000000000","message":"Patch Set 6: Workflow+1","accounts_in_message":[],"_revision_number":6},{"id":"5592a268129b34e865f1f98ffdad5a1d148b6ee7","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-02 12:13:20.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"4badb216435bf40e8187ab562a62b5cabea5cd8a","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-02 17:00:50.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/12e3d8a04a2c4e5995f5c633e1a0c778 : SUCCESS in 7m 10s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/a2154404d8954bd5bde691a5c97981b9 : SUCCESS in 5m 10s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/3638d23003b74806addd5cec1b549889 : SUCCESS in 4m 27s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/98d23c6d587a4953b1908574b3294784 : SUCCESS in 4m 38s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/1052f3fddd904e4d8088bf7261209dd8 : SUCCESS in 5m 01s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/0c4b956901a744d193cbb516e17dc881 : SUCCESS in 5m 08s\n- requirements-check https://zuul.opendev.org/t/openstack/build/0dd417a476ba4bb09428725ea5f527cb : SUCCESS in 2m 52s\n- osc-upload-image https://zuul.opendev.org/t/openstack/build/2cd3233ecc8f4e8ca589d3433476a80d : SUCCESS in 7m 03s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/2223174ac3ea43e8920a82735cad6097 : SUCCESS in 58m 03s","accounts_in_message":[],"_revision_number":6},{"id":"63e5904fdbbea2410f7452990f5cf1a65e33bc50","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-02 17:00:51.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":6},{"id":"0df5d2d499a2c90424a7a076a128265aaa5a17c2","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-02 17:02:23.000000000","message":"Patch Set 6:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2972fe077d714b90a18648532348589a : SUCCESS in 1m 06s\n- osc-promote-image https://zuul.opendev.org/t/openstack/build/8727cf25ed6349a9b5bbeff6370bf36c : SUCCESS in 51s","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"e05d39abb56b2bf048533b84f44406d415210ffd","revisions":{"d4a1fe5d2c02c8667ac53ac5801c80b2d68a6c1e":{"kind":"REWORK","_number":1,"created":"2020-09-11 12:34:05.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/62/751262/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/62/751262/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/1"}}},"commit":{"parents":[{"commit":"ced30769f7be1edd5bffa82226cf0f5f48b50f5f","subject":"Switch openstack console log show operation to use OpenStackSDK","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/ced30769f7be1edd5bffa82226cf0f5f48b50f5f"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:31:18.000000000","tz":120},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:33:09.000000000","tz":120},"subject":"Switch console url show operations to SDK","message":"Switch console url show operations to SDK\n\nSwitch from using novaclient to SDK for openstack console url show operation.\n\nDepends-On: https://review.opendev.org/#/c/751234/\nChange-Id: Ibe247825148788c549c2c1e991aae92338cdf557\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d4a1fe5d2c02c8667ac53ac5801c80b2d68a6c1e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d4a1fe5d2c02c8667ac53ac5801c80b2d68a6c1e"}]},"branch":"refs/heads/master"},"08932d262aae1e860b157c6b31ea2ce2f6cc244b":{"kind":"REWORK","_number":2,"created":"2020-09-12 08:59:09.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/62/751262/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/62/751262/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/2"}}},"commit":{"parents":[{"commit":"dde28c17efd8adad7142d214c3dbef771b30aad0","subject":"Switch openstack console log show operation to use OpenStackSDK","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/dde28c17efd8adad7142d214c3dbef771b30aad0"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:31:18.000000000","tz":120},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-12 08:58:50.000000000","tz":120},"subject":"Switch console url show operations to SDK","message":"Switch console url show operations to SDK\n\nSwitch from using novaclient to SDK for openstack console url show operation.\n\nDepends-On: https://review.opendev.org/#/c/751234/\nChange-Id: Ibe247825148788c549c2c1e991aae92338cdf557\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/08932d262aae1e860b157c6b31ea2ce2f6cc244b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/08932d262aae1e860b157c6b31ea2ce2f6cc244b"}]},"branch":"refs/heads/master"},"2f1d93c86ca48ec1038b0e357808ac6bfb83d141":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2020-10-01 11:40:29.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/62/751262/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/62/751262/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/3"}}},"commit":{"parents":[{"commit":"97b91aa985a5b9ba0e507e69ac45a39174d6f135","subject":"Switch openstack console log show operation to use OpenStackSDK","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/97b91aa985a5b9ba0e507e69ac45a39174d6f135"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:31:18.000000000","tz":120},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2020-10-01 11:40:29.000000000","tz":0},"subject":"Switch console url show operations to SDK","message":"Switch console url show operations to SDK\n\nSwitch from using novaclient to SDK for openstack console url show operation.\n\nDepends-On: https://review.opendev.org/#/c/751234/\nChange-Id: Ibe247825148788c549c2c1e991aae92338cdf557\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/2f1d93c86ca48ec1038b0e357808ac6bfb83d141"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/2f1d93c86ca48ec1038b0e357808ac6bfb83d141"}]},"branch":"refs/heads/master"},"41cd69bf0a47e8c0f7e631665934f1190c74777e":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2020-10-06 12:59:08.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/62/751262/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/62/751262/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/4"}}},"commit":{"parents":[{"commit":"97b91aa985a5b9ba0e507e69ac45a39174d6f135","subject":"Switch openstack console log show operation to use OpenStackSDK","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/97b91aa985a5b9ba0e507e69ac45a39174d6f135"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:31:18.000000000","tz":120},"committer":{"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","date":"2020-10-06 12:58:59.000000000","tz":0},"subject":"Switch console url show operations to SDK","message":"Switch console url show operations to SDK\n\nSwitch from using novaclient to SDK for openstack console url show operation.\n\nDepends-On: https://review.opendev.org/756286\nChange-Id: Ibe247825148788c549c2c1e991aae92338cdf557\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/41cd69bf0a47e8c0f7e631665934f1190c74777e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/41cd69bf0a47e8c0f7e631665934f1190c74777e"}]},"branch":"refs/heads/master"},"f725ee76e76697a3c2984253cedcfbe96f745a9e":{"kind":"TRIVIAL_REBASE","_number":5,"created":"2020-10-08 16:10:07.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/62/751262/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/62/751262/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/5"}}},"commit":{"parents":[{"commit":"74db8dd65d35b326d3fa1c680b04a668a3f66bdc","subject":"Switch openstack console log show operation to use OpenStackSDK","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/74db8dd65d35b326d3fa1c680b04a668a3f66bdc"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:31:18.000000000","tz":120},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2020-10-08 16:10:07.000000000","tz":0},"subject":"Switch console url show operations to SDK","message":"Switch console url show operations to SDK\n\nSwitch from using novaclient to SDK for openstack console url show operation.\n\nDepends-On: https://review.opendev.org/756286\nChange-Id: Ibe247825148788c549c2c1e991aae92338cdf557\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/f725ee76e76697a3c2984253cedcfbe96f745a9e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/f725ee76e76697a3c2984253cedcfbe96f745a9e"}]},"branch":"refs/heads/master"},"e05d39abb56b2bf048533b84f44406d415210ffd":{"kind":"REWORK","_number":6,"created":"2020-10-24 08:20:09.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/62/751262/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/62/751262/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/62/751262/6"}}},"commit":{"parents":[{"commit":"66f5fa18d46cf34a42a0f13cbd3c35d9379c6ff0","subject":"Merge \"Remove the unused coding style modules\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/66f5fa18d46cf34a42a0f13cbd3c35d9379c6ff0"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-09-11 12:31:18.000000000","tz":120},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-10-24 08:19:52.000000000","tz":120},"subject":"Switch console url show operations to SDK","message":"Switch console url show operations to SDK\n\nSwitch from using novaclient to SDK for openstack console url show operation.\n\nDepends-On: https://review.opendev.org/756286\nChange-Id: Ibe247825148788c549c2c1e991aae92338cdf557\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e05d39abb56b2bf048533b84f44406d415210ffd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e05d39abb56b2bf048533b84f44406d415210ffd"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
