Cc pdlogs 61

Updated after review comments for the ENT feature.  
Modified the messages to capitalize acronyms.  
Removed unnecessary whitespace.  
Added full stop to end of each part of the message.  
Actions are numbered if there is more than one action.  This separates the individual actions.
Separated the log definitions with a blank line to make them more readable.  
Ran astyle (according to the coding standards) against sprout_ent_definitions.h.
Fixed header file inconsistencies.
The review comment had: "A lot of the logs still say 'report issue to support' or something similar. This just isn't suitable for Project Clearwater."
The language was kept the same as this is what is done for the SNMP alarm definitions.  The associated documentation should identify the Metaswitch support team for ClearwaterCore and the Clearwater mailing list for Clearwater.
Removed signalnames.cpp.  The signal call name is now obtained from the strsignal call.
Replaced hs_realm with hs_hostname.  Removed the zone.
David

From: Eleanor Merry [mailto:notifications@github.com]
Sent: Wednesday, December 17, 2014 10:05 AM
To: Metaswitch/sprout
Cc: David Collins
Subject: Re: [sprout] Cc pdlogs 61 (#895)


In include/sprout_ent_definitions.h<https://github.com/Metaswitch/sprout/pull/895#discussion-diff-21981369>:

> +  "Fatal - Must enable S-CSCF or I-CSCF in /etc/clearwater/config.",

> +  "Neither an S-CSCF nor an I-CSCF was configured in the /etc/clearwater/config.",

> +  "The application will exit until the problem is fixed.",

> +  "The S-CSCF is configured by setting the scscf=<port> option. "

> +  "The I-CSCF is configured by setting the icscf=<port> option."

> +);

> +

> +static const PDLog CL_SPROUT_SI_CSCF_NO_HOMESTEAD

> +(

> +  PDLogBase::CL_SPROUT_ID + 7,

> +  PDLOG_ERR,

> +  "Fatal - S/I-CSCF enabled with no Homestead server specified in /etc/clearwater/config.",

> +  "The S-CSCF and/or the I-CSCF options (scscf=<port>, icscf=<port>) were configured in "

> +  "the /etc/clearwater/config file but no Homestead was configured in the same file.",

> +  "The application will exit until the problem is fixed.",

> +  "Set the hs_realm=<hostname>.<zone> option in the /etc/clearwater/config file. "

It should be hs_hostname, not hs_realm, and it won't necessarily be hostname.zone

—
Reply to this email directly or view it on GitHub<https://github.com/Metaswitch/sprout/pull/895/files#r21981369>.