)]}'
{"id":"openstack%2Fceilometer~65671","triplet_id":"openstack%2Fceilometer~master~I18dee7552836c724798ea1340e2e933c09d8a294","project":"openstack/ceilometer","branch":"master","hashtags":[],"change_id":"I18dee7552836c724798ea1340e2e933c09d8a294","subject":"Remove redundant meter (name,type,unit) tuples from Resource model","status":"MERGED","created":"2014-01-09 11:30:46.000000000","updated":"2014-01-10 13:15:26.000000000","submitted":"2014-01-10 13:15:25.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"bc1565eb69953d191fac3a9dd80d3164f7d72733","_number":65671,"virtual_id_number":65671,"owner":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"value":0,"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},{"value":0,"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},{"value":0,"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":2,"date":"2014-01-10 13:15:25.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"recommended":{"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},"all":[{"value":0,"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"value":1,"date":"2014-01-09 19:28:23.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},{"value":2,"date":"2014-01-09 21:53:12.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":2,"date":"2014-01-09 16:50:55.000000000","_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":1,"date":"2014-01-09 14:59:00.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},{"value":1,"date":"2014-01-09 18:47:48.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"all":[{"value":0,"date":"2014-01-09 11:34:05.000000000","_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"value":0,"date":"2014-01-09 11:34:21.000000000","_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},{"value":1,"date":"2014-01-09 21:53:12.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"date":"2014-01-09 11:33:59.000000000","_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},{"value":0,"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-01-09 11:33:59.000000000","updated_by":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"reviewer":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"state":"REVIEWER"},{"updated":"2014-01-09 11:34:05.000000000","updated_by":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"reviewer":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"state":"REVIEWER"},{"updated":"2014-01-09 11:34:21.000000000","updated_by":{"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},"reviewer":{"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},"state":"REVIEWER"},{"updated":"2014-01-09 14:59:00.000000000","updated_by":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"reviewer":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"state":"REVIEWER"},{"updated":"2014-01-09 18:47:48.000000000","updated_by":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"reviewer":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"state":"REVIEWER"},{"updated":"2014-01-09 21:53:12.000000000","updated_by":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"reviewer":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"state":"REVIEWER"},{"updated":"2014-01-10 13:15:25.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"75be4b34748671f9ee74bbd93a7fbf10c3c4d38e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-09 13:10:06.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-ceilometer-pep8 http://logs.openstack.org/71/65671/1/check/gate-ceilometer-pep8/5200de0 : SUCCESS in 1m 28s\n- gate-ceilometer-docs http://docs-draft.openstack.org/71/65671/1/check/gate-ceilometer-docs/1c25d0a/doc/build/html/ : SUCCESS in 3m 55s\n- gate-ceilometer-python26 http://logs.openstack.org/71/65671/1/check/gate-ceilometer-python26/ffc87ed : SUCCESS in 7m 16s\n- gate-ceilometer-python27 http://logs.openstack.org/71/65671/1/check/gate-ceilometer-python27/6a328a8 : SUCCESS in 5m 36s\n- check-tempest-dsvm-full http://logs.openstack.org/71/65671/1/check/check-tempest-dsvm-full/983d482 : SUCCESS in 37m 19s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/71/65671/1/check/check-tempest-dsvm-postgres-full/6585d54 : SUCCESS in 42m 45s\n- check-tempest-dsvm-neutron http://logs.openstack.org/71/65671/1/check/check-tempest-dsvm-neutron/ef30c73 : SUCCESS in 34m 47s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/71/65671/1/check/gate-tempest-dsvm-large-ops/1f27d81 : SUCCESS in 15m 42s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/71/65671/1/check/gate-tempest-dsvm-neutron-large-ops/a42f986 : SUCCESS in 15m 01s\n- check-swift-dsvm-functional http://logs.openstack.org/71/65671/1/check/check-swift-dsvm-functional/264ab23 : SUCCESS in 12m 51s\n","accounts_in_message":[],"_revision_number":1},{"id":"6d871f830b1267e4a5344056fbef157b63166b98","author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"date":"2014-01-09 14:59:00.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"14b941955195c3c5052f4add5c3ced9eeeed2f3d","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2014-01-09 16:50:55.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\nYay less code.","accounts_in_message":[],"_revision_number":1},{"id":"bd696b3e676362caafff0e17a265e16b4ff9fade","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2014-01-09 18:47:48.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nYAY!","accounts_in_message":[],"_revision_number":1},{"id":"4b58fd789e3eec5688b7654cb1c61467c6042ed8","author":{"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},"date":"2014-01-09 19:28:23.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nYou\u0027re my hero.","accounts_in_message":[],"_revision_number":1},{"id":"5d4e49f4b782134cade830bef09536b07b7d7d55","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-09 21:53:12.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"8807c1123833b12889eaf5d246137dc62cbebc0b","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2014-01-09 21:53:12.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\nwasn\u0027t even aware we were building list of related meters on backend as well.\n\nsince we do it on API layer, this change is fine for me.","accounts_in_message":[],"_revision_number":1},{"id":"7dcd461a83d6c1c2625f6068d737634683d405f8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-10 13:15:25.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-ceilometer-docs http://docs-draft.openstack.org/71/65671/1/gate/gate-ceilometer-docs/13659b4/doc/build/html/ : SUCCESS in 3m 24s\n- gate-ceilometer-pep8 http://logs.openstack.org/71/65671/1/gate/gate-ceilometer-pep8/f13124a : SUCCESS in 2m 14s\n- gate-ceilometer-python26 http://logs.openstack.org/71/65671/1/gate/gate-ceilometer-python26/d69a0f5 : SUCCESS in 9m 32s\n- gate-ceilometer-python27 http://logs.openstack.org/71/65671/1/gate/gate-ceilometer-python27/593861a : SUCCESS in 8m 02s\n- gate-tempest-dsvm-full http://logs.openstack.org/71/65671/1/gate/gate-tempest-dsvm-full/444d07b : SUCCESS in 42m 33s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/71/65671/1/gate/gate-tempest-dsvm-postgres-full/10ccff4 : SUCCESS in 1h 02m 48s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/71/65671/1/gate/gate-tempest-dsvm-neutron/d013991 : SUCCESS in 39m 40s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/71/65671/1/gate/gate-tempest-dsvm-large-ops/fbb4992 : SUCCESS in 11m 48s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/71/65671/1/gate/gate-tempest-dsvm-neutron-large-ops/d527c4c : SUCCESS in 14m 41s\n- gate-swift-dsvm-functional http://logs.openstack.org/71/65671/1/gate/gate-swift-dsvm-functional/5aa3e42 : SUCCESS in 13m 04s\n","accounts_in_message":[],"_revision_number":1},{"id":"bc1565eb69953d191fac3a9dd80d3164f7d72733","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-10 13:15:26.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"7c4c0e8fc098fbadec2e30ea2bdf1068b5b4288a","revisions":{"7c4c0e8fc098fbadec2e30ea2bdf1068b5b4288a":{"kind":"REWORK","_number":1,"created":"2014-01-09 11:30:46.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/71/65671/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/71/65671/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/71/65671/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/71/65671/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/71/65671/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/71/65671/1"}}},"commit":{"parents":[{"commit":"25e7dafec0dabca21f6fb99710de6d959d85cfc1","subject":"Merge \"Fixed spelling mistake\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/25e7dafec0dabca21f6fb99710de6d959d85cfc1"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2014-01-09 11:11:43.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2014-01-09 11:29:52.000000000","tz":0},"subject":"Remove redundant meter (name,type,unit) tuples from Resource model","message":"Remove redundant meter (name,type,unit) tuples from Resource model\n\nFixes bug 1267162\n\nPreviously, a list of meter (name,type,unit) tuples with per-sample\ncardinality was added to the Resource model by the storage driver\nget_resource method.\n\nHowever this potential huge and highly repetitive list was then\npromptly discarded by the API layer, as the meter links in the API\nResource representation are populated via subsequent calls into\nthe storage driver get_meters method.\n\nNow, we avoid the scaling issue by suppressing the creation of\nthis unused meter list.\n\nChange-Id: I18dee7552836c724798ea1340e2e933c09d8a294\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/7c4c0e8fc098fbadec2e30ea2bdf1068b5b4288a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/7c4c0e8fc098fbadec2e30ea2bdf1068b5b4288a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}