)]}'
{"id":"openstack%2Fglance~617229","triplet_id":"openstack%2Fglance~master~If6d0348346d2086a2500b0012a0e81e80cea7395","project":"openstack/glance","branch":"master","topic":"bug/1802587","hashtags":[],"change_id":"If6d0348346d2086a2500b0012a0e81e80cea7395","subject":"Make location API compatible with multiple store","status":"MERGED","created":"2018-11-12 06:51:59.000000000","updated":"2019-09-25 14:24:02.000000000","submitted":"2019-09-25 14:22:28.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":15,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"617229-1569421348289-5622d755","meta_rev_id":"c1c66eac02bdcdaa382ec1caf4b6082546d24d69","_number":617229,"virtual_id_number":617229,"owner":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"date":"2019-08-09 05:20:23.000000000","_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},{"value":0,"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"value":0,"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},{"value":0,"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-09-25 14:22:27.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":2,"date":"2019-09-12 14:34:19.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2019-08-08 04:54:53.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},{"value":1,"date":"2019-08-13 15:55:53.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"value":0,"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},{"value":0,"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"}],"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":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":1,"date":"2019-09-25 12:28:11.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},{"value":0,"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"value":0,"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},{"value":0,"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"recommended":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"all":[{"value":0,"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},{"value":0,"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"value":1,"date":"2019-08-08 04:54:53.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},{"value":0,"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Top Priority / Holds Gate"},"description":"","value":1,"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},{"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-26 05:54:21.000000000","updated_by":{"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},"reviewer":{"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},"state":"REVIEWER"},{"updated":"2019-07-11 09:44:38.000000000","updated_by":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"reviewer":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"state":"REVIEWER"},{"updated":"2019-07-25 16:34:59.000000000","updated_by":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"reviewer":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"state":"REVIEWER"},{"updated":"2019-08-01 10:59:00.000000000","updated_by":{"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},"reviewer":{"_account_id":9373,"name":"Vlad Gusev","email":"vlad.esten@gmail.com","username":"s10"},"state":"REVIEWER"},{"updated":"2019-08-05 12:51:01.000000000","updated_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"reviewer":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"state":"REVIEWER"},{"updated":"2019-08-09 05:20:23.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-08-13 15:55:53.000000000","updated_by":{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},"reviewer":{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},"state":"REVIEWER"},{"updated":"2019-09-12 14:34:19.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2019-09-25 12:28:11.000000000","updated_by":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"reviewer":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"state":"REVIEWER"},{"updated":"2019-09-25 14:22:27.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":"764dac07eec97dbba8f140bc14a772e5f0f848a1","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2018-11-12 06:51:59.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b8286b67200c3041008b5ad4560217e3aeea15b4","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2018-11-12 07:29:22.000000000","message":"Patch Set 1:\n\nBuild succeeded\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/1/check-not-vote/ext-glance-zuul/7b00dfc : SUCCESS in 36m 57s","accounts_in_message":[],"_revision_number":1},{"id":"95235505947420fc8e851a8cc48142b6d70cb74d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-11-12 08:38:31.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 http://logs.openstack.org/29/617229/1/check/tempest-full/f2bea5d/ : SUCCESS in 1h 40m 59s\n- neutron-grenade http://logs.openstack.org/29/617229/1/check/neutron-grenade/3e8ecce/ : SUCCESS in 59m 21s\n- tempest-full-py3 http://logs.openstack.org/29/617229/1/check/tempest-full-py3/fe51e1e/ : SUCCESS in 1h 25m 06s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/1/check/openstack-tox-lower-constraints/cf5dd36/ : SUCCESS in 3m 21s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/1/check/openstack-tox-pep8/7521699/ : SUCCESS in 4m 14s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/1/check/openstack-tox-py27/9941837/ : SUCCESS in 4m 27s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/1/check/openstack-tox-py35/1184166/ : SUCCESS in 4m 16s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/1/check/openstack-tox-py36/79eec45/ : SUCCESS in 3m 26s\n- openstack-tox-docs http://logs.openstack.org/29/617229/1/check/openstack-tox-docs/fbde2d0/html/ : SUCCESS in 4m 16s\n- openstack-tox-functional http://logs.openstack.org/29/617229/1/check/openstack-tox-functional/a66e73b/ : TIMED_OUT in 30m 41s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/1/check/openstack-tox-functional-py35/89506c1/ : SUCCESS in 7m 51s\n- glance-eventlet-ssl-handshake-broken-py35 http://logs.openstack.org/29/617229/1/check/glance-eventlet-ssl-handshake-broken-py35/80deacb/ : FAILURE in 4m 01s (non-voting)\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/1/check/devstack-plugin-ceph-tempest/2463449/ : FAILURE in 21m 45s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"61f0524147cdaa2ac1dcfa2ff6d781813f379eb0","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2018-11-12 08:40:40.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"4e4de1feab7b423468e07d3fdb783c50a9adebe7","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2018-11-12 09:15:53.000000000","message":"Patch Set 1:\n\nBuild succeeded\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/1/check-not-vote/ext-glance-zuul/4c90e04 : SUCCESS in 34m 54s","accounts_in_message":[],"_revision_number":1},{"id":"05d94d3019b48554d7869878eb7f2e190bd8435c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-11-12 10:09:10.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/29/617229/1/check/tempest-full/e7cc3c4/ : SUCCESS in 1h 27m 35s\n- neutron-grenade http://logs.openstack.org/29/617229/1/check/neutron-grenade/2a0e465/ : SUCCESS in 51m 55s\n- tempest-full-py3 http://logs.openstack.org/29/617229/1/check/tempest-full-py3/85c1b11/ : SUCCESS in 1h 19m 11s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/1/check/openstack-tox-lower-constraints/7815fb8/ : SUCCESS in 3m 31s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/1/check/openstack-tox-pep8/b2e954c/ : SUCCESS in 4m 24s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/1/check/openstack-tox-py27/4359a3e/ : SUCCESS in 4m 35s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/1/check/openstack-tox-py35/eef654e/ : SUCCESS in 4m 13s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/1/check/openstack-tox-py36/a06298c/ : SUCCESS in 2m 54s\n- openstack-tox-docs http://logs.openstack.org/29/617229/1/check/openstack-tox-docs/567f5c5/html/ : SUCCESS in 4m 53s\n- openstack-tox-functional http://logs.openstack.org/29/617229/1/check/openstack-tox-functional/9ee2765/ : SUCCESS in 11m 32s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/1/check/openstack-tox-functional-py35/fa29ae1/ : SUCCESS in 13m 44s\n- glance-eventlet-ssl-handshake-broken-py35 http://logs.openstack.org/29/617229/1/check/glance-eventlet-ssl-handshake-broken-py35/cca3c49/ : FAILURE in 5m 05s (non-voting)\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/1/check/devstack-plugin-ceph-tempest/6fca9e5/ : FAILURE in 23m 38s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"658deaef9f61f8cd07e72426d6d4ebd31a2f5abb","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2018-11-16 13:17:56.000000000","message":"Patch Set 1: Code-Review+1\n\nCode LGTM.  Do we need a test for this to prevent regressions?  (No is an acceptable answer, I\u0027m just wondering about the refactoring due to happen to make multiple backends the default configuration in Stein.)","accounts_in_message":[],"_revision_number":1},{"id":"b5aca866059f4cf1127f1fdf334c089bf0babb44","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2018-11-16 13:49:30.000000000","message":"Patch Set 1:\n\nBrian,\n\nThank you for review. I have tried to write a unit test for it but it is difficult. I will try again.","accounts_in_message":[],"_revision_number":1},{"id":"ce74dbbfdb4c9efecb0724d79937b528b0823f20","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2018-11-16 20:53:45.000000000","message":"Patch Set 1:\n\nDon\u0027t go to crazy on the unit test.  I\u0027m wondering whether we should just reject the request if no backend is specified.  It might be safer than using the default store.","accounts_in_message":[],"_revision_number":1},{"id":"1bf49b8175c12dfd3b775f684ea4abc9c500503a","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2018-11-16 22:07:20.000000000","message":"Patch Set 1:\n\nPunting to the default backend doesn\u0027t seem like the right solution to me - e.g. I may have swift as my default backend (for general purpose - user creates and image and wants it stored somewhere), but also have the http store enabled for access to published images.\n\nI wonder if get_size_from_uri_and_backend() could apply the same logic as get() (in glance_store.multi_backend) - i.e. iterate over all backends until it finds one that can handle the scheme and get a size?\n\nFailing that, forcing the specification in the location add request seems like the safest option, although it will break compatibility - it\u0027s possible to add a location without specify the backend in current releases, but that will change....","accounts_in_message":[],"_revision_number":1},{"id":"cff25e0b18efff9bbe9be521672e1fcc2fd5c45f","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2018-11-16 22:39:04.000000000","message":"Patch Set 1: Code-Review-1\n\nFurther; this patch doesn\u0027t seem to solve the problem anyway - it just pushes it back a bit. The next failure occurs at:\n\n 2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi Traceback (most recent call last):\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/common/wsgi.py\", line 1274, in __call__\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     request, **action_args)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/common/wsgi.py\", line 1317, in dispatch\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     return method(*args, **kwargs)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/common/utils.py\", line 417, in wrapped\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     return func(self, req, *args, **kwargs)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/api/v2/images.py\", line 249, in update\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     change_method(req, image, change)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/api/v2/images.py\", line 302, in _do_add\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     self._do_add_locations(image, path[1], value)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/api/v2/images.py\", line 497, in _do_add_locations\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     image.locations.insert(pos, value)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/api/policy.py\", line 282, in _checker\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     return method(*args, **kwargs)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/quota/__init__.py\", line 284, in insert\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     return self.locations.insert(index, object)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/location.py\", line 232, in insert\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     [location])\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/location.py\", line 137, in _set_image_size\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     context\u003dcontext)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance_store/multi_backend.py\", line 487, in get_size_from_uri_and_backend\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     uri, backend, conf\u003dCONF)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance_store/location.py\", line 109, in get_location_from_uri_and_backend\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi     raise exceptions.UnknownScheme(scheme\u003dbackend)\n2018-11-16 22:34:58.758 8091 ERROR glance.common.wsgi UnknownScheme: Unknown scheme \u0027None\u0027 found in URI\n\n\nI tried my suggested approach, and it gets a bit further, but then trips up at:\n\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi Traceback (most recent call last):\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/common/wsgi.py\", line 1274, in __call__\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     request, **action_args)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/common/wsgi.py\", line 1317, in dispatch\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     return method(*args, **kwargs)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/common/utils.py\", line 417, in wrapped\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     return func(self, req, *args, **kwargs)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/api/v2/images.py\", line 252, in update\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     image_repo.save(image)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/domain/proxy.py\", line 99, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     result \u003d self.base.save(base_item, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/notifier.py\", line 515, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     super(ImageRepoProxy, self).save(image, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/domain/proxy.py\", line 99, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     result \u003d self.base.save(base_item, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/api/policy.py\", line 120, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     return super(ImageRepoProxy, self).save(image, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/domain/proxy.py\", line 99, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     result \u003d self.base.save(base_item, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/quota/__init__.py\", line 117, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     return super(ImageRepoProxy, self).save(image, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/domain/proxy.py\", line 99, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     result \u003d self.base.save(base_item, from_state\u003dfrom_state)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/location.py\", line 80, in save\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     self._set_acls(image)\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi   File \"/usr/lib/python2.7/site-packages/glance/location.py\", line 64, in _set_acls\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi     location[\u0027url\u0027], location[\u0027metadata\u0027][\u0027backend\u0027],\n2018-11-16 22:38:10.267 8126 ERROR glance.common.wsgi KeyError: \u0027backend\u0027\n\n\nSo there\u0027s more to be done ... or just require the backend to be specified in metadata :/","accounts_in_message":[],"_revision_number":1},{"id":"85d670c7ff91730ecbead8f0c9b3cc76aa1d3158","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2018-11-17 00:28:33.000000000","message":"Patch Set 1:\n\nAs an experiment / PoC, I added this to glance_store.multi_backend:\n\ndef get_backend_for_uri(uri, context\u003dNone):\n    backends \u003d CONF.enabled_backends\n    for backend in backends:\n        try:\n            if not uri.startswith(backends[backend]):\n                continue\n            loc \u003d location.get_location_from_uri_and_backend(\n                uri, backend, conf\u003dCONF)\n            return backend\n        except (exceptions.UnknownScheme):\n            continue\n    raise exceptions.NotFound(_(\"Scheme cannot be handled by any configured \"\n                                \"backend\"))\n\n\nand this to _do_add_locations() in ImagesController (glance/api/v2/images.py):\n\n        if CONF.enabled_backends and \u0027backend\u0027 not in value[\u0027metadata\u0027]:\n            store \u003d glance_store.get_backend_for_uri(value[\u0027url\u0027])\n            value[\u0027metadata\u0027][\u0027backend\u0027] \u003d unicode(store)\n\n\nand that allows me to add an http location to a queued image, when the default store is a file one, and it correctly picks my http store.\n\nNot sure if it\u0027s the right solution, but something to consider....","accounts_in_message":[],"_revision_number":1},{"id":"92d299220e374746b3f7cb6556c5b0505e5fa3f4","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2018-11-19 05:11:25.000000000","message":"Patch Set 1: Workflow-1","accounts_in_message":[],"_revision_number":1},{"id":"af01f691aff5d12023a3830c706fcd69ea5bb518","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2018-11-19 05:22:17.000000000","message":"Patch Set 1:\n\n\u003e As an experiment / PoC, I added this to glance_store.multi_backend:\n \u003e \n \u003e def get_backend_for_uri(uri, context\u003dNone):\n \u003e backends \u003d CONF.enabled_backends\n \u003e for backend in backends:\n \u003e try:\n \u003e if not uri.startswith(backends[backend]):\n \u003e continue\n \u003e loc \u003d location.get_location_from_uri_and_backend(\n \u003e uri, backend, conf\u003dCONF)\n \u003e return backend\n \u003e except (exceptions.UnknownScheme):\n \u003e continue\n \u003e raise exceptions.NotFound(_(\"Scheme cannot be handled by any\n \u003e configured \"\n \u003e \"backend\"))\n \u003e \n \u003e \n \u003e and this to _do_add_locations() in ImagesController\n \u003e (glance/api/v2/images.py):\n \u003e \n \u003e if CONF.enabled_backends and \u0027backend\u0027 not in value[\u0027metadata\u0027]:\n \u003e store \u003d glance_store.get_backend_for_uri(value[\u0027url\u0027])\n \u003e value[\u0027metadata\u0027][\u0027backend\u0027] \u003d unicode(store)\n \u003e \n \u003e \n \u003e and that allows me to add an http location to a queued image, when\n \u003e the default store is a file one, and it correctly picks my http\n \u003e store.\n \u003e \n \u003e Not sure if it\u0027s the right solution, but something to consider....\n\nHi Ian,\n\nThank you for your rigorous testing, your solution sounds logical to me. It will be nice if you propose those patches. (I will abandon this once your patches are up).","accounts_in_message":[],"_revision_number":1},{"id":"3a2d3f9e2002b2a79d7c75406c5941bb92449b6f","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2018-11-20 02:08:44.000000000","message":"Patch Set 1:\n\nPut these up for discussion:\n\nhttps://review.openstack.org/618932\nhttps://review.openstack.org/618933","accounts_in_message":[],"_revision_number":1},{"id":"891245f3c030e47223711b067f9646cfcf14d440","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-05-27 07:08:20.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"fa2c17e0a022d3f0eac6c560251155e183478e65","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-27 09:09:26.000000000","message":"Patch Set 2: 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 http://logs.openstack.org/29/617229/2/check/tempest-full/02f30ed/ : SUCCESS in 1h 38m 41s\n- neutron-grenade http://logs.openstack.org/29/617229/2/check/neutron-grenade/044553a/ : SUCCESS in 57m 27s\n- grenade-py3 http://logs.openstack.org/29/617229/2/check/grenade-py3/7d4166e/ : SUCCESS in 55m 31s\n- tempest-full-py3 http://logs.openstack.org/29/617229/2/check/tempest-full-py3/a88d7b1/ : SUCCESS in 1h 42m 58s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/2/check/openstack-tox-lower-constraints/9719ec8/ : SUCCESS in 3m 54s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/2/check/openstack-tox-pep8/8d2a167/ : SUCCESS in 3m 57s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/2/check/openstack-tox-py27/130253f/ : FAILURE in 4m 16s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/2/check/openstack-tox-py35/3f68c93/ : FAILURE in 4m 30s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/2/check/openstack-tox-py36/d913c3b/ : FAILURE in 4m 38s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/2/check/openstack-tox-py37/a92f3a3/ : FAILURE in 4m 21s\n- openstack-tox-docs http://logs.openstack.org/29/617229/2/check/openstack-tox-docs/29bf3b8/html/ : SUCCESS in 4m 26s\n- openstack-tox-functional http://logs.openstack.org/29/617229/2/check/openstack-tox-functional/55b1252/ : FAILURE in 8m 40s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/2/check/openstack-tox-functional-py35/bb10596/ : FAILURE in 9m 59s\n- glance-eventlet-ssl-handshake-broken-py35 http://logs.openstack.org/29/617229/2/check/glance-eventlet-ssl-handshake-broken-py35/14452d5/ : FAILURE in 3m 11s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/2/check/glance-code-constants-check/44676a3/ : SUCCESS in 4m 15s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/2/check/devstack-plugin-ceph-tempest/e26ccb1/ : SUCCESS in 1h 26m 19s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"b9b194f4778a4a4e02d5337098f3ea46b468bd02","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-05-27 11:24:35.000000000","message":"Patch Set 2:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/2/check-not-vote/ext-glance-zuul/53857a9 : FAILURE in 47m 11s","accounts_in_message":[],"_revision_number":2},{"id":"a0b189690cb63a9e89751182980e9f7c98d9e85f","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-04 05:42:35.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"268757ed0e4a52c55f681c2683af4d969150d993","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-04 05:43:00.000000000","message":"Patch Set 3: Review-Priority+1","accounts_in_message":[],"_revision_number":3},{"id":"aae139ffbd9a504e27802a8a6286fa97e2a571cf","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-04 07:33:32.000000000","message":"Patch Set 3: 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 http://logs.openstack.org/29/617229/3/check/tempest-full/2784e0c/ : SUCCESS in 1h 45m 09s\n- neutron-grenade http://logs.openstack.org/29/617229/3/check/neutron-grenade/3d7ef46/ : SUCCESS in 1h 09m 46s\n- grenade-py3 http://logs.openstack.org/29/617229/3/check/grenade-py3/55b4232/ : SUCCESS in 56m 54s\n- tempest-full-py3 http://logs.openstack.org/29/617229/3/check/tempest-full-py3/e2e43a7/ : SUCCESS in 1h 49m 32s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/3/check/openstack-tox-lower-constraints/fd5877f/ : SUCCESS in 3m 21s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/3/check/openstack-tox-pep8/39ec371/ : SUCCESS in 4m 46s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/3/check/openstack-tox-py27/a3e5373/ : FAILURE in 4m 27s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/3/check/openstack-tox-py35/f1f0602/ : FAILURE in 5m 25s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/3/check/openstack-tox-py36/7b72fa9/ : FAILURE in 4m 44s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/3/check/openstack-tox-py37/9525f65/ : FAILURE in 5m 21s\n- openstack-tox-docs http://logs.openstack.org/29/617229/3/check/openstack-tox-docs/1cd07a3/html/ : SUCCESS in 5m 44s\n- openstack-tox-functional http://logs.openstack.org/29/617229/3/check/openstack-tox-functional/3568550/ : FAILURE in 8m 15s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/3/check/openstack-tox-functional-py35/63f7b76/ : FAILURE in 8m 22s\n- glance-eventlet-ssl-handshake-broken-py35 http://logs.openstack.org/29/617229/3/check/glance-eventlet-ssl-handshake-broken-py35/9f1ead6/ : FAILURE in 2m 32s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/3/check/glance-code-constants-check/829a818/ : SUCCESS in 3m 35s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/3/check/devstack-plugin-ceph-tempest/38d1eb2/ : SUCCESS in 1h 26m 29s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"b2accf0ea1b89b8123d7868b54b3aadb28bd53d7","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-06-04 20:43:15.000000000","message":"Patch Set 3:\n\nBuild succeeded\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/3/check-not-vote/ext-glance-zuul/c841725 : SUCCESS in 34m 48s","accounts_in_message":[],"_revision_number":3},{"id":"82e4d2ea45654305b0b848b13c1c8cd1ac56ed6f","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-10 06:29:16.000000000","message":"Uploaded patch set 4: Patch Set 3 was rebased.","accounts_in_message":[],"_revision_number":4},{"id":"1bdfd492c805f6bfe30aecec93a552968ee98a71","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-10 08:22:02.000000000","message":"Patch Set 4: 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 http://logs.openstack.org/29/617229/4/check/tempest-full/54da557/ : SUCCESS in 1h 39m 24s\n- neutron-grenade http://logs.openstack.org/29/617229/4/check/neutron-grenade/5d1c59f/ : SUCCESS in 1h 01m 48s\n- grenade-py3 http://logs.openstack.org/29/617229/4/check/grenade-py3/1471d9f/ : SUCCESS in 1h 00m 56s\n- tempest-full-py3 http://logs.openstack.org/29/617229/4/check/tempest-full-py3/0b6b430/ : SUCCESS in 1h 47m 02s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/4/check/openstack-tox-lower-constraints/b4b3240/ : SUCCESS in 4m 25s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/4/check/openstack-tox-pep8/008fd5a/ : SUCCESS in 4m 23s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/4/check/openstack-tox-py27/3e2a419/ : FAILURE in 4m 33s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/4/check/openstack-tox-py35/250d5e6/ : FAILURE in 4m 44s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/4/check/openstack-tox-py36/9480948/ : FAILURE in 4m 38s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/4/check/openstack-tox-py37/07f0f6f/ : FAILURE in 4m 47s\n- openstack-tox-docs http://logs.openstack.org/29/617229/4/check/openstack-tox-docs/ce114a4/html/ : SUCCESS in 5m 00s\n- openstack-tox-functional http://logs.openstack.org/29/617229/4/check/openstack-tox-functional/60e40bf/ : FAILURE in 8m 51s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/4/check/openstack-tox-functional-py35/1862443/ : FAILURE in 21m 40s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/4/check/glance-eventlet-ssl-handshake-broken-py3/b7d49cd/ : FAILURE in 4m 08s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/4/check/glance-code-constants-check/1530af0/ : SUCCESS in 4m 33s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/4/check/devstack-plugin-ceph-tempest/605ad5c/ : SUCCESS in 1h 18m 31s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"e929080ccca820551fbd60bbcdb88de0cb01dc5a","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-06-10 11:07:48.000000000","message":"Patch Set 4:\n\nBuild succeeded\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/4/check-not-vote/ext-glance-zuul/d41e4f6 : SUCCESS in 42m 40s","accounts_in_message":[],"_revision_number":4},{"id":"a65ea23e2713d8ea42e055fa5c2b45266b0ebe8c","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-18 05:01:38.000000000","message":"Patch Set 4:\n\nrecheck","accounts_in_message":[],"_revision_number":4},{"id":"fe27d8ed4e68d7aeeb71c5257a67fd30dcdb1632","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-18 05:23:08.000000000","message":"Uploaded patch set 5: Patch Set 4 was rebased.","accounts_in_message":[],"_revision_number":5},{"id":"b99ae84ea10dec945c9ed4966659c64dc2a10115","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-18 05:58:23.000000000","message":"Uploaded patch set 6: Patch Set 5 was rebased.","accounts_in_message":[],"_revision_number":6},{"id":"55130f0230668f7f67f02aa99b60ea4987c8a615","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-18 08:00:45.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/29/617229/6/check/tempest-full/e4146c1/ : SUCCESS in 1h 52m 10s\n- neutron-grenade http://logs.openstack.org/29/617229/6/check/neutron-grenade/1a5cb1a/ : SUCCESS in 58m 33s\n- grenade-py3 http://logs.openstack.org/29/617229/6/check/grenade-py3/ae1beee/ : SUCCESS in 1h 00m 30s\n- tempest-full-py3 http://logs.openstack.org/29/617229/6/check/tempest-full-py3/0b519dd/ : SUCCESS in 1h 53m 08s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/6/check/openstack-tox-lower-constraints/9694052/ : SUCCESS in 4m 12s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/6/check/openstack-tox-pep8/0784a28/ : SUCCESS in 5m 40s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/6/check/openstack-tox-py27/ec15d26/ : SUCCESS in 4m 59s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/6/check/openstack-tox-py35/7712e34/ : SUCCESS in 4m 11s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/6/check/openstack-tox-py36/45b82fd/ : SUCCESS in 4m 54s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/6/check/openstack-tox-py37/d2e70e8/ : SUCCESS in 6m 21s\n- openstack-tox-docs http://logs.openstack.org/29/617229/6/check/openstack-tox-docs/0ac6a65/html/ : SUCCESS in 5m 14s\n- openstack-tox-functional http://logs.openstack.org/29/617229/6/check/openstack-tox-functional/2d0a671/ : SUCCESS in 8m 59s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/6/check/openstack-tox-functional-py35/271fff6/ : SUCCESS in 9m 19s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/6/check/glance-eventlet-ssl-handshake-broken-py3/aace71a/ : FAILURE in 4m 34s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/6/check/glance-code-constants-check/da3091c/ : SUCCESS in 4m 19s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/6/check/devstack-plugin-ceph-tempest/9603071/ : SUCCESS in 1h 29m 37s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"2e0f972404099e4cd71df0bbf39494311e1c3e52","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-06-18 10:49:26.000000000","message":"Patch Set 6:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/6/check-not-vote/ext-glance-zuul/2dbc42f : FAILURE in 13m 52s","accounts_in_message":[],"_revision_number":6},{"id":"d361c451732833e06d22452b2968c4d85b6a3d48","author":{"_account_id":26233,"name":"zengjia","email":"zengjia@awcloud.com","username":"zengjia87"},"date":"2019-06-26 05:54:21.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"586b45c162c609fa873f5ffed87f574b77fffa79","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2019-06-26 17:23:01.000000000","message":"Patch Set 6: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"002556f1650bd03ce55b62feda902da82664edb5","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-27 14:01:17.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nHi Ian,\nPlease see the inline comment, also if possible kindly verify it fixes your issue. Thank you for review.","accounts_in_message":[],"_revision_number":6},{"id":"7e3ee00d199c0e7047699106dad73e2c0905e7fc","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2019-06-27 18:10:12.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"676ab333ae5b05f3d93c0ecace6b9e4025777ef9","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-06-27 18:20:46.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nHi Ian,\n\nPlease find the inline comment.","accounts_in_message":[],"_revision_number":6},{"id":"242967df4901ce61445938ff49234848d5cd52c2","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2019-07-10 20:19:34.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nLGTM, but we definitly need unit test for this function _update_store_in_location()","accounts_in_message":[],"_revision_number":6},{"id":"d2d4263bd3cd2167a18f9559b118065b34478a99","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-11 04:45:09.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nHi Victor,\n\nThank you for review. I will try to add unit test for the same.","accounts_in_message":[],"_revision_number":6},{"id":"0dfeff39d5f634f7de30dd2b4b21ce4ab92849d4","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2019-07-11 06:37:11.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nHere is the unit test I write in glance/tests/unit/v2/test_images_resource.py from line 2669:\n\n    def _test_update_store_in_location(self, metadata, store_id, expected):\n        locations \u003d [dict({\n            \u0027url\u0027: \u0027rbd://aaaaaaaa/images/id\u0027\n        }, **metadata)]\n        with mock.patch.object(\n                store_utils, \u0027get_store_id_from_uri\u0027) as mock_get_store_id:\n            mock_get_store_id.return_value \u003d store_id\n            self.controller._update_store_in_location(mock.Mock(), locations)\n            self.assertEqual(locations[0][\u0027metadata\u0027][\u0027store\u0027], expected)\n\n    def test_update_store_location_with_no_metadata(self):\n        self._test_update_store_in_location({}, \u0027rbd1\u0027, \u0027rbd1\u0027)\n\n    def test_update_store_location_with_no_store(self):\n        self._test_update_store_in_location({\u0027metadata\u0027: {}}, \u0027rbd1\u0027, \u0027rbd1\u0027)\n\n    def test_update_store_location_with_different_store(self):\n        metadata \u003d {\u0027metadata\u0027: {\u0027store\u0027: \u0027rbd2\u0027}}\n        self._test_update_store_in_location(metadata, \u0027rbd1\u0027, \u0027rbd1\u0027)\n\n    def test_update_store_location_with_same_store(self):\n        metadata \u003d {\u0027metadata\u0027: {\u0027store\u0027: \u0027rbd1\u0027}}\n        self._test_update_store_in_location(metadata, \u0027rbd1\u0027, \u0027rbd1\u0027)\n\nI think it will be clearer to understand all the case of your method by reading the UT","accounts_in_message":[],"_revision_number":6},{"id":"1f9ffc32e0391361ac73a75411409f2fd85dbdd4","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-11 06:48:58.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nHi victor,\n\nthank you for review, go ahead and submit a patch with making you as co-author.\n\nThank you for your efforts.","accounts_in_message":[],"_revision_number":6},{"id":"1959be1749b2f4cc2f621c806e0e0d85969f192e","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-11 07:14:12.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"73c971bc39117758515622defa3fc4b5a46afe53","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-11 07:14:42.000000000","message":"Patch Set 7:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":7},{"id":"ed6d4e4a608d0d8fe80b60178208e6e35fea0e54","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-11 07:14:54.000000000","message":"Patch Set 7: Review-Priority+1","accounts_in_message":[],"_revision_number":7},{"id":"1505e4c1f570c3752d5a737015f4b440d9c9366f","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2019-07-11 08:16:33.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"fac97944ba8f789650a3dc5e77072f4237284988","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-11 08:16:55.000000000","message":"Patch Set 8:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":8},{"id":"1680966833c7ed8a97990a71dce983eff0b55fcd","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2019-07-11 08:20:49.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"311b1d908c2d76a8dd38343d4c70d0b3dd926b00","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-11 08:21:12.000000000","message":"Patch Set 9:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":9},{"id":"524e842eda92537f813567f160195d7d9cde3997","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-11 08:40:44.000000000","message":"Patch Set 9: Review-Priority+1","accounts_in_message":[],"_revision_number":9},{"id":"f26d61e3d6cb47ea03827dd616ba91369c0b1a1e","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2019-07-11 09:44:38.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"336e3363ce6b9acc23cc0a028a25df71844792eb","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-11 10:19:06.000000000","message":"Patch Set 9: 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 http://logs.openstack.org/29/617229/9/check/tempest-full/9e87525/ : SUCCESS in 1h 55m 48s\n- neutron-grenade http://logs.openstack.org/29/617229/9/check/neutron-grenade/c202477/ : SUCCESS in 59m 05s\n- grenade-py3 http://logs.openstack.org/29/617229/9/check/grenade-py3/4a22257/ : SUCCESS in 59m 54s\n- tempest-full-py3 http://logs.openstack.org/29/617229/9/check/tempest-full-py3/f4f96e1/ : SUCCESS in 1h 41m 12s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/9/check/openstack-tox-lower-constraints/a2dde43/ : SUCCESS in 3m 51s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/9/check/openstack-tox-pep8/1cfeca4/ : FAILURE in 4m 15s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/9/check/openstack-tox-py27/3ba7f2c/ : SUCCESS in 4m 04s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/9/check/openstack-tox-py35/c75a1ca/ : SUCCESS in 4m 55s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/9/check/openstack-tox-py36/a37dcb1/ : SUCCESS in 4m 14s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/9/check/openstack-tox-py37/e858f27/ : SUCCESS in 5m 36s\n- openstack-tox-docs http://logs.openstack.org/29/617229/9/check/openstack-tox-docs/6e7e125/html/ : SUCCESS in 4m 44s\n- openstack-tox-functional http://logs.openstack.org/29/617229/9/check/openstack-tox-functional/69ca872/ : SUCCESS in 8m 59s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/9/check/openstack-tox-functional-py35/85ce4fe/ : SUCCESS in 9m 06s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/9/check/glance-eventlet-ssl-handshake-broken-py3/cafe27f/ : FAILURE in 5m 00s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/9/check/glance-code-constants-check/8f9d4e3/ : SUCCESS in 4m 09s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/9/check/devstack-plugin-ceph-tempest/81e1019/ : SUCCESS in 1h 31m 42s (non-voting)","accounts_in_message":[],"_revision_number":9},{"id":"9199b47f760b5791a858874f5b5c85a6d82772e3","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2019-07-11 12:05:06.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"51883f75857c71ec0b6c6d8c7e88ae30f052d3e4","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-11 12:05:29.000000000","message":"Patch Set 10:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":10},{"id":"8b9c7aa18d60cad61047b9d70ac585c14c75258d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-11 15:28:59.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/29/617229/10/check/tempest-full/3cf31f7/ : SUCCESS in 1h 56m 05s\n- neutron-grenade http://logs.openstack.org/29/617229/10/check/neutron-grenade/983d1d0/ : SUCCESS in 58m 54s\n- grenade-py3 http://logs.openstack.org/29/617229/10/check/grenade-py3/836912c/ : SUCCESS in 1h 04m 55s\n- tempest-full-py3 http://logs.openstack.org/29/617229/10/check/tempest-full-py3/fd759fe/ : SUCCESS in 1h 33m 35s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/10/check/openstack-tox-lower-constraints/fef8d01/ : SUCCESS in 3m 34s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/10/check/openstack-tox-pep8/440f4fb/ : SUCCESS in 4m 16s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/10/check/openstack-tox-py27/cfb8571/ : SUCCESS in 6m 33s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/10/check/openstack-tox-py35/e2f5ddb/ : SUCCESS in 7m 29s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/10/check/openstack-tox-py36/17ad779/ : SUCCESS in 6m 00s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/10/check/openstack-tox-py37/2b0e8f0/ : SUCCESS in 6m 38s\n- openstack-tox-docs http://logs.openstack.org/29/617229/10/check/openstack-tox-docs/cb0d58a/html/ : SUCCESS in 6m 12s\n- openstack-tox-functional http://logs.openstack.org/29/617229/10/check/openstack-tox-functional/c884031/ : SUCCESS in 8m 55s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/10/check/openstack-tox-functional-py35/a8c6b0c/ : SUCCESS in 10m 23s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/10/check/glance-eventlet-ssl-handshake-broken-py3/1ec06f8/ : FAILURE in 5m 16s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/10/check/glance-code-constants-check/a5cbe95/ : SUCCESS in 4m 05s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/10/check/devstack-plugin-ceph-tempest/010796a/ : SUCCESS in 1h 19m 33s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"017fb9969f10d37652b69859df5c567e69834540","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2019-07-19 14:26:00.000000000","message":"Patch Set 10: Code-Review+2\n\nLGTM.  Good collaboration on the tests.","accounts_in_message":[],"_revision_number":10},{"id":"3e47e42b80dcddb3367f76ab3d042716dd41ae8f","author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"date":"2019-07-25 11:29:13.000000000","message":"Patch Set 10: Code-Review-1\n\n(1 comment)\n\nThanks Guys!\n\nI have concern about the level the operation is logging but otherwise looks good to me.","accounts_in_message":[],"_revision_number":10},{"id":"e0381a9bbe13cb20cad2018e3a65f01e821682fc","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-25 12:42:38.000000000","message":"Patch Set 11: Published edit on patch set 10.","accounts_in_message":[],"_revision_number":11},{"id":"72f3e22b4b0d0fc592bfee40972de787ed542a49","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-25 14:03:22.000000000","message":"Uploaded patch set 12.","accounts_in_message":[],"_revision_number":12},{"id":"981d8fe87145dc36abbabb1f559f98238a624778","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-25 14:17:42.000000000","message":"Uploaded patch set 13.","accounts_in_message":[],"_revision_number":13},{"id":"3e894f9157d2fd6f571f1ae4a69992aa3bb24160","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2019-07-25 16:34:59.000000000","message":"Patch Set 13: Code-Review-1\n\nThis still looks wrong to me. It appears that it will clobber any explicit store specification in metadata supplied by the user. If I\u0027m just \"not getting it\", override my -1 with your +2\u0027s.","accounts_in_message":[],"_revision_number":13},{"id":"c3587e9cec83a57b2bb2cdaca2dc14cd662d2e2e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-25 17:02:01.000000000","message":"Patch Set 13: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/29/617229/13/check/tempest-full/6cf817e/ : SUCCESS in 1h 41m 37s\n- neutron-grenade http://logs.openstack.org/29/617229/13/check/neutron-grenade/e329562/ : SUCCESS in 1h 01m 28s\n- grenade-py3 http://logs.openstack.org/29/617229/13/check/grenade-py3/fd53c10/ : SUCCESS in 55m 44s\n- tempest-full-py3 http://logs.openstack.org/29/617229/13/check/tempest-full-py3/78e56d1/ : SUCCESS in 1h 42m 24s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/13/check/openstack-tox-lower-constraints/d1e9002/ : SUCCESS in 6m 10s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/13/check/openstack-tox-pep8/e549da9/ : SUCCESS in 5m 16s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/13/check/openstack-tox-py27/e81e0f4/ : SUCCESS in 5m 07s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/13/check/openstack-tox-py35/e237ca4/ : SUCCESS in 5m 57s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/13/check/openstack-tox-py36/789e274/ : SUCCESS in 7m 41s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/13/check/openstack-tox-py37/feb5218/ : SUCCESS in 6m 01s\n- openstack-tox-docs http://logs.openstack.org/29/617229/13/check/openstack-tox-docs/ef540b8/html/ : SUCCESS in 7m 28s\n- openstack-tox-functional http://logs.openstack.org/29/617229/13/check/openstack-tox-functional/3963735/ : SUCCESS in 8m 55s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/13/check/openstack-tox-functional-py35/4ad7227/ : SUCCESS in 13m 45s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/13/check/glance-eventlet-ssl-handshake-broken-py3/99beea7/ : FAILURE in 4m 32s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/13/check/glance-code-constants-check/d8b5d28/ : SUCCESS in 4m 44s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/13/check/devstack-plugin-ceph-tempest/0d5a21b/ : SUCCESS in 1h 16m 57s (non-voting)","accounts_in_message":[],"_revision_number":13},{"id":"39c74c4cd64ef13694dcb71f2ebfe287a231e804","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-25 17:49:59.000000000","message":"Patch Set 13:\n\nBuild succeeded\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/13/check-not-vote/ext-glance-zuul/222e9fb : SUCCESS in 37m 04s","accounts_in_message":[],"_revision_number":13},{"id":"4ecf702e4e09b0436acd0e579b0d6c044c2eb767","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-26 07:06:13.000000000","message":"Patch Set 13: Review-Priority+1","accounts_in_message":[],"_revision_number":13},{"id":"b2a4550a976868458a4a77bd5e75d4ce2a494e22","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2019-07-29 11:48:20.000000000","message":"Patch Set 13: Code-Review-1\n\nI have a general concern about this patch.  It was introduced to handle the situation where enabled_backends is set but no store is included in the API request.  It goes well beyond that to duplicating the lazy update of store locations that\u0027s already taking place during the image-list call.\n\nI guess you\u0027re doing this because at this point in the code, you can\u0027t tell a new location from the old ones?\n\nTo Iain\u0027s point, I think the user-supplied store could be overwritten, but that would be like autocorrect (which may not be what we want to happen when someone is setting/updating locations).  Unless someone tries to configure multiple http stores (which we have decided not to allow).\n\n@Iain: you are the most serious consumer of the http store, do you have any use cases that would be impacted by glance only allowing one http store?\n\nSo to get to the point here: (1) I\u0027m worried about the \"autocorrect\" behavior for bad store names with good URLs; (2) If we have to do the lazy update here (in addition to the image-list call), we need to use a utility function so that the store update code can\u0027t get out of sync in multiple places.","accounts_in_message":[],"_revision_number":13},{"id":"20a24660ab60a5ac52c3b9ae28ce89021494726e","author":{"_account_id":23561,"name":"iain MacDonnell","email":"iain.macdonnell@oracle.com","username":"imacdonn"},"date":"2019-07-29 17:12:17.000000000","message":"Patch Set 13:\n\n\u003e @Iain: you are the most serious consumer of the http store, do you\n \u003e have any use cases that would be impacted by glance only allowing\n \u003e one http store?\n\nNegative. My use-case is pretty simple; The images are already published on HTTP servers on the intranet, and I just need to register them with glance by external URL.\n\nI can\u0027t think of any valid case for multiple HTTP \"stores\" (it\u0027s really sortof a virtual store) - what would be different between them? (rhetorical question, I think)\n\nMy concern wasn\u0027t selfish (this time) - I don\u0027t think I\u0027d be affected by this change - it just doesn\u0027t seem like the right thing to do.","accounts_in_message":[],"_revision_number":13},{"id":"909d5d91ca31a42d81d19e403bd6c18477672e37","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 06:17:59.000000000","message":"Uploaded patch set 14.","accounts_in_message":[],"_revision_number":14},{"id":"637187d6deb86486ecb081d61774bbf260bfb5a5","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 06:20:21.000000000","message":"Patch Set 14: Review-Priority+1\n\nHi Brian,\n\nAs per suggestion move code to common place.\nAlso made this patch dependent on https://review.opendev.org/#/c/658962/","accounts_in_message":[],"_revision_number":14},{"id":"aeea1e02b64a455953d03e470776ba397b8370d1","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-30 09:25:35.000000000","message":"Patch Set 14: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/29/617229/14/check/tempest-full/035dca0/ : SUCCESS in 1h 51m 40s\n- neutron-grenade http://logs.openstack.org/29/617229/14/check/neutron-grenade/537e2ee/ : SUCCESS in 54m 26s\n- grenade-py3 http://logs.openstack.org/29/617229/14/check/grenade-py3/6080b57/ : SUCCESS in 1h 05m 58s\n- tempest-full-py3 http://logs.openstack.org/29/617229/14/check/tempest-full-py3/ef091f9/ : SUCCESS in 1h 24m 29s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/14/check/openstack-tox-lower-constraints/70fb703/ : SUCCESS in 4m 53s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/14/check/openstack-tox-pep8/16aa6aa/ : SUCCESS in 5m 34s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/14/check/openstack-tox-py27/be83b22/ : SUCCESS in 4m 42s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/14/check/openstack-tox-py35/10ebe06/ : SUCCESS in 6m 14s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/14/check/openstack-tox-py36/c78f5e7/ : SUCCESS in 4m 57s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/14/check/openstack-tox-py37/4394fbb/ : SUCCESS in 5m 22s\n- openstack-tox-docs http://logs.openstack.org/29/617229/14/check/openstack-tox-docs/f5d3980/html/ : SUCCESS in 5m 46s\n- openstack-tox-functional http://logs.openstack.org/29/617229/14/check/openstack-tox-functional/48e7119/ : SUCCESS in 9m 56s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/14/check/openstack-tox-functional-py35/23c4984/ : SUCCESS in 9m 39s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/14/check/glance-eventlet-ssl-handshake-broken-py3/6fdc660/ : FAILURE in 4m 44s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/14/check/glance-code-constants-check/98c0ac7/ : SUCCESS in 4m 52s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/14/check/devstack-plugin-ceph-tempest/745a92f/ : SUCCESS in 1h 09m 06s (non-voting)","accounts_in_message":[],"_revision_number":14},{"id":"f3dd4aea0fbe9d87d48dbe389004c28d7beffb20","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-30 10:56:54.000000000","message":"Patch Set 14:\n\nBuild succeeded\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/14/check-not-vote/ext-glance-zuul/d608cd4 : SUCCESS in 44m 19s","accounts_in_message":[],"_revision_number":14},{"id":"600a0bf49b6118a979237c1db67c73501640e04f","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2019-07-30 14:10:01.000000000","message":"Patch Set 14: Code-Review-1\n\n(2 comments)\n\nThanks for refactoring!  Some comments inline about improving the tests (though feel free to disagree).","accounts_in_message":[],"_revision_number":14},{"id":"38fdf6fa385800160f17c20e246f5f5e19faf77f","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 16:56:41.000000000","message":"Patch Set 14:\n\n(2 comments)\n\nHi Brian,\n\nThank you for review, made changes as per your suggestion.","accounts_in_message":[],"_revision_number":14},{"id":"31da39244c58a8605d28b2d5e2959fa017fd5c82","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 17:00:29.000000000","message":"Uploaded patch set 15.","accounts_in_message":[],"_revision_number":15},{"id":"4e80cec048316d2b1af771a3c08f6b1e29d5359d","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 17:23:12.000000000","message":"Uploaded patch set 16: Patch Set 15 was rebased.","accounts_in_message":[],"_revision_number":16},{"id":"1bbaf7accb3a5934d3eeba579bf62516c16a25fa","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 17:24:43.000000000","message":"Patch Set 16: Review-Priority+1","accounts_in_message":[],"_revision_number":16},{"id":"898a22c30e1b643fdedb5ceca65f62d44ac6bb77","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-30 21:01:11.000000000","message":"Patch Set 16: 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 http://logs.openstack.org/29/617229/16/check/tempest-full/2a33f81/ : FAILURE in 23m 10s\n- neutron-grenade http://logs.openstack.org/29/617229/16/check/neutron-grenade/d8e5c0c/ : SUCCESS in 57m 45s\n- grenade-py3 http://logs.openstack.org/29/617229/16/check/grenade-py3/5aaf1d1/ : SUCCESS in 57m 10s\n- tempest-full-py3 http://logs.openstack.org/29/617229/16/check/tempest-full-py3/44fc8f4/ : SUCCESS in 1h 22m 00s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/16/check/openstack-tox-lower-constraints/665e5eb/ : SUCCESS in 3m 28s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/16/check/openstack-tox-pep8/47bec61/ : SUCCESS in 6m 42s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/16/check/openstack-tox-py27/f5a6eaa/ : SUCCESS in 5m 02s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/16/check/openstack-tox-py35/a2acd92/ : FAILURE in 7m 28s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/16/check/openstack-tox-py36/7cf692e/ : SUCCESS in 5m 14s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/16/check/openstack-tox-py37/1fa0b1b/ : SUCCESS in 5m 33s\n- openstack-tox-docs http://logs.openstack.org/29/617229/16/check/openstack-tox-docs/72ed452/html/ : SUCCESS in 5m 42s\n- openstack-tox-functional http://logs.openstack.org/29/617229/16/check/openstack-tox-functional/55d4b99/ : SUCCESS in 12m 03s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/16/check/openstack-tox-functional-py35/1f378cc/ : SUCCESS in 10m 23s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/16/check/glance-eventlet-ssl-handshake-broken-py3/6e025d5/ : FAILURE in 5m 33s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/16/check/glance-code-constants-check/0c5c2a7/ : SUCCESS in 4m 13s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/16/check/devstack-plugin-ceph-tempest/ab553cf/ : SUCCESS in 1h 11m 12s (non-voting)","accounts_in_message":[],"_revision_number":16},{"id":"c91e5c438103af80dbf7570175f732bcefe384f4","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-30 21:20:49.000000000","message":"Patch Set 16:\n\nrecheck","accounts_in_message":[],"_revision_number":16},{"id":"f85056921652b575f7cda67410102b8c772a0014","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-07-30 21:35:08.000000000","message":"Patch Set 16:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/16/check-not-vote/ext-glance-zuul/1390dc8 : FAILURE in 16m 10s","accounts_in_message":[],"_revision_number":16},{"id":"e5e90d991a931b7b0c67c78a696a509b255fb9d2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-31 00:24:46.000000000","message":"Patch Set 16: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/29/617229/16/check/tempest-full/3fba6f4/ : SUCCESS in 1h 33m 53s\n- neutron-grenade http://logs.openstack.org/29/617229/16/check/neutron-grenade/718546b/ : SUCCESS in 56m 58s\n- grenade-py3 http://logs.openstack.org/29/617229/16/check/grenade-py3/4040440/ : SUCCESS in 1h 00m 14s\n- tempest-full-py3 http://logs.openstack.org/29/617229/16/check/tempest-full-py3/65c0fb0/ : SUCCESS in 1h 22m 09s\n- openstack-tox-lower-constraints http://logs.openstack.org/29/617229/16/check/openstack-tox-lower-constraints/2388dcc/ : SUCCESS in 4m 05s\n- openstack-tox-pep8 http://logs.openstack.org/29/617229/16/check/openstack-tox-pep8/8c8c77c/ : SUCCESS in 5m 15s\n- openstack-tox-py27 http://logs.openstack.org/29/617229/16/check/openstack-tox-py27/14daee6/ : SUCCESS in 6m 58s\n- openstack-tox-py35 http://logs.openstack.org/29/617229/16/check/openstack-tox-py35/1b04f75/ : SUCCESS in 5m 12s\n- openstack-tox-py36 http://logs.openstack.org/29/617229/16/check/openstack-tox-py36/ef4df35/ : SUCCESS in 7m 05s\n- openstack-tox-py37 http://logs.openstack.org/29/617229/16/check/openstack-tox-py37/cf6f5f5/ : SUCCESS in 5m 23s\n- openstack-tox-docs http://logs.openstack.org/29/617229/16/check/openstack-tox-docs/e8af65a/html/ : SUCCESS in 7m 15s\n- openstack-tox-functional http://logs.openstack.org/29/617229/16/check/openstack-tox-functional/ca6ceeb/ : SUCCESS in 8m 25s\n- openstack-tox-functional-py35 http://logs.openstack.org/29/617229/16/check/openstack-tox-functional-py35/2d0d8b0/ : SUCCESS in 7m 51s\n- glance-eventlet-ssl-handshake-broken-py3 http://logs.openstack.org/29/617229/16/check/glance-eventlet-ssl-handshake-broken-py3/859d8a0/ : FAILURE in 4m 53s (non-voting)\n- glance-code-constants-check http://logs.openstack.org/29/617229/16/check/glance-code-constants-check/8f53ada/ : SUCCESS in 4m 08s\n- devstack-plugin-ceph-tempest http://logs.openstack.org/29/617229/16/check/devstack-plugin-ceph-tempest/4d53a4e/ : SUCCESS in 1h 10m 58s (non-voting)","accounts_in_message":[],"_revision_number":16},{"id":"14daab338a27fb56a1d3ee2a4a7b4cd3b1d09522","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-31 06:55:20.000000000","message":"Patch Set 16: Review-Priority+2\n\nNeed to merge as with glance_store v1.0 and latest master user will not be able to create the image.\n\nhttps://bugs.launchpad.net/glance/+bug/1802587/comments/8","accounts_in_message":[],"_revision_number":16},{"id":"a09f1b8ab74992b1fef4588ba5c115e7c095f29c","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-07-31 06:56:03.000000000","message":"Patch Set 16: Review-Priority+1\n\nIgnore previous comment, meant for parent patch.","accounts_in_message":[],"_revision_number":16},{"id":"3fcd2beb6ae494ff173b94a19c602deb91ace410","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-08-01 18:00:14.000000000","message":"Uploaded patch set 17: Patch Set 16 was rebased.","accounts_in_message":[],"_revision_number":17},{"id":"dda02e251c0e46bc64d4ddc20ce8c49cae302c7a","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-08-01 21:05:17.000000000","message":"Patch Set 17:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/17/check-not-vote/ext-glance-zuul/96cad97 : FAILURE in 51m 19s","accounts_in_message":[],"_revision_number":17},{"id":"b17c4cabcf8d37798ef864c72f9c0394669dff25","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-01 23:33:19.000000000","message":"Patch Set 17: 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://logs.opendev.org/29/617229/17/check/tempest-full/91fd5b0/ : FAILURE in 1h 32m 09s\n- neutron-grenade https://logs.opendev.org/29/617229/17/check/neutron-grenade/09da65d/ : SUCCESS in 55m 41s\n- grenade-py3 https://logs.opendev.org/29/617229/17/check/grenade-py3/e4734c5/ : SUCCESS in 50m 59s\n- tempest-full-py3 https://logs.opendev.org/29/617229/17/check/tempest-full-py3/237005e/ : SUCCESS in 1h 15m 13s\n- openstack-tox-lower-constraints https://logs.opendev.org/29/617229/17/check/openstack-tox-lower-constraints/54113ed/ : SUCCESS in 4m 04s\n- openstack-tox-pep8 https://logs.opendev.org/29/617229/17/check/openstack-tox-pep8/fd685e9/ : SUCCESS in 5m 24s\n- openstack-tox-py27 https://logs.opendev.org/29/617229/17/check/openstack-tox-py27/8a037c3/ : SUCCESS in 4m 30s\n- openstack-tox-py35 https://logs.opendev.org/29/617229/17/check/openstack-tox-py35/ac3da5e/ : SUCCESS in 4m 52s\n- openstack-tox-py36 https://logs.opendev.org/29/617229/17/check/openstack-tox-py36/a2fe848/ : SUCCESS in 6m 02s\n- openstack-tox-py37 https://logs.opendev.org/29/617229/17/check/openstack-tox-py37/125c1cb/ : SUCCESS in 4m 48s\n- openstack-tox-docs https://logs.opendev.org/29/617229/17/check/openstack-tox-docs/d056381/html/ : SUCCESS in 4m 41s\n- openstack-tox-functional https://logs.opendev.org/29/617229/17/check/openstack-tox-functional/c25d46f/ : SUCCESS in 9m 31s\n- openstack-tox-functional-py35 https://logs.opendev.org/29/617229/17/check/openstack-tox-functional-py35/2c86143/ : SUCCESS in 9m 47s\n- glance-eventlet-ssl-handshake-broken-py3 https://logs.opendev.org/29/617229/17/check/glance-eventlet-ssl-handshake-broken-py3/8efa619/ : FAILURE in 5m 29s (non-voting)\n- glance-code-constants-check https://logs.opendev.org/29/617229/17/check/glance-code-constants-check/2f4cac9/ : SUCCESS in 3m 44s\n- devstack-plugin-ceph-tempest https://logs.opendev.org/29/617229/17/check/devstack-plugin-ceph-tempest/879d0e4/ : SUCCESS in 1h 11m 54s (non-voting)","accounts_in_message":[],"_revision_number":17},{"id":"33ef7005c1bbcde8af72ad02a375c6a611e396fc","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-08-02 02:34:20.000000000","message":"Patch Set 17:\n\nrecheck","accounts_in_message":[],"_revision_number":17},{"id":"974d7b2bdad805e470aeb3c83f55aaa13186df8e","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-08-02 03:20:15.000000000","message":"Patch Set 17:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/17/check-not-vote/ext-glance-zuul/14827c9 : FAILURE in 45m 26s","accounts_in_message":[],"_revision_number":17},{"id":"a49fcbb810d4f3cca37e093c4a0dcdb63464b8b2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-02 04:11:38.000000000","message":"Patch Set 17: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://logs.opendev.org/29/617229/17/check/tempest-full/43faf1a/ : SUCCESS in 1h 31m 37s\n- neutron-grenade https://logs.opendev.org/29/617229/17/check/neutron-grenade/0b7f3f4/ : SUCCESS in 1h 01m 16s\n- grenade-py3 https://logs.opendev.org/29/617229/17/check/grenade-py3/5f662a4/ : SUCCESS in 57m 34s\n- tempest-full-py3 https://logs.opendev.org/29/617229/17/check/tempest-full-py3/07d525f/ : SUCCESS in 1h 13m 33s\n- openstack-tox-lower-constraints https://logs.opendev.org/29/617229/17/check/openstack-tox-lower-constraints/741a09b/ : SUCCESS in 3m 35s\n- openstack-tox-pep8 https://logs.opendev.org/29/617229/17/check/openstack-tox-pep8/ded3b1b/ : SUCCESS in 4m 21s\n- openstack-tox-py27 https://logs.opendev.org/29/617229/17/check/openstack-tox-py27/1479f7b/ : SUCCESS in 4m 54s\n- openstack-tox-py35 https://logs.opendev.org/29/617229/17/check/openstack-tox-py35/c7924f3/ : SUCCESS in 4m 17s\n- openstack-tox-py36 https://logs.opendev.org/29/617229/17/check/openstack-tox-py36/e2b52dc/ : SUCCESS in 5m 09s\n- openstack-tox-py37 https://logs.opendev.org/29/617229/17/check/openstack-tox-py37/d10cac8/ : SUCCESS in 4m 56s\n- openstack-tox-docs https://logs.opendev.org/29/617229/17/check/openstack-tox-docs/a7c1034/html/ : SUCCESS in 5m 01s\n- openstack-tox-functional https://logs.opendev.org/29/617229/17/check/openstack-tox-functional/1981a4e/ : SUCCESS in 8m 40s\n- openstack-tox-functional-py35 https://logs.opendev.org/29/617229/17/check/openstack-tox-functional-py35/a942333/ : SUCCESS in 11m 19s\n- glance-eventlet-ssl-handshake-broken-py3 https://logs.opendev.org/29/617229/17/check/glance-eventlet-ssl-handshake-broken-py3/907c908/ : FAILURE in 4m 15s (non-voting)\n- glance-code-constants-check https://logs.opendev.org/29/617229/17/check/glance-code-constants-check/e54b4bf/ : SUCCESS in 3m 36s\n- devstack-plugin-ceph-tempest https://logs.opendev.org/29/617229/17/check/devstack-plugin-ceph-tempest/37a34c6/ : SUCCESS in 1h 13m 41s (non-voting)","accounts_in_message":[],"_revision_number":17},{"id":"be3e25fdaa80b5a3c5c363ef356ecfdd98ea35ce","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2019-08-05 12:51:01.000000000","message":"Patch Set 17: Code-Review+2\n\n(1 comment)\n\nAll my concerns have been addressed.  Some nits noted inline if you have to put up another PS.","accounts_in_message":[],"_revision_number":17},{"id":"07786249a32682669d4c3bc19e45720c1b528512","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-08-06 05:42:04.000000000","message":"Uploaded patch set 18: Patch Set 17 was rebased.","accounts_in_message":[],"_revision_number":18},{"id":"3e663d2c20f676697918950da3a6af8c493ad267","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-08-06 07:16:44.000000000","message":"Patch Set 18:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/18/check-not-vote/ext-glance-zuul/99934d2 : FAILURE in 1m 40s","accounts_in_message":[],"_revision_number":18},{"id":"a401fec2ad3eb3e317c191b4c29ed37eac5a5e41","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 09:41:18.000000000","message":"Patch Set 18: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://logs.opendev.org/29/617229/18/check/tempest-full/9ae7bec/ : SUCCESS in 1h 38m 00s\n- neutron-grenade https://logs.opendev.org/29/617229/18/check/neutron-grenade/07e56c2/ : SUCCESS in 1h 02m 46s\n- grenade-py3 https://logs.opendev.org/29/617229/18/check/grenade-py3/8d8e2a3/ : SUCCESS in 1h 02m 01s\n- tempest-full-py3 https://logs.opendev.org/29/617229/18/check/tempest-full-py3/23bf8fd/ : SUCCESS in 1h 19m 58s\n- openstack-tox-lower-constraints https://logs.opendev.org/29/617229/18/check/openstack-tox-lower-constraints/0019a23/ : SUCCESS in 3m 35s\n- openstack-tox-pep8 https://logs.opendev.org/29/617229/18/check/openstack-tox-pep8/ccdc196/ : SUCCESS in 5m 19s\n- openstack-tox-py27 https://logs.opendev.org/29/617229/18/check/openstack-tox-py27/4460d20/ : SUCCESS in 5m 22s\n- openstack-tox-py35 https://logs.opendev.org/29/617229/18/check/openstack-tox-py35/6520680/ : SUCCESS in 11m 00s\n- openstack-tox-py36 https://logs.opendev.org/29/617229/18/check/openstack-tox-py36/8e1accc/ : SUCCESS in 6m 26s\n- openstack-tox-py37 https://logs.opendev.org/29/617229/18/check/openstack-tox-py37/cb9556b/ : SUCCESS in 10m 25s\n- openstack-tox-docs https://logs.opendev.org/29/617229/18/check/openstack-tox-docs/f6a91a0/html/ : SUCCESS in 6m 54s\n- openstack-tox-functional https://logs.opendev.org/29/617229/18/check/openstack-tox-functional/40dd3fd/ : SUCCESS in 15m 32s\n- openstack-tox-functional-py35 https://logs.opendev.org/29/617229/18/check/openstack-tox-functional-py35/53e0a6c/ : SUCCESS in 10m 01s\n- glance-eventlet-ssl-handshake-broken-py3 https://logs.opendev.org/29/617229/18/check/glance-eventlet-ssl-handshake-broken-py3/4989767/ : FAILURE in 10m 16s (non-voting)\n- glance-code-constants-check https://logs.opendev.org/29/617229/18/check/glance-code-constants-check/74c397b/ : SUCCESS in 5m 50s\n- devstack-plugin-ceph-tempest https://logs.opendev.org/29/617229/18/check/devstack-plugin-ceph-tempest/6c45276/ : FAILURE in 1h 23m 59s (non-voting)","accounts_in_message":[],"_revision_number":18},{"id":"aabd20a21c209b28cc88faedc55d4f73d758cc26","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2019-08-08 04:54:53.000000000","message":"Uploaded patch set 19: Patch Set 18 was rebased.","accounts_in_message":[],"_revision_number":19},{"id":"ab90da335866e841132c7b9f8d2e9fedf75dc92a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-08 06:37:03.000000000","message":"Patch Set 19: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full https://logs.opendev.org/29/617229/19/check/tempest-full/505e852/ : SUCCESS in 1h 36m 35s\n- neutron-grenade https://logs.opendev.org/29/617229/19/check/neutron-grenade/d88af43/ : SUCCESS in 54m 43s\n- grenade-py3 https://logs.opendev.org/29/617229/19/check/grenade-py3/e31b251/ : SUCCESS in 1h 03m 52s\n- tempest-full-py3 https://logs.opendev.org/29/617229/19/check/tempest-full-py3/27d0fe2/ : SUCCESS in 1h 28m 19s\n- openstack-tox-lower-constraints https://logs.opendev.org/29/617229/19/check/openstack-tox-lower-constraints/4fdd3d1/ : SUCCESS in 3m 41s\n- openstack-tox-pep8 https://logs.opendev.org/29/617229/19/check/openstack-tox-pep8/8b789de/ : SUCCESS in 4m 26s\n- openstack-tox-py27 https://logs.opendev.org/29/617229/19/check/openstack-tox-py27/9c65e93/ : SUCCESS in 4m 47s\n- openstack-tox-py35 https://logs.opendev.org/29/617229/19/check/openstack-tox-py35/02899cc/ : SUCCESS in 4m 40s\n- openstack-tox-py36 https://logs.opendev.org/29/617229/19/check/openstack-tox-py36/0165525/ : SUCCESS in 6m 56s\n- openstack-tox-py37 https://logs.opendev.org/29/617229/19/check/openstack-tox-py37/c56fd02/ : SUCCESS in 6m 03s\n- openstack-tox-docs https://logs.opendev.org/29/617229/19/check/openstack-tox-docs/0a2cdd1/html/ : SUCCESS in 8m 02s\n- openstack-tox-functional https://logs.opendev.org/29/617229/19/check/openstack-tox-functional/3af285e/ : SUCCESS in 11m 11s\n- openstack-tox-functional-py35 https://logs.opendev.org/29/617229/19/check/openstack-tox-functional-py35/076b2df/ : SUCCESS in 12m 23s\n- glance-eventlet-ssl-handshake-broken-py3 https://logs.opendev.org/29/617229/19/check/glance-eventlet-ssl-handshake-broken-py3/a45255d/ : FAILURE in 4m 08s (non-voting)\n- glance-code-constants-check https://logs.opendev.org/29/617229/19/check/glance-code-constants-check/f4af01a/ : SUCCESS in 3m 52s\n- devstack-plugin-ceph-tempest https://logs.opendev.org/29/617229/19/check/devstack-plugin-ceph-tempest/019e770/ : SUCCESS in 1h 06m 24s (non-voting)","accounts_in_message":[],"_revision_number":19},{"id":"1c629c4d06f7456c4a0e2d3c91b1d5574557f0de","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2019-08-09 05:20:23.000000000","message":"Patch Set 19:\n\nBuild failed\n\n- dsvm-glance http://207.189.188.190/logs/29/617229/19/check-not-vote/ext-glance-zuul/d555a47 : FAILURE in 2m 09s","accounts_in_message":[],"_revision_number":19},{"id":"0edcae42a3bc1419fb9a75005d3251fbb285bd88","author":{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},"date":"2019-08-13 15:55:53.000000000","message":"Patch Set 19: Code-Review+1","accounts_in_message":[],"_revision_number":19},{"id":"4ccf813c87e6c777b5a3cb620a3c03a7a343cb20","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-09-12 14:34:19.000000000","message":"Patch Set 19: Code-Review+2\n\n(1 comment)\n\nThis looks good to me and the test coverage seems good. I will leave it to another core to actually approve though since both Brian and Erno have been more involved in this.","accounts_in_message":[],"_revision_number":19},{"id":"e6864f6e050c8da1380bc659cdfa571a727a9dc5","author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"date":"2019-09-25 12:28:11.000000000","message":"Patch Set 19: Workflow+1","accounts_in_message":[],"_revision_number":19},{"id":"1f0851c780ed955d9d027536b2b3da8aca983e4d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 12:28:25.000000000","message":"Patch Set 19: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":19},{"id":"ffb0420c291ce3a39df7f3ab283157dba1216f4e","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 14:22:27.000000000","message":"Patch Set 19: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tempest-full https://zuul.opendev.org/t/openstack/build/8bea0a79b5e5412584677ff1532c8ad8 : SUCCESS in 1h 43m 06s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/51dd4182285c4f1390664e6f57722d38 : SUCCESS in 57m 38s\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/2975231475e243968cad93752003c109 : SUCCESS in 1h 05m 35s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/85b9e7bcb15344dfa05a30b301472e07 : SUCCESS in 1h 22m 03s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/fb8183cfbfed4eb2aec25837ebd255cc : SUCCESS in 3m 53s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/66123ebbb40e4fb583286599210ae6b1 : SUCCESS in 4m 11s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/677d39fc5148411d9e014b14226b4a0a : SUCCESS in 4m 15s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/fb6c9cdc753246989ce533b310ac0ff0 : SUCCESS in 5m 59s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/4c3371403b484610bd7d956cf24dfdce : SUCCESS in 5m 42s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d972653e041f479d8e6587ade3053222 : SUCCESS in 7m 35s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/e541e86a4c344b3cb26464b8233e1971 : SUCCESS in 7m 49s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/b64894a3d5ab473f8968f014ea46779e : SUCCESS in 8m 20s","accounts_in_message":[],"_revision_number":19},{"id":"deeb0bd9023753688427b89072dde4cd95cbed6f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 14:22:28.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":19},{"id":"3b63663434c7e54f68565bdf79fae2503b7762fd","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 14:24:02.000000000","message":"Patch Set 19:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b76d8433b9a846bdb1fbceb61bcb7d1c : SUCCESS in 1m 10s","accounts_in_message":[],"_revision_number":19}],"current_revision_number":19,"current_revision":"4e070fd6e0346948898b751516234ba62121e2a8","revisions":{"6ce9e54240bcd29f4804682ab18ffb2115e855ff":{"kind":"REWORK","_number":1,"created":"2018-11-12 06:51:59.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/1"}}},"commit":{"parents":[{"commit":"d2e62d43975dab346fb65abf5bcb2d38de06d2e0","subject":"Merge \"Embed validation data in locations\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/d2e62d43975dab346fb65abf5bcb2d38de06d2e0"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2018-11-12 06:31:37.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2018-11-12 06:51:52.000000000","tz":0},"subject":"Add location to default backend if backend is not provided","message":"Add location to default backend if backend is not provided\n\nIn case of Multiple backend while adding location to image if backend is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade provision to pass the default backend if backend is not provided\nwhile adding location to the image.\n\nDocImpact\nCloses-Bug: #1802587\n\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/6ce9e54240bcd29f4804682ab18ffb2115e855ff"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/6ce9e54240bcd29f4804682ab18ffb2115e855ff"}]},"branch":"refs/heads/master"},"ca5b9e56fce4ae1b2291a33ef6c6e81f2a249563":{"kind":"REWORK","_number":2,"created":"2019-05-27 07:08:20.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/2"}}},"commit":{"parents":[{"commit":"55d2a2ab6b51a59de3625852585d2d40664d8409","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/55d2a2ab6b51a59de3625852585d2d40664d8409"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-27 07:07:42.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/ca5b9e56fce4ae1b2291a33ef6c6e81f2a249563"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/ca5b9e56fce4ae1b2291a33ef6c6e81f2a249563"}]},"branch":"refs/heads/master"},"8bbc427a98b0de04ad7966a27f7b4ca12df0600d":{"kind":"REWORK","_number":3,"created":"2019-06-04 05:42:35.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/3"}}},"commit":{"parents":[{"commit":"1945344de5308fd713409efd8b589c084e7c2084","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/1945344de5308fd713409efd8b589c084e7c2084"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-06-04 05:34:54.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/8bbc427a98b0de04ad7966a27f7b4ca12df0600d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/8bbc427a98b0de04ad7966a27f7b4ca12df0600d"}]},"branch":"refs/heads/master"},"40042d1443388c254c353a9d7d02ecbdb103f495":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2019-06-10 06:29:16.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/4"}}},"commit":{"parents":[{"commit":"af8cdd90504ec80cff3c776f951870b5b5be7989","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/af8cdd90504ec80cff3c776f951870b5b5be7989"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-06-10 06:25:16.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/40042d1443388c254c353a9d7d02ecbdb103f495"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/40042d1443388c254c353a9d7d02ecbdb103f495"}]},"branch":"refs/heads/master"},"2855078f3704406e1918ed45361fb7d076cc4f13":{"kind":"TRIVIAL_REBASE","_number":5,"created":"2019-06-18 05:23:08.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/5"}}},"commit":{"parents":[{"commit":"7128e8295edf46a91804bb81a357080277579016","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/7128e8295edf46a91804bb81a357080277579016"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-06-18 05:22:38.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/2855078f3704406e1918ed45361fb7d076cc4f13"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/2855078f3704406e1918ed45361fb7d076cc4f13"}]},"branch":"refs/heads/master"},"bc55a06fcf6d3c1be14714766498a4300d9e7b0f":{"kind":"TRIVIAL_REBASE","_number":6,"created":"2019-06-18 05:58:23.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/6"}}},"commit":{"parents":[{"commit":"69bef0982f330b68170c6d77b6996b35dcfad9f5","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/69bef0982f330b68170c6d77b6996b35dcfad9f5"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-06-18 05:55:26.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/bc55a06fcf6d3c1be14714766498a4300d9e7b0f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/bc55a06fcf6d3c1be14714766498a4300d9e7b0f"}]},"branch":"refs/heads/master"},"74132605bb9b1584abfb5db62d147224bb37ef0c":{"kind":"REWORK","_number":7,"created":"2019-07-11 07:14:12.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/7"}}},"commit":{"parents":[{"commit":"32598c624e760ef66c130f668a9e729207efb7ac","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/32598c624e760ef66c130f668a9e729207efb7ac"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-11 07:13:55.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/74132605bb9b1584abfb5db62d147224bb37ef0c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/74132605bb9b1584abfb5db62d147224bb37ef0c"}]},"branch":"refs/heads/master"},"f5b0d579f3ab6728ae1ae962e165e9ba0c7bcdbc":{"kind":"REWORK","_number":8,"created":"2019-07-11 08:16:33.000000000","uploader":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"ref":"refs/changes/29/617229/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/8"}}},"commit":{"parents":[{"commit":"32598c624e760ef66c130f668a9e729207efb7ac","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/32598c624e760ef66c130f668a9e729207efb7ac"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","date":"2019-07-11 08:13:13.000000000","tz":120},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f5b0d579f3ab6728ae1ae962e165e9ba0c7bcdbc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/f5b0d579f3ab6728ae1ae962e165e9ba0c7bcdbc"}]},"branch":"refs/heads/master"},"121010a8b7a8cdb80f0f0f030bd8f906ecf11139":{"kind":"REWORK","_number":9,"created":"2019-07-11 08:20:49.000000000","uploader":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"ref":"refs/changes/29/617229/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/9"}}},"commit":{"parents":[{"commit":"32598c624e760ef66c130f668a9e729207efb7ac","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/32598c624e760ef66c130f668a9e729207efb7ac"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","date":"2019-07-11 08:20:35.000000000","tz":120},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/121010a8b7a8cdb80f0f0f030bd8f906ecf11139"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/121010a8b7a8cdb80f0f0f030bd8f906ecf11139"}]},"branch":"refs/heads/master"},"70a543acbff841bbb34b545df8282f0c277ab043":{"kind":"REWORK","_number":10,"created":"2019-07-11 12:05:06.000000000","uploader":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"ref":"refs/changes/29/617229/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/10"}}},"commit":{"parents":[{"commit":"32598c624e760ef66c130f668a9e729207efb7ac","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/32598c624e760ef66c130f668a9e729207efb7ac"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","date":"2019-07-11 12:04:52.000000000","tz":120},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/70a543acbff841bbb34b545df8282f0c277ab043"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/70a543acbff841bbb34b545df8282f0c277ab043"}]},"branch":"refs/heads/master"},"4abdb84e2ac6845fcd06b7888f48785dedc637ce":{"kind":"REWORK","_number":11,"created":"2019-07-25 12:42:38.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/11"}}},"commit":{"parents":[{"commit":"32598c624e760ef66c130f668a9e729207efb7ac","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/32598c624e760ef66c130f668a9e729207efb7ac"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-25 12:38:52.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/4abdb84e2ac6845fcd06b7888f48785dedc637ce"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/4abdb84e2ac6845fcd06b7888f48785dedc637ce"}]},"branch":"refs/heads/master"},"70a2a705c37852656d7c068cd11b5b3156d15909":{"kind":"REWORK","_number":12,"created":"2019-07-25 14:03:22.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/12"}}},"commit":{"parents":[{"commit":"d1e85a4c05152d1ceecea39e7ff1efe6b9f7db13","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/d1e85a4c05152d1ceecea39e7ff1efe6b9f7db13"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-25 14:03:03.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/70a2a705c37852656d7c068cd11b5b3156d15909"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/70a2a705c37852656d7c068cd11b5b3156d15909"}]},"branch":"refs/heads/master"},"d912bb3b5a1f082742d752aff9ac94f18e1f01e3":{"kind":"REWORK","_number":13,"created":"2019-07-25 14:17:42.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/13"}}},"commit":{"parents":[{"commit":"d1e85a4c05152d1ceecea39e7ff1efe6b9f7db13","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/d1e85a4c05152d1ceecea39e7ff1efe6b9f7db13"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-25 14:16:56.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/d912bb3b5a1f082742d752aff9ac94f18e1f01e3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/d912bb3b5a1f082742d752aff9ac94f18e1f01e3"}]},"branch":"refs/heads/master"},"a9ea77f7c566ff3966926c22b626a3f1e6db4eab":{"kind":"REWORK","_number":14,"created":"2019-07-30 06:17:59.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/14"}}},"commit":{"parents":[{"commit":"5cab654f6e5c110e8917e819e53d8f33e5d6fc34","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/5cab654f6e5c110e8917e819e53d8f33e5d6fc34"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-30 06:15:56.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/a9ea77f7c566ff3966926c22b626a3f1e6db4eab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/a9ea77f7c566ff3966926c22b626a3f1e6db4eab"}]},"branch":"refs/heads/master"},"3e8dcdadcfab8e7f662c28a2fb27422fd13ca23f":{"kind":"REWORK","_number":15,"created":"2019-07-30 17:00:29.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/15","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/15","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/15 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/15 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/15 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/15"}}},"commit":{"parents":[{"commit":"397f8b5bb93fcbefe5071fc9ebd2897db8d515a7","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/397f8b5bb93fcbefe5071fc9ebd2897db8d515a7"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-30 16:59:50.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/3e8dcdadcfab8e7f662c28a2fb27422fd13ca23f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/3e8dcdadcfab8e7f662c28a2fb27422fd13ca23f"}]},"branch":"refs/heads/master"},"27d78914df1a50d23e9f3c4d9f097468aa910771":{"kind":"TRIVIAL_REBASE","_number":16,"created":"2019-07-30 17:23:12.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/16","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/16","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/16 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/16 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/16 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/16"}}},"commit":{"parents":[{"commit":"ffb277d97751d334ae4ae5665854495239200861","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/ffb277d97751d334ae4ae5665854495239200861"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-07-30 17:21:50.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/27d78914df1a50d23e9f3c4d9f097468aa910771"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/27d78914df1a50d23e9f3c4d9f097468aa910771"}]},"branch":"refs/heads/master"},"85576902a6bb496bc4182f1f1b21232ae02ba339":{"kind":"TRIVIAL_REBASE","_number":17,"created":"2019-08-01 18:00:14.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/17","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/17","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/17 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/17 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/17 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/17"}}},"commit":{"parents":[{"commit":"859b6541ee0a49e569074b18c708985fdc930b5b","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/859b6541ee0a49e569074b18c708985fdc930b5b"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-08-01 17:58:08.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/85576902a6bb496bc4182f1f1b21232ae02ba339"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/85576902a6bb496bc4182f1f1b21232ae02ba339"}]},"branch":"refs/heads/master"},"36e8aaf33f30bd688390529e13330ba9d44283c6":{"kind":"TRIVIAL_REBASE","_number":18,"created":"2019-08-06 05:42:04.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/18","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/18","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/18 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/18 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/18 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/18"}}},"commit":{"parents":[{"commit":"9cf96d44dc06a8248c4fd12dc57db73307721235","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/9cf96d44dc06a8248c4fd12dc57db73307721235"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-08-06 05:41:53.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/36e8aaf33f30bd688390529e13330ba9d44283c6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/36e8aaf33f30bd688390529e13330ba9d44283c6"}]},"branch":"refs/heads/master"},"4e070fd6e0346948898b751516234ba62121e2a8":{"kind":"TRIVIAL_REBASE","_number":19,"created":"2019-08-08 04:54:53.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/29/617229/19","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/29/617229/19","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/19 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/19 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/29/617229/19 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/29/617229/19"}}},"commit":{"parents":[{"commit":"0e5984edc707a980028e1aa9b0d48d57ed3908d5","subject":"Lazy update stores information","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/0e5984edc707a980028e1aa9b0d48d57ed3908d5"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-05-24 07:53:16.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2019-08-08 04:54:29.000000000","tz":0},"subject":"Make location API compatible with multiple store","message":"Make location API compatible with multiple store\n\nIn case of Multiple store while adding location to image if backend store is\nnot provided it returns 400 BadRequest saying \u0027Invalid location\u0027.\n\nMade changes to loaction API to findout the store from location URI\nand add it as a backend store to the location metadata.\n\nDocImpact\nCloses-Bug: #1802587\nCo-Authored-By: Victor Coutellier \u003cvictor.coutellier@gmail.com\u003e\nChange-Id: If6d0348346d2086a2500b0012a0e81e80cea7395\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/4e070fd6e0346948898b751516234ba62121e2a8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/4e070fd6e0346948898b751516234ba62121e2a8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}