replace logger with logrus for logging

[suhussai]

Description

Removes the logger package and related tests.

Related Issue

#615

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Example logs with logrus:

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-io]

Current coverage is 22.79% (diff: 10.76%)

Merging #623 into master will decrease coverage by 0.22%

@@             master       #623   diff @@
==========================================
  Files            26         25     -1   
  Lines          1972       2009    +37   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            454        458     +4   
- Misses         1463       1496    +33   
  Partials         55         55          

Powered by Codecov. Last update d8d8863...6b697d2

[palourde]

Thank you @suhussai for this PR, I really appreciate it!

The only problem I see so far with the logrus package is the fact that there's no way, out of the box, to identify the Remote Caller (see sirupsen/logrus#63) but it's an issue with the package itself, not your implementation.

Removing that capability would make debugging much more harder, so we'll have to think of a workaround before merging this change. I'll look around, let me know if you find something!