)]}'
{"cinder/volume/drivers/netapp/dataontap/ssc_cmode.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b6c47ffaddc33c72d3694da9800dee194bc6e50b","unresolved":false,"context_lines":[{"line_number":173,"context_line":"    vols \u003d set()"},{"line_number":174,"context_line":"    for res in result:"},{"line_number":175,"context_line":"        records \u003d res.get_child_content(\u0027num-records\u0027)"},{"line_number":176,"context_line":"        if int(records) \u003e 0:"},{"line_number":177,"context_line":"            attr_list \u003d res.get_child_by_name(\u0027attributes-list\u0027)"},{"line_number":178,"context_line":"            if attr_list:"},{"line_number":179,"context_line":"                vol_attrs \u003d attr_list.get_children()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_d854aecb","line":176,"updated":"2015-10-02 15:46:30.000000000","message":"The reviews variable contains a string (\u00274\u0027 in the unit test).\n\nOn Python 2, str \u003e int is always True:\n\n\u003e\u003e\u003e \u00270\u0027 \u003e 0\nTrue\n\u003e\u003e\u003e \u00271\u0027 \u003e 0\nTrue\n\u003e\u003e\u003e \u00272\u0027 \u003e 0\nTrue\n\nInternally, Python compares the *type* names... \u0027str\u0027 \u003e \u0027int\u0027.\n\nIt looks like an obvious bug in the code, or maybe the test mock incorrectly the object?\n\nI\u0027m not sure that my change is correct :-/","commit_id":"b1eca7f735fde5203694d84f4dc8958b846903d9"}]}