Make check add_* methods more flexible

## Overview

The ComplianceCheck `add_*` functions are pretty simplistic & free form - a key/label for the failure and things that failed/warned/succeeded in that way. This has been fine while the compliance tool was a "self contained thing", but as we integrate into other components it becomes a bit too simple.

It would be nice to allow failures/warnings/successes to be recorded with richer information, for example a CRN, the account or instance that had the problem etc. I think making the `add_*` functions take an optional kwargs which propagate to the internal data structures & then out into `check_results.json` would give us more fidelity in the data & more flexibility.

- `add_warnings`
- `add_failures`
- `add_successes`

## Requirements

TBD

## Approach

TBD

## Security and Privacy

N/A

## Test Plan

TBD