Remove vcf annotator healthcheck

[theferrit32]

Close #530

[larrybabb]

@jsstevenson would you please review this. the think-tank group decided the healthcheck() was not necessary so we removed it. If you have a counter arg, please make it, otherwise we will move forward.

[jsstevenson]

I'd be fine with removal for now. The bigger problem is that the VCF annotator suppresses all exceptions and just dumps them verbatim onto the VCF in an ERRORS info field. I think we ideally should be more specific about exception handling, such that some things get recognized as annotation-related errors (like coordinate wonkiness or something) and suppressed + logged, and other things should just interrupt annotation entirely (like a SeqRepoRestDataProxy HTTP error). This was a bandaid for the latter.

[theferrit32]

After discussion we decided that at least the filesystem and the REST based dataproxies should work the same way, in that they will fail at construction time if the underlying data source is not accessible. This latest commit adds a call to the /ping endpoint in the __init__ function of the REST dataproxy and throws an exception if it fails

[theferrit32]

I have no idea why the tests are failing because they run locally with I do pytest --vcr-record=none

[theferrit32]

I think the problem is the vcr decorator on test functions does not record any requests made by fixture initialization functions run prior to the test function itself. So when we construct a REST dataproxy in a test fixture in conftest.py, the ping request is not being recorded.