I haven't looked at the client code to see if it would be weird, but could it make sense to actually just use `serviceName` instead of introducing another duplicated value?

Or, if the intent is to potentially have serviceName and metricsSource be different, then I think this needs a

```
config.fromEnvironment('METRICS_SOURCE', env.serviceName);
```