)]}'
{"nova/tests/db/test_db_api.py":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"8fc080d9a9384bde2d0251976f19d622e5e9669f","unresolved":false,"context_lines":[{"line_number":4228,"context_line":"        self.assertRaises(exception.AgentBuildNotFound,"},{"line_number":4229,"context_line":"            db.agent_build_update, self.ctxt, agent_build.id, {\u0027os\u0027: \u0027OS/2\u0027})"},{"line_number":4230,"context_line":""},{"line_number":4231,"context_line":"    def test_agebt_build_exists(self):"},{"line_number":4232,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4233,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4234,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2FQrQ%3D","line":4231,"updated":"2013-07-05 19:33:42.000000000","message":"typo here ... s/agebt/agent/","commit_id":"29f74d989db866bf5e53cd8a96cc362ec37afd64"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"ba7bfbb25fc9b03e587decf5dd9067f0809c26db","unresolved":false,"context_lines":[{"line_number":4228,"context_line":"        self.assertRaises(exception.AgentBuildNotFound,"},{"line_number":4229,"context_line":"            db.agent_build_update, self.ctxt, agent_build.id, {\u0027os\u0027: \u0027OS/2\u0027})"},{"line_number":4230,"context_line":""},{"line_number":4231,"context_line":"    def test_agebt_build_exists(self):"},{"line_number":4232,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4233,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4234,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAP3%2F%2F%2Bl0%3D","line":4231,"in_reply_to":"AAAAPn%2F%2FQrQ%3D","updated":"2013-07-08 07:49:18.000000000","message":"Done","commit_id":"29f74d989db866bf5e53cd8a96cc362ec37afd64"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"91f599f34b5972b5873ca0200a2c1183fd36325f","unresolved":false,"context_lines":[{"line_number":4311,"context_line":"        self.assertRaises(exception.AgentBuildNotFound,"},{"line_number":4312,"context_line":"            db.agent_build_update, self.ctxt, agent_build.id, {\u0027os\u0027: \u0027OS/2\u0027})"},{"line_number":4313,"context_line":""},{"line_number":4314,"context_line":"    def test_agent_build_exists(self):"},{"line_number":4315,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4316,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4317,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2F6RY%3D","line":4314,"updated":"2013-07-09 11:24:02.000000000","message":"this is actually a race waiting to happen. This only will fail if this runs in the same testr process as L4283, and after it.\n\nThis test case needs issolation and to not depend on any other test functions.","commit_id":"a61536b033e2da1b231ca105e9d07902b2d281b7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"7e42cb3e8b1dd36d170a0d72dfaeef2cec43ad6e","unresolved":false,"context_lines":[{"line_number":4311,"context_line":"        self.assertRaises(exception.AgentBuildNotFound,"},{"line_number":4312,"context_line":"            db.agent_build_update, self.ctxt, agent_build.id, {\u0027os\u0027: \u0027OS/2\u0027})"},{"line_number":4313,"context_line":""},{"line_number":4314,"context_line":"    def test_agent_build_exists(self):"},{"line_number":4315,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4316,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4317,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2F5yY%3D","line":4314,"in_reply_to":"AAAAP3%2F%2F59Q%3D","updated":"2013-07-09 14:21:09.000000000","message":"oh, sorry, I see now. The exception is on the second build. My bad.","commit_id":"a61536b033e2da1b231ca105e9d07902b2d281b7"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"028dde81aea28a71b19e76e473c5cdb7c74ffc23","unresolved":false,"context_lines":[{"line_number":4311,"context_line":"        self.assertRaises(exception.AgentBuildNotFound,"},{"line_number":4312,"context_line":"            db.agent_build_update, self.ctxt, agent_build.id, {\u0027os\u0027: \u0027OS/2\u0027})"},{"line_number":4313,"context_line":""},{"line_number":4314,"context_line":"    def test_agent_build_exists(self):"},{"line_number":4315,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4316,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4317,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2F59Q%3D","line":4314,"in_reply_to":"AAAAP3%2F%2F6RY%3D","updated":"2013-07-09 13:34:33.000000000","message":"Now DB API tests run on SQLite in-memory DB, setUp() create connection for each test, database exists only within the scope of that connection, race impossible.\n\nThere is a patch https://review.openstack.org/#/c/33236/ for run DB API tests on another backends. With this patch tests also run concurrently without any data races\n(see commit message).","commit_id":"a61536b033e2da1b231ca105e9d07902b2d281b7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"7e42cb3e8b1dd36d170a0d72dfaeef2cec43ad6e","unresolved":false,"context_lines":[{"line_number":4314,"context_line":"    def test_agent_build_exists(self):"},{"line_number":4315,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4316,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4317,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"},{"line_number":4318,"context_line":"        self.assertRaises(exception.AgentBuildExists, db.agent_build_create,"},{"line_number":4319,"context_line":"                          self.ctxt, values)"},{"line_number":4320,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2F5yM%3D","line":4317,"updated":"2013-07-09 14:21:09.000000000","message":"nit: can we get rid of agent_build here, as it\u0027s an unused variable?","commit_id":"a61536b033e2da1b231ca105e9d07902b2d281b7"},{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"8cda081d899f56dbc0298c425429184ddff91dea","unresolved":false,"context_lines":[{"line_number":4314,"context_line":"    def test_agent_build_exists(self):"},{"line_number":4315,"context_line":"        values \u003d {\u0027hypervisor\u0027: \u0027kvm\u0027, \u0027os\u0027: \u0027FreeBSD\u0027,"},{"line_number":4316,"context_line":"                  \u0027architecture\u0027: \u0027x86_64\u0027}"},{"line_number":4317,"context_line":"        agent_build \u003d db.agent_build_create(self.ctxt, values)"},{"line_number":4318,"context_line":"        self.assertRaises(exception.AgentBuildExists, db.agent_build_create,"},{"line_number":4319,"context_line":"                          self.ctxt, values)"},{"line_number":4320,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2F5tM%3D","line":4317,"in_reply_to":"AAAAP3%2F%2F5yM%3D","updated":"2013-07-09 14:42:22.000000000","message":"Done","commit_id":"a61536b033e2da1b231ca105e9d07902b2d281b7"}]}