Add missing alarm options to the documentation

Partial-Bug: 1380533
Change-Id: I9b6242bc19e299951a62c15cf3a3c9f35857a8c4
Clarify deprecation comment to be accurate

The planned change in functionality was delayed until Juno,
the original comment was written in IceHouse. There needs to be
a period of two cycles for deprecation.

N.B: No functional change in this patch, just change of comment.

Change-Id: I3f452fbda59358fe85c1d7060a4abebd2f7c311a
Avoid clobbering existing class definition

Using the same variable name for a new class implies that an
existing class is being modified whereas what's happening is that a
subclass is being created of an existing class and having a
different value for an attribute. Such an implication is confusing
for maintainability.

Change-Id: If784949e8b5706e881bc702ff0b30848bca17d5c
refactor hbase storage code

create a common hbase base connection that alarm, event, and
metering dbs can inherit.

Change-Id: Iaefcb17373539885499a64563aa13b8124c53c8d
Merge "Add missing alarm options to the documentation"
Handle poorly formed individual sensor readings

ipmitool returns a series of multiple readings per run. Some of
these readings may be of a form that parser code cannot handle. The
desired behavior in this case is for the single reading to be
dropped and for parsing to continue. This change ensures that
behavior by continuing through the loop instead of allowing
exceptions to cause the loop to exit.

Tests have been added which verify the correct behavior.

Note that though there was an opportunity to log each skipped
reading this would lead to a very large number of log messages as
many or even most samples from ipmitool are not parsed.

Change-Id: I6a3193d5a6e12c69ca5c548e5fb58d8bc9646348
Closes-Bug: #1381600
Merge "Avoid clobbering existing class definition"
Merge "Clarify deprecation comment to be accurate"
Merge "refactor hbase storage code"
Merge "Handle poorly formed individual sensor readings"