)]}'
{"nova/tests/unit/compute/test_compute_api.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"b3b4e987733c8ef1068766a912dacbb0bdee24d6","unresolved":false,"context_lines":[{"line_number":51,"context_line":"from nova.tests.unit.objects import test_flavor"},{"line_number":52,"context_line":"from nova.tests.unit.objects import test_migration"},{"line_number":53,"context_line":"from nova.tests.unit.objects import test_service"},{"line_number":54,"context_line":"from nova.tests import uuidsentinel"},{"line_number":55,"context_line":"from nova import utils"},{"line_number":56,"context_line":"from nova.volume import cinder"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_b4a6e4a8","line":54,"updated":"2015-12-14 15:52:12.000000000","message":"from nova.tests import uuidsentinel as uuids \n\nwould make this more readable later.","commit_id":"ee11a379bb2e9ce75b7e0189c73734e9c0de427a"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"b3b4e987733c8ef1068766a912dacbb0bdee24d6","unresolved":false,"context_lines":[{"line_number":1739,"context_line":"                    \u0027launched_at\u0027: timeutils.utcnow(),"},{"line_number":1740,"context_line":"                    \u0027locked\u0027: False,"},{"line_number":1741,"context_line":"                    \u0027availability_zone\u0027: \u0027fake_az\u0027,"},{"line_number":1742,"context_line":"                    \u0027uuid\u0027: uuidsentinel.fake})"},{"line_number":1743,"context_line":"        volumes \u003d {}"},{"line_number":1744,"context_line":"        old_volume_id \u003d uuidutils.generate_uuid()"},{"line_number":1745,"context_line":"        volumes[old_volume_id] \u003d {\u0027id\u0027: old_volume_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_94d96824","line":1742,"updated":"2015-12-14 15:52:12.000000000","message":"so, I really don\u0027t like calling this uuidsentinel.fake, can we change it to uuidsentinel.instance or something. Part of the point of the sentinels is to make them semantically meaningful for review and debug, and stop using \"fake\" as a name.","commit_id":"ee11a379bb2e9ce75b7e0189c73734e9c0de427a"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"317b8f754f52aa390c2d9abe0634cd365454d3da","unresolved":false,"context_lines":[{"line_number":1752,"context_line":"        volumes[new_volume_id] \u003d {\u0027id\u0027: new_volume_id,"},{"line_number":1753,"context_line":"                                  \u0027display_name\u0027: \u0027new_volume\u0027,"},{"line_number":1754,"context_line":"                                  \u0027attach_status\u0027: \u0027detached\u0027,"},{"line_number":1755,"context_line":"                                  \u0027instance_uuid\u0027: uuidsentinel.none,"},{"line_number":1756,"context_line":"                                  \u0027size\u0027: 5,"},{"line_number":1757,"context_line":"                                  \u0027status\u0027: \u0027available\u0027}"},{"line_number":1758,"context_line":"        self.assertRaises(exception.InstanceInvalidState,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_8904f117","line":1755,"updated":"2015-12-14 15:43:57.000000000","message":"I assume instance_uuid can actually be None on a volume, so I\u0027m not sure this change is good (this changes it from None to a real uuid). Is this getting set on an actual UUID field somewhere?","commit_id":"ee11a379bb2e9ce75b7e0189c73734e9c0de427a"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"b3b4e987733c8ef1068766a912dacbb0bdee24d6","unresolved":false,"context_lines":[{"line_number":2559,"context_line":""},{"line_number":2560,"context_line":"    def test_external_instance_event(self):"},{"line_number":2561,"context_line":"        instances \u003d ["},{"line_number":2562,"context_line":"            objects.Instance(uuid\u003duuidsentinel.uuid1, host\u003d\u0027host1\u0027),"},{"line_number":2563,"context_line":"            objects.Instance(uuid\u003duuidsentinel.uuid2, host\u003d\u0027host1\u0027),"},{"line_number":2564,"context_line":"            objects.Instance(uuid\u003duuidsentinel.uuid3, host\u003d\u0027host2\u0027),"},{"line_number":2565,"context_line":"            ]"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_74307c56","line":2562,"updated":"2015-12-14 15:52:12.000000000","message":"can we use inst1 -\u003e inst3 instead of uuid1 -\u003e 3","commit_id":"ee11a379bb2e9ce75b7e0189c73734e9c0de427a"}]}