OpenCue
OpenCue copied to clipboard
cuebot: Handle exceptions when moving hosts with running cores
Link the Issue(s) this Pull Request is related to. When you attempt to change allocation of a hoist with running cores, cuebot will throw an error that is not handled properly on cuegui side. This will render the application unusable until you relaunch it.
Summarize your change.
When we make a cuebot call, there is a second layer for exception that will take every possible exception. Just like with CueException
, an error message will appear with explanaition and the user can continue using the application after pressing OK. This will make sure it takes into account any unexpected error thrown by cuebot.
Do you have an example of the error you're trying to catch here? I'm wondering, if the error is on the cuebot side, why it's not coming back as
CueException
.
@bcipriano an example could be com.imageworks.spcue.EntityModificationError
. This error is thrown when you try to change allocation of a host with running cores. To replicate:
- Launch opencue
- Go to the "Manage Hosts" plugin
- Filter the hosts to "a" allocation
- Select a host that is running frames (the "Idle" is less than the "Cores")
- Right click the host and select "Change Allocation"
- Select any other allocation and click "Ok"
- Check the shell for the error
- The gui freezes and needs to be restarted to continue
Found an actual reason for the problem. setAllocation
API call was not encapsulated with @util.grpcExceptionParser
decorator. This means that error were not parsed correctly so they could not get caught correctly either.