Code review comment for ~xavpaice/charm-nrpe:nrpe-address

> For the moment this is handy, it will allow nagios servers to monitor, even if
> they are not part of the juju model

For a temporary fork this seems ok, but I tend to think this isn't the correct fix for the upstream charm. Wouldn't the correct fix here to be implement support for spaces so that relations work as expected rather than add a configuration to work without relations?

Is the root cause something that can't be fixed due to some limitation?