Issue 220: Fix for intermittent bookkeeper cluster upgrade failure

[anishakj]

Change log description

Fixing intermittent upgrade failure of bookkeeper.
Added some more conditions to handle error scenarios

Purpose of the change

Fixes #220

What the code does

while during bookkeeper update, it fails to get the pod sometimes. Issue is happening because while it tries to get an outdated pod, it has already updated. So added more checks to post error only if all pods were not updated

How to verify it

Peformed upgrade multiple time and issue is not seen

[codecov-commenter]

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (5257dd4) 78.84% compared to head (24be144) 78.59%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   78.84%   78.59%   -0.26%     
==========================================
  Files          11       11              
  Lines        1872     1878       +6     
==========================================
  Hits         1476     1476              
- Misses        307      313       +6     
  Partials       89       89              

Files	Coverage Δ
controllers/upgrade.go	68.94% <0.00%> (-1.95%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Amit-Singh40]

LGTM