)]}'
{"id":"openstack%2Fkeystoneauth~751237","triplet_id":"openstack%2Fkeystoneauth~master~I988df57271b57af60fd191b340a8bc735a4549bc","project":"openstack/keystoneauth","branch":"master","topic":"bad-placement","hashtags":[],"change_id":"I988df57271b57af60fd191b340a8bc735a4549bc","subject":"Add support for invalid version documents","status":"ABANDONED","created":"2020-09-11 10:31:58.000000000","updated":"2021-10-05 10:55:47.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d97f6af5e1339dd099361aedc750033293d01cc4","_number":751237,"virtual_id_number":751237,"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":[{"tag":"autogenerated:zuul:check","value":1,"date":"2020-09-11 11:59:37.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2020-09-11 10:34:56.000000000","_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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"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":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"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"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-09-11 11:59: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"}],"messages":[{"id":"39a412404deb47ffd47e798ff654df1e7a1cd34f","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-09-11 10:31:58.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"56a96b5068c66f9ac6830efbceb6c61c6b3380a5","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-09-11 10:34:56.000000000","message":"Patch Set 1:\n\nThis is highly speculative. This has come out of my work on [1]. In short, because these older versions of placement (Queens and earlier) didn\u0027t expose the status field, it\u0027s not possible use any of the version discovery. I\u0027m pretty sure it would be possible to side-step this and force a given version of the API but I have yet to figure out how.\n\n[1] https://review.opendev.org/745375","accounts_in_message":[],"_revision_number":1},{"id":"fc32c1670fb15c34e38bb4b37e0f1135b2f7e405","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-11 11:59:37.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/d6c5562ad23a43d6a5cf1a0e2ad78482 : SUCCESS in 1h 23m 41s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/5b28c3203f8f448fb60070db90034dec : SUCCESS in 7m 13s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b6d7cc9745f0470cbedb9fc4679d5030 : SUCCESS in 3m 57s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/63b9996d5ee14150a60a8ee16f5556c7 : SUCCESS in 5m 12s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/6623fd4b14ca4451a13d5144de2b2ca4 : SUCCESS in 5m 09s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c3e62231c30b4f4697f4882b19063ef4 : SUCCESS in 6m 02s (non-voting)\n- openstacksdk-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/83f4ce39d41c4996857393ccbf591e00 : SUCCESS in 58m 32s\n- openstacksdk-tox-py38-tips https://zuul.opendev.org/t/openstack/build/6178e377bd214c57a433f2e54be601d9 : SUCCESS in 5m 13s\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/14dfdb535d374ee5b8ebdd6d834f40f4 : SUCCESS in 6m 46s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/95ae00f0decd41179c1150a382f97e0b : SUCCESS in 15m 55s","accounts_in_message":[],"_revision_number":1},{"id":"d97f6af5e1339dd099361aedc750033293d01cc4","tag":"autogenerated:gerrit:abandon","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-10-05 10:55:47.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"1ba47e17e4de321e2dbe8fb8be9f8f0b83cd7db6","revisions":{"1ba47e17e4de321e2dbe8fb8be9f8f0b83cd7db6":{"kind":"REWORK","_number":1,"created":"2020-09-11 10:31:58.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/37/751237/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystoneauth","ref":"refs/changes/37/751237/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/37/751237/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/37/751237/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystoneauth refs/changes/37/751237/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystoneauth refs/changes/37/751237/1"}}},"commit":{"parents":[{"commit":"c2d2e7af2989a81104739f7bb0c397c9e98a3949","subject":"Merge \"Correct major version discovery for non-keystone plugins\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/c2d2e7af2989a81104739f7bb0c397c9e98a3949"}]}],"author":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2020-09-11 10:06:55.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2020-09-11 10:31:56.000000000","tz":60},"subject":"Add support for invalid version documents","message":"Add support for invalid version documents\n\nOlder versions of placement didn\u0027t include the status field in their\nversion discovery documents. This was fixed in placement in [1] but\nthere are clouds out there in the wild running this version and it\u0027s not\npossible to use keystoneauth to talk to them. Resolve this by allowing\nus to continue functioning in the absence of this field.\n\n[1] https://review.opendev.org/575117\n\nChange-Id: I988df57271b57af60fd191b340a8bc735a4549bc\nSigned-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/1ba47e17e4de321e2dbe8fb8be9f8f0b83cd7db6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystoneauth/commit/1ba47e17e4de321e2dbe8fb8be9f8f0b83cd7db6"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}