Closed mrbarge closed 2 years ago
Merging #646 (99afb43) into master (1ec35c0) will not change coverage. The diff coverage is
0.00%
.
@@ Coverage Diff @@
## master #646 +/- ##
=======================================
Coverage 31.00% 31.00%
=======================================
Files 46 46
Lines 4638 4638
=======================================
Hits 1438 1438
Misses 3105 3105
Partials 95 95
Impacted Files | Coverage Δ | |
---|---|---|
pkg/controller/account/account_controller.go | 13.48% <0.00%> (ø) |
Thanks for the fix @mrbarge !
@dkeohane can you open another PR to take care of the logging issue on line 1315?
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dkeohane, iamkirkbater, mrbarge
The full list of commands accepted by this bot can be found here.
The pull request process is described here
This PR addresses some minor logging nits observed where the parameters to the logging call were not being supplied in properly-balanced key/value pairs.
This fixes the following error being observed in the operator logs: