Comment 6 for bug 1423952

Revision history for this message
Matt Riedemann (mriedem) wrote :

I think a see a possible bug in the @reverts_task_state decorator in the compute manager code, it assumes there is an 'instance' key in kwargs to the method and it gets the uuid from that for the state update. However, if kwargs['instance'] raises a KeyError, we swallow it and continue on our merry way.

There are other decorators that use utilities to get all of the args into kwargs dict and then we can be sure we're getting the right thing. I'm going to play with some patches to see how often we hit this in a normal Jenkins run and were ignoring it.