Deployment subsystem

Never have an "else" that is after a return, and always make the error case the if:

```
if !allReplControllersProcessed {
  return ...
}
return deployapi.DeploymentStatusComplete
```