Terminal status for goals

[MoffKalast]

It seems that occasionally when using move_basic through the action client it's possible to get the following result when preempting:

[ WARN] [1613663891.808006341, 985.273000000]: Your executeCallback did not set the goal to a terminal status.
This is a bug in your ActionServer implementation. Fix your code!
For now, the ActionServer will set this goal to aborted

Upon which move_basic seems to lock up sometimes. The solution would be to check in which case it's possible to end the callback without calling set_something for the goal before returning. Found in nav-features-aislemode branch, possibly a problem in others as well.

[MoffKalast]

After more research it seems to be a bug in the action client itself, so we'll have to watch to never get into situations where this can occur instead.

ros/actionlib#126

[dorkamotorka]

From my experience this is usually not a bug in the actionServer but rather our code that set a goal two time successful or aborting the same goal multiple times.

[MoffKalast]

Well in this case it may just be both 😑