Bed 4752

[stephanieslamb]

Description

Updated specified log errors to warns in order to cut down noise.

Motivation and Context

This PR addresses: BED-4752

Why is this change required? What problem does it solve?
This change was requested to lessen confusion post releases or while investigating certain issues.

How Has This Been Tested?

No real code changes. Unit tests still pass.

Screenshots (optional):

Types of changes

• Chore (a change that does not modify the application functionality)

Checklist:

• I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: Issues and Pull Requests README #672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed

[mvlipka]

LGTM
Couple of nits, but, it doesn't appear we really have as much of a standard when it comes to logging warnings.

Errors are generally prefixed with one of [Error, Failed, Unable], but we don't seem to have as much consensus when it comes to warn level messages.

[zinic]

One request for refactor, changeset otherwise looks solid