)]}'
{"id":"openstack%2Fpython-novaclient~687141","triplet_id":"openstack%2Fpython-novaclient~master~Id354ccfa99e500a598685e6b794c12160ea2a990","project":"openstack/python-novaclient","branch":"master","topic":"bp/image-precache-support","hashtags":[],"change_id":"Id354ccfa99e500a598685e6b794c12160ea2a990","subject":"Add aggregate-cache-images command and client routines","status":"MERGED","created":"2019-10-07 19:08:12.000000000","updated":"2019-10-19 17:48:53.000000000","submitted":"2019-10-19 17:48:52.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":36,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"687141-1571507333098-d71617f4","meta_rev_id":"d348da89bb35d70ea8d41788e8ffeedb974446af","_number":687141,"virtual_id_number":687141,"owner":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"date":"2019-10-18 18:05:29.000000000","_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"date":"2019-10-18 17:54:21.000000000","_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-10-19 17:48:52.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":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"all":[{"value":2,"date":"2019-10-19 15:11:23.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":2,"date":"2019-10-19 09:52:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"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":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"all":[{"value":1,"date":"2019-10-19 15:11:23.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"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},"Review-Priority":{"all":[{"value":0,"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},{"value":0,"_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":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-10-16 14:17:52.000000000","updated_by":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"reviewer":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"state":"REVIEWER"},{"updated":"2019-10-16 18:20:23.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2019-10-18 03:18:59.000000000","updated_by":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"reviewer":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"state":"REVIEWER"},{"updated":"2019-10-18 18:05:29.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2019-10-19 09:52:21.000000000","updated_by":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"reviewer":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"state":"REVIEWER"},{"updated":"2019-10-19 15:11:23.000000000","updated_by":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"reviewer":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"state":"REVIEWER"},{"updated":"2019-10-19 17:48:52.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":"fc2e266c6bbc71c3d84aa60303e01cd600714430","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-07 19:08:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f7729296f5d992160137e69ce3464c5a7c949d0f","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2019-10-07 19:35:47.000000000","message":"Patch Set 1:\n\nWhen was this REST call added to the API?  Has this always been there, and just never supported by the client?  Trying to determine if a microversion is needed here…","accounts_in_message":[],"_revision_number":1},{"id":"f38017507f8f2724c1a2d8afc07896d73b112156","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-07 21:01:44.000000000","message":"Patch Set 1:\n\n\u003e When was this REST call added to the API?  Has this always been\n \u003e there, and just never supported by the client?  Trying to determine\n \u003e if a microversion is needed here…\n\nIt\u0027s proposed and yes, I\u0027ve got microversion stuff to figure out yet:\n\nhttps://review.opendev.org/#/q/topic:bp/image-precache-support+(status:open+OR+status:merged)","accounts_in_message":[],"_revision_number":1},{"id":"d5f31609ffaac097f9d20194bb43b4f84183f8d6","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-07 21:03:47.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nHaving a depends-on in the commit message would have probably avoided the confusion. I realize this is a WIP and such, but release note and all that.","accounts_in_message":[],"_revision_number":1},{"id":"81f2058ce30c5c64d58ae00fe9add4a9dfa2c77f","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-07 21:05:53.000000000","message":"Patch Set 1:\n\n\u003e (1 comment)\n \u003e \n \u003e Having a depends-on in the commit message would have probably\n \u003e avoided the confusion. I realize this is a WIP and such, but\n \u003e release note and all that.\n\nPretty sure it\u0027s not going to land with almost no tests, and WIP in the subject line, but sure, I\u0027ll throw that into the next rev.","accounts_in_message":[],"_revision_number":1},{"id":"6884ddfd3924fce57c2e6b70c493556d689d6093","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-07 21:17:55.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/fb934e91ae4f4328a2eb0f23a84b8663 : TIMED_OUT in 2h 07m 44s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/50747efdf5134f57a4b75acbc4426a53 : FAILURE in 1h 28m 47s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/dc35413f2ca14495af1a89e4dac5ab77 : FAILURE in 5m 10s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ad72a3500e2d4de9a97676e9abf06538 : FAILURE in 4m 39s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/214e4c67880c49ac9ca7c4bdb2e0548d : FAILURE in 4m 04s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/67bd0f793e2a4227adf65fcb5c41d04b : FAILURE in 5m 14s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f2b0948eaa644d038e468589a5c2c067 : FAILURE in 4m 14s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/10004000c0d44ebca3e0879c93a27f91 : FAILURE in 4m 59s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ea008aaa12b145a1a58d8f864dc8f220 : SUCCESS in 3m 54s\n- novaclient-dsvm-functional https://zuul.opendev.org/t/openstack/build/a60f98ff7d24454dac5c04bf4c5bca6e : SUCCESS in 1h 24m 11s","accounts_in_message":[],"_revision_number":1},{"id":"f28661bdb49b7603b6ce69cdb4b031eab2dd026e","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2019-10-07 21:23:49.000000000","message":"Patch Set 1:\n\n*shrug* better safe than sorry; I actually missed the WIP myself, though the missing testing surely would have stopped me :)","accounts_in_message":[],"_revision_number":1},{"id":"d2b76cd2dd24f01ada1d02175846a55850ec1ca0","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-15 18:01:25.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"bddc09a8fbb03eeca3a290cb3ffd93dfa2b9f689","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-15 20:01:20.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/a23e3e2fb0b94f8d88e7357dc7495b86 : SUCCESS in 1h 48m 17s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/91612d2b47a8431d94206bdce6408e12 : SUCCESS in 1h 40m 11s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/88a3d0eb8bfc47ccac74715b05c5a1c9 : SUCCESS in 5m 53s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b60bb61fb766436bbd2e17fa2b9fe63b : SUCCESS in 5m 57s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/14500729fa414b12a001ac5913ac7a69 : SUCCESS in 3m 45s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/fe860ca093014b3d935d2ff0ebbfd594 : SUCCESS in 4m 45s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0e713fc094854344912e61988f6208d9 : SUCCESS in 5m 36s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/f55153f094964290bd17a5237fa1ef47 : SUCCESS in 5m 01s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4b364aefb42c4e449d2c44cf0e06713f : SUCCESS in 6m 38s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7b0d5df1e9b742a6accb85f27d2b4d7a : SUCCESS in 6m 50s\n- novaclient-dsvm-functional https://zuul.opendev.org/t/openstack/build/5fb5e0e672604093b822a145d80f7edb : SUCCESS in 1h 00m 03s","accounts_in_message":[],"_revision_number":2},{"id":"95394a6130f18c941bcc86984fc524d29ddfc26d","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2019-10-15 23:05:46.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"a37b0b2e4750275729ed90c2c6c98b0364ab78d2","author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"date":"2019-10-16 05:03:51.000000000","message":"Patch Set 2: Code-Review-1\n\n(4 comments)\n\ndoc/source/cli/nova.rst should be updated.","accounts_in_message":[],"_revision_number":2},{"id":"7448f80e421eaa4fa1add1c2456892f19c11aae1","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-16 14:06:20.000000000","message":"Patch Set 2:\n\n(4 comments)","accounts_in_message":[],"_revision_number":2},{"id":"0e6a7f0a19ddd9b3b5c24ec925259b6c4ce8b5d4","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-16 14:06:29.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"4fb9ad726c8d5b6deb9abfcab71874d45d652f8f","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-16 14:09:29.000000000","message":"Patch Set 4: Patch Set 3 was rebased","accounts_in_message":[],"_revision_number":4},{"id":"7aca9398ef61637bb52f022aff02bb5a1d23c7c9","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2019-10-16 14:17:52.000000000","message":"Patch Set 4: Code-Review+2","accounts_in_message":[],"_revision_number":4},{"id":"4602719b6cc41fcec81058e13e4080423e0f3edc","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-16 14:45:13.000000000","message":"Patch Set 4: Code-Review-1\n\n(9 comments)","accounts_in_message":[],"_revision_number":4},{"id":"e8cd214242d9afd689976a8b432b1bc397553c8a","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-16 15:40:36.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"11f20234bf2ecd30a074cb0c97d07aeff8ddde4c","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-16 15:48:34.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"75312e29386d61af2b69f9912d6a7ca13fd1ff0e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-16 15:55:29.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/3e5af2e477d2432d9f2f603c0f519655 : SUCCESS in 1h 38m 37s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/698ed2728673430aba7b765e66afcaaf : SUCCESS in 1h 36m 40s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2b588b6fb03b45a28b7c8db677067b20 : SUCCESS in 4m 55s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/16fb1231ed814663b0e52c893b592556 : SUCCESS in 5m 48s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1cd9b7e3af4444688bbbfe73d0939a4f : SUCCESS in 4m 00s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/2fd6da673b294bdfa7379be92b873ad2 : SUCCESS in 3m 52s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/368f90dbffad45e1a7a99e87855e2b43 : SUCCESS in 4m 20s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/e9e69aebd01941ad8b8edab578e5fc15 : SUCCESS in 5m 18s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b86e4fdbbdba491ea4a81b7f1cc1e540 : SUCCESS in 7m 03s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/425eebd9eec64fce91b3107548380c2d : SUCCESS in 7m 02s\n- novaclient-dsvm-functional https://zuul.opendev.org/t/openstack/build/f67310da1878437ab0d3433137be9d5f : SUCCESS in 1h 07m 45s","accounts_in_message":[],"_revision_number":4},{"id":"502890f878aedbbbd6f13b4bf0b95898c788d075","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-16 16:46:29.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"98c82efac5e0c4988621a40ead0f8075afc82c10","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-16 18:18:31.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"442fac3d301f842d92b859a117376c0d6c5e436e","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-16 18:19:57.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"7646fc210e62b8be622124272a6baf0b2a8a59ad","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-16 18:20:23.000000000","message":"Patch Set 6: Code-Review+2\n\nlgtm now","accounts_in_message":[],"_revision_number":6},{"id":"f99355d6839a6fc80922e4fbdea2647ea20bf6f7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-16 21:39:35.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/634fb293a91e41a8aceae363c10dee70 : SUCCESS in 1h 35m 07s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/17153ce1a1eb4c14a4ec3c8e58f55a2f : SUCCESS in 1h 25m 53s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/e6a8aa19281d4ec0b3d095a9ed336e39 : SUCCESS in 6m 34s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/629c7c378c4d4163aaf3dcb71730b742 : SUCCESS in 7m 01s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/22711acd7f4747a8adf7acf5d9eb6997 : SUCCESS in 5m 57s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/1b50d2ab09fa4da39567cb9abe0acd36 : SUCCESS in 5m 17s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/29461473972e49daa1baf69db12e4838 : SUCCESS in 5m 36s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/a5d0bf1af5be43df9b9e197839e093e5 : SUCCESS in 6m 00s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c602d01faf274c858621b2dff42e8838 : SUCCESS in 7m 09s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/d5f7923c8d8f4bcba67028daf46e5c63 : SUCCESS in 4m 53s\n- novaclient-dsvm-functional https://zuul.opendev.org/t/openstack/build/ca3f8d58ee1f41b6baf27a1ff36f289e : SUCCESS in 1h 08m 20s","accounts_in_message":[],"_revision_number":6},{"id":"5ba0cd65dd0fdc26268514b549dce64e688c705e","author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"date":"2019-10-17 02:46:05.000000000","message":"Patch Set 6: Code-Review+2\n\n(3 comments)\n\nLGTM. Thanks. But some nits.","accounts_in_message":[],"_revision_number":6},{"id":"b0d3cedff4bb22718a2dfd03f3f2a85ffa422305","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2019-10-17 11:03:49.000000000","message":"Patch Set 6: Code-Review-1\n\n(1 comment)\n\n`aggregate-cache-images` command allows requesting precaching of several glance images at once. The current implementation of novaclient.v2.images.GlanceManager.find_image leads to a separate API calls for each of the requested images, which is not very cool and can slow down the execution of the command.","accounts_in_message":[],"_revision_number":6},{"id":"52ac7f051b42b603c4d69883153944dd343f0b11","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-17 13:52:41.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"24967a453fe4ee52115e928105985d00a35756fc","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-17 16:33:03.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"1b4c1dd6abed1d5bf7960930d6bc93eba3670252","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2019-10-17 21:25:57.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"e891fe3b807e814ec49fdffed833bbe47c403d08","author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"date":"2019-10-18 03:18:59.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"541f9264ecd29af12c3cabdfec48416d32aa6fed","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 13:52:52.000000000","message":"Patch Set 6:\n\nI can see about adding the find_images() thing after this lands and updating it, but I really don\u0027t think that it there\u0027s a major concern over the number of images. The server side will do the same thing, checking each image sequentially, so it needs the same treatment.","accounts_in_message":[],"_revision_number":6},{"id":"0f589512fcb860fb2cac0d3f83e7d6353147f107","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-18 13:57:10.000000000","message":"Patch Set 6:\n\n\u003e I can see about adding the find_images() thing after this lands and updating it, but I really don\u0027t think that it there\u0027s a major concern over the number of images. The server side will do the same thing, checking each image sequentially, so it needs the same treatment.\n\nAlso note this is an admin-only operation (by default policy anyway) and likely going to be part of a cron or some sort of image build pipeline so I would expect automation to fire this off and just make sure it didn\u0027t blow up but performance is a secondary concern at the moment.","accounts_in_message":[],"_revision_number":6},{"id":"bf9b6aee808a03e2ef8a2f05f1c6eec4f7c9bb51","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-10-18 14:50:08.000000000","message":"Patch Set 6:\n\nFWIW, this discussion reminded me of a performance related issue I dealt with recently in OSC [1] where the problem involved doing thousands of image/flavor lookups (they have \u003e 3K images) and the command taking up to 60 seconds as a result.\n\nI think there should definitely at least be a follow up here. Maybe Andrey could stack a patch on top of this one now since he already knows what to do, as a compromise?\n\nAnd then since Dan mentioned there is similar logic on the server side, we should add something similar there too.\n\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1724167","accounts_in_message":[],"_revision_number":6},{"id":"9fde9ccdae6224568274670c53a4544822814ef7","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 15:02:00.000000000","message":"Patch Set 6:\n\n\u003e FWIW, this discussion reminded me of a performance related issue I\n \u003e dealt with recently in OSC [1] where the problem involved doing\n \u003e thousands of image/flavor lookups (they have \u003e 3K images) and the\n \u003e command taking up to 60 seconds as a result.\n \u003e \n \u003e I think there should definitely at least be a follow up here. Maybe\n \u003e Andrey could stack a patch on top of this one now since he already\n \u003e knows what to do, as a compromise?\n \u003e \n \u003e And then since Dan mentioned there is similar logic on the server\n \u003e side, we should add something similar there too.\n \u003e \n \u003e [1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1724167\n\nI\u0027m not sure this is similar as this is only looking up images that you ask it to in a given instantiation of this command. And, pre-caching 3K (or likely even 1K) images wouldn\u0027t really make sense in any scenario I can think of. We\u0027re only doing the image lookup as a courtesy to the user in case they fat-finger an image name or id on the command line. The API version of this doesn\u0027t do the checking.\n\nI\u0027ve already started mangling my patch locally to do what Andrey has asked. His code always makes two calls, even if I just provide one image, so that is materially worse for the basic case and needs some more logic to make it better. It also doesn\u0027t work well with the current fake/fixture stuff here in the client, so I\u0027m trying to figure out how to unwind that.\n\nI of course think that I\u0027m not really introducing anything new here that means this should be blocked on a performance improvement, but I\u0027m trying.","accounts_in_message":[],"_revision_number":6},{"id":"65b3107508827b7d64caeb91bd7bf777bf46af71","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-10-18 15:30:09.000000000","message":"Patch Set 6:\n\n\u003e I\u0027m not sure this is similar as this is only looking up images that\n \u003e you ask it to in a given instantiation of this command. And,\n \u003e pre-caching 3K (or likely even 1K) images wouldn\u0027t really make\n \u003e sense in any scenario I can think of. We\u0027re only doing the image\n \u003e lookup as a courtesy to the user in case they fat-finger an image\n \u003e name or id on the command line. The API version of this doesn\u0027t do\n \u003e the checking.\n\n\"And, pre-caching 3K (or likely even 1K) images wouldn\u0027t really make sense in any scenario I can think of.\"\n\nThat\u0027s fair, and I can\u0027t claim to know or predict whether someone will want to cache 1000 images as I am not part of The Edge(tm). But, since I have familiarity with Yahoo/Oath/VerizonMedia, I hesitate to assume too much.\n\nOK, makes sense that this is a courtesy thing for the user (same as the rhbz I linked earlier) and not an issue on the API side. When you said, \"The server side will do the same thing, checking each image sequentially, so it needs the same treatment,\" I had thought that meant that there was batching we could do there as well.\n\n \u003e I\u0027ve already started mangling my patch locally to do what Andrey\n \u003e has asked. His code always makes two calls, even if I just provide\n \u003e one image, so that is materially worse for the basic case and needs\n \u003e some more logic to make it better. It also doesn\u0027t work well with\n \u003e the current fake/fixture stuff here in the client, so I\u0027m trying to\n \u003e figure out how to unwind that.\n \u003e \n \u003e I of course think that I\u0027m not really introducing anything new here\n \u003e that means this should be blocked on a performance improvement, but\n \u003e I\u0027m trying.\n\nAck. FWIW I thought a follow up would have been OK here as I think first iterations don\u0027t generally have to be optimized. Though, in this case, I\u0027d want to add the follow up right away because the performance issue is known and the general approach of the solution is also known.","accounts_in_message":[],"_revision_number":6},{"id":"aff9b22f55c4787bb201763a123822a5b208fadb","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 16:04:10.000000000","message":"Patch Set 6:\n\n\u003e OK, makes sense that this is a courtesy thing for the user (same as\n \u003e the rhbz I linked earlier)\n\nBut that scales with the number of images in the response from a server list. That means if I\u0027m listing all servers, the client will look up every image when it first encounters it in the list. Thus there\u0027s not much I can do about it as a user if I still want to see the listing.\n\nIn the case with this image precache command, it is a \"I provided you with N images and you did N lookups. If I don\u0027t like that, I could provide fewer images per request\".\n\n \u003e and not an issue on the API side. When\n \u003e you said, \"The server side will do the same thing, checking each\n \u003e image sequentially, so it needs the same treatment,\" I had thought\n \u003e that meant that there was batching we could do there as well.\n\nThere is and I did. My point was, the server side already does the lookup of each image oneshot-style (just like this) and it needs batching as well. The server side is already merged and we discussed follow-ups for a few situations like this.\n\n \u003e Ack. FWIW I thought a follow up would have been OK here as I think\n \u003e first iterations don\u0027t generally have to be optimized. Though, in\n \u003e this case, I\u0027d want to add the follow up right away because the\n \u003e performance issue is known and the general approach of the solution\n \u003e is also known.\n\nYes, well, Andrey took the shot that it would never happen as a follow-up and the other cores didn\u0027t +W so I took that to mean it needs to happen now.","accounts_in_message":[],"_revision_number":6},{"id":"d28db9e35cbc08d67979b534faf39ac55faca121","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 16:16:53.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"120fe38b29b4130169ea10d4c10684db122f4bde","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-10-18 17:36:26.000000000","message":"Patch Set 6:\n\n\u003e But that scales with the number of images in the response from a\n \u003e server list. That means if I\u0027m listing all servers, the client will\n \u003e look up every image when it first encounters it in the list. Thus\n \u003e there\u0027s not much I can do about it as a user if I still want to see\n \u003e the listing.\n \u003e \n \u003e In the case with this image precache command, it is a \"I provided\n \u003e you with N images and you did N lookups. If I don\u0027t like that, I\n \u003e could provide fewer images per request\".\n\nOK, I think I see your point. The server list doesn\u0027t \"have to\" lookup every image (if there are dupes), but the image precache must lookup every image by definition. It\u0027s a matter of whether that lookup can be batched or not.\n\n \u003e \u003e and not an issue on the API side. When\n \u003e \u003e you said, \"The server side will do the same thing, checking each\n \u003e \u003e image sequentially, so it needs the same treatment,\" I had\n \u003e thought\n \u003e \u003e that meant that there was batching we could do there as well.\n \u003e \n \u003e There is and I did. My point was, the server side already does the\n \u003e lookup of each image oneshot-style (just like this) and it needs\n \u003e batching as well. The server side is already merged and we\n \u003e discussed follow-ups for a few situations like this.\n\nOK, I guess it\u0027s just me having trouble parsing your words.\n\n \u003e \u003e Ack. FWIW I thought a follow up would have been OK here as I\n \u003e think\n \u003e \u003e first iterations don\u0027t generally have to be optimized. Though, in\n \u003e \u003e this case, I\u0027d want to add the follow up right away because the\n \u003e \u003e performance issue is known and the general approach of the\n \u003e solution\n \u003e \u003e is also known.\n \u003e \n \u003e Yes, well, Andrey took the shot that it would never happen as a\n \u003e follow-up and the other cores didn\u0027t +W so I took that to mean it\n \u003e needs to happen now.\n\nFair. I thought maybe he might be convinced upon review of the latest comments and the idea of a \"right now\" follow up rather than a \"later\" follow up.","accounts_in_message":[],"_revision_number":6},{"id":"bcaec758a007fbd8aca8a8948fe6f9e7d78ed286","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 17:49:35.000000000","message":"Uploaded patch set 8: Patch Set 7 was rebased.","accounts_in_message":[],"_revision_number":8},{"id":"f0ef14bd4438f7ccb0f2179451ee706475b34d9e","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 17:50:05.000000000","message":"Uploaded patch set 9: Patch Set 8 was rebased.","accounts_in_message":[],"_revision_number":9},{"id":"fd5b24ba889f2ec70961a72680d6a8c2e2aa52c6","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-10-18 17:54:21.000000000","message":"Patch Set 9:\n\n\u003e OK, I think I see your point. The server list doesn\u0027t \"have to\"\n \u003e lookup every image (if there are dupes), but the image precache\n \u003e must lookup every image by definition. It\u0027s a matter of whether\n \u003e that lookup can be batched or not.\n\nNo, that\u0027s not what I\u0027m saying. The bug you referenced is not about lots of servers, it\u0027s about lots of images. Presumably the problem is if I list 1000 servers, and each has a unique image, then server list is just constantly going to glance to show info about the image during a list. My point is that server list doing that inefficiently is a real problem because ... I just need to list my servers and the performance is terrible. That affects everyone, admins, users, and isn\u0027t really workaroundable in a reasonable way.\n\nThis image pre-cache thing is just providing a list of images to pre-cache on computes. It\u0027s a thing that would likely run from cron, or some script. It has no output. There\u0027s no real waiting around for it, it queues a background process on the server (after it has validated the images). If I _need_ to request a thousand images be pre-cached and that is for some reason too slow (which doesn\u0027t really make sense since this would be run non-interactively in most cases), then I could split it into pieces and request 10 images at a time or something, which would create parallel tasks on the server side and split up the work. That\u0027s not a reasonable thing I can do for listing servers.","accounts_in_message":[],"_revision_number":9},{"id":"add4bd4d763a299017e2bb3edd410fcc78201062","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2019-10-18 18:05:29.000000000","message":"Patch Set 9:\n\n\u003e \u003e OK, I think I see your point. The server list doesn\u0027t \"have to\"\n \u003e \u003e lookup every image (if there are dupes), but the image precache\n \u003e \u003e must lookup every image by definition. It\u0027s a matter of whether\n \u003e \u003e that lookup can be batched or not.\n \u003e \n \u003e No, that\u0027s not what I\u0027m saying. The bug you referenced is not about\n \u003e lots of servers, it\u0027s about lots of images. Presumably the problem\n \u003e is if I list 1000 servers, and each has a unique image, then server\n \u003e list is just constantly going to glance to show info about the\n \u003e image during a list. My point is that server list doing that\n \u003e inefficiently is a real problem because ... I just need to list my\n \u003e servers and the performance is terrible. That affects everyone,\n \u003e admins, users, and isn\u0027t really workaroundable in a reasonable way.\n \u003e \n \u003e This image pre-cache thing is just providing a list of images to\n \u003e pre-cache on computes. It\u0027s a thing that would likely run from\n \u003e cron, or some script. It has no output. There\u0027s no real waiting\n \u003e around for it, it queues a background process on the server (after\n \u003e it has validated the images). If I _need_ to request a thousand\n \u003e images be pre-cached and that is for some reason too slow (which\n \u003e doesn\u0027t really make sense since this would be run non-interactively\n \u003e in most cases), then I could split it into pieces and request 10\n \u003e images at a time or something, which would create parallel tasks on\n \u003e the server side and split up the work. That\u0027s not a reasonable\n \u003e thing I can do for listing servers.\n\nOK, I see what you mean now. Thanks for explaining with the examples, that helped a lot.","accounts_in_message":[],"_revision_number":9},{"id":"4f5b1a1f067881d17145114f09e4784f6a534497","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-18 19:49:19.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/2b1e4eaf678d42c1a3caeb39db8fc76d : SUCCESS in 1h 57m 41s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/fc16671434b44b499ea1ded2e18e2093 : SUCCESS in 1h 24m 47s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/af92e4cd4fb64ec19d05d7b95ba23003 : SUCCESS in 4m 58s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/51e6d507e79a4eb9840078dfb174eee9 : SUCCESS in 5m 13s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/819143681694432cb04ee5b88239176d : SUCCESS in 3m 55s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/4fb65ebf76b642e2ae1117bfb8085519 : SUCCESS in 4m 01s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/780b4a3a90d142fba3aab83b155c4e1f : SUCCESS in 3m 46s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/77fed2306be34d779faccd9e9b6c0e67 : SUCCESS in 4m 32s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1d3537193d5c41fe9bb87cd89df2b2f0 : SUCCESS in 5m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/1ad40673339644e3bc0b60c07eca002c : SUCCESS in 7m 29s\n- novaclient-dsvm-functional https://zuul.opendev.org/t/openstack/build/8cc2363ce3cc4905bfc0b89704057a6a : SUCCESS in 1h 05m 28s","accounts_in_message":[],"_revision_number":9},{"id":"021d059db5d00b0b08240e7f2357ffde5d428daa","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2019-10-19 09:52:21.000000000","message":"Patch Set 9: Code-Review+2","accounts_in_message":[],"_revision_number":9},{"id":"9a90798aa758b073da6e84d5216ac84f610514fb","author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"date":"2019-10-19 15:11:23.000000000","message":"Patch Set 9: Code-Review+2 Workflow+1\n\n(2 comments)","accounts_in_message":[],"_revision_number":9},{"id":"bceb773330dca7b5506d8a3d80b2ff26d5775bf6","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-19 15:11:35.000000000","message":"Patch Set 9: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":9},{"id":"29938b118f1097ffc3f7e792e200eada1f90b432","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-19 17:48:52.000000000","message":"Patch Set 9: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/ad015610d0064c5c9ac1d06749be777a : SUCCESS in 1h 27m 00s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/80b4ce359f1d4eeea3617985bc85af0f : SUCCESS in 1h 22m 18s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/6f9ce29e919b4f3cb2857368fe31ea90 : SUCCESS in 4m 21s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/09046f9823be4d839e49ac2cd48e2597 : SUCCESS in 3m 33s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/ccc733281cd647e9926653c00437e0e3 : SUCCESS in 3m 44s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/58b74034933b4705b768e80633aa05af : SUCCESS in 3m 31s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/441f0b98ddc24cc6864c96878fc09d44 : SUCCESS in 4m 53s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/582d5adc0ac4485a80fb8ea39ca78b98 : SUCCESS in 5m 05s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/21f6d36bc81f479298927b897290bb3b : SUCCESS in 4m 53s\n- novaclient-dsvm-functional https://zuul.opendev.org/t/openstack/build/9ee59fea09b4416ca96c58d2dfae9392 : SUCCESS in 59m 05s","accounts_in_message":[],"_revision_number":9},{"id":"e89a0fee02d8fa4113e5b3e1302f21294fbd0dae","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-19 17:48:53.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":9}],"current_revision_number":9,"current_revision":"71c29a184b388b38d81dc609b9883c5e8f7166cf","revisions":{"53b9dddb956f17aed982e1511b67330a79a3ce2c":{"kind":"REWORK","_number":1,"created":"2019-10-07 19:08:12.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/1"}}},"commit":{"parents":[{"commit":"48634ed51e64f10424eef5bfb18dc63eda051c1d","subject":"Add release note for bug 1845322","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/48634ed51e64f10424eef5bfb18dc63eda051c1d"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 19:08:04.000000000","tz":-420},"subject":"WIP: Add aggregate-cache-images command and client routines","message":"WIP: Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/53b9dddb956f17aed982e1511b67330a79a3ce2c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/53b9dddb956f17aed982e1511b67330a79a3ce2c"}]},"branch":"refs/heads/master"},"68915059736069e2ce4a8f838c16c7e87ae78974":{"kind":"REWORK","_number":2,"created":"2019-10-15 18:01:25.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/2"}}},"commit":{"parents":[{"commit":"f0ed6b6ae823e9ab9fae0ceffa126204d00cac0e","subject":"Microversion 2.80: Add user_id/project_id to migration-list API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/f0ed6b6ae823e9ab9fae0ceffa126204d00cac0e"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-15 18:01:14.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/68915059736069e2ce4a8f838c16c7e87ae78974"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/68915059736069e2ce4a8f838c16c7e87ae78974"}]},"branch":"refs/heads/master"},"81f889f39b3b811676c3a6f9cbccd8815017112d":{"kind":"REWORK","_number":3,"created":"2019-10-16 14:06:29.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/3"}}},"commit":{"parents":[{"commit":"f0ed6b6ae823e9ab9fae0ceffa126204d00cac0e","subject":"Microversion 2.80: Add user_id/project_id to migration-list API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/f0ed6b6ae823e9ab9fae0ceffa126204d00cac0e"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-16 14:05:38.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/81f889f39b3b811676c3a6f9cbccd8815017112d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/81f889f39b3b811676c3a6f9cbccd8815017112d"}]},"branch":"refs/heads/master"},"18b3f9e5e770fda2824be23b30b9cce68a507bb8":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2019-10-16 14:09:29.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/41/687141/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/4"}}},"commit":{"parents":[{"commit":"8744bea0e3ebe5bc4d0d899189bfa0bcdcb0a08f","subject":"Microversion 2.80: Add user_id/project_id to migration-list API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/8744bea0e3ebe5bc4d0d899189bfa0bcdcb0a08f"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-10-16 14:09:29.000000000","tz":0},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/18b3f9e5e770fda2824be23b30b9cce68a507bb8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/18b3f9e5e770fda2824be23b30b9cce68a507bb8"}]},"branch":"refs/heads/master"},"3124945e20e3e8b34000623674abfb4e929a8158":{"kind":"REWORK","_number":5,"created":"2019-10-16 16:46:29.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/5"}}},"commit":{"parents":[{"commit":"8744bea0e3ebe5bc4d0d899189bfa0bcdcb0a08f","subject":"Microversion 2.80: Add user_id/project_id to migration-list API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/8744bea0e3ebe5bc4d0d899189bfa0bcdcb0a08f"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-16 16:46:11.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/3124945e20e3e8b34000623674abfb4e929a8158"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/3124945e20e3e8b34000623674abfb4e929a8158"}]},"branch":"refs/heads/master"},"07190f91e1f0f4d0701ecfda4de5f104cc322e4c":{"kind":"REWORK","_number":6,"created":"2019-10-16 18:19:57.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/6"}}},"commit":{"parents":[{"commit":"8744bea0e3ebe5bc4d0d899189bfa0bcdcb0a08f","subject":"Microversion 2.80: Add user_id/project_id to migration-list API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/8744bea0e3ebe5bc4d0d899189bfa0bcdcb0a08f"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-16 18:19:54.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/07190f91e1f0f4d0701ecfda4de5f104cc322e4c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/07190f91e1f0f4d0701ecfda4de5f104cc322e4c"}]},"branch":"refs/heads/master"},"cffb1b56519700c28596f78584b8528839ddde05":{"kind":"REWORK","_number":7,"created":"2019-10-18 16:16:53.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/7"}}},"commit":{"parents":[{"commit":"558a84767f55594770b49c9ddf5ae870559b569e","subject":"Add images.GlanceManager.find_images() bulk query","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/558a84767f55594770b49c9ddf5ae870559b569e"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-18 16:16:47.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/cffb1b56519700c28596f78584b8528839ddde05"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/cffb1b56519700c28596f78584b8528839ddde05"}]},"branch":"refs/heads/master"},"34174dc63fdecf7d9535c3aaf92f7602d0d07543":{"kind":"TRIVIAL_REBASE","_number":8,"created":"2019-10-18 17:49:35.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/8"}}},"commit":{"parents":[{"commit":"d17c14e256cc2b1a5a51b5595bcb1e22e2f27c09","subject":"Add images.GlanceManager.find_images() bulk query","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/d17c14e256cc2b1a5a51b5595bcb1e22e2f27c09"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-18 17:49:27.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/34174dc63fdecf7d9535c3aaf92f7602d0d07543"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/34174dc63fdecf7d9535c3aaf92f7602d0d07543"}]},"branch":"refs/heads/master"},"71c29a184b388b38d81dc609b9883c5e8f7166cf":{"kind":"NO_CHANGE","_number":9,"created":"2019-10-18 17:50:05.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/41/687141/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/41/687141/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/41/687141/9"}}},"commit":{"parents":[{"commit":"e1bb4378db1e5be85a59861c2d473388fa336da8","subject":"Add images.GlanceManager.find_images() bulk query","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/e1bb4378db1e5be85a59861c2d473388fa336da8"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-07 18:23:28.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2019-10-18 17:50:01.000000000","tz":-420},"subject":"Add aggregate-cache-images command and client routines","message":"Add aggregate-cache-images command and client routines\n\nThis adds the ability to request image precache support for an aggregate\nin support of the matching server feature.\n\nRelated to blueprint image-precache-support\n\nDepends-On: https://review.opendev.org/#/c/687140\nChange-Id: Id354ccfa99e500a598685e6b794c12160ea2a990\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/71c29a184b388b38d81dc609b9883c5e8f7166cf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/71c29a184b388b38d81dc609b9883c5e8f7166cf"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}