)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"49c56dc91fb88a8ca1fadc252453b20f3c831161","unresolved":false,"context_lines":[{"line_number":13,"context_line":"objects."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I62057e2bc1690566d2fa10b41a19fd541d6e42c3"},{"line_number":16,"context_line":"Fixes-Bug: #1470153"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba3cc151_b13695e5","line":16,"updated":"2015-06-30 16:54:43.000000000","message":"Should be Closes-Bug: so that gerrit does the right launchpad fu.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references","commit_id":"5fe19b760770a91050d1671c926865fdf345350a"}],"nova/tests/unit/objects/test_objects.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"906bc09507a58200496c372fe48597e37d0e51ce","unresolved":false,"context_lines":[{"line_number":1248,"context_line":""},{"line_number":1249,"context_line":"        obj_classes \u003d base.NovaObjectRegistry.obj_classes()"},{"line_number":1250,"context_line":"        for name, field in obj_class.fields.items():"},{"line_number":1251,"context_line":"            # Notes(yjiang5): ObjectListBase should be covered by"},{"line_number":1252,"context_line":"            # child_versions test"},{"line_number":1253,"context_line":"            if (issubclass(obj_class, base.ObjectListBase) and"},{"line_number":1254,"context_line":"                    name \u003d\u003d \u0027objects\u0027):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_7147ad4b","side":"PARENT","line":1251,"updated":"2015-06-30 16:55:12.000000000","message":"Can we get an explanation in the commit message about why we weren\u0027t covered by child_versions testing?","commit_id":"9cb24b31680833647a3bb8821e03674b44bda71b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"263d5ca0f7c2859065bf5f70a8d946ba5672d6b3","unresolved":false,"context_lines":[{"line_number":1300,"context_line":"                         \u0027Some objects have changed dependencies. \u0027"},{"line_number":1301,"context_line":"                         \u0027Please make sure to bump the versions of \u0027"},{"line_number":1302,"context_line":"                         \u0027parent objects and provide a rule in their \u0027"},{"line_number":1303,"context_line":"                         \u0027obj_make_compatible() routines to backlevel \u0027"},{"line_number":1304,"context_line":"                         \u0027the child object.\u0027)"},{"line_number":1305,"context_line":""},{"line_number":1306,"context_line":"    def test_obj_make_compatible(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba3cc151_0216dcf0","line":1303,"updated":"2015-07-01 15:20:12.000000000","message":"We need to update this error message to also say to make sure the versions in the object_relationships mapping in this test are updated for any object version changes, since I had a change which has all of the right dependency version updates but forgot to update that mapping now that this change is merged, so I got this:\n\nhttp://paste.openstack.org/show/332379/\n\nThe output on that error made me think that I needed to potentially update something in the fixedip or instance objects, which I don\u0027t for bdm changes.  The problem was I didn\u0027t update this mapping in object_relationships:\n\n\u0027BlockDeviceMappingList\u0027: {\u0027BlockDeviceMapping\u0027: \u00271.9\u0027},","commit_id":"eafe8ae066f14a6b3e562a5a25edb20e617fbee2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"50f911badff795ddaa844ec92ed0f0c7fc1ee41b","unresolved":false,"context_lines":[{"line_number":1300,"context_line":"                         \u0027Some objects have changed dependencies. \u0027"},{"line_number":1301,"context_line":"                         \u0027Please make sure to bump the versions of \u0027"},{"line_number":1302,"context_line":"                         \u0027parent objects and provide a rule in their \u0027"},{"line_number":1303,"context_line":"                         \u0027obj_make_compatible() routines to backlevel \u0027"},{"line_number":1304,"context_line":"                         \u0027the child object.\u0027)"},{"line_number":1305,"context_line":""},{"line_number":1306,"context_line":"    def test_obj_make_compatible(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba3cc151_78ef33b1","line":1303,"in_reply_to":"ba3cc151_0216dcf0","updated":"2015-07-08 21:16:29.000000000","message":"Yar: https://review.openstack.org/#/c/197684/","commit_id":"eafe8ae066f14a6b3e562a5a25edb20e617fbee2"}]}
