)]}'
{"id":"openstack%2Fnova~803756","triplet_id":"openstack%2Fnova~master~I8775565081960cd6445e94011c5796250c87ed6b","project":"openstack/nova","branch":"master","topic":"bp/configurable-instance-hostnames","hashtags":[],"change_id":"I8775565081960cd6445e94011c5796250c87ed6b","subject":"api: Remove various legacy extension prefixes","status":"ABANDONED","created":"2021-08-06 14:08:13.000000000","updated":"2021-08-19 11:11:19.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"695ed036d0cf1e0e3cc61511bd27d7be621b1c9b","_number":803756,"virtual_id_number":803756,"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-08-06 15:37:41.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"all":[{"value":-1,"date":"2021-08-06 16:58:34.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":0,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-08-06 14:39:29.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":"CC"},{"updated":"2021-08-06 15:37:41.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-08-06 16:31:55.000000000","updated_by":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-08-06 16:58:34.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2021-08-06 17:21:33.000000000","updated_by":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"reviewer":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"state":"CC"}],"messages":[{"id":"469fee1d8de047e24c35670c79c524f1439a4c53","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-08-06 14:08:13.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"248ff07b4fbd8c13d46a5e262abc184115b56e19","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-06 14:39:29.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/cd034232a9df495592ca74f801e4b4dc : SUCCESS in 20m 26s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/c74d21f636c6486baa091aeddc4cff94 : SUCCESS in 28m 13s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"46dda75649dd95f4edf1300c50f7da55233ee5d0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-06 15:37:41.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/926cad62b7ee4d059c70531b2de2343d : SUCCESS in 1h 07m 28s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/a0b3d7ecde664b3cb2e26dbad79dd855 : SUCCESS in 16m 10s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/0f48fba80c3e44be92206c0ec9a662d9 : SUCCESS in 14m 15s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ad4efa6552314f66a721c3fb27b3218c : SUCCESS in 9m 04s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0f10e6d5977045f7a58554f71db9528a : SUCCESS in 12m 33s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d3820b45633348fc8ce9f3ee50c88a05 : SUCCESS in 12m 10s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/785c679c58cd461bb363dd105e54a5a5 : SUCCESS in 13m 15s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/20ac9677312247698ecf956ce24cc7fd : SUCCESS in 9m 15s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/8648fe5585c04a818925a89013eeb470 : SUCCESS in 1h 09m 02s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/17b8a637a78a4c05a5497636c1f0e6bb : SUCCESS in 1h 12m 17s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/f876bdc0ace24f1ba15f0420cab8b419 : SUCCESS in 51m 23s (non-voting)\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/9b5bf33356b845c59a28e694bb534f2a : SUCCESS in 56m 22s (non-voting)\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/571237fffb5546459bf99c658a461645 : SUCCESS in 1h 25m 53s\n- nova-next https://zuul.opendev.org/t/openstack/build/1a1811a4776a422186bb16f2890a16ed : SUCCESS in 1h 15m 21s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/bafe1bf29fc44c76ab62fb9c4139ab0d : SUCCESS in 6m 18s (non-voting)\n- nova-tox-functional-centos8-py36 https://zuul.opendev.org/t/openstack/build/88a2d9c9ab38428c8f95016614bbb63e : SUCCESS in 18m 36s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/2e814ff0bd834f2c89582bb44040de4e : SUCCESS in 18m 52s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/d584d4f46c784b08a1ebccfbaf29eecc : SUCCESS in 16m 09s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/2d1de51c18764cd7a231ffc2c0666993 : SUCCESS in 1h 11m 07s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/3cb6520735bf49debb0228d7628093b3 : SUCCESS in 48m 25s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/3c994313859648b6b42b84fc74a2c749 : SUCCESS in 50m 14s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/5573c075e04e4bf4adc46e50a9d58e2b : SUCCESS in 30m 40s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/beefff4aa68649fd84c80ba7c50dabdf : SUCCESS in 42m 52s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"fe5479dc46787acd9c5ae8ebed20e32a81f1f8fd","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-08-06 16:31:55.000000000","message":"Patch Set 1:\n\nBuild failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/56/803756/1/check/tempest-dsvm-full-focal-py3/363c22e/ : FAILURE in 2h 04m 15s","accounts_in_message":[],"_revision_number":1},{"id":"b08102c7a21682c69244228707eebf1370229177","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-08-06 16:58:34.000000000","message":"Patch Set 1: Code-Review-1\n\nthis is not mentioned in the spec - https://specs.openstack.org/openstack/nova-specs/specs/xena/approved/configurable-instance-hostnames.html\n\nJust to note, we tried to remove these legacy extension from API in train but we agreed not to do those as they are not much benefit of doing it than breaking users\n - Item# 3, L46 in https://etherpad.opendev.org/p/nova-api-cleanup\n\nif we are touching the same API or field like \u0027OS-EXT-SRV-ATTR:hostname instance\u0027 in this BP proposal then we can improve the field by removing the legacy prefix but for other unrelated API we should not. It will unnecessary ask users to change the things with not much gain.","accounts_in_message":[],"_revision_number":1},{"id":"ab5bccc5e21cdc376b0e337d9bf3bbf5167eddf0","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-08-06 17:21:33.000000000","message":"Patch Set 1:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/803756/1/iscsi : SUCCESS in 2h 10m 10s","accounts_in_message":[],"_revision_number":1},{"id":"695ed036d0cf1e0e3cc61511bd27d7be621b1c9b","tag":"autogenerated:gerrit:abandon","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-08-19 11:11:19.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"874826aad1deaa3b0302eccac9d77e3f6c4f4a0c","revisions":{"874826aad1deaa3b0302eccac9d77e3f6c4f4a0c":{"kind":"REWORK","_number":1,"created":"2021-08-06 14:08:13.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/56/803756/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/56/803756/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/56/803756/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/56/803756/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/56/803756/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/56/803756/1"}}},"commit":{"parents":[{"commit":"df88552090ccc087bc44f7a04de81aa4420840a4","subject":"tests: Speed up \u0027servers\u0027 API tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/df88552090ccc087bc44f7a04de81aa4420840a4"}]}],"author":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-08-05 11:52:23.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-08-06 14:07:49.000000000","tz":60},"subject":"api: Remove various legacy extension prefixes","message":"api: Remove various legacy extension prefixes\n\nMultiple APIs accept and return field with prefixes like\n\u0027OS-FLV-EXT-DATA\u0027. These prefixes are a hangover from the days before\nmicroversions, when nova\u0027s API supported extensions. They no longer make\nmuch sense and can and should be dropped. Start doing this now. Given we\ndon\u0027t actually bump the max API microversion, much of what\u0027s added here\nis effectively dead code and not easily testable via functional tests\nbut (a) we\u0027ll address that in future patches and (b) we have existing\ntests to ensure we haven\u0027t broken the existing behavior.\n\nWhile we\u0027re here, we move around some calls in the view code to get\nthings in the order of the microversion that added them.\n\nChange-Id: I8775565081960cd6445e94011c5796250c87ed6b\nSigned-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e\nBlueprint: configurable-instance-hostnames\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/874826aad1deaa3b0302eccac9d77e3f6c4f4a0c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/874826aad1deaa3b0302eccac9d77e3f6c4f4a0c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
