)]}'
{"id":"openstack%2Fglance~740997","triplet_id":"openstack%2Fglance~stable%2Fussuri~I789fa7adfb459e7861c90a51f418a635c0c22244","project":"openstack/glance","branch":"stable/ussuri","topic":"bug/1886374-stable/ussuri","hashtags":[],"change_id":"I789fa7adfb459e7861c90a51f418a635c0c22244","subject":"Improve lazy loading mechanism for multiple stores","status":"MERGED","created":"2020-07-14 16:20:22.000000000","updated":"2020-07-15 05:19:13.000000000","submitted":"2020-07-15 02:11:34.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"740997-1594779094545-41dc37f0","meta_rev_id":"f86ef12b3ff6caae38ac97d65b39c57b750de7f6","_number":740997,"virtual_id_number":740997,"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,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-07-15 02:11:34.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2020-07-14 19:56:31.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2020-07-15 00:18:20.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":1,"date":"2020-07-15 00:18:20.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Top Priority / Holds Gate"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-07-14 19:56:31.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":"2020-07-15 00:18:20.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":"2020-07-15 02:11:34.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":"24b9fe7a12064004c449d0bdb1fc08464adab5c8","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2020-07-14 16:20:22.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"c4d1dbdb52bfaf33a48943bef0b39b2f787fab48","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-14 17:49:44.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/13e6b447da524a7db0e14e4a51ef4033 : SUCCESS in 1h 15m 42s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/e1f7f8f0c1b2423593a8668b2579f518 : SUCCESS in 1h 09m 44s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f80e9d97b9a44c59bbd5387c300fd361 : SUCCESS in 4m 48s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/dfbd1ed5910f4752989a97e74689f23b : SUCCESS in 4m 51s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5b33f48e367b4b509eaea81e09a4241b : SUCCESS in 7m 39s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/bec293be51214d95918a05a551a8855a : SUCCESS in 6m 10s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/fb902c63fe4f48aaa9309e5057483ad2 : SUCCESS in 7m 09s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/491bcf633aaa457fb2ec0d0b20a8c52c : SUCCESS in 8m 21s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/bc9aa3042d004e10bccd836a7e0adcac : SUCCESS in 11m 46s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/0f14f4adee464f66b7683adabe074400 : SUCCESS in 5m 09s\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/a4f0adc6ffb74deab8611a3afcb16fe3 : SUCCESS in 59m 41s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c4befeaff43c431f8172356f5e57a528 : SUCCESS in 1h 04m 27s","accounts_in_message":[],"_revision_number":1},{"id":"a6091a34e1276e52aea70f6a7d70acdb1e1a0a61","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-07-14 19:56:31.000000000","message":"Patch Set 1: Code-Review+2\n\nLooks safe to backport.","accounts_in_message":[],"_revision_number":1},{"id":"81045e9d5512137d3c83fb6ac147b0bc0eb3e8fb","author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"date":"2020-07-15 00:18:20.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"e5d54991c020546347ac79fb47bec07a5e1d7892","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-15 00:18:35.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"2a98ae8c6272452a9bb4142f90f4e8b834743fa8","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-15 02:11:34.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/e4805c0b6e3d4f9a90d9e85aa43863a7 : SUCCESS in 1h 29m 11s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/550ab11be9ee4981a494775d6e170445 : SUCCESS in 1h 12m 53s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/d47fcf96043a468ab182cda17836d6b2 : SUCCESS in 6m 05s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9bf8bee21ba94709966bed58569b8559 : SUCCESS in 5m 27s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/084c07362230490a802ca7eef1a93cee : SUCCESS in 4m 49s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/67b0dfb40fd54ca8a349daeba0f09096 : SUCCESS in 5m 31s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c3402125473f4effa857202a144e55f3 : SUCCESS in 6m 50s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/2dbbef167a684307adca8811cacb6452 : SUCCESS in 15m 15s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/1af0e3213ee34cb1a755c4fa9a96656a : SUCCESS in 7m 30s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/60cae9bc85a04945a040f25fa8649700 : SUCCESS in 1h 03m 34s","accounts_in_message":[],"_revision_number":1},{"id":"1b6c5c8c4625dfcdad8e72e0e3a89ec2981bfc63","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-15 02:11:34.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"f09b8b06fbf80f2a8c5822fe8db6329f38f594e5","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-15 02:12:48.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b8ed68cae7394fe4b50a58696fd37cde : SUCCESS in 58s","accounts_in_message":[],"_revision_number":1},{"id":"65e4f212ff48dc7401a6357b1be2319d0dd5b4a4","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2020-07-15 05:19:13.000000000","message":"Patch Set 1: Cherry Picked\n\nThis patchset was cherry picked to branch stable/train as commit baf6b5ebf05ce7a2fb012f4a90cf34c61059d97a","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"9a104acbae328788451442f71aefbc5c20a98f3c","revisions":{"9a104acbae328788451442f71aefbc5c20a98f3c":{"kind":"REWORK","_number":1,"created":"2020-07-14 16:20:22.000000000","uploader":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"ref":"refs/changes/97/740997/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/97/740997/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/97/740997/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/97/740997/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/97/740997/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/97/740997/1"}}},"commit":{"parents":[{"commit":"2df2792fee66c32a8b04bab8e2a4e1880c248ef1","subject":"Don\u0027t include plugins on \u0027copy-image\u0027 import","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/2df2792fee66c32a8b04bab8e2a4e1880c248ef1"}]}],"author":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2020-07-06 07:49:31.000000000","tz":0},"committer":{"name":"Abhishek Kekane","email":"akekane@redhat.com","date":"2020-07-14 16:20:22.000000000","tz":0},"subject":"Improve lazy loading mechanism for multiple stores","message":"Improve lazy loading mechanism for multiple stores\n\nGlance has a facility lazy loading for legacy images which will be called\non get/list api calls to add store information in image\u0027s location metadata\nbased on location URL of image. Even if admin decides to change the store\nnames in glance-api.conf same will also be updated in location metadata\nfor all images related to that particular store. Current implementation of\nlegacy image performs this operation on each get/list call as location metadata\nis not getting updated in database or it doesn\u0027t handle to perform store name\ncheck in glance-api.conf.\n\nImprovements done:\n1. Save updated location metadata information in database permenantly\n2. Add logic to perform lazy loading only if store information is not present\nin location metadata or store present in location metadata is not defined in\nglance\u0027s enbaled_backends configuration option.\n\nChange-Id: I789fa7adfb459e7861c90a51f418a635c0c22244\nCloses-Bug: #1886374\n(cherry picked from commit ab0e5268a9c2614572659d763b3c0b6fc36dd0cf)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/9a104acbae328788451442f71aefbc5c20a98f3c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/9a104acbae328788451442f71aefbc5c20a98f3c"}]},"branch":"refs/heads/stable/ussuri"}},"requirements":[],"submit_records":[],"submit_requirements":[]}