)]}'
{"id":"openstack%2Fceilometer~196330","triplet_id":"openstack%2Fceilometer~master~I88cbd83910dcf79bed8a0f9616f98e4346362ab3","project":"openstack/ceilometer","branch":"master","topic":"bp/api-no-pipeline","hashtags":[],"change_id":"I88cbd83910dcf79bed8a0f9616f98e4346362ab3","subject":"Add support for posting samples to notification-agent via API","status":"MERGED","created":"2015-06-27 02:22:28.000000000","updated":"2015-06-29 17:00:37.000000000","submitted":"2015-06-29 17:00:35.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"83e9a63c3521a526e3c2c6cc30051b542aa4f3d1","_number":196330,"virtual_id_number":196330,"owner":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},{"value":2,"date":"2015-06-29 17:00:35.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"}],"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":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2015-06-29 15:07:05.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":2,"date":"2015-06-27 20:36:56.000000000","_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"}],"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":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"all":[{"value":0,"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2015-06-29 15:07:05.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"}],"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":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-06-27 08:36:24.000000000","updated_by":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"reviewer":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"state":"REVIEWER"},{"updated":"2015-06-27 20:36:56.000000000","updated_by":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"reviewer":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"state":"REVIEWER"},{"updated":"2015-06-29 15:07:05.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":"2015-06-29 17:00:35.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5ae3daa93c32f05bb9c8e40d2ad5da90e447040f","author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"date":"2015-06-27 02:22:28.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6931bf4f6faa6fc23af8d7d00c67b6fd3b7782d4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-27 03:13:01.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/30/196330/1/check/gate-ceilometer-pep8/1a76d8e/ : SUCCESS in 3m 17s\n- gate-ceilometer-docs http://docs-draft.openstack.org/30/196330/1/check/gate-ceilometer-docs/f8ac483//doc/build/html/ : SUCCESS in 5m 17s\n- gate-ceilometer-python27 http://logs.openstack.org/30/196330/1/check/gate-ceilometer-python27/a73d501/ : SUCCESS in 8m 09s\n- gate-ceilometer-python34 http://logs.openstack.org/30/196330/1/check/gate-ceilometer-python34/7e1b771/ : SUCCESS in 6m 45s\n- check-tempest-dsvm-full http://logs.openstack.org/30/196330/1/check/check-tempest-dsvm-full/35cf713/ : SUCCESS in 43m 31s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/30/196330/1/check/check-tempest-dsvm-postgres-full/867b6f1/ : SUCCESS in 40m 57s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/30/196330/1/check/check-tempest-dsvm-neutron-full/99234c7/ : SUCCESS in 50m 24s\n- check-grenade-dsvm http://logs.openstack.org/30/196330/1/check/check-grenade-dsvm/b721e3e/ : SUCCESS in 44m 39s\n- check-devstack-dsvm-cells http://logs.openstack.org/30/196330/1/check/check-devstack-dsvm-cells/10f073b/ : SUCCESS in 18m 19s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/30/196330/1/check/gate-tempest-dsvm-ceilometer-mongodb-full/5267fff/ : SUCCESS in 44m 55s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"fb61fb14162749cef0effac8f83889bee2d4749c","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2015-06-27 08:36:24.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"7c99578f6e8728daa3375cfda4ac254d730c4679","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2015-06-27 12:00:55.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nTiny nit in the commit message that you might as well fix, otherwise good IMO.\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":1},{"id":"93d5baa56f524087f63f9955c4a44bac2b573d73","author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"date":"2015-06-27 12:46:12.000000000","message":"Patch Set 1:\n\nEoghan Glynn, thanks,done.","accounts_in_message":[],"_revision_number":1},{"id":"44007fb4e880d8ba8259e0b2c406e0c45e3ee29a","author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"date":"2015-06-27 12:46:28.000000000","message":"Patch Set 2: Commit message was updated","accounts_in_message":[],"_revision_number":2},{"id":"38980fd40f774d5946351c8961987754f04db860","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-27 13:31:50.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/30/196330/2/check/gate-ceilometer-pep8/16b9af7/ : SUCCESS in 3m 21s\n- gate-ceilometer-docs http://docs-draft.openstack.org/30/196330/2/check/gate-ceilometer-docs/6a6d49f//doc/build/html/ : SUCCESS in 6m 10s\n- gate-ceilometer-python27 http://logs.openstack.org/30/196330/2/check/gate-ceilometer-python27/727b6c6/ : SUCCESS in 6m 53s\n- gate-ceilometer-python34 http://logs.openstack.org/30/196330/2/check/gate-ceilometer-python34/3c0ca11/ : SUCCESS in 7m 02s\n- check-tempest-dsvm-full http://logs.openstack.org/30/196330/2/check/check-tempest-dsvm-full/ecce6da/ : SUCCESS in 43m 39s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/30/196330/2/check/check-tempest-dsvm-postgres-full/7a91330/ : SUCCESS in 42m 31s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/30/196330/2/check/check-tempest-dsvm-neutron-full/7415b03/ : SUCCESS in 44m 58s\n- check-grenade-dsvm http://logs.openstack.org/30/196330/2/check/check-grenade-dsvm/5280cb3/ : SUCCESS in 38m 25s\n- check-devstack-dsvm-cells http://logs.openstack.org/30/196330/2/check/check-devstack-dsvm-cells/7cab138/ : SUCCESS in 19m 20s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/30/196330/2/check/gate-tempest-dsvm-ceilometer-mongodb-full/51c24e0/ : SUCCESS in 45m 11s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"80a6223e82f275c2d9849ad641bd97cf84ccc301","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2015-06-27 20:36:56.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"31de2cf9da2bd695ec592dfa44965496c390265d","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2015-06-29 15:07:05.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nlooks good in Heat integration tests","accounts_in_message":[],"_revision_number":2},{"id":"d07ea4a4b27080d3b9dae04f6b5e49bda1111af8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-29 15:07:24.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"d6c9c12fa0b3d7f3396f93f5730ca578dce41a48","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-29 17:00:35.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-ceilometer-docs http://docs-draft.openstack.org/30/196330/2/gate/gate-ceilometer-docs/94edd78//doc/build/html/ : SUCCESS in 3m 55s\n- gate-ceilometer-pep8 http://logs.openstack.org/30/196330/2/gate/gate-ceilometer-pep8/c13369b/ : SUCCESS in 3m 44s\n- gate-ceilometer-python27 http://logs.openstack.org/30/196330/2/gate/gate-ceilometer-python27/2e55566/ : SUCCESS in 7m 32s\n- gate-ceilometer-python34 http://logs.openstack.org/30/196330/2/gate/gate-ceilometer-python34/a104953/ : SUCCESS in 6m 47s\n- gate-tempest-dsvm-full http://logs.openstack.org/30/196330/2/gate/gate-tempest-dsvm-full/fe4b711/ : SUCCESS in 40m 24s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/30/196330/2/gate/gate-tempest-dsvm-postgres-full/609586a/ : SUCCESS in 34m 07s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/30/196330/2/gate/gate-tempest-dsvm-neutron-full/33a901a/ : SUCCESS in 42m 58s\n- gate-grenade-dsvm http://logs.openstack.org/30/196330/2/gate/gate-grenade-dsvm/7f9144e/ : SUCCESS in 42m 17s\n- gate-devstack-dsvm-cells http://logs.openstack.org/30/196330/2/gate/gate-devstack-dsvm-cells/f369dd6/ : SUCCESS in 20m 28s","accounts_in_message":[],"_revision_number":2},{"id":"e4f7c8230b781cc2d3f34655c6870ba8e04809b8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-29 17:00:37.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"9ad75552c54279e21eb301e5f029280879240cfe","revisions":{"cc94e0fe871fa77b808cf6b4b65aea87207967d8":{"kind":"REWORK","_number":1,"created":"2015-06-27 02:22:28.000000000","uploader":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"ref":"refs/changes/30/196330/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/30/196330/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/1"}}},"commit":{"parents":[{"commit":"917fea99bca3604ccb8fd5bed3e4523d23f861e7","subject":"Merge \"Revert \"Add support for posting samples to notification-agent via API\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/917fea99bca3604ccb8fd5bed3e4523d23f861e7"}]}],"author":{"name":"liu-sheng","email":"liusheng@huawei.com","date":"2015-06-27 01:25:31.000000000","tz":480},"committer":{"name":"liu-sheng","email":"liusheng@huawei.com","date":"2015-06-27 01:42:20.000000000","tz":480},"subject":"Add support for posting samples to notification-agent via API","message":"Add support for posting samples to notification-agent via API\n\nThis change make the ceilometer-api getting rid the pipline. samples will\nbe push to notification bus when creating via /v2/meters API.\n\nTo avoid the ceilometer-api\u0027s dependency of notification-agent, especially\nin some testing scenarios, this change also provide a optional parameter\nthat allow samples posted to storage directly.\n\nNOTE: This change reverting the reverted patch[1] and merging it with\npatch[2], the original patch[1] has caused the related bug so has been\nreverted by patch[3], the patch[2] tried to fix the related bug before\nthe patch[1] been reverted.\n\n[1] https://review.openstack.org/#/c/193472/\n\n[2] https://review.openstack.org/#/c/195866/\n\n[3] https://review.openstack.org/#/c/195570/\n\nRelated-bug: #1468697\nPartially implements: blueprint api-no-pipeline\n\nChange-Id: I88cbd83910dcf79bed8a0f9616f98e4346362ab3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/cc94e0fe871fa77b808cf6b4b65aea87207967d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/cc94e0fe871fa77b808cf6b4b65aea87207967d8"}]},"branch":"refs/heads/master"},"9ad75552c54279e21eb301e5f029280879240cfe":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2015-06-27 12:46:28.000000000","uploader":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"ref":"refs/changes/30/196330/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/30/196330/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/30/196330/2"}}},"commit":{"parents":[{"commit":"917fea99bca3604ccb8fd5bed3e4523d23f861e7","subject":"Merge \"Revert \"Add support for posting samples to notification-agent via API\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/917fea99bca3604ccb8fd5bed3e4523d23f861e7"}]}],"author":{"name":"liu-sheng","email":"liusheng@huawei.com","date":"2015-06-27 01:25:31.000000000","tz":480},"committer":{"name":"liusheng","email":"liusheng2048@gmail.com","date":"2015-06-27 12:46:28.000000000","tz":0},"subject":"Add support for posting samples to notification-agent via API","message":"Add support for posting samples to notification-agent via API\n\nThis change make the ceilometer-api getting rid the pipeline. samples will\nbe push to notification bus when creating via /v2/meters API.\n\nTo avoid the ceilometer-api\u0027s dependency of notification-agent, especially\nin some testing scenarios, this change also provide a optional parameter\nthat allow samples posted to storage directly.\n\nNOTE: This change reverting the reverted patch[1] and merging it with\npatch[2], the original patch[1] has caused the related bug so has been\nreverted by patch[3], the patch[2] tried to fix the related bug before\nthe patch[1] been reverted.\n\n[1] https://review.openstack.org/#/c/193472/\n\n[2] https://review.openstack.org/#/c/195866/\n\n[3] https://review.openstack.org/#/c/195570/\n\nRelated-bug: #1468697\nPartially implements: blueprint api-no-pipeline\n\nChange-Id: I88cbd83910dcf79bed8a0f9616f98e4346362ab3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/9ad75552c54279e21eb301e5f029280879240cfe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/9ad75552c54279e21eb301e5f029280879240cfe"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}