I see Chris's merge above made it into the charm, but with a default setting of "_sub", none of our expected default check names exist on a fresh deploy.
I've proposed this MP to remove that default suffix - those who need it can obviously still set it wherever needed. But I'd really like to know the reasoning behind setting this default behaviour in the first place, as I may be missing something important here.
I see Chris's merge above made it into the charm, but with a default setting of "_sub", none of our expected default check names exist on a fresh deploy.
I've proposed this MP to remove that default suffix - those who need it can obviously still set it wherever needed. But I'd really like to know the reasoning behind setting this default behaviour in the first place, as I may be missing something important here.
https:/ /code.launchpad .net/~barrypric e/nrpe- charm/+ git/nrpe- charm/+ merge/316532