)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"f421cdbcee0cc2f4120e6da4a6da0776bf245f26","unresolved":false,"context_lines":[{"line_number":9,"context_line":"this ports the instance_usage_audit_log tests and corresponding"},{"line_number":10,"context_line":"client into v3. There is no the url:"},{"line_number":11,"context_line":"GET /os-instance-usage-audit-log/:(id)."},{"line_number":12,"context_line":"Some related tests are removed in the v3 tests."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Partially implements blueprint nova-v3-api-tests"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F6FI%3D","line":12,"updated":"2013-12-27 11:49:30.000000000","message":"So GET /os-instance-usage-audit-log/:(id) was replaced in V3 by adding a query \u0027before\u0027 parameter instead. Perhaps the tests which you removed could be altered to use that form of query instead rather than deleting them?\n\nSee commit 78e769d26258f27c4875997ed611e6f06bfce4be in Nova.","commit_id":"bc02d6030294b761abf7393d08270c7c7e1cb98f"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"747b1e690b81e6932ee768c79acbd71b5e193c0a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"this ports the instance_usage_audit_log tests and corresponding"},{"line_number":10,"context_line":"client into v3. There is no the url:"},{"line_number":11,"context_line":"GET /os-instance-usage-audit-log/:(id)."},{"line_number":12,"context_line":"Some related tests are removed in the v3 tests."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Partially implements blueprint nova-v3-api-tests"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F55Y%3D","line":12,"in_reply_to":"AAAAT3%2F%2F6FI%3D","updated":"2013-12-27 12:53:51.000000000","message":"thanks for your reminder. will add it.","commit_id":"bc02d6030294b761abf7393d08270c7c7e1cb98f"}],"tempest/api/compute/v3/admin/test_instance_usage_audit_log.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"0a44edd847f467a89bd8c18f542b6ad3d7624953","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                          \u0027period_beginning\u0027, \u0027num_hosts_not_run\u0027]"},{"line_number":59,"context_line":"        for item in expected_items:"},{"line_number":60,"context_line":"            self.assertIn(item, body)"},{"line_number":61,"context_line":"        self.assertEqual(body[\u0027period_ending\u0027], \"2012-12-23 23:00:00\")"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"class InstanceUsageAuditLogV3TestXML(InstanceUsageAuditLogV3TestJSON):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2F8b0%3D","line":61,"updated":"2014-01-06 12:45:41.000000000","message":"Why it is not 2012-12-24 00:00:00 ?","commit_id":"56156aec7054d8055d84c0f6c0bad011b8438c85"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"2f1522f6c03b56cf41ce008a5785bfad771f3855","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                          \u0027period_beginning\u0027, \u0027num_hosts_not_run\u0027]"},{"line_number":59,"context_line":"        for item in expected_items:"},{"line_number":60,"context_line":"            self.assertIn(item, body)"},{"line_number":61,"context_line":"        self.assertEqual(body[\u0027period_ending\u0027], \"2012-12-23 23:00:00\")"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"class InstanceUsageAuditLogV3TestXML(InstanceUsageAuditLogV3TestJSON):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2FScc%3D","line":61,"in_reply_to":"AAAAUH%2F%2F0PA%3D","updated":"2014-01-15 13:03:06.000000000","message":"I think its a Nova bug. Unfortunately we may need to preserve the V2 API behaviour because its likely that any clients programs that use this interface rely on this bug now. \n\nIvan - could you bring this up on the mailing list (re nova behavior)? For now I\u0027m fine with the tempest test going through as-is as long as afazekas is ok with it (so I\u0027ll +2 but not approve for now). Then we\u0027ll update the test when get a consensus that its ok to change the behavior for V3 (and maybe for V2).","commit_id":"56156aec7054d8055d84c0f6c0bad011b8438c85"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"37f9af737251314eabaa081c70348ae24405d5ea","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                          \u0027period_beginning\u0027, \u0027num_hosts_not_run\u0027]"},{"line_number":59,"context_line":"        for item in expected_items:"},{"line_number":60,"context_line":"            self.assertIn(item, body)"},{"line_number":61,"context_line":"        self.assertEqual(body[\u0027period_ending\u0027], \"2012-12-23 23:00:00\")"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"class InstanceUsageAuditLogV3TestXML(InstanceUsageAuditLogV3TestJSON):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAUH%2F%2F0PA%3D","line":61,"in_reply_to":"AAAAUH%2F%2F8b0%3D","updated":"2014-01-08 05:50:52.000000000","message":"Good question.\nBecause it\u0027s end \u003e\u003d rightnow, not end\u003erightnow \nin https://github.com/openstack/nova/blob/master/nova/utils.py#L293\n\nI\u0027m not sure whether  it\u0027s a bug or not.","commit_id":"56156aec7054d8055d84c0f6c0bad011b8438c85"}]}