delete _remove roles from the run_list

When a `$barclamp-$role_remove` role (and its removal recipe) has successfully finished, delete it from the node's run_list.
Why do we need this loop here, instead of doing it inside `delete_remove_role_from_runlist`? This would avoid having to do the loop twice.