Best not to synchronize on `this`. We had deadlocks in DynamicGroup [1]. I suspect this won't deadlock because we're only doing getMembers() rather than addMember(), but still best to synchronize on a private final object instead.

[1] https://github.com/brooklyncentral/brooklyn/pull/464