)]}'
{"id":"openstack%2Fceilometer~167370","triplet_id":"openstack%2Fceilometer~master~Icab626b28d14514b0f024df447a8e7f35c52257c","project":"openstack/ceilometer","branch":"master","topic":"bug/1435855","hashtags":[],"change_id":"Icab626b28d14514b0f024df447a8e7f35c52257c","subject":"Adds support for default rule in ceilometer policy.json.","status":"MERGED","created":"2015-03-24 18:59:08.000000000","updated":"2015-04-09 10:31:11.000000000","submitted":"2015-04-09 10:31:08.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":8,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"10f0a09feea8c9c01da697e83500a02918e18a8d","_number":167370,"virtual_id_number":167370,"owner":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},{"value":0,"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"value":0,"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},{"value":0,"date":"2015-04-08 13:07:42.000000000","_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":0,"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},{"value":0,"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},{"value":0,"date":"2015-04-08 13:35:00.000000000","_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},{"value":0,"date":"2015-04-08 13:05:50.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":2,"date":"2015-04-09 10:31:08.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"value":0,"date":"2015-04-08 13:08:02.000000000","_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"value":0,"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"}],"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":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"all":[{"value":0,"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},{"value":2,"date":"2015-04-08 10:39:54.000000000","_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"value":0,"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},{"value":0,"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":2,"date":"2015-04-07 19:43:34.000000000","_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},{"value":2,"date":"2015-04-09 01:31:25.000000000","_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},{"value":0,"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2015-04-09 08:31:09.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"value":0,"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"value":1,"date":"2015-04-07 19:59:41.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"}],"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":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"all":[{"value":0,"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},{"value":1,"date":"2015-04-08 10:39:54.000000000","_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"value":0,"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},{"value":0,"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"value":0,"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"value":0,"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},{"value":0,"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},{"value":0,"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"value":0,"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"value":0,"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"}],"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":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},{"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-24 18:59:14.000000000","updated_by":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"reviewer":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"state":"REVIEWER"},{"updated":"2015-03-25 07:54:20.000000000","updated_by":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"reviewer":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"state":"REVIEWER"},{"updated":"2015-03-26 05:33:51.000000000","updated_by":{"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},"reviewer":{"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},"state":"REVIEWER"},{"updated":"2015-03-28 01:32:10.000000000","updated_by":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"reviewer":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"state":"REVIEWER"},{"updated":"2015-03-31 14:06:08.000000000","updated_by":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"reviewer":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"state":"REVIEWER"},{"updated":"2015-04-07 19:43:34.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-04-07 19:59:41.000000000","updated_by":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"reviewer":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"state":"REVIEWER"},{"updated":"2015-04-08 10:39:54.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-04-08 13:05:50.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2015-04-08 13:07:42.000000000","updated_by":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"reviewer":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"state":"REVIEWER"},{"updated":"2015-04-08 13:08:02.000000000","updated_by":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"reviewer":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"state":"REVIEWER"},{"updated":"2015-04-09 01:31:25.000000000","updated_by":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"reviewer":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"state":"REVIEWER"},{"updated":"2015-04-09 08:31:09.000000000","updated_by":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"reviewer":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"state":"REVIEWER"},{"updated":"2015-04-09 10:31:08.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d5762450b8101072e9308373e033d4ed68434fee","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-24 18:59:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"1cf74a6b693d94791d347acf22a139f61ad68be4","author":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"date":"2015-03-24 18:59:14.000000000","message":"Patch Set 1:\n\nThank you for your first contribution to OpenStack.\n\n    Your patch will now be tested automatically by OpenStack testing frameworks\n    and once the automatic tests pass, it will be reviewed by other friendly\n    developers. They will give you feedback and may require you to refine it.\n\n    People seldom get their patch approved on the first try, so don\u0027t be\n    concerned if requested to make corrections. Feel free to modify your patch\n    and resubmit a new change-set.\n\n    Patches usually take 3 to 7 days to be reviewed so be patient and be\n    available on IRC to ask and answer questions about your work. The more you\n    participate in the community the more rewarding it is for you. You may also\n    notice that the more you get to know people and get to be known, the faster\n    your patches will be reviewed and eventually approved. Get to know others\n    and become known by doing code reviews: anybody can do it, and it\u0027s a\n    great way to learn the code base.\n\n    Thanks again for supporting OpenStack, we look forward to working with you.\n\n    IRC: https://wiki.openstack.org/wiki/IRC\n    Workflow: http://docs.openstack.org/infra/manual/developers.html\n    Commit Messages: https://wiki.openstack.org/wiki/GitCommitMessages","accounts_in_message":[],"_revision_number":1},{"id":"8f00d3f7a47c68849088094c22268a6a6ec04899","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-24 20:02:56.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/70/167370/1/check/gate-ceilometer-pep8/da5983b/ : SUCCESS in 2m 59s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/1/check/gate-ceilometer-docs/3c5d6d6//doc/build/html/ : SUCCESS in 4m 40s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/1/check/gate-ceilometer-python27/c96f9c8/ : FAILURE in 6m 59s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/1/check/gate-ceilometer-python34/9107346/ : SUCCESS in 12m 26s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/1/check/check-tempest-dsvm-full/4b4af13/ : FAILURE in 47m 46s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/1/check/check-tempest-dsvm-postgres-full/b2595ec/ : FAILURE in 37m 46s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/1/check/check-tempest-dsvm-neutron-full/6995907/ : FAILURE in 54m 04s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/1/check/check-grenade-dsvm/310c417/ : FAILURE in 38m 20s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/1/check/check-devstack-dsvm-cells/946e472/ : SUCCESS in 19m 34s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/1/check/check-swift-dsvm-functional/276e948/ : SUCCESS in 25m 02s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/70/167370/1/check/gate-tempest-dsvm-ceilometer-mongodb/e386c44/ : FAILURE in 25m 58s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/1/check/gate-tempest-dsvm-ceilometer-mongodb-full/16ff760/ : FAILURE in 38m 42s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"09d9d058a32e3fb4c7ebe618fdf7ba163c970f46","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-03-24 22:34:03.000000000","message":"Patch Set 1: Code-Review-1\n\nThe commit message isn\u0027t properly formatted.\n\nAlso, I suspect we need to at least add the \"default\" rule in policy.json for this to work, and may need to specify others as well.","accounts_in_message":[],"_revision_number":1},{"id":"f98a10cb81e8644d67c97d1376a25aec865489a2","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2015-03-25 02:57:14.000000000","message":"Patch Set 1: Code-Review-1\n\nagree with Matthew Edmonds","accounts_in_message":[],"_revision_number":1},{"id":"459db3513f15b6c7c605b3a073e3e6a26c1034c9","author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"date":"2015-03-25 07:54:20.000000000","message":"Patch Set 1: Code-Review-1\n\nplease take a look at link https://wiki.openstack.org/wiki/GitCommitMessages it could be helpful for you to format commit message in proper way.","accounts_in_message":[],"_revision_number":1},{"id":"a90bc71aa35bf1418305e02088f12f37a20c7369","author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"date":"2015-03-25 10:40:56.000000000","message":"Patch Set 1: Code-Review-1","accounts_in_message":[],"_revision_number":1},{"id":"3cf3b75eaf3a92a261b3c9dfd5863f405c85ce96","author":{"_account_id":9775,"name":"Sudipta Biswas","email":"sbiswas7@in.ibm.com","username":"sudswas"},"date":"2015-03-26 05:41:37.000000000","message":"Patch Set 1: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":1},{"id":"3ba593cb091f7c42f8e1a28a875a6669d454f710","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-27 13:35:45.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"57a6ab47a62ce8427aa819dd9ac4d8290e49db4c","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-27 13:38:46.000000000","message":"Patch Set 2:\n\nFixed the commit message. Added default entry in the policy.json . UTs run fine in my local","accounts_in_message":[],"_revision_number":2},{"id":"503ff27748ba522dcdeab47f99059def6393a008","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-27 14:52:02.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/70/167370/2/check/gate-ceilometer-pep8/6a9bb92/ : SUCCESS in 3m 24s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/2/check/gate-ceilometer-docs/1759fd5//doc/build/html/ : SUCCESS in 5m 23s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/2/check/gate-ceilometer-python27/0424dd2/ : SUCCESS in 6m 19s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/2/check/gate-ceilometer-python34/e9b1197/ : SUCCESS in 5m 23s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/2/check/check-tempest-dsvm-full/edb8c94/ : FAILURE in 48m 25s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/2/check/check-tempest-dsvm-postgres-full/cf1c17a/ : SUCCESS in 40m 09s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/2/check/check-tempest-dsvm-neutron-full/35f43bd/ : SUCCESS in 1h 11m 28s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/2/check/check-grenade-dsvm/5c069b0/ : SUCCESS in 33m 52s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/2/check/check-devstack-dsvm-cells/6d2e97d/ : SUCCESS in 19m 05s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/2/check/check-swift-dsvm-functional/40bdfb8/ : SUCCESS in 22m 39s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/70/167370/2/check/gate-tempest-dsvm-ceilometer-mongodb/78f9ff8/ : SUCCESS in 27m 26s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/2/check/gate-tempest-dsvm-ceilometer-mongodb-full/121e104/ : SUCCESS in 41m 09s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"a8afebc8c935f8967f995312de13c67a96e64e7c","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-03-27 16:03:05.000000000","message":"Patch Set 2: Code-Review-1\n\nyou lost the rbac.py changes from your first patch set.","accounts_in_message":[],"_revision_number":2},{"id":"06869773c6135162f62b84eaf8412d87834fd30c","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-28 01:20:30.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"e4f2bf74026c5ae8a5f0f21407e6f7f3efd96880","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2015-03-28 01:32:10.000000000","message":"Patch Set 3: Code-Review-1\n\n(1 comment)\n\ncould you add some unit test to verify it?","accounts_in_message":[],"_revision_number":3},{"id":"1aeee8bc561358d1c72912ea8757c128e4d3b407","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-28 04:41:58.000000000","message":"Patch Set 3: 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/70/167370/3/check/gate-ceilometer-pep8/7726b16/ : FAILURE in 3m 41s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/3/check/gate-ceilometer-docs/8c35ed4//doc/build/html/ : SUCCESS in 4m 01s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/3/check/gate-ceilometer-python27/67d9d5d/ : SUCCESS in 6m 43s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/3/check/gate-ceilometer-python34/321614c/ : SUCCESS in 5m 15s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/3/check/check-tempest-dsvm-full/8126c57/ : SUCCESS in 44m 36s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/3/check/check-tempest-dsvm-postgres-full/996158a/ : SUCCESS in 36m 42s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/3/check/check-tempest-dsvm-neutron-full/8e78b46/ : SUCCESS in 52m 09s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/3/check/check-grenade-dsvm/b28ba45/ : FAILURE in 36m 04s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/3/check/check-devstack-dsvm-cells/8a7139e/ : SUCCESS in 19m 10s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/3/check/check-swift-dsvm-functional/2bb24b8/ : SUCCESS in 22m 51s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/70/167370/3/check/gate-tempest-dsvm-ceilometer-mongodb/a0a3f1e/ : SUCCESS in 25m 29s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/3/check/gate-tempest-dsvm-ceilometer-mongodb-full/89ccd78/ : SUCCESS in 42m 44s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"b0e29f6a40002c1e182a8696fb98db3e75c197eb","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-03-30 11:28:24.000000000","message":"Patch Set 3: Code-Review-1\n\nNeed to resolve pep8 errors:\n\n./ceilometer/api/rbac.py:57:1: E303 too many blank lines (3)\n./ceilometer/api/rbac.py:60:1: E302 expected 2 blank lines, found 3","accounts_in_message":[],"_revision_number":3},{"id":"cd37d9d3bc09ef84cf2449ca9d6e925e45c359af","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-30 11:42:38.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"68690516426aef4d1a15a8bf0ecea0b0f2e65b3f","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-30 11:50:40.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"7a5547005d2056b3c82c0c0beffff128d46f390e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-30 12:44:28.000000000","message":"Patch Set 5: 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/70/167370/5/check/gate-ceilometer-pep8/084adc7/ : SUCCESS in 3m 25s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/5/check/gate-ceilometer-docs/9aeb742//doc/build/html/ : SUCCESS in 4m 08s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/5/check/gate-ceilometer-python27/7f59001/ : SUCCESS in 6m 24s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/5/check/gate-ceilometer-python34/7c9b53c/ : SUCCESS in 6m 42s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/5/check/check-tempest-dsvm-full/eca7054/ : SUCCESS in 43m 06s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/5/check/check-tempest-dsvm-postgres-full/9527732/ : SUCCESS in 38m 37s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/5/check/check-tempest-dsvm-neutron-full/24061ac/ : SUCCESS in 50m 05s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/5/check/check-grenade-dsvm/7767ba3/ : FAILURE in 37m 05s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/5/check/check-devstack-dsvm-cells/c6328fe/ : SUCCESS in 19m 36s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/5/check/check-swift-dsvm-functional/cb73be7/ : SUCCESS in 21m 50s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/70/167370/5/check/gate-tempest-dsvm-ceilometer-mongodb/597332b/ : SUCCESS in 25m 26s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/5/check/gate-tempest-dsvm-ceilometer-mongodb-full/373cf8f/ : SUCCESS in 39m 34s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"4937dd59afdef451c6a097c71131c40e1ea6aedd","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-03-31 01:29:11.000000000","message":"Patch Set 5: Code-Review-1\n\nstarted this thread on the mailing list regarding the grenade failures: http://lists.openstack.org/pipermail/openstack-dev/2015-March/060209.html","accounts_in_message":[],"_revision_number":5},{"id":"2db2dc56fde3ddcfe389cc3961d1b98ce0c3bf7b","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2015-03-31 14:31:57.000000000","message":"Patch Set 5:\n\njust to clarify, is the issue that if the policy.json file doesn\u0027t have a \u0027default\u0027 value, it breaks?","accounts_in_message":[],"_revision_number":5},{"id":"92d95510e70d5f813545853f2ee65cf78c9a43a7","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-31 17:03:34.000000000","message":"Patch Set 5:\n\nwell..The problem with the current code is that if a specific key (for eg. get_meters) is not explicitly mentioned in the policy.json , it does not check for the default value. So , let\u0027s say I make v2/meters call and \"get_meters\" is not mentioned among the set of rules in the policy.json , ideally it show consider the default value (i e if default value is defined) or return false (as per the logic in oslo_policy) . However, the current code does none. The current code does absolutely nothing (i e no rbac related validation) if the rule is not specified in the policy.json. This occurs because the rbac.enforce gets all the rule keys and loops through them and checks if the rule key corresponding to the requested rest api is present in policy.json . Instead , it should just pass the control to oslo_policy because oslo_policy already has all that logic implemeted.","accounts_in_message":[],"_revision_number":5},{"id":"a90569031ce1d7c31027d51e1aeca77c432b40cc","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-03-31 17:09:21.000000000","message":"Patch Set 5:\n\nThe changes made as part of this review for rbac.py removes the looping through the rule keys and forwards the control to oslo_policy so that it uses oslo_policy to implement rbac validation .With this change, if a rule key for a rest api is not defined in the policy.json , the default value will be used and if there is no default value , the oslo_policy enforce method will return a false.","accounts_in_message":[],"_revision_number":5},{"id":"3da2a33ac1d39dc650516226665dc978d6564079","author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"date":"2015-03-31 17:33:28.000000000","message":"Patch Set 5: Code-Review-1\n\nDivya,\n  if you don\u0027t specify any corresponding policy method in the policy file you really want to preserve the old behavior (backward compatible) where the only thing that Ceilometer checks is that you are an admin or not. This is taken care in the two legacy methods where the segregation policy is used. If you introduce a default that is empty, like you do, you are not adding anything to what it is currently doing, but if you then change the value of that default to something then you break the old behavior. Project access scope in Ceilometer API is messy to say the least.","accounts_in_message":[],"_revision_number":5},{"id":"5c80a15827784f358dfffcbbff06d924f5525d33","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-03-31 19:30:34.000000000","message":"Patch Set 5:\n\n@Fabio, the introduction of the segregation rule actually broke backward compat, since that didn\u0027t exist prior to Juno. But we can fix that by modifying the get_limited_to method to use the context_is_admin rule if the segregation rule doesn\u0027t exist. As for the addition of a default rule... that doesn\u0027t have to break backward compat either, if we do things your way when the default rule is missing and Divya\u0027s way when the default rule is present.\n\n@Divya, I\u0027ve coded up a fix for both the segregation and default rule usage to make them backward compatible. Works in my testing. I\u0027ll email to you so you can update your patch here.","accounts_in_message":[],"_revision_number":5},{"id":"87dffa2acab4badeb85c2b1480212ecd9728deb2","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2015-03-31 19:30:43.000000000","message":"Patch Set 5:\n\nDivya,\n\nThanks for your patch, please see the associated discussion on the ML:\n\n  http://lists.openstack.org/pipermail/openstack-dev/2015-March/060209.html\n\nWith the proposed changes, I believe we can land this for kilo-rc1.\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":5},{"id":"6fd66e4f87195a706f07cfd4d4d16439253a1e6c","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-04-01 14:59:17.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"0b4e1442741950338b82f88d17c751f451d274f5","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-01 15:02:07.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"663d91379c53b072167ffe5c5b313da252d27fb3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-01 16:11:45.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/6/check/gate-ceilometer-pep8/f0bb784/ : SUCCESS in 3m 35s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/6/check/gate-ceilometer-docs/cf1e81b//doc/build/html/ : SUCCESS in 4m 05s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/6/check/gate-ceilometer-python27/9e8cef6/ : SUCCESS in 8m 32s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/6/check/gate-ceilometer-python34/dab0c0b/ : SUCCESS in 4m 57s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/6/check/check-tempest-dsvm-full/9924145/ : SUCCESS in 51m 24s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/6/check/check-tempest-dsvm-postgres-full/78022b8/ : SUCCESS in 41m 35s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/6/check/check-tempest-dsvm-neutron-full/a9b4b44/ : SUCCESS in 53m 42s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/6/check/check-grenade-dsvm/21d302a/ : SUCCESS in 44m 30s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/6/check/check-devstack-dsvm-cells/24cfd65/ : SUCCESS in 26m 03s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/6/check/check-swift-dsvm-functional/eb97038/ : SUCCESS in 22m 28s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/70/167370/6/check/gate-tempest-dsvm-ceilometer-mongodb/cc7e0e6/ : SUCCESS in 29m 15s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/6/check/gate-tempest-dsvm-ceilometer-mongodb-full/e0115d1/ : SUCCESS in 46m 40s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"5f73eadf3a12ccff8ec23537455e0a8ebc01aae0","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2015-04-01 17:55:06.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nThanks for the quick turnaround Divya, a minor suggestion inline, otherwise good to fly IMO.\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":6},{"id":"5d837a31005673b4cb6fe4b3f454d82aa08ad57f","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-04-02 17:37:35.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"b414d17b74d2929f43a069e8aa4413683aad2be1","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-02 18:05:06.000000000","message":"Patch Set 7: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"e031d047e01f8b37679defe46824693c353de4be","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-02 18:46:29.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/7/check/gate-ceilometer-pep8/9f5e73e/ : SUCCESS in 3m 59s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/7/check/gate-ceilometer-docs/d3c58e4//doc/build/html/ : SUCCESS in 3m 56s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/7/check/gate-ceilometer-python27/cac3c56/ : SUCCESS in 8m 28s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/7/check/gate-ceilometer-python34/88e4c30/ : SUCCESS in 5m 14s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/7/check/check-tempest-dsvm-full/3f2805b/ : SUCCESS in 46m 00s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/7/check/check-tempest-dsvm-postgres-full/22434fa/ : SUCCESS in 45m 36s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/7/check/check-tempest-dsvm-neutron-full/def62d4/ : SUCCESS in 1h 04m 05s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/7/check/check-grenade-dsvm/ac558ef/ : SUCCESS in 45m 32s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/7/check/check-devstack-dsvm-cells/e1cb3d3/ : SUCCESS in 23m 13s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/7/check/check-swift-dsvm-functional/3b147d4/ : SUCCESS in 26m 17s\n- gate-tempest-dsvm-ceilometer-mongodb http://logs.openstack.org/70/167370/7/check/gate-tempest-dsvm-ceilometer-mongodb/7943832/ : SUCCESS in 31m 16s (non-voting)\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/7/check/gate-tempest-dsvm-ceilometer-mongodb-full/1e83747/ : SUCCESS in 49m 05s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"a57fd7a6e879eed196320cec484eb551d35a09c6","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-04-06 17:24:20.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"50425dae88dece7ebb34adfc1616ec8016dbe9d9","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-06 17:26:47.000000000","message":"Patch Set 8: Code-Review+1","accounts_in_message":[],"_revision_number":8},{"id":"cd3d8cd7a5fbc79eee2579aa2abb50b55e4913b9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-06 19:57:49.000000000","message":"Patch Set 8: 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/70/167370/8/check/gate-ceilometer-pep8/fdac622/ : SUCCESS in 3m 38s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/8/check/gate-ceilometer-docs/7ac22ea//doc/build/html/ : SUCCESS in 16m 07s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/8/check/gate-ceilometer-python27/c54b5bf/ : SUCCESS in 18m 25s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/8/check/gate-ceilometer-python34/a536d67/ : SUCCESS in 10m 13s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/8/check/check-tempest-dsvm-full/af0d014/ : SUCCESS in 1h 02m 33s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/8/check/check-tempest-dsvm-postgres-full/bce29f1/ : SUCCESS in 41m 32s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/8/check/check-tempest-dsvm-neutron-full/b9c4668/ : FAILURE in 1h 57m 16s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/8/check/check-grenade-dsvm/bdbbfc3/ : SUCCESS in 35m 55s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/8/check/check-devstack-dsvm-cells/505deb7/ : SUCCESS in 23m 04s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/8/check/check-swift-dsvm-functional/bd3c19f/ : SUCCESS in 22m 27s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/8/check/gate-tempest-dsvm-ceilometer-mongodb-full/0e1055b/ : SUCCESS in 40m 50s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"7802b66748440f6573a29f07d0a03c7d52c4624d","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2015-04-06 19:59:23.000000000","message":"Patch Set 8:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- check-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1311066\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":8},{"id":"0a366a8baaff43261f3dea54400eeae3af31e2fc","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-06 21:00:34.000000000","message":"Patch Set 8:\n\nrecheck","accounts_in_message":[],"_revision_number":8},{"id":"871d51485a282b70ce1dbd907c5de5cb82c992ca","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-06 21:52:35.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/8/check/gate-ceilometer-pep8/026bb44/ : SUCCESS in 4m 16s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/8/check/gate-ceilometer-docs/3b83eff//doc/build/html/ : SUCCESS in 4m 06s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/8/check/gate-ceilometer-python27/6a7ee90/ : SUCCESS in 6m 02s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/8/check/gate-ceilometer-python34/68d65f4/ : SUCCESS in 5m 09s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/8/check/check-tempest-dsvm-full/d3d3f09/ : SUCCESS in 45m 25s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/8/check/check-tempest-dsvm-postgres-full/633419d/ : SUCCESS in 39m 48s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/8/check/check-tempest-dsvm-neutron-full/63d045c/ : SUCCESS in 51m 43s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/8/check/check-grenade-dsvm/8fbd8c7/ : SUCCESS in 42m 18s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/8/check/check-devstack-dsvm-cells/242efbd/ : SUCCESS in 21m 13s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/8/check/check-swift-dsvm-functional/4900d5e/ : SUCCESS in 22m 02s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/8/check/gate-tempest-dsvm-ceilometer-mongodb-full/44c1771/ : SUCCESS in 45m 10s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"f34c179a9169474dfc94ec6119998f65aa215197","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2015-04-07 08:24:39.000000000","message":"Patch Set 8: Code-Review-1\n\n(1 comment)\n\nThanks Divya, a minor style nitpick inline, otherwise good to land IMO.\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":8},{"id":"e9aeb57d39ed9b4f2fad71eb160ae12c63529e10","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-04-07 11:43:21.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"6bf3af210871f52ac1e7cc6544a3dd4593dc8a66","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-07 12:44:04.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/9/check/gate-ceilometer-pep8/0258e71/ : SUCCESS in 4m 01s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/9/check/gate-ceilometer-docs/beb18dd//doc/build/html/ : SUCCESS in 6m 07s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/9/check/gate-ceilometer-python27/6723cd7/ : SUCCESS in 6m 26s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/9/check/gate-ceilometer-python34/0083efd/ : SUCCESS in 7m 02s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/9/check/check-tempest-dsvm-full/99dbd7b/ : SUCCESS in 48m 43s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/9/check/check-tempest-dsvm-postgres-full/1b8c255/ : SUCCESS in 42m 17s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/9/check/check-tempest-dsvm-neutron-full/4f95db8/ : SUCCESS in 59m 34s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/9/check/check-grenade-dsvm/53dd473/ : SUCCESS in 45m 28s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/9/check/check-devstack-dsvm-cells/0bc8c0f/ : SUCCESS in 20m 04s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/9/check/check-swift-dsvm-functional/d2af0ea/ : SUCCESS in 22m 56s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/9/check/gate-tempest-dsvm-ceilometer-mongodb-full/9d8821a/ : SUCCESS in 46m 18s (non-voting)","accounts_in_message":[],"_revision_number":9},{"id":"c21cb135641afd9da258563bc9fdf684abc932e1","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-07 12:57:28.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"0342223109d3773b90a4962e11c69f89d5125943","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2015-04-07 13:05:01.000000000","message":"Patch Set 9: Code-Review-1\n\n(1 comment)\n\nminor issue as the logic seems to be repeated.","accounts_in_message":[],"_revision_number":9},{"id":"64c1046dabb0710a40c14650ea6939778ebbbfdb","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2015-04-07 13:32:25.000000000","message":"Patch Set 9: -Code-Review\n\nremoving -1 as it\u0027s not a blocker... i think it should be cleaned up though.","accounts_in_message":[],"_revision_number":9},{"id":"1599da293d13dc040123e1a6a1ddfeab8281aeb8","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-04-07 13:43:36.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"48bd8537079df38ab8633055af7742654d46d4b7","author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"date":"2015-04-07 14:32:38.000000000","message":"Patch Set 10: Code-Review-1\n\nHi Divya,\n\nThe change itself looks good to me. Could you please provide tests too for it?\n\nThanks and Best Regards,\nIldiko","accounts_in_message":[],"_revision_number":10},{"id":"ef9ac4fb6387d8be4e24270c49ddd05835123c53","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-07 15:06:21.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-pep8/ea07c28/ : SUCCESS in 3m 44s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/check/gate-ceilometer-docs/cd4abc5//doc/build/html/ : SUCCESS in 6m 00s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python27/b64af3b/ : SUCCESS in 6m 08s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python34/7b20e35/ : SUCCESS in 5m 03s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-full/404f3d2/ : SUCCESS in 47m 37s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-postgres-full/492f5d4/ : SUCCESS in 43m 13s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-neutron-full/8699ae5/ : SUCCESS in 1h 01m 57s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/10/check/check-grenade-dsvm/5d881b1/ : SUCCESS in 45m 27s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/check/check-devstack-dsvm-cells/66ca922/ : SUCCESS in 19m 44s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/10/check/check-swift-dsvm-functional/25496a6/ : SUCCESS in 23m 26s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/10/check/gate-tempest-dsvm-ceilometer-mongodb-full/bdf7362/ : SUCCESS in 44m 12s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"4d7a9f187b33ba98f246a227ad289f3f0087005d","author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"date":"2015-04-07 19:43:34.000000000","message":"Patch Set 10: Code-Review+2\n\ni\u0027m ok with this. the code is tested via the addition of default to etc/ceilometer/policy.json (UT breaks without it) and the backward compatibility is contained by grenade tests.","accounts_in_message":[],"_revision_number":10},{"id":"60869b9b6f7cc2e16d37e619dad3c9c45fff2444","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-07 19:59:41.000000000","message":"Patch Set 10: Code-Review+1\n\nThe case where default exists in the policy is tested by UT, and the case where it doesn\u0027t is tested by grenade.","accounts_in_message":[],"_revision_number":10},{"id":"da1649ccd550fafa04cab67d0c2d56eb11fe3886","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2015-04-08 10:39:54.000000000","message":"Patch Set 10: Code-Review+2 Workflow+1\n\nAs agreed on #openstack-ceilometer IRC with ildikov \u0026 dikonoor, let\u0027s punt the resolution of the testing issue to a subsequent patch.\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":10},{"id":"55fece8ebff3b0173b28a8122f3cc4b036db605f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 10:39:59.000000000","message":"Patch Set 10: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":10},{"id":"43044212f3f40fe3d815400c99885885459d35af","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 13:04:52.000000000","message":"Patch Set 10: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/gate/gate-ceilometer-docs/aaf6f0d//doc/build/html/ : SUCCESS in 4m 16s\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-pep8/c0aea76/ : SUCCESS in 3m 11s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-python27/97aa8ac/ : FAILURE in 17m 31s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-python34/efeed04/ : SUCCESS in 6m 22s\n- gate-tempest-dsvm-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-full/3ceebd1/ : SUCCESS in 42m 32s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-postgres-full/5736746/ : FAILURE in 1h 34m 32s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-neutron-full/c4b169a/ : FAILURE in 58m 27s\n- gate-grenade-dsvm http://logs.openstack.org/70/167370/10/gate/gate-grenade-dsvm/597c724/ : SUCCESS in 40m 30s\n- gate-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/gate/gate-devstack-dsvm-cells/20ddacc/ : SUCCESS in 19m 39s\n- gate-swift-dsvm-functional http://logs.openstack.org/70/167370/10/gate/gate-swift-dsvm-functional/86dc612/ : SUCCESS in 22m 35s","accounts_in_message":[],"_revision_number":10},{"id":"e912b9c3d0d05de43ff8c33e8e91b0c9570464b5","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2015-04-08 13:05:50.000000000","message":"Patch Set 10:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- gate-ceilometer-python27: unrecognized error\n- gate-tempest-dsvm-neutron-full: unrecognized error\n- gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1421895 https://bugs.launchpad.net/bugs/1311066 https://bugs.launchpad.net/bugs/1352498\n\nSome of the tests failed in a way that we did not understand. Please help us classify these issues so that they can be part of Elastic Recheck http://status.openstack.org/elastic-recheck/\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":10},{"id":"b6774625c4edea0d696200685fe383a13c2dfe06","author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"date":"2015-04-08 13:07:42.000000000","message":"Patch Set 10:\n\nrecheck","accounts_in_message":[],"_revision_number":10},{"id":"d9ff22bd78072c57161a5f8f2253a35850a5a3c0","author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"date":"2015-04-08 13:08:02.000000000","message":"Patch Set 10:\n\nrecheck bug 1352498","accounts_in_message":[],"_revision_number":10},{"id":"82c6ceb7f8163a0e39e73a8ce97e14305ffcc60a","author":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"date":"2015-04-08 13:35:00.000000000","message":"Patch Set 10:\n\nrecheck","accounts_in_message":[],"_revision_number":10},{"id":"ee5b71b5a2efb5dd75ef487a7cfaae99068d0b96","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 14:12:32.000000000","message":"Patch Set 10: 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/70/167370/10/check/gate-ceilometer-pep8/7d8ef72/ : SUCCESS in 3m 19s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/check/gate-ceilometer-docs/06c5a85//doc/build/html/ : SUCCESS in 5m 44s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python27/e24f829/ : FAILURE in 17m 33s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python34/045f5bb/ : SUCCESS in 6m 42s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-full/4aa932e/ : SUCCESS in 45m 57s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-postgres-full/db2aee7/ : SUCCESS in 57m 44s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-neutron-full/bddcb49/ : SUCCESS in 1h 01m 29s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/10/check/check-grenade-dsvm/680f77c/ : SUCCESS in 40m 48s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/check/check-devstack-dsvm-cells/610ec2c/ : SUCCESS in 18m 07s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/10/check/check-swift-dsvm-functional/6f6093e/ : SUCCESS in 23m 36s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/10/check/gate-tempest-dsvm-ceilometer-mongodb-full/a70ffac/ : SUCCESS in 41m 49s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"348f68e646e4c2132b732f708d094d9ce1643a02","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-08 18:47:13.000000000","message":"Patch Set 10:\n\nrecheck bug 1441393","accounts_in_message":[],"_revision_number":10},{"id":"1f5daab34eb730f21b392f4249a1e67776230998","author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"date":"2015-04-08 18:52:13.000000000","message":"Patch Set 10:\n\nrecheck bug 1421895","accounts_in_message":[],"_revision_number":10},{"id":"c956e59a1581047eb3fa15740021de82d3c03992","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-08 19:47:11.000000000","message":"Patch Set 10:\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/70/167370/10/check/gate-ceilometer-pep8/75e1620/ : SUCCESS in 3m 17s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/check/gate-ceilometer-docs/f2b3879//doc/build/html/ : SUCCESS in 5m 00s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python27/9256eeb/ : FAILURE in 19m 16s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python34/9b250f3/ : SUCCESS in 4m 53s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-full/10e681e/ : SUCCESS in 59m 49s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-postgres-full/b2cc7bf/ : SUCCESS in 47m 09s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-neutron-full/63ee851/ : SUCCESS in 54m 48s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/10/check/check-grenade-dsvm/ea8dee2/ : SUCCESS in 38m 58s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/check/check-devstack-dsvm-cells/6d66ec6/ : SUCCESS in 20m 17s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/10/check/check-swift-dsvm-functional/55b3f89/ : SUCCESS in 24m 50s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/10/check/gate-tempest-dsvm-ceilometer-mongodb-full/ea12aee/ : SUCCESS in 44m 47s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"f405c5f6f47f37f412fe4ff6cd8bdf4b6a9cf1af","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-04-09 01:31:25.000000000","message":"Patch Set 10: Code-Review+2\n\nlet\u0027s wait https://review.openstack.org/171458 land in first to unblock the gate","accounts_in_message":[],"_revision_number":10},{"id":"8ff2d17090995932a40c8508ed27ecbcd3ca685b","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-04-09 04:53:01.000000000","message":"Patch Set 10:\n\nreverify bug 1441393","accounts_in_message":[],"_revision_number":10},{"id":"199cf22645216e9e0309fad3a2615c27f4cc5b6f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 05:57:04.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-pep8/3f75815/ : SUCCESS in 3m 23s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/check/gate-ceilometer-docs/0bbab56//doc/build/html/ : SUCCESS in 4m 07s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python27/1806edc/ : SUCCESS in 7m 17s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python34/a2c0a82/ : SUCCESS in 5m 17s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-full/b0ad998/ : SUCCESS in 45m 22s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-postgres-full/a4c7465/ : SUCCESS in 49m 54s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-neutron-full/37c15bf/ : SUCCESS in 1h 00m 04s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/10/check/check-grenade-dsvm/df2843d/ : SUCCESS in 40m 19s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/check/check-devstack-dsvm-cells/fbcc869/ : SUCCESS in 18m 25s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/10/check/check-swift-dsvm-functional/d5712d2/ : SUCCESS in 23m 44s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/10/check/gate-tempest-dsvm-ceilometer-mongodb-full/fb846a0/ : SUCCESS in 46m 46s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"a6baa090de524513e41eba23bce8e71c72413d64","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 05:57:10.000000000","message":"Patch Set 10: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":10},{"id":"4156bbd470c90a28db5676e8287e952e7f6fa869","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 07:12:45.000000000","message":"Patch Set 10: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/gate/gate-ceilometer-docs/f435899//doc/build/html/ : SUCCESS in 4m 54s\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-pep8/5cba782/ : SUCCESS in 4m 03s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-python27/f5e3b49/ : FAILURE in 9m 23s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-python34/0f9c9bb/ : SUCCESS in 6m 52s\n- gate-tempest-dsvm-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-full/e00bf66/ : SUCCESS in 1h 02m 23s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-postgres-full/d01733f/ : SUCCESS in 41m 26s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-neutron-full/c130ead/ : SUCCESS in 1h 14m 17s\n- gate-grenade-dsvm http://logs.openstack.org/70/167370/10/gate/gate-grenade-dsvm/f2dffd4/ : SUCCESS in 43m 47s\n- gate-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/gate/gate-devstack-dsvm-cells/7e5f6dd/ : SUCCESS in 23m 32s\n- gate-swift-dsvm-functional http://logs.openstack.org/70/167370/10/gate/gate-swift-dsvm-functional/eb7dd10/ : SUCCESS in 23m 00s","accounts_in_message":[],"_revision_number":10},{"id":"375ed3229f9d147bc3e19e0510468dae5db35dc3","author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"date":"2015-04-09 08:31:09.000000000","message":"Patch Set 10: -Code-Review\n\nreverify","accounts_in_message":[],"_revision_number":10},{"id":"23452be1cf67d7a9b0db256513ad54c2a599acbc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 09:37:00.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-pep8/21c2111/ : SUCCESS in 3m 26s\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/check/gate-ceilometer-docs/26f6a1e//doc/build/html/ : SUCCESS in 4m 14s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python27/5a0537c/ : SUCCESS in 6m 27s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/check/gate-ceilometer-python34/587a772/ : SUCCESS in 5m 22s\n- check-tempest-dsvm-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-full/2e1a7b1/ : SUCCESS in 48m 32s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-postgres-full/15f0562/ : SUCCESS in 49m 42s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/check/check-tempest-dsvm-neutron-full/97a66f0/ : SUCCESS in 56m 31s\n- check-grenade-dsvm http://logs.openstack.org/70/167370/10/check/check-grenade-dsvm/8f86605/ : SUCCESS in 43m 46s\n- check-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/check/check-devstack-dsvm-cells/ebc0e17/ : SUCCESS in 20m 38s\n- check-swift-dsvm-functional http://logs.openstack.org/70/167370/10/check/check-swift-dsvm-functional/b1d2a61/ : SUCCESS in 22m 14s\n- gate-tempest-dsvm-ceilometer-mongodb-full http://logs.openstack.org/70/167370/10/check/gate-tempest-dsvm-ceilometer-mongodb-full/bc795e3/ : SUCCESS in 42m 04s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"e2dc13c48e4856fe8d1f77032df5079dec11af61","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 09:37:07.000000000","message":"Patch Set 10: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":10},{"id":"7bb787a50e9684b7f01d025eea61cac78be6d2af","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 10:31:08.000000000","message":"Patch Set 10: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-ceilometer-docs http://docs-draft.openstack.org/70/167370/10/gate/gate-ceilometer-docs/90b20c1//doc/build/html/ : SUCCESS in 5m 57s\n- gate-ceilometer-pep8 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-pep8/a4984ee/ : SUCCESS in 3m 31s\n- gate-ceilometer-python27 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-python27/b503b18/ : SUCCESS in 6m 28s\n- gate-ceilometer-python34 http://logs.openstack.org/70/167370/10/gate/gate-ceilometer-python34/5314eeb/ : SUCCESS in 5m 30s\n- gate-tempest-dsvm-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-full/58537b4/ : SUCCESS in 49m 53s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-postgres-full/ba4a488/ : SUCCESS in 48m 30s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/70/167370/10/gate/gate-tempest-dsvm-neutron-full/895c89f/ : SUCCESS in 50m 29s\n- gate-grenade-dsvm http://logs.openstack.org/70/167370/10/gate/gate-grenade-dsvm/df006fd/ : SUCCESS in 46m 03s\n- gate-devstack-dsvm-cells http://logs.openstack.org/70/167370/10/gate/gate-devstack-dsvm-cells/b4c9516/ : SUCCESS in 21m 20s\n- gate-swift-dsvm-functional http://logs.openstack.org/70/167370/10/gate/gate-swift-dsvm-functional/48bc329/ : SUCCESS in 27m 00s","accounts_in_message":[],"_revision_number":10},{"id":"d4ff6a524fcc47c99377f96b15aa67bbf9c38e95","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-09 10:31:11.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":10}],"current_revision_number":10,"current_revision":"aa78d70df29c3927c032ef49079011fd9c937f73","revisions":{"4bc6ac5e680ee9d5e91b3673fba591501cd581b6":{"kind":"REWORK","_number":1,"created":"2015-03-24 18:59:08.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/1"}}},"commit":{"parents":[{"commit":"27ca273c049eb253fff603f15f54da78c0fb7abb","subject":"Merge \"refuse to post sample which is not supported\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/27ca273c049eb253fff603f15f54da78c0fb7abb"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-24 18:43:03.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-24 18:43:03.000000000","tz":60},"subject":"Adds support for default rule in ceilometer policy.json. Fixes Bug:1435855","message":"Adds support for default rule in ceilometer policy.json. Fixes Bug:1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/4bc6ac5e680ee9d5e91b3673fba591501cd581b6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/4bc6ac5e680ee9d5e91b3673fba591501cd581b6"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"27ca273c049eb253fff603f15f54da78c0fb7abb","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"2b4defc90cf2afefa2c1d1052f44109868f8c8fb":{"kind":"REWORK","_number":2,"created":"2015-03-27 13:35:45.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/2"}}},"commit":{"parents":[{"commit":"e14a4edc81c365a84289d72f34703eff03a37e8b","subject":"Merge \"drop deprecated novaclient.v1_1\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/e14a4edc81c365a84289d72f34703eff03a37e8b"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 13:34:18.000000000","tz":60},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2b4defc90cf2afefa2c1d1052f44109868f8c8fb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2b4defc90cf2afefa2c1d1052f44109868f8c8fb"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"e14a4edc81c365a84289d72f34703eff03a37e8b","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"62f528f2a23bb8baaaafd7b236c84c30e2c507ce":{"kind":"REWORK","_number":3,"created":"2015-03-28 01:20:30.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/3"}}},"commit":{"parents":[{"commit":"c8243a90c46ec321e596ded8d1b7b122f2b478ad","subject":"Merge \"Optimize the flow of getting pollster resources\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/c8243a90c46ec321e596ded8d1b7b122f2b478ad"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-28 01:20:22.000000000","tz":60},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/62f528f2a23bb8baaaafd7b236c84c30e2c507ce"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/62f528f2a23bb8baaaafd7b236c84c30e2c507ce"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"c8243a90c46ec321e596ded8d1b7b122f2b478ad","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"e65a25aa2b589da4272e8fa36b270a5999828787":{"kind":"REWORK","_number":4,"created":"2015-03-30 11:42:38.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/4"}}},"commit":{"parents":[{"commit":"c8243a90c46ec321e596ded8d1b7b122f2b478ad","subject":"Merge \"Optimize the flow of getting pollster resources\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/c8243a90c46ec321e596ded8d1b7b122f2b478ad"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-30 11:41:50.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/e65a25aa2b589da4272e8fa36b270a5999828787"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/e65a25aa2b589da4272e8fa36b270a5999828787"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"c8243a90c46ec321e596ded8d1b7b122f2b478ad","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"883c77691818f000d56383ea07c3473911d88bda":{"kind":"REWORK","_number":5,"created":"2015-03-30 11:50:40.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/5"}}},"commit":{"parents":[{"commit":"c8243a90c46ec321e596ded8d1b7b122f2b478ad","subject":"Merge \"Optimize the flow of getting pollster resources\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/c8243a90c46ec321e596ded8d1b7b122f2b478ad"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-30 11:49:56.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/883c77691818f000d56383ea07c3473911d88bda"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/883c77691818f000d56383ea07c3473911d88bda"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"c8243a90c46ec321e596ded8d1b7b122f2b478ad","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"5a5d515fa7b24549adcdbeb8ea3f921e80fb9fb4":{"kind":"REWORK","_number":6,"created":"2015-04-01 14:59:17.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/6"}}},"commit":{"parents":[{"commit":"65099e6aae75b7ef22625e2c3c40b0528f3f7b6d","subject":"Merge \"remove instance:\u003cflavor\u003e meter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/65099e6aae75b7ef22625e2c3c40b0528f3f7b6d"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-04-01 14:59:06.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/5a5d515fa7b24549adcdbeb8ea3f921e80fb9fb4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/5a5d515fa7b24549adcdbeb8ea3f921e80fb9fb4"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"65099e6aae75b7ef22625e2c3c40b0528f3f7b6d","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"c70a513bae16adfdfbc2959fc26ed994df91ffa0":{"kind":"REWORK","_number":7,"created":"2015-04-02 17:37:35.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/7"}}},"commit":{"parents":[{"commit":"8fc8dd176c796ec7792aa7c10e06958a80d4f6f1","subject":"Fix the event type for trusts","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/8fc8dd176c796ec7792aa7c10e06958a80d4f6f1"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-04-02 17:37:24.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/c70a513bae16adfdfbc2959fc26ed994df91ffa0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/c70a513bae16adfdfbc2959fc26ed994df91ffa0"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"8fc8dd176c796ec7792aa7c10e06958a80d4f6f1","is_merged_in_target_branch":true,"change_id":"I93175f7beaca3b7961d0431fd8c6ba819b3f164b","change_number":169929,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"aa75aaf42d077acec95910729e71b77d4caf7e50":{"kind":"REWORK","_number":8,"created":"2015-04-06 17:24:20.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/8"}}},"commit":{"parents":[{"commit":"2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2","subject":"Revert \"remove instance:\u003cflavor\u003e meter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-04-06 17:24:10.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/aa75aaf42d077acec95910729e71b77d4caf7e50"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/aa75aaf42d077acec95910729e71b77d4caf7e50"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2","is_merged_in_target_branch":true,"change_id":"I59938b9129b23d140a443cf055ea73b3568f00d4","change_number":170374,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"8e912653522a51eb80f749cfa217c779888dec27":{"kind":"REWORK","_number":9,"created":"2015-04-07 11:43:21.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/9"}}},"commit":{"parents":[{"commit":"2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2","subject":"Revert \"remove instance:\u003cflavor\u003e meter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-04-07 11:42:40.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/8e912653522a51eb80f749cfa217c779888dec27"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/8e912653522a51eb80f749cfa217c779888dec27"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2","is_merged_in_target_branch":true,"change_id":"I59938b9129b23d140a443cf055ea73b3568f00d4","change_number":170374,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"aa78d70df29c3927c032ef49079011fd9c937f73":{"kind":"REWORK","_number":10,"created":"2015-04-07 13:43:36.000000000","uploader":{"_account_id":13637,"name":"Divya K Konoor","email":"dikonoor@in.ibm.com","username":"dikonoor"},"ref":"refs/changes/70/167370/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/70/167370/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/70/167370/10"}}},"commit":{"parents":[{"commit":"2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2","subject":"Revert \"remove instance:\u003cflavor\u003e meter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2"}]}],"author":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-03-27 08:27:35.000000000","tz":60},"committer":{"name":"Divya","email":"dikonoor@in.ibm.com","date":"2015-04-07 13:43:06.000000000","tz":120},"subject":"Adds support for default rule in ceilometer policy.json.","message":"Adds support for default rule in ceilometer policy.json.\n\nThe default rule is broken in the current implementation of\nceilometer rbac, because ceilometer rbac.py does not leverage\nthe support provided by oslo_policy . It instead tries to\nloop through all the rules in the policy.json to check if the\nrule corresponding to the requested REST api matches with the\nany in the policy.json. In this process, it completely ignores\nthe existence of the default rule.\n\nCloses-Bug: 1435855\n\nChange-Id: Icab626b28d14514b0f024df447a8e7f35c52257c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/aa78d70df29c3927c032ef49079011fd9c937f73"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/aa78d70df29c3927c032ef49079011fd9c937f73"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"2fb046fb6682ceadc82acdbfd3d708ac23fe9fa2","is_merged_in_target_branch":true,"change_id":"I59938b9129b23d140a443cf055ea73b3568f00d4","change_number":170374,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}