)]}'
{"ceilometerclient/tests/v2/test_shell.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"939d90ebe18e356014720f6d3c91ae8f1fd92eef","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        if repeat_actions is not None:"},{"line_number":178,"context_line":"            self.args.repeat_actions \u003d repeat_actions"},{"line_number":179,"context_line":"        orig \u003d sys.stdout"},{"line_number":180,"context_line":"        sys.stdout \u003d six.StringIO()"},{"line_number":181,"context_line":"        alarm \u003d [alarms.Alarm(mock.Mock(), self.ALARM)]"},{"line_number":182,"context_line":"        self.cc.alarms.get.return_value \u003d alarm"},{"line_number":183,"context_line":"        self.cc.alarms.update.return_value \u003d alarm[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BuCA%3D","line":180,"updated":"2013-11-21 05:03:06.000000000","message":"i\u0027m probably missing this somewhere, but it doesn\u0027t look like sys stuff is used anywhere.","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"bb406fa4a4c4796e732d1faa48c760be3ea228bb","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        if repeat_actions is not None:"},{"line_number":178,"context_line":"            self.args.repeat_actions \u003d repeat_actions"},{"line_number":179,"context_line":"        orig \u003d sys.stdout"},{"line_number":180,"context_line":"        sys.stdout \u003d six.StringIO()"},{"line_number":181,"context_line":"        alarm \u003d [alarms.Alarm(mock.Mock(), self.ALARM)]"},{"line_number":182,"context_line":"        self.cc.alarms.get.return_value \u003d alarm"},{"line_number":183,"context_line":"        self.cc.alarms.update.return_value \u003d alarm[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BsRs%3D","line":180,"in_reply_to":"AAAATn%2F%2BuCA%3D","updated":"2013-11-21 13:53:16.000000000","message":"You\u0027re right that the stream isn\u0027t used, I just wanted to consume the output that\u0027s not relevant to the test assertions.","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"}],"ceilometerclient/v2/shell.py":[{"author":{"_account_id":5313,"name":"David Peraza","email":"david_peraza@persistentsys.com","username":"dperaza"},"change_message_id":"3a77055d148b67af7978fcba1a0f84e46f85c751","unresolved":false,"context_lines":[{"line_number":295,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":296,"context_line":"           default\u003dFalse,"},{"line_number":297,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":298,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":299,"context_line":"def do_alarm_create(cc, args\u003d{}):"},{"line_number":300,"context_line":"    \u0027\u0027\u0027Create a new alarm (Deprecated).\u0027\u0027\u0027"},{"line_number":301,"context_line":"    fields \u003d dict(filter(lambda x: not (x[1] is None), vars(args).items()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BvJ8%3D","line":298,"updated":"2013-11-20 21:52:30.000000000","message":"should you specify that False is the default? Same below?","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":5313,"name":"David Peraza","email":"david_peraza@persistentsys.com","username":"dperaza"},"change_message_id":"569bf63e820d3116694782277c2522be76ec3276","unresolved":false,"context_lines":[{"line_number":295,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":296,"context_line":"           default\u003dFalse,"},{"line_number":297,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":298,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":299,"context_line":"def do_alarm_create(cc, args\u003d{}):"},{"line_number":300,"context_line":"    \u0027\u0027\u0027Create a new alarm (Deprecated).\u0027\u0027\u0027"},{"line_number":301,"context_line":"    fields \u003d dict(filter(lambda x: not (x[1] is None), vars(args).items()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Br7k%3D","line":298,"in_reply_to":"AAAATn%2F%2BuEM%3D","updated":"2013-11-21 15:01:32.000000000","message":"cool","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"bb406fa4a4c4796e732d1faa48c760be3ea228bb","unresolved":false,"context_lines":[{"line_number":295,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":296,"context_line":"           default\u003dFalse,"},{"line_number":297,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":298,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":299,"context_line":"def do_alarm_create(cc, args\u003d{}):"},{"line_number":300,"context_line":"    \u0027\u0027\u0027Create a new alarm (Deprecated).\u0027\u0027\u0027"},{"line_number":301,"context_line":"    fields \u003d dict(filter(lambda x: not (x[1] is None), vars(args).items()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BsRY%3D","line":298,"in_reply_to":"AAAATn%2F%2BvD4%3D","updated":"2013-11-21 13:53:16.000000000","message":"What Gordon said.","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"939d90ebe18e356014720f6d3c91ae8f1fd92eef","unresolved":false,"context_lines":[{"line_number":295,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":296,"context_line":"           default\u003dFalse,"},{"line_number":297,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":298,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":299,"context_line":"def do_alarm_create(cc, args\u003d{}):"},{"line_number":300,"context_line":"    \u0027\u0027\u0027Create a new alarm (Deprecated).\u0027\u0027\u0027"},{"line_number":301,"context_line":"    fields \u003d dict(filter(lambda x: not (x[1] is None), vars(args).items()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BuEM%3D","line":298,"in_reply_to":"AAAATn%2F%2BvD4%3D","updated":"2013-11-21 05:03:06.000000000","message":"dperaza, if you run \u0027ceilometer help \u003ccommand\u003e\u0027 it will show you the default value along with the help text.","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":5313,"name":"David Peraza","email":"david_peraza@persistentsys.com","username":"dperaza"},"change_message_id":"265042d56a7e9b9f30183965fffc8431e71c3625","unresolved":false,"context_lines":[{"line_number":295,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":296,"context_line":"           default\u003dFalse,"},{"line_number":297,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":298,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":299,"context_line":"def do_alarm_create(cc, args\u003d{}):"},{"line_number":300,"context_line":"    \u0027\u0027\u0027Create a new alarm (Deprecated).\u0027\u0027\u0027"},{"line_number":301,"context_line":"    fields \u003d dict(filter(lambda x: not (x[1] is None), vars(args).items()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BvD4%3D","line":298,"in_reply_to":"AAAATn%2F%2BvG4%3D","updated":"2013-11-20 22:29:02.000000000","message":"I mean in the Help text","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"1ff541e49d0fcf0126907c4508fec6112db1e9fd","unresolved":false,"context_lines":[{"line_number":295,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":296,"context_line":"           default\u003dFalse,"},{"line_number":297,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":298,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":299,"context_line":"def do_alarm_create(cc, args\u003d{}):"},{"line_number":300,"context_line":"    \u0027\u0027\u0027Create a new alarm (Deprecated).\u0027\u0027\u0027"},{"line_number":301,"context_line":"    fields \u003d dict(filter(lambda x: not (x[1] is None), vars(args).items()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BvG4%3D","line":298,"in_reply_to":"AAAATn%2F%2BvJ8%3D","updated":"2013-11-20 22:17:16.000000000","message":"The original problem was that false was specified as the default for *both* create and update.\n\nI\u0027m keeping false as the default for create, but need to have *no default* for update, as the current value of this attribute is retrieved first and a specific default would have the effect of over-writing that (which is the bug I\u0027m addressing with this patch).","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"939d90ebe18e356014720f6d3c91ae8f1fd92eef","unresolved":false,"context_lines":[{"line_number":458,"context_line":"               ALARM_COMBINATION_OPERATORS))"},{"line_number":459,"context_line":"@utils.arg(\u0027--repeat-actions\u0027, dest\u003d\u0027repeat_actions\u0027,"},{"line_number":460,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":461,"context_line":"           default\u003dFalse,"},{"line_number":462,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":463,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":464,"context_line":"def do_alarm_combination_update(cc, args\u003d{}):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BuDg%3D","line":461,"updated":"2013-11-21 05:03:06.000000000","message":"should this default be here?","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"bb406fa4a4c4796e732d1faa48c760be3ea228bb","unresolved":false,"context_lines":[{"line_number":458,"context_line":"               ALARM_COMBINATION_OPERATORS))"},{"line_number":459,"context_line":"@utils.arg(\u0027--repeat-actions\u0027, dest\u003d\u0027repeat_actions\u0027,"},{"line_number":460,"context_line":"           metavar\u003d\u0027{True|False}\u0027, type\u003dutils.string_to_bool,"},{"line_number":461,"context_line":"           default\u003dFalse,"},{"line_number":462,"context_line":"           help\u003d(\u0027True if actions should be repeatedly notified \u0027"},{"line_number":463,"context_line":"                \u0027while alarm remains in target state\u0027))"},{"line_number":464,"context_line":"def do_alarm_combination_update(cc, args\u003d{}):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BsRQ%3D","line":461,"in_reply_to":"AAAATn%2F%2BuDg%3D","updated":"2013-11-21 13:53:16.000000000","message":"Nope, good spot!","commit_id":"fd06346a434b80bd1b5236f65d5cc821253475c1"}]}