ModelControllerClient leaks memory

AbstractModelControllerClient.activeRequests is never cleared, as remove() is called with the wrong value.

This is a significant memory leak, as it means that it can hold onto complete deployments.