)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"dee6684f1af69b32831dc3d8e5648c8da0445b26","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ghanshyam \u003cghanshyam.mann@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-03-10 16:50:43 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the Bug # 1290221"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch correct the Nova V3 test_image_oneserver according to new image_client."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAVn%2F%2FDIk%3D","line":7,"updated":"2014-03-10 12:50:15.000000000","message":"A commit message subject should have the summary. But I think this doesn\u0027t have it.\n\nI think the line.9 has the summary. So how about\nlike this \"Fix test_image_oneserver according to new image_client\" ?\n\nAnd please see\nhttps://wiki.openstack.org/wiki/GitCommitMessages\n\nand add \n\n Closes-Bug: #1290221\n\nto the commit log.","commit_id":"275218e063ea7446ecb07b068f3ce6df6dc2b50f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"361d8746a0ebfcc82d99053d8a5e6512b3b6384a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ghanshyam \u003cghanshyam.mann@nectechnologies.in\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-03-10 16:50:43 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the Bug # 1290221"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch correct the Nova V3 test_image_oneserver according to new image_client."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAVn%2F%2B%2BLQ%3D","line":7,"in_reply_to":"AAAAVn%2F%2FDIk%3D","updated":"2014-03-10 23:32:53.000000000","message":"My bad. I will correct it. Thanks.","commit_id":"275218e063ea7446ecb07b068f3ce6df6dc2b50f"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"dee6684f1af69b32831dc3d8e5648c8da0445b26","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the Bug # 1290221"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch correct the Nova V3 test_image_oneserver according to new image_client."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Iadaf311e37a42b2adaa9d980a10ae251450030d0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAVn%2F%2FDGU%3D","line":9,"updated":"2014-03-10 12:50:15.000000000","message":"Commit message lines except the first line should be wrapped at 72 characters. \nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_GIT_commit_message_structure","commit_id":"275218e063ea7446ecb07b068f3ce6df6dc2b50f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"361d8746a0ebfcc82d99053d8a5e6512b3b6384a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the Bug # 1290221"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch correct the Nova V3 test_image_oneserver according to new image_client."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Iadaf311e37a42b2adaa9d980a10ae251450030d0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAVn%2F%2B%2BLM%3D","line":9,"in_reply_to":"AAAAVn%2F%2FDGU%3D","updated":"2014-03-10 23:32:53.000000000","message":"My bad. I will correct it. Thanks.","commit_id":"275218e063ea7446ecb07b068f3ce6df6dc2b50f"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0f0b2f1bea179d02fa35d324a02dc147972a764b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix V3 image tests according to new image_client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Bug # 1290221"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This patch correct the Nova V3 test_image_oneserver"},{"line_number":12,"context_line":"and test_images_negative according to new image_client."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAVn%2F%2B9WQ%3D","line":9,"updated":"2014-03-11 02:11:33.000000000","message":"Oh, I think this line is not needed because there is a same line following.","commit_id":"b02d800530bfc315dc726faebe539d422d855616"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"22234956a1a288f7e1452e8ed6a8ac68a57bf117","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix V3 image tests according to new image_client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Bug # 1290221"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This patch correct the Nova V3 test_image_oneserver"},{"line_number":12,"context_line":"and test_images_negative according to new image_client."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAVn%2F%2B9Uw%3D","line":9,"in_reply_to":"AAAAVn%2F%2B9WQ%3D","updated":"2014-03-11 02:15:19.000000000","message":"Yes.","commit_id":"b02d800530bfc315dc726faebe539d422d855616"}],"tempest/api/compute/v3/images/test_images_negative.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"dee6684f1af69b32831dc3d8e5648c8da0445b26","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        resp, body \u003d self.servers_client.create_image(server_id, name, meta)"},{"line_number":36,"context_line":"        image_id \u003d data_utils.parse_image_id(resp[\u0027location\u0027])"},{"line_number":37,"context_line":"        self.addCleanup(self.client.delete_image, image_id)"},{"line_number":38,"context_line":"        self.client.wait_for_image_status(image_id, \u0027active\u0027)"},{"line_number":39,"context_line":"        return resp, body"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2FDDc%3D","line":38,"updated":"2014-03-10 12:50:15.000000000","message":"It\u0027s unclear to me here. I don\u0027t understand why this test is succeeded in the gate now? For example\ntest_create_image_from_deleted_server() and test_create_image_from_nonexistent_server() are passing the test in http://logs.openstack.org/26/78526/2/check/check-tempest-dsvm-full/ef874f9/logs/testr_results.html.gz .","commit_id":"275218e063ea7446ecb07b068f3ce6df6dc2b50f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"361d8746a0ebfcc82d99053d8a5e6512b3b6384a","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        resp, body \u003d self.servers_client.create_image(server_id, name, meta)"},{"line_number":36,"context_line":"        image_id \u003d data_utils.parse_image_id(resp[\u0027location\u0027])"},{"line_number":37,"context_line":"        self.addCleanup(self.client.delete_image, image_id)"},{"line_number":38,"context_line":"        self.client.wait_for_image_status(image_id, \u0027active\u0027)"},{"line_number":39,"context_line":"        return resp, body"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @test.attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2B%2BLU%3D","line":38,"in_reply_to":"AAAAVn%2F%2FDDc%3D","updated":"2014-03-10 23:32:53.000000000","message":"Yes, Those will pass because when they call __create_image__() with deleted and non existent server, it will trow the exception of not_found from very first line (create_image function). So it never reach to wait_for_image_status() line. if from any of the test it comes to this line then it would have been failed.","commit_id":"275218e063ea7446ecb07b068f3ce6df6dc2b50f"}]}