)]}'
{"id":"openstack%2Fpython-openstackclient~762134","triplet_id":"openstack%2Fpython-openstackclient~master~Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80","project":"openstack/python-openstackclient","branch":"master","topic":"osc-first","hashtags":[],"change_id":"Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80","subject":"Switch compute aggregate functions to SDK","status":"MERGED","created":"2020-11-10 12:57:12.000000000","updated":"2020-12-05 10:07:22.000000000","submitted":"2020-12-05 10:04:28.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":39,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"762134-osc-first","meta_rev_id":"e1e25c4f98f660e8860925c3c4e8afc0d0a4236a","_number":762134,"virtual_id_number":762134,"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,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-12-05 10:04:27.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":"","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":2,"date":"2020-12-02 11:29:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"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"]}],"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":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"all":[{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":1,"date":"2020-12-03 16:00:54.000000000","permitted_voting_range":{"min":1,"max":1},"_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"]}],"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-18 11:48:36.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-19 17:22:08.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":"1b53ff54668f04b9b1d373fa5320f42357970126","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-10 12:57:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c9439db5a9eaff256c203e52df1b128b29602474","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-10 22:21:54.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- test-release-openstack https://zuul.opendev.org/t/openstack/build/da9a58a6ef1a43c78db8c79e07e94198 : SUCCESS in 3m 34s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/06d11fef6bdb41439d9bdb1967723301 : SUCCESS in 8m 39s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/6ee746666d33471ea55dc95b60b0c296 : SUCCESS in 4m 53s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/bad1f83e85ae4587a50b1d7879b1058c : SUCCESS in 4m 27s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/cfcfb274092a44ad8c614300d34ea2db : SUCCESS in 4m 22s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/cac2ad5a77b84d42a7327815bec55a92 : SUCCESS in 8m 09s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5ca31087fdcc449585992bdf97008437 : SUCCESS in 4m 27s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/013f64a203034959b82cd8a62f9d0422 : SUCCESS in 7m 44s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/38fb353b8bc54701842d436e29af58be : SUCCESS in 4m 11s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5efe36f3f69a4406944d2804cb9c8054 : SUCCESS in 11m 46s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/92a647d8f4cd406a9561be33e1351006 : SUCCESS in 9m 08s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/f745d183f54c4dffb737675289450eb3 : SUCCESS in 14m 49s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/40fb9b39269f457ab87ea4efbc7acb51 : FAILURE in 56m 43s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/a6139cea2f424e5ea9861b8e7217dc10 : FAILURE in 49m 10s","accounts_in_message":[],"_revision_number":1},{"id":"6be467983a79bb3fb42a0624b8fdd15a20d691fd","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-12 14:47:21.000000000","message":"Patch Set 1: Code-Review-1\n\n(6 comments)\n\nPersonally, I\u0027d rather we added the new cache command in a separate commit, even if it meant using novaclient temporarily. Any chance you could do that?\n\nIn other news, the functional test failure is wrong and you forgot to mention the new command in the reno","accounts_in_message":[],"_revision_number":1},{"id":"c6fecc53de7b42b543c02ad95d2870ae13c93529","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-12 14:47:53.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"38edbbe85c757c134de5a5fd93711edf0ea3bc37","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-12 16:42:46.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"242d4b27c4f8b96d249f1b5b7fb62243e1b12111","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-12 16:44:36.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"eb8c8512828e789410ee6d765b309e775a098e5b","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-12 16:47:50.000000000","message":"Patch Set 2:\n\n(5 comments)\n\n\u003e Personally, I\u0027d rather we added the new cache command in a separate commit, even if it meant using novaclient temporarily. Any chance you could do that?\n\nI agree, that might be it have been better separate (follow-up) change (after switch to SDK), but investing now additional time for intermediate change to revert it immediately back is also a no go.\nI am trying currently to pretty much just switch stuff resource by resource without investing too much into optimizations of the existing code. In this case adding missing op appeared to be easy to be included instead of again stacking changes.","accounts_in_message":[],"_revision_number":2},{"id":"ab6dac53cf15cf6bba34505748ee6bd7acf29ef3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-12 21:55:58.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- test-release-openstack https://zuul.opendev.org/t/openstack/build/68326dfebd63440589b17c1c9906b8c5 : SUCCESS in 3m 03s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/fe0be38d783d41e1b89acea161afa208 : SUCCESS in 4m 38s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/30dc23adea894c8da8ec28d29b92c903 : SUCCESS in 4m 55s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b989549792054fd69d59097b3c005206 : SUCCESS in 5m 02s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/7540eb0b4997429588f6808476f7b538 : SUCCESS in 6m 16s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f19c4e5675ed490ba69ea5deb2c0e9b4 : FAILURE in 3m 51s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d8511fbc3872447a955ac58751103979 : SUCCESS in 4m 14s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/de4ebb3c5ec849c4b5fffffe9bbaf20c : SUCCESS in 5m 05s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/a601f4886ea9426b90e2a88bc0925a9c : SUCCESS in 5m 19s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1d22361907a04d92b31b9c5112af7673 : SUCCESS in 7m 50s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/a0a613eee66b416e8644d9c609e5e2cb : SUCCESS in 4m 21s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/e130a83de6194b9c8fac2d68e28135c6 : SUCCESS in 7m 16s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/8bc424c6f18046899849b1654c0a78a5 : FAILURE in 1h 15m 04s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/4d8fbab6cd1e46829976f8c6dc095b54 : FAILURE in 1h 17m 02s\n\nWarning:\n  Comments left for invalid file openstackclient/tests/unit/volume/v2/test_volume_backend.py","accounts_in_message":[],"_revision_number":3},{"id":"a8dda635123e4b2aeee96c59cfa0867f3474a3cc","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-16 14:32:30.000000000","message":"Patch Set 3:\n\n(3 comments)\n\nThis looks pretty good. Just need to validate [1] myself.\n\n[1] https://review.opendev.org/#/c/762134/3/openstackclient/compute/v2/aggregate.py@185","accounts_in_message":[],"_revision_number":3},{"id":"0febae36c1f1db38ea7d96834b00c8a7c7da95c4","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-16 16:35:19.000000000","message":"Patch Set 3: Code-Review-1\n\n(6 comments)","accounts_in_message":[],"_revision_number":3},{"id":"06f1b88c1efe869371293e8227c6a928dd769b81","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-16 17:00:43.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"f59b0ed12c119ec5f4b35737cae7313ad737f65c","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 11:51:20.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"fd7cb990f45f8e201950da1eff52036271c035cc","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 11:55:23.000000000","message":"Patch Set 3:\n\n(3 comments)","accounts_in_message":[],"_revision_number":3},{"id":"18108f3c0f73fa4be085db9edaa92bdc535ee610","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-17 15:42:32.000000000","message":"Patch Set 4: 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- test-release-openstack https://zuul.opendev.org/t/openstack/build/fb29e923ed6b42b49e46fab61b2af2bb : SUCCESS in 2m 43s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/6e71aaf5463f40c98b1d8b825b73e75a : SUCCESS in 4m 52s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/3f1cabb76dee4fa2b8c940c976334e92 : SUCCESS in 5m 33s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/e31ce36a45e84359aa0b2ef249a29ed9 : SUCCESS in 5m 06s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/6928e8dcc11d42588ea17c43c99eb375 : SUCCESS in 6m 14s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/68ea74c8721645d5a1690bc76d5f2cec : SUCCESS in 4m 04s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/ac5a8eb76d1f4dcf8e4fbe734e8e2ac6 : SUCCESS in 3m 46s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/bcea17605e9748e19d3bd56aa5c2ef2e : SUCCESS in 4m 26s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/0bcbf9cd12c6408890203a52b09f944c : SUCCESS in 4m 46s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/74172f8d3fd64282987bc514c83192c2 : SUCCESS in 7m 17s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/2dbf97a8c3ec4b6c9120d05d40780103 : SUCCESS in 4m 44s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/f146eafd33714d8694ac31e94f0ce6e8 : SUCCESS in 6m 19s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/b48179560fea479c9ca01a445ce6b8aa : FAILURE in 1h 11m 43s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/e0280d3f2e98494ca20af1269a50e520 : FAILURE in 1h 01m 56s","accounts_in_message":[],"_revision_number":4},{"id":"b31b05ea11a76290252a04496fb68d840846ec80","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 16:55:46.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"b1dfd390e2e8e8dfc6484554c78a6716a3ee1575","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 16:56:20.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"4c23327094e42df1684a47b3499147ba3aed1bef","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-17 18:08:25.000000000","message":"Patch Set 5: Code-Review-1\n\n(8 comments)\n\nThis is much better and almost there. Couple of changes inline, mostly to do with what we return to the user via the console","accounts_in_message":[],"_revision_number":5},{"id":"1e282e8f83e40f8eb5545821684e9d9ddd4a48db","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 18:19:32.000000000","message":"Patch Set 5:\n\nuhm, biggest part of your comments are about really changing how the operation work and not related to original intend to do switch to SDK. While I can agree on reworking operations while we are on them, this should be a clear decision and will increase patch and commit message (pretty much in can happen in any other patch in a series). Are we sure we want that?","accounts_in_message":[],"_revision_number":5},{"id":"688c85de27670a5c9da8d1f25202b2a3fce2e7e6","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-17 18:39:58.000000000","message":"Patch Set 5:\n\n\u003e uhm, biggest part of your comments are about really changing how\n \u003e the operation work and not related to original intend to do switch\n \u003e to SDK. While I can agree on reworking operations while we are on\n \u003e them, this should be a clear decision and will increase patch and\n \u003e commit message (pretty much in can happen in any other patch in a\n \u003e series). Are we sure we want that?\n\nI should have been clearer. Apologies. The -1 is for the design of \u0027openstack aggregate cache image\u0027 command (we should use positional args instead). Everything else can and should be done as a follow-up to keep this patch small, yes","accounts_in_message":[],"_revision_number":5},{"id":"53bd776706f2aa06a02bf601c32b9bfcf55a202d","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 18:56:42.000000000","message":"Patch Set 5:\n\n(1 comment)\n\nyeah, good.\nDo you think \u0027openstack aggregate cache image _aggregate_ _image1_ _image2_ _image3_\u0027 is good? I personally feel this is bad UX. I guess we are not using anywhere positional arguments if a list of items is supported (except this is a single argument).","accounts_in_message":[],"_revision_number":5},{"id":"59a5621332aa057f8b41594c1c43026de89c1c6e","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-17 19:03:19.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"48f544cbed1510952a2559ce29c8a42057cca884","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-17 22:48:13.000000000","message":"Patch Set 6: 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- test-release-openstack https://zuul.opendev.org/t/openstack/build/61c57efbcc1447c2a594dd2131c4f00c : SUCCESS in 2m 23s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/75596d96a43d48c5aaadf5706e3135c2 : SUCCESS in 4m 46s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/d98bad978e69449c95adf941a6e28ff1 : SUCCESS in 4m 45s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/12c0c86fc9eb4b6ca0bee112d2c073f8 : SUCCESS in 5m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/cfe0c329f8514fdd8ecccf1c74f0f5c8 : SUCCESS in 6m 32s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2b9cfeb5fd424bf7a1e45d0328b40953 : SUCCESS in 3m 39s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/1e068b5118e84cdba496bf8786e753a0 : SUCCESS in 4m 39s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/2674a1549935453998ff8eda55bc27fd : SUCCESS in 3m 57s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/79c582d8df564f69810c9c0461154093 : SUCCESS in 4m 44s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2cdab9f9bf1f41509224691faec20ea8 : SUCCESS in 8m 25s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/2610173a49ec4355a213cfe657976c1e : SUCCESS in 4m 39s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/2a3fa6c5541e47e582ce83c5bc485daa : SUCCESS in 9m 06s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/5af53bb45fbd4765833b73fed730c240 : FAILURE in 55m 09s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/1c2965e879b4480a9dc3c604a8ebcfa8 : SUCCESS in 1h 00m 58s","accounts_in_message":[],"_revision_number":6},{"id":"e4c22dcfe2032a61920e575c297eea5a11b09526","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-18 09:53:30.000000000","message":"Patch Set 6: Code-Review-1\n\n\u003e (1 comment)\n \u003e \n \u003e yeah, good.\n \u003e Do you think \u0027openstack aggregate cache image _aggregate_ _image1_\n \u003e _image2_ _image3_\u0027 is good? I personally feel this is bad UX. I\n \u003e guess we are not using anywhere positional arguments if a list of\n \u003e items is supported (except this is a single argument).\n\nYes, I think it is, and I\u0027m not the only one [1]. We require at least one image, so a positional argument makes sense. In terms of general UX, we use this same pattern (or accepting one *or more* resources as positional arguments) for almost all \u0027\u003cresource\u003e delete\u0027 commands, e.g. \u0027server delete\u0027:\n\n  openstack server delete .. \u003cserver\u003e [\u003cserver\u003e ...]\n\nwe also use it for \u0027object create\u0027:\n\n  openstack object create ... \u003ccontainer\u003e \u003cfilename\u003e [\u003cfilename\u003e ...]\n\nplus commands like \u0027start server\u0027, \u0027stop server\u0027. There are plenty of examples of this. No reason not to use it here too.\n\n[1] https://review.opendev.org/#/c/688960/2/openstackclient/compute/v2/aggregate.py@422","accounts_in_message":[],"_revision_number":6},{"id":"9cd30cdbf4b00e7653bc32b5281ce07fa5c63a60","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-18 09:54:21.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"495e959405e92a2fd36ac33e0870d586bc3e689e","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-18 10:26:42.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"e5da2578f296548e3ffa35b279b7667219d76833","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-18 10:46:31.000000000","message":"Patch Set 7: Code-Review+2\n\n(2 comments)\n\nOkay, my two must-fix comments are address. This LGTM now","accounts_in_message":[],"_revision_number":7},{"id":"2dd352773e05b8c22caf26585d995f8565557ba4","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-11-18 11:48:36.000000000","message":"Patch Set 7: Code-Review+1\n\nActually, let\u0027s wait for a new release of openstacksdk containing [1] before merging this\n\n[1] https://review.opendev.org/#/c/762131/","accounts_in_message":[],"_revision_number":7},{"id":"4ce345a134b3bea5067f18a353b388ac7e5ed6eb","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-18 15:30:47.000000000","message":"Patch Set 7: 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- test-release-openstack https://zuul.opendev.org/t/openstack/build/2a92345b8f0944d3bbd942f20f26451d : SUCCESS in 2m 44s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/f4c207d7e347407e98c329f38c559439 : SUCCESS in 5m 57s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/808632f7045c4d3c8ae0ccdab0efecc4 : SUCCESS in 6m 32s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/fd2070625fdd43669e8c17338c82de12 : SUCCESS in 4m 53s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/3be76a67835c440eaef154f3bf41062e : SUCCESS in 5m 18s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/becbf31146734d6283d832db449dcb45 : SUCCESS in 3m 44s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/9c26002104194dec9edc3963f9641da6 : SUCCESS in 3m 36s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/794dc07c770c4947b3d873132f8e820d : SUCCESS in 3m 52s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/59f7f40f59004287a5f9c31c0d4d648b : SUCCESS in 4m 45s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3b16c5c2fafa483984e795ec67cbd93e : SUCCESS in 8m 46s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/386bd49aadf14613ad06aee411392bb3 : SUCCESS in 4m 46s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/32f37c20471e48f29de7b95e891dcef3 : SUCCESS in 7m 02s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/b0a3e27ef3334131bf5dd6f7b9366716 : FAILURE in 54m 26s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/d7263563165e4126a4d5c34175447961 : SUCCESS in 53m 13s","accounts_in_message":[],"_revision_number":7},{"id":"2f1464aacf357641a5f332991ea96bdb5021d4b5","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-18 15:32:01.000000000","message":"Patch Set 7:\n\nexpected failure until new SDK release is there","accounts_in_message":[],"_revision_number":7},{"id":"d5266582e2956e9df90c8476400fe879da96d6af","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-11-19 12:09:42.000000000","message":"Patch Set 7:\n\nrecheck","accounts_in_message":[],"_revision_number":7},{"id":"a2737ce560d87358ee4519456387a27a0935676b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-11-19 17:22:08.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/7e1b8fd89b3a4552989e90f3cbc10b94 : SUCCESS in 2m 43s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/0e15878a494548389b25b95ec6b95d58 : SUCCESS in 5m 09s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/055ddbbc678c479497a2578f52b521d1 : SUCCESS in 4m 46s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/f577a288f15f4d90809d1f51e6177e38 : SUCCESS in 5m 54s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/dfdb91a697674870a3a776346645ebc3 : SUCCESS in 5m 12s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/31738019d2ca4f59b35947c855fa383c : SUCCESS in 5m 30s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/e491a64183e7450fa320462b37b0bd72 : SUCCESS in 3m 51s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/6df0362d1f194efa80e214139c8e1290 : SUCCESS in 3m 59s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/bae33b49b89841af9999af0eb4a69b47 : SUCCESS in 3m 57s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5aaeff5ff5fe4f7fa93ca6ed806d02d6 : SUCCESS in 11m 09s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/15573ba247f546989a2ce036d8e4ba72 : SUCCESS in 6m 57s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/5b174905414d434ebf49f55b6a05a5e4 : SUCCESS in 7m 22s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/5fe9d2efe3894998a1a72e869f5cc2ca : SUCCESS in 56m 41s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/d455947fca8644ae92e21276efabec65 : SUCCESS in 51m 49s","accounts_in_message":[],"_revision_number":7},{"id":"02acbe4f928cc42740b0e22426f7306c0071e464","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-12-01 14:56:27.000000000","message":"Patch Set 7: Code-Review-1\n\nopenstacksdk 0.52.0 is out. Can you bump this to reflect that or rebase onto https://review.opendev.org/c/openstack/python-openstackclient/+/750151?","accounts_in_message":[],"_revision_number":7},{"id":"138e16945747c89f14cc1dfdc192cdc2eefc1f0d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-12-02 11:26:54.000000000","message":"Uploaded patch set 8: Patch Set 7 was rebased.","accounts_in_message":[],"_revision_number":8},{"id":"384f527d8a99e7b0cf4e7d1cd77cbc62fa240cb6","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2020-12-02 11:29:55.000000000","message":"Patch Set 8: Code-Review+2\n\nI rebased onto https://review.opendev.org/c/openstack/python-openstackclient/+/750151 which brings in openstacksdk 0.52.0","accounts_in_message":[],"_revision_number":8},{"id":"2c8ae5a9002b056d18bbdbc1a2de9d3f0f71a916","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-02 20:08:43.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/9e440c2125f940648818fc4a9a6b2e55 : SUCCESS in 2m 39s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/3d0eefe930744821a2ac9737f50a998f : SUCCESS in 4m 55s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/0996a3009b3743cca887a5437b6a487c : SUCCESS in 4m 37s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/54f3776d0a26478d9982eacf4f454b37 : SUCCESS in 4m 41s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/cceaab9451e84d6db52c5bf85cfe7974 : SUCCESS in 5m 22s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/6ed3926dd85647b69a83c4cc85ef7688 : SUCCESS in 5m 05s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/e501f20027ba4139b846ac76461cee99 : SUCCESS in 3m 57s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/2d206b016b3d462bace9269d23aa1cf3 : SUCCESS in 4m 13s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/c01472ccc5e04a469f7a10c7b86ce25c : SUCCESS in 4m 34s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a0bbc213be484dd99388e933d9640461 : SUCCESS in 6m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/f66fcbf44d7d4605b64313105848d81f : SUCCESS in 4m 08s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/97c9f3bea163444c93ee8fd93741f0e1 : SUCCESS in 6m 49s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/858b4e29e76b4d468f11a6dea78b9cba : SUCCESS in 1h 01m 42s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/a9fa64f3be414d22bab951ce25b4d0d7 : SUCCESS in 1h 01m 40s","accounts_in_message":[],"_revision_number":8},{"id":"a083e58efb0373adc39e5ada62f8d6e25a69bca1","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-12-03 16:00:54.000000000","message":"Patch Set 8: Workflow+1","accounts_in_message":[],"_revision_number":8},{"id":"261f8c703562c2900fdbc4b04a83cd4115b0f9f0","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-12-04 15:39:06.000000000","message":"Patch Set 8:\n\nrecheck","accounts_in_message":[],"_revision_number":8},{"id":"99c205c741bf04a2d30350dc2fadb76540270f4c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-04 15:40:51.000000000","message":"Patch Set 8: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":8},{"id":"a76d69c689d6fbf5528a47ef17c33540d7de01ef","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-04 19:02:52.000000000","message":"Patch Set 8: Verified-2\n\nThis change depends on a change that failed to merge.","accounts_in_message":[],"_revision_number":8},{"id":"57995cc97a7f15ed4c9c54ba4ed816db858418a0","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-12-04 19:06:41.000000000","message":"Patch Set 8:\n\nrecheck","accounts_in_message":[],"_revision_number":8},{"id":"ce183a47d0455d75bacb331033d38e08e9a3403b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-05 01:27:05.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/26db04a73d7b4ffb84ad7b0b0b0311fe : SUCCESS in 2m 01s\n- openstackclient-check-plugins https://zuul.opendev.org/t/openstack/build/bd8b706218d2441b8c92c6b940d1fcdb : SUCCESS in 5m 18s (non-voting)\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/75f9c731832647e9a8ff78b9b6ffbba3 : SUCCESS in 5m 09s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/592796819d324c608d6fe203c9f5a0db : SUCCESS in 4m 37s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/e46696a30db44ad08106d02a7c5c2b9c : SUCCESS in 5m 17s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5b2a10c775954d379843e34abdd08162 : SUCCESS in 4m 51s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/15653c56a466457aa3d88db28c11feb4 : SUCCESS in 3m 40s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/3afe6060c7ab4173b94bcd3559e5f0b8 : SUCCESS in 4m 12s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/e4f35b9ed8634e6ea25b7087790df736 : SUCCESS in 4m 38s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/44f28c07a1124f3d8fd9e69c699d6523 : SUCCESS in 6m 14s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/881b9264af25474297d0ed33c2177566 : SUCCESS in 4m 40s\n- osc-build-image https://zuul.opendev.org/t/openstack/build/6e9a0d31b8e041fcb909b72b1a9b9d82 : SUCCESS in 6m 38s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/5390941a80f3415ca61309b1e41eca18 : SUCCESS in 56m 41s\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/f04ff2bb40af4f099dd555947557d5cf : SUCCESS in 1h 11m 50s","accounts_in_message":[],"_revision_number":8},{"id":"7964bd8074ce0391c1ab21cf30020714b5b24055","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-05 08:57:17.000000000","message":"Patch Set 8: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":8},{"id":"ac2b4a955d3f1d268502aadbeec089099fd9efc8","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-05 10:04:27.000000000","message":"Patch Set 8: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/379dbcd9b78b4b40b5acb1867a6b1b53 : SUCCESS in 2m 57s\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/2ff43b6982dd453c9d68ad0946b685dc : SUCCESS in 6m 22s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/acc3db589ceb43bdbd73928a11d3e5ca : SUCCESS in 4m 46s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/118d2bc5e0304c4b9e4b976b7db4e5a7 : SUCCESS in 3m 39s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c0875a87a59940a98a11f5c5cefbf5a9 : SUCCESS in 3m 48s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/971738c6b12c43d482b2f1f0144166ad : SUCCESS in 3m 39s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6b66e353bbd94b00bf8612ca452ad31b : SUCCESS in 6m 04s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/45c1b8d47563429fb5e191801bd2fbbe : SUCCESS in 4m 09s\n- osc-upload-image https://zuul.opendev.org/t/openstack/build/dae93599aa05440eae199de0c85c2fba : SUCCESS in 7m 03s\n- osc-functional-devstack https://zuul.opendev.org/t/openstack/build/7d57803e1c7e4440a0951c40b6477cf0 : SUCCESS in 46m 49s","accounts_in_message":[],"_revision_number":8},{"id":"7dc71d2ad21ed6e53aeb180aeab38417e1e1c877","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-05 10:04:28.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":8},{"id":"e1e25c4f98f660e8860925c3c4e8afc0d0a4236a","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-12-05 10:07:22.000000000","message":"Patch Set 8:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/13a0026a0c774b92805c2594a64b16e7 : SUCCESS in 1m 07s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7fb8adcd08754b66a0d963f8541e10ec : SUCCESS in 55s\n- osc-promote-image https://zuul.opendev.org/t/openstack/build/e74e3198c28e441489b01c56a3ea332e : SUCCESS in 40s","accounts_in_message":[],"_revision_number":8}],"current_revision_number":8,"current_revision":"f36a34b675e69a811d5cd48f6bcfd6ce7bda6a5a","revisions":{"428951ed7ac3dcc752def352b021b586265f9e48":{"kind":"REWORK","_number":1,"created":"2020-11-10 12:57:12.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/1"}}},"commit":{"parents":[{"commit":"e2bd1530cf1fc5a84af4dc88829f3c73cdf694f2","subject":"Merge \"functional: Remove test for \u0027quota set --force\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e2bd1530cf1fc5a84af4dc88829f3c73cdf694f2"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/428951ed7ac3dcc752def352b021b586265f9e48"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/428951ed7ac3dcc752def352b021b586265f9e48"}]},"branch":"refs/heads/master"},"af6c447ef7b9aeb6be1b84c573d810d459df8432":{"kind":"REWORK","_number":2,"created":"2020-11-12 16:42:46.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/2"}}},"commit":{"parents":[{"commit":"e2bd1530cf1fc5a84af4dc88829f3c73cdf694f2","subject":"Merge \"functional: Remove test for \u0027quota set --force\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e2bd1530cf1fc5a84af4dc88829f3c73cdf694f2"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-12 16:42:29.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/af6c447ef7b9aeb6be1b84c573d810d459df8432"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/af6c447ef7b9aeb6be1b84c573d810d459df8432"}]},"branch":"refs/heads/master"},"d6dfab81f15fe0baf314a7d8ed6a698666489f34":{"kind":"REWORK","_number":3,"created":"2020-11-12 16:44:36.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/3"}}},"commit":{"parents":[{"commit":"e2bd1530cf1fc5a84af4dc88829f3c73cdf694f2","subject":"Merge \"functional: Remove test for \u0027quota set --force\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e2bd1530cf1fc5a84af4dc88829f3c73cdf694f2"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-12 16:44:04.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d6dfab81f15fe0baf314a7d8ed6a698666489f34"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d6dfab81f15fe0baf314a7d8ed6a698666489f34"}]},"branch":"refs/heads/master"},"288adb69e8587659a168e16c0fac46a18db95a14":{"kind":"REWORK","_number":4,"created":"2020-11-17 11:51:20.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/4"}}},"commit":{"parents":[{"commit":"24edeb24a678b8cf8a8e296b303b367c45678a82","subject":"Merge \"tests: Remove \u0027agent\u0027 functional tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/24edeb24a678b8cf8a8e296b303b367c45678a82"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-17 11:51:02.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/288adb69e8587659a168e16c0fac46a18db95a14"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/288adb69e8587659a168e16c0fac46a18db95a14"}]},"branch":"refs/heads/master"},"b5162191ea994984cb678c903774823449cfb43e":{"kind":"REWORK","_number":5,"created":"2020-11-17 16:55:46.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/5"}}},"commit":{"parents":[{"commit":"24edeb24a678b8cf8a8e296b303b367c45678a82","subject":"Merge \"tests: Remove \u0027agent\u0027 functional tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/24edeb24a678b8cf8a8e296b303b367c45678a82"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-17 16:55:31.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/b5162191ea994984cb678c903774823449cfb43e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/b5162191ea994984cb678c903774823449cfb43e"}]},"branch":"refs/heads/master"},"94b62e5f1cc67a93d33f17d046f09c782de2341a":{"kind":"REWORK","_number":6,"created":"2020-11-17 19:03:19.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/6"}}},"commit":{"parents":[{"commit":"24edeb24a678b8cf8a8e296b303b367c45678a82","subject":"Merge \"tests: Remove \u0027agent\u0027 functional tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/24edeb24a678b8cf8a8e296b303b367c45678a82"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-17 19:02:17.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/94b62e5f1cc67a93d33f17d046f09c782de2341a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/94b62e5f1cc67a93d33f17d046f09c782de2341a"}]},"branch":"refs/heads/master"},"12803d7b5d8ef98ad15dcc5ab6cd9db9392493c7":{"kind":"REWORK","_number":7,"created":"2020-11-18 10:26:42.000000000","uploader":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"ref":"refs/changes/34/762134/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/7"}}},"commit":{"parents":[{"commit":"24edeb24a678b8cf8a8e296b303b367c45678a82","subject":"Merge \"tests: Remove \u0027agent\u0027 functional tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/24edeb24a678b8cf8a8e296b303b367c45678a82"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-18 10:26:26.000000000","tz":60},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/12803d7b5d8ef98ad15dcc5ab6cd9db9392493c7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/12803d7b5d8ef98ad15dcc5ab6cd9db9392493c7"}]},"branch":"refs/heads/master"},"f36a34b675e69a811d5cd48f6bcfd6ce7bda6a5a":{"kind":"TRIVIAL_REBASE","_number":8,"created":"2020-12-02 11:26:54.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/34/762134/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/34/762134/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/34/762134/8"}}},"commit":{"parents":[{"commit":"0f4f42b65281b9b8b4f8fc3e58da8c9d8b68ee08","subject":"Switch compute flavors from novaclient/direct to SDK","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/0f4f42b65281b9b8b4f8fc3e58da8c9d8b68ee08"}]}],"author":{"name":"Artem Goncharov","email":"Artem.goncharov@gmail.com","date":"2020-11-10 12:56:11.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"sfinucan@redhat.com","date":"2020-12-02 11:24:51.000000000","tz":0},"subject":"Switch compute aggregate functions to SDK","message":"Switch compute aggregate functions to SDK\n\nContinue journey towards having OSC consuming SDK for nova part.\n\nDepends-On: https://review.opendev.org/#/c/762131/\nChange-Id: Id16e6c47aa93f02f15f49e1f59f73fecaa3e3b80\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/f36a34b675e69a811d5cd48f6bcfd6ce7bda6a5a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/f36a34b675e69a811d5cd48f6bcfd6ce7bda6a5a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}