)]}'
{"senlin/policies/lb_policy.py":[{"author":{"_account_id":27224,"name":"Duc Truong","email":"duc.openstack@gmail.com","username":"dtruong2"},"change_message_id":"35b62bde645389dbb6f9a5bf9e5b8a2712ab99e6","unresolved":false,"context_lines":[{"line_number":466,"context_line":"            else:  # action.action \u003d\u003d consts.CLUSTER_SCALE_IN"},{"line_number":467,"context_line":"                count \u003d 1"},{"line_number":468,"context_line":"        elif action.action \u003d\u003d consts.CLUSTER_REPLACE_NODES:"},{"line_number":469,"context_line":"            candidates \u003d list(action.inputs[\u0027candidates\u0027].keys())"},{"line_number":470,"context_line":"            count \u003d len(candidates)"},{"line_number":471,"context_line":"        else:"},{"line_number":472,"context_line":"            count \u003d deletion.get(\u0027count\u0027, 0)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfbec78f_4159b132","line":469,"range":{"start_line":469,"start_character":25,"end_line":469,"end_character":29},"updated":"2019-05-13 22:32:06.000000000","message":"nit: Do you actually need to explicitly convert to a list?  Doesn\u0027t keys() already return a list?","commit_id":"51f2d9b060960d986c87aeb110498a45291ed985"},{"author":{"_account_id":27224,"name":"Duc Truong","email":"duc.openstack@gmail.com","username":"dtruong2"},"change_message_id":"35b62bde645389dbb6f9a5bf9e5b8a2712ab99e6","unresolved":false,"context_lines":[{"line_number":559,"context_line":"                action.action \u003d\u003d consts.NODE_RECOVER):"},{"line_number":560,"context_line":"            candidates \u003d [action.entity.id]"},{"line_number":561,"context_line":"        elif action.action \u003d\u003d consts.CLUSTER_REPLACE_NODES:"},{"line_number":562,"context_line":"            candidates \u003d list(action.inputs[\u0027candidates\u0027].values())"},{"line_number":563,"context_line":"        else:"},{"line_number":564,"context_line":"            creation \u003d action.data.get(\u0027creation\u0027, None)"},{"line_number":565,"context_line":"            candidates \u003d creation.get(\u0027nodes\u0027, []) if creation else []"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfbec78f_c144c149","line":562,"range":{"start_line":562,"start_character":25,"end_line":562,"end_character":29},"updated":"2019-05-13 22:32:06.000000000","message":"Same question as above.  Is this conversion necessary?","commit_id":"51f2d9b060960d986c87aeb110498a45291ed985"}]}