)]}'
{"nova/api/openstack/common.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"89cfc491ce383d344b27b5fa57fa4b47d8525561","unresolved":false,"context_lines":[{"line_number":364,"context_line":"                \"%(state)s\") % {\u0027action\u0027: action, \u0027attr\u0027: attr, \u0027state\u0027: state}"},{"line_number":365,"context_line":"    elif not_launched:"},{"line_number":366,"context_line":"        msg \u003d _(\"Cannot \u0027%(action)s\u0027 an instance which has never been \""},{"line_number":367,"context_line":"                \"active\") % {\u0027action\u0027: action}"},{"line_number":368,"context_line":"    else:"},{"line_number":369,"context_line":"        # At least give some meaningful message"},{"line_number":370,"context_line":"        msg \u003d _(\"Instance is in an invalid state for \u0027%s\u0027\") % action"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPX%2F%2FzhM%3D","line":367,"updated":"2013-06-10 08:41:05.000000000","message":"do you need a dictionary if there is only one parameter?","commit_id":"d2e56ed60ca2874773cfda9bd1f00be7fd5a0483"}],"nova/tests/compute/test_compute.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"f923885f8fd218bcf2d345248cdd521aa9745d81","unresolved":false,"context_lines":[{"line_number":5731,"context_line":""},{"line_number":5732,"context_line":"        db.instance_destroy(self.context, instance[\u0027uuid\u0027])"},{"line_number":5733,"context_line":""},{"line_number":5734,"context_line":"    def test_rebuild(self, vm_state\u003dvm_states.ACTIVE):"},{"line_number":5735,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"},{"line_number":5736,"context_line":"        instance_uuid \u003d instance[\u0027uuid\u0027]"},{"line_number":5737,"context_line":"        self.compute.run_instance(self.context, instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAPX%2F%2FvZ0%3D","line":5734,"updated":"2013-06-11 16:07:43.000000000","message":"I think it\u0027s confusing to have this be both an actual test method and a helper that\u0027s called by other test methods.  It would be more readable to make this _test_rebuild, remove the default vm_state, and then have another short test method to call it with ACTIVE.","commit_id":"a8df2e5498d96219f50c3425fb8784b8d3342e92"},{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"f923885f8fd218bcf2d345248cdd521aa9745d81","unresolved":false,"context_lines":[{"line_number":5950,"context_line":"        self.stubs.Set(nova.virt.fake.FakeDriver, \u0027legacy_nwinfo\u0027,"},{"line_number":5951,"context_line":"                       lambda x: False)"},{"line_number":5952,"context_line":""},{"line_number":5953,"context_line":"    def test_reboot_soft(self, vm_state\u003dvm_states.ACTIVE):"},{"line_number":5954,"context_line":"        # Ensure instance can be soft rebooted."},{"line_number":5955,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"},{"line_number":5956,"context_line":"        self.compute.run_instance(self.context, instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAPX%2F%2FvZk%3D","line":5953,"updated":"2013-06-11 16:07:43.000000000","message":"Same as above comment.","commit_id":"a8df2e5498d96219f50c3425fb8784b8d3342e92"}]}