)]}'
{"id":"openstack%2Fceilometer~30729","triplet_id":"openstack%2Fceilometer~master~I28f079a3466a247bce7b4614db3e4c960d769ed7","project":"openstack/ceilometer","branch":"master","topic":"bug/1183106","hashtags":[],"change_id":"I28f079a3466a247bce7b4614db3e4c960d769ed7","subject":"Update oslo, use new configuration generator","status":"MERGED","created":"2013-05-28 14:13:32.000000000","updated":"2013-05-29 00:19:36.000000000","submitted":"2013-05-29 00:19:36.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c60ba92d0d44be0a3bfde546006e391e54a0e506","_number":30729,"virtual_id_number":30729,"owner":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-05-29 00:19:36.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":0,"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"}],"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"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":2,"date":"2013-05-28 15:51:12.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":2,"date":"2013-05-29 00:13:02.000000000","_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":-1,"date":"2013-05-28 15:51:12.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"}],"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":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-05-28 15:50:58.000000000","_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":1,"date":"2013-05-29 00:13:02.000000000","_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":0,"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"}],"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":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-05-28 15:51:12.000000000","updated_by":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"reviewer":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"state":"REVIEWER"},{"updated":"2013-05-28 15:51: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":"2013-05-29 00:13:02.000000000","updated_by":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"reviewer":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"state":"REVIEWER"},{"updated":"2013-05-29 00:19:36.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"48aa2d1eeffdc1d2f9fab9947060b0c4ee4408fa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-28 14:20:06.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/30729/1/check/gate-ceilometer-pep8/2558 : SUCCESS in 1m 06s\n- http://docs-draft.openstack.org/30729/1/check/gate-ceilometer-docs/1364/doc/build/html/ : SUCCESS in 3m 19s\n- http://logs.openstack.org/30729/1/check/gate-ceilometer-python26/2492 : SUCCESS in 6m 24s\n- http://logs.openstack.org/30729/1/check/gate-ceilometer-python27/2490 : SUCCESS in 3m 57s\n","accounts_in_message":[],"_revision_number":1},{"id":"aa20bd83c33dd60c96ed67c5d5d3408fd5c65c71","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2013-05-28 14:34:04.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"273532a416612030217d2619441ea205d85a81ad","author":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"date":"2013-05-28 15:16:31.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nDo not think the common.config should be imported.\nThat package should stay outside of ceilometer. Do not think\nwe need to pull more than we actually need. Other parts of\nceilometer code already uses oslo.config.","accounts_in_message":[],"_revision_number":1},{"id":"181e51770f24cfa65a75960ead41febf9b953914","author":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"date":"2013-05-28 15:22:02.000000000","message":"Patch Set 1:\n\nIf the script needs to use the generator which does not exist\nin the ceilometer dependent oslo.config, seems to me we are\ndepending on multiple versions of the oslo.config, feeling\nhurt. Should update the dependency to require a higher level\noslo.config to have generator being available.","accounts_in_message":[],"_revision_number":1},{"id":"c84da67ea4954fcc57e2cd44db38f2610d89bb07","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2013-05-28 15:50:58.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a15f423b78b2b15a908f4c2728c3754ca10c8710","author":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"date":"2013-05-28 15:51:12.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":2},{"id":"bef25d51f2e579ea610e14773e0221d2830b3f75","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2013-05-28 15:51:12.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":2},{"id":"5455b000f9b2ca9bacc0cb2c22a2d90ba8d918a2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-28 15:57:29.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/30729/2/check/gate-ceilometer-pep8/2560 : SUCCESS in 2m 05s\n- http://docs-draft.openstack.org/30729/2/check/gate-ceilometer-docs/1366/doc/build/html/ : SUCCESS in 4m 16s\n- http://logs.openstack.org/30729/2/check/gate-ceilometer-python26/2494 : SUCCESS in 6m 05s\n- http://logs.openstack.org/30729/2/check/gate-ceilometer-python27/2492 : SUCCESS in 5m 03s\n","accounts_in_message":[],"_revision_number":2},{"id":"cc90b8af5d7a7bdcf3bb2481bff8bf533bbd456e","author":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"date":"2013-05-29 00:13:02.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\nlitong01: the config.cfg is now in oslo, we are not importing that part at all. We are only importing a module (generator.py) to help parse the Cfg() definitions. So there is still only one config.cfg used directly by ceilometer (the oslo version). I don\u0027t see a problem with this.","accounts_in_message":[],"_revision_number":2},{"id":"13714015a2821e344f146f6365333684adc8e4da","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-29 00:13:04.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"ed8d7d496776751d14b27fca1a68d996c49c83ae","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-29 00:19:36.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://docs-draft.openstack.org/30729/2/gate/gate-ceilometer-docs/1373/doc/build/html/ : SUCCESS in 4m 21s\n- http://logs.openstack.org/30729/2/gate/gate-ceilometer-pep8/2567 : SUCCESS in 2m 38s\n- http://logs.openstack.org/30729/2/gate/gate-ceilometer-python26/2501 : SUCCESS in 6m 22s\n- http://logs.openstack.org/30729/2/gate/gate-ceilometer-python27/2499 : SUCCESS in 4m 13s\n","accounts_in_message":[],"_revision_number":2},{"id":"b1b86f462b4c2a16c7fe9dba047338532da58a0e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-29 00:19:36.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"4afe645fdc76af2f9bf3ff94f355a81547c5176f","revisions":{"18bffb3ce7d8d9a4291aef66a2b922af6121c29d":{"kind":"REWORK","_number":1,"created":"2013-05-28 14:13:32.000000000","uploader":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"ref":"refs/changes/29/30729/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/29/30729/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/1"}}},"commit":{"parents":[{"commit":"a18c1676e2d80c078b9ad3aa38dd8adcc19f1357","subject":"Merge \"Don\u0027t use pecan to configure logging\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/a18c1676e2d80c078b9ad3aa38dd8adcc19f1357"}]}],"author":{"name":"Julien Danjou","email":"julien@danjou.info","date":"2013-05-28 14:01:57.000000000","tz":120},"committer":{"name":"Julien Danjou","email":"julien@danjou.info","date":"2013-05-28 14:13:16.000000000","tz":120},"subject":"Update oslo, use new configuration generator","message":"Update oslo, use new configuration generator\n\nChange-Id: I28f079a3466a247bce7b4614db3e4c960d769ed7\nSigned-off-by: Julien Danjou \u003cjulien@danjou.info\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/18bffb3ce7d8d9a4291aef66a2b922af6121c29d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/18bffb3ce7d8d9a4291aef66a2b922af6121c29d"}]},"branch":"refs/heads/master"},"4afe645fdc76af2f9bf3ff94f355a81547c5176f":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2013-05-28 15:50:58.000000000","uploader":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"ref":"refs/changes/29/30729/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/29/30729/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/29/30729/2"}}},"commit":{"parents":[{"commit":"45640c55ccb93d6b5ee596937d8592662df6c25a","subject":"Merge \"Imported Translations from Transifex\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/45640c55ccb93d6b5ee596937d8592662df6c25a"}]}],"author":{"name":"Julien Danjou","email":"julien@danjou.info","date":"2013-05-28 14:01:57.000000000","tz":120},"committer":{"name":"Gordon Chung","email":"chungg@ca.ibm.com","date":"2013-05-28 15:50:23.000000000","tz":-240},"subject":"Update oslo, use new configuration generator","message":"Update oslo, use new configuration generator\n\nChange-Id: I28f079a3466a247bce7b4614db3e4c960d769ed7\nSigned-off-by: Julien Danjou \u003cjulien@danjou.info\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/4afe645fdc76af2f9bf3ff94f355a81547c5176f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/4afe645fdc76af2f9bf3ff94f355a81547c5176f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}