)]}'
{"id":"openstack%2Fpython-ceilometerclient~230272","triplet_id":"openstack%2Fpython-ceilometerclient~master~I2d29ebbc983ecedc0467ad3527491dc4f6eeeb74","project":"openstack/python-ceilometerclient","branch":"master","topic":"bug/1501958","hashtags":[],"change_id":"I2d29ebbc983ecedc0467ad3527491dc4f6eeeb74","subject":"defaulting cli alarm-history output with N/A if data is missing","status":"ABANDONED","created":"2015-10-02 05:07:18.000000000","updated":"2016-04-23 04:43:07.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"e51e77aa65100e57625e260d1a783dfca64913c1","_number":230272,"virtual_id_number":230272,"owner":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2015-10-03 01:00:38.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"date":"2015-10-03 00:23:12.000000000","_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},{"date":"2015-10-05 14:57:12.000000000","_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true}],"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":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-10-02 05:07:29.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-10-02 09:40:31.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-10-03 01:00:38.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-10-05 14:57:12.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"}],"messages":[{"id":"4387024818f0ab20f20aad2030302d4a1f3232ea","author":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"date":"2015-10-02 05:07:18.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c435185b4585af6a393bbbc52dc8cc99193d0da4","author":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"date":"2015-10-02 05:07:29.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. Also it\n    takes generally at least a couple of weeks for cores to get around to\n    reviewing code. The more you participate in the community the more\n    rewarding it is for you. You may also notice that the more you get to know\n    people and get to be known, the faster your patches will be reviewed and\n    eventually approved. Get to know others and become known by doing code\n    reviews: anybody can do it, and it\u0027s a 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":"a01437d880621f314ab11c46a503b3dbe182aaed","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-02 05:39:41.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-ceilometerclient-pep8 http://logs.openstack.org/72/230272/1/check/gate-python-ceilometerclient-pep8/6a52ecc/ : SUCCESS in 50s\n- gate-python-ceilometerclient-docs http://docs-draft.openstack.org/72/230272/1/check/gate-python-ceilometerclient-docs/39650ef//doc/build/html/ : SUCCESS in 1m 04s\n- gate-python-ceilometerclient-python27 http://logs.openstack.org/72/230272/1/check/gate-python-ceilometerclient-python27/3a2a3b9/ : SUCCESS in 1m 06s\n- gate-python-ceilometerclient-python34 http://logs.openstack.org/72/230272/1/check/gate-python-ceilometerclient-python34/fdbbbaf/ : SUCCESS in 1m 36s\n- gate-python-ceilometerclient-pypy http://logs.openstack.org/72/230272/1/check/gate-python-ceilometerclient-pypy/50be605/ : SUCCESS in 1m 54s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient http://logs.openstack.org/72/230272/1/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient/515f8f0/ : FAILURE in 31m 25s\n- gate-ceilometerclient-dsvm-functional http://logs.openstack.org/72/230272/1/check/gate-ceilometerclient-dsvm-functional/46cd813/ : FAILURE in 32m 17s","accounts_in_message":[],"_revision_number":1},{"id":"5acc3e197b86cb56459871c7d4702804cb39d87e","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2015-10-02 09:40:31.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nI think there are more work to do, alarm-show, alarm-list should handle this issue as well, they are all severity backward compatibility issue.","accounts_in_message":[],"_revision_number":1},{"id":"7bb97268f0da0c3628e158d7ea3797c6818edb8a","author":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"date":"2015-10-02 21:14:57.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"15a47a1c96f1fc16658e32c7737cfb8e80566991","author":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"date":"2015-10-03 00:22:25.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"9ac5be624a3df55d168970351b9151deae17833e","author":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"date":"2015-10-03 00:23:12.000000000","message":"Patch Set 2:\n\nalarm_change_detail_formatter is used by do_query_alarm_history and do_alarm_history, which this fix addresses\n\nthe other calls eventually use getattr with a default of \u0027\u0027 (so you see the field name \"Severity\" but no value.\n\nI resubmitted with this behavior","accounts_in_message":[],"_revision_number":2},{"id":"1aadf7d4604c17a212dffca024d911a89cc9f3f9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-03 01:00:38.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-python-ceilometerclient-pep8 http://logs.openstack.org/72/230272/2/check/gate-python-ceilometerclient-pep8/d8a41f6/ : SUCCESS in 1m 06s\n- gate-python-ceilometerclient-docs http://docs-draft.openstack.org/72/230272/2/check/gate-python-ceilometerclient-docs/1466af4//doc/build/html/ : SUCCESS in 1m 07s\n- gate-python-ceilometerclient-python27 http://logs.openstack.org/72/230272/2/check/gate-python-ceilometerclient-python27/ca09115/ : FAILURE in 1m 06s\n- gate-python-ceilometerclient-python34 http://logs.openstack.org/72/230272/2/check/gate-python-ceilometerclient-python34/e87ce5b/ : FAILURE in 1m 20s\n- gate-python-ceilometerclient-pypy http://logs.openstack.org/72/230272/2/check/gate-python-ceilometerclient-pypy/aa8d4e2/ : FAILURE in 1m 25s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient http://logs.openstack.org/72/230272/2/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient/89a3874/ : FAILURE in 32m 36s\n- gate-ceilometerclient-dsvm-functional http://logs.openstack.org/72/230272/2/check/gate-ceilometerclient-dsvm-functional/4b69a9b/ : FAILURE in 21m 48s","accounts_in_message":[],"_revision_number":2},{"id":"2f67288df472d517d93633fec67ccfd59f8a610e","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2016-04-23 04:43:07.000000000","message":"Abandoned\n\nno active for over 2 months","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"597ce401143604eb05f75d3c82d83f6310c6efb1","revisions":{"f43a88b73688187bec373e80b69625ebab57e419":{"kind":"REWORK","_number":1,"created":"2015-10-02 05:07:18.000000000","uploader":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"ref":"refs/changes/72/230272/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-ceilometerclient","ref":"refs/changes/72/230272/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/1"}}},"commit":{"parents":[{"commit":"6d70e8efba26710534b5b0b003b2568295c6111a","subject":"Merge \"Add statistic in rule information\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/6d70e8efba26710534b5b0b003b2568295c6111a"}]}],"author":{"name":"Chris Plock","email":"plockc@gmail.com","date":"2015-10-02 05:06:51.000000000","tz":-420},"committer":{"name":"Chris Plock","email":"plockc@gmail.com","date":"2015-10-02 05:06:51.000000000","tz":-420},"subject":"defaulting cli alarm-history output with N/A if data is missing","message":"defaulting cli alarm-history output with N/A if data is missing\n\nThis is to resolve an introduced bug in juno (and resolves for icehouse)\nfrom an alarm-history client request, by providing a default output of\n\"N/A\" for severity which was not available prior to Kilo.\n\nChange-Id: I2d29ebbc983ecedc0467ad3527491dc4f6eeeb74\nCloses-Bug: #1501958\nRelated-Bug: #1468295\nRelated-Blueprint: ceilometer-alarm-level\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/f43a88b73688187bec373e80b69625ebab57e419"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/f43a88b73688187bec373e80b69625ebab57e419"}]},"branch":"refs/heads/master"},"597ce401143604eb05f75d3c82d83f6310c6efb1":{"kind":"REWORK","_number":2,"created":"2015-10-03 00:22:25.000000000","uploader":{"_account_id":18666,"name":"Chris Plock","email":"chrisplo@cisco.com","username":"plockc"},"ref":"refs/changes/72/230272/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-ceilometerclient","ref":"refs/changes/72/230272/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-ceilometerclient refs/changes/72/230272/2"}}},"commit":{"parents":[{"commit":"6d70e8efba26710534b5b0b003b2568295c6111a","subject":"Merge \"Add statistic in rule information\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/6d70e8efba26710534b5b0b003b2568295c6111a"}]}],"author":{"name":"Chris Plock","email":"plockc@gmail.com","date":"2015-10-02 05:06:51.000000000","tz":-420},"committer":{"name":"Chris Plock","email":"plockc@gmail.com","date":"2015-10-03 00:22:33.000000000","tz":-420},"subject":"defaulting cli alarm-history output with N/A if data is missing","message":"defaulting cli alarm-history output with N/A if data is missing\n\nThis is to resolve an introduced bug in juno (and resolves for icehouse)\nfrom an alarm-history client request, by providing a default output of\n\u0027\u0027 for severity, which was not available prior to Kilo.  This follows\nthe pattern used for displaying severity in other client commands such\nas alarm-list and alarm-show\n\nChange-Id: I2d29ebbc983ecedc0467ad3527491dc4f6eeeb74\nCloses-Bug: #1501958\nRelated-Bug: #1468295\nRelated-Blueprint: ceilometer-alarm-level\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/597ce401143604eb05f75d3c82d83f6310c6efb1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/597ce401143604eb05f75d3c82d83f6310c6efb1"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}