)]}'
{"id":"openstack%2Fceilometer~227708","triplet_id":"openstack%2Fceilometer~master~Id21147afbda2eb08a5f3f64506efc6d02314e848","project":"openstack/ceilometer","branch":"master","topic":"bug/1480333","hashtags":[],"change_id":"Id21147afbda2eb08a5f3f64506efc6d02314e848","subject":"Preparation for collector to store event/meter only","status":"ABANDONED","created":"2015-09-25 07:50:29.000000000","updated":"2015-09-28 05:59:49.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"af63b4876e5cedfc35ebcd26ff1ce18391415bb1","_number":227708,"virtual_id_number":227708,"owner":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2015-09-25 11:01:49.000000000","_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},{"value":-1,"date":"2015-09-25 10:57:45.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"rejected":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-2,"date":"2015-09-25 12:39:40.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"}],"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":{"all":[{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"}],"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":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-09-25 10:57:45.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-09-25 11:01:49.000000000","updated_by":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"reviewer":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"state":"REVIEWER"},{"updated":"2015-09-25 12:39:40.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"}],"messages":[{"id":"31599f682620be94098daedab9a82935723ef4af","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-09-25 07:50:29.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3a48f41b2aa254faee0d543e4a7caa53775de9a6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-25 08:41:41.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ceilometer-pep8 http://logs.openstack.org/08/227708/1/check/gate-ceilometer-pep8/58bed8e/ : SUCCESS in 3m 52s\n- gate-ceilometer-docs http://docs-draft.openstack.org/08/227708/1/check/gate-ceilometer-docs/59cc943//doc/build/html/ : SUCCESS in 4m 53s\n- gate-ceilometer-python27 http://logs.openstack.org/08/227708/1/check/gate-ceilometer-python27/0a33ce1/ : SUCCESS in 4m 39s\n- gate-ceilometer-python34 http://logs.openstack.org/08/227708/1/check/gate-ceilometer-python34/d53a342/ : SUCCESS in 5m 09s\n- gate-tempest-dsvm-full http://logs.openstack.org/08/227708/1/check/gate-tempest-dsvm-full/396c95a/ : SUCCESS in 51m 00s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/08/227708/1/check/gate-tempest-dsvm-postgres-full/ee9a38f/ : SUCCESS in 41m 49s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/08/227708/1/check/gate-tempest-dsvm-neutron-full/591e23e/ : SUCCESS in 48m 32s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/08/227708/1/check/gate-tempest-dsvm-ceilometer-mongodb-full/dd003ab/ : SUCCESS in 42m 19s (non-voting)\n- gate-ceilometer-dsvm-functional-mongodb http://logs.openstack.org/08/227708/1/check/gate-ceilometer-dsvm-functional-mongodb/56441d9/ : FAILURE in 28m 50s\n- gate-ceilometer-dsvm-functional-postgresql http://logs.openstack.org/08/227708/1/check/gate-ceilometer-dsvm-functional-postgresql/d88f7ad/ : FAILURE in 29m 57s\n- gate-grenade-dsvm-ceilometer http://logs.openstack.org/08/227708/1/check/gate-grenade-dsvm-ceilometer/486cc48/ : SUCCESS in 39m 34s\n- gate-ceilometer-dsvm-integration http://logs.openstack.org/08/227708/1/check/gate-ceilometer-dsvm-integration/7bf2f9c/ : SUCCESS in 30m 26s","accounts_in_message":[],"_revision_number":1},{"id":"9e890f46bdd74186536ae8bdcab9c6dec1ba63df","author":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"date":"2015-09-25 09:19:38.000000000","message":"Patch Set 1:\n\n* ipmi-test http://52.27.155.124/ipmi/227708/1 : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"6fb3c9c73fb8a1791a2b9cd5cd174244f954605c","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-09-25 09:20:48.000000000","message":"Patch Set 1:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":1},{"id":"da841a911c61b1c9b4917bdecd72a3aa83692117","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2015-09-25 09:46:30.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nReally like code churn for nothing at this point.","accounts_in_message":[],"_revision_number":1},{"id":"e1e2a2e98b9cc92d01e9632a2fa7c6565eb3af16","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-09-25 09:58:46.000000000","message":"Patch Set 1:\n\n@jd, I split this change from another patch due to Chris Dent\u0027s comment on https://review.openstack.org/#/c/214982/1/ceilometer/collector.py","accounts_in_message":[],"_revision_number":1},{"id":"8bc90e9b742eb65db5943b50739c46659c7f2fdf","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-09-25 10:02:49.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"661977443197655847084d140763f09e3458ae5a","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-09-25 10:03:09.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"80776bf6003f469e1a3856b2a4211cb41a8cc5b5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-25 10:57:45.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ceilometer-pep8 http://logs.openstack.org/08/227708/2/check/gate-ceilometer-pep8/c54e060/ : SUCCESS in 2m 50s\n- gate-ceilometer-docs http://docs-draft.openstack.org/08/227708/2/check/gate-ceilometer-docs/c238919//doc/build/html/ : SUCCESS in 5m 51s\n- gate-ceilometer-python27 http://logs.openstack.org/08/227708/2/check/gate-ceilometer-python27/5c77a6b/ : SUCCESS in 3m 37s\n- gate-ceilometer-python34 http://logs.openstack.org/08/227708/2/check/gate-ceilometer-python34/6ad6f9b/ : SUCCESS in 4m 27s\n- gate-tempest-dsvm-full http://logs.openstack.org/08/227708/2/check/gate-tempest-dsvm-full/d9f1706/ : SUCCESS in 52m 47s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/08/227708/2/check/gate-tempest-dsvm-postgres-full/721d442/ : SUCCESS in 48m 22s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/08/227708/2/check/gate-tempest-dsvm-neutron-full/2288659/ : SUCCESS in 53m 02s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/08/227708/2/check/gate-tempest-dsvm-ceilometer-mongodb-full/4ecb1f3/ : SUCCESS in 52m 50s (non-voting)\n- gate-ceilometer-dsvm-functional-mongodb http://logs.openstack.org/08/227708/2/check/gate-ceilometer-dsvm-functional-mongodb/b907372/ : FAILURE in 44m 26s\n- gate-ceilometer-dsvm-functional-postgresql http://logs.openstack.org/08/227708/2/check/gate-ceilometer-dsvm-functional-postgresql/26247e1/ : FAILURE in 38m 55s\n- gate-grenade-dsvm-ceilometer http://logs.openstack.org/08/227708/2/check/gate-grenade-dsvm-ceilometer/c6ede5e/ : SUCCESS in 40m 19s\n- gate-ceilometer-dsvm-integration http://logs.openstack.org/08/227708/2/check/gate-ceilometer-dsvm-integration/53c961f/ : SUCCESS in 38m 11s","accounts_in_message":[],"_revision_number":2},{"id":"8270f2d26228412e6e3f7b044bb3f204127ad900","author":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"date":"2015-09-25 11:01:49.000000000","message":"Patch Set 2:\n\n* ipmi-test http://52.27.155.124/ipmi/227708/2 : SUCCESS","accounts_in_message":[],"_revision_number":2},{"id":"293ac16ecff38d98f50ad2134fc4b169b7ff3a57","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2015-09-25 12:39:40.000000000","message":"Patch Set 2: Code-Review-2\n\nI\u0027m really going to stick to my bad opinion on this – this is bad.\n\nThe commit message is bad because it references something in the future that might or might not be merged.\n\nThe idea of the code is not crazy, but it\u0027s not even a change to a better solution – which would be NOT using the global conf object.\n\nI don\u0027t think doing code churn like that is going to improve anything. I understand you might have done that to please pep8, but it\u0027s a terrible idea. Just hack around it if you need on one line until we clear that global CONF object for good, that\u0027s enough.","accounts_in_message":[],"_revision_number":2},{"id":"10f1bfbb66d8776eb10982ff85d70dc4fbed889e","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-09-28 05:59:49.000000000","message":"Abandoned\n\nhmm, merged to https://review.openstack.org/c/214982","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"a437cde71ce1a3bfe86b7b75556f393deb63b5bc","revisions":{"3604c5ebbdf8ce4f9b0ab99309aa17272836e263":{"kind":"REWORK","_number":1,"created":"2015-09-25 07:50:29.000000000","uploader":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"ref":"refs/changes/08/227708/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/08/227708/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/1"}}},"commit":{"parents":[{"commit":"b4a704af0db7c03fc88b328718d2a932cfba6d79","subject":"Open Mitaka development","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/b4a704af0db7c03fc88b328718d2a932cfba6d79"}]}],"author":{"name":"Lianhao Lu","email":"lianhao.lu@intel.com","date":"2015-09-25 06:56:31.000000000","tz":480},"committer":{"name":"Lianhao Lu","email":"lianhao.lu@intel.com","date":"2015-09-25 07:05:19.000000000","tz":480},"subject":"Preparation for collector to store event/meter only","message":"Preparation for collector to store event/meter only\n\nUse CONF to dereference oslo config to minimize the code length.\n\nRelated-Bug: #1480333\nChange-Id: Id21147afbda2eb08a5f3f64506efc6d02314e848\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/3604c5ebbdf8ce4f9b0ab99309aa17272836e263"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/3604c5ebbdf8ce4f9b0ab99309aa17272836e263"}]},"branch":"refs/heads/master"},"a437cde71ce1a3bfe86b7b75556f393deb63b5bc":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2015-09-25 10:03:09.000000000","uploader":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"ref":"refs/changes/08/227708/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/08/227708/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/08/227708/2"}}},"commit":{"parents":[{"commit":"b4a704af0db7c03fc88b328718d2a932cfba6d79","subject":"Open Mitaka development","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/b4a704af0db7c03fc88b328718d2a932cfba6d79"}]}],"author":{"name":"Lianhao Lu","email":"lianhao.lu@intel.com","date":"2015-09-25 06:56:31.000000000","tz":480},"committer":{"name":"Lianhao Lu","email":"lianhao.lu@intel.com","date":"2015-09-25 10:02:02.000000000","tz":480},"subject":"Preparation for collector to store event/meter only","message":"Preparation for collector to store event/meter only\n\nUse CONF to dereference oslo config to minimize the code length.\n\nChange-Id: Id21147afbda2eb08a5f3f64506efc6d02314e848\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/a437cde71ce1a3bfe86b7b75556f393deb63b5bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/a437cde71ce1a3bfe86b7b75556f393deb63b5bc"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}