Remove reliance on controller.ComputeHash() for determining changes to pod template spec

[jessesuen]

Resolves: #88

The implementation is complete, but I still need to fix and write unit tests for this.

[dthomson25]

I think we have the potential for a nil pointer exception.

@alexmt can you also take a look since this is a bigger change

[codecov-io]

Codecov Report

Merging #104 into master will increase coverage by 0.34%.
The diff coverage is 75.4%.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
+ Coverage   79.09%   79.43%   +0.34%     
==========================================
  Files          15       15              
  Lines        1808     1838      +30     
==========================================
+ Hits         1430     1460      +30     
- Misses        278      280       +2     
+ Partials      100       98       -2

Impacted Files	Coverage Δ
utils/conditions/conditions.go	89.4% <100%> (ø)	⬆️
controller/controller.go	52.6% <100%> (ø)	⬆️
controller/canary.go	85.71% <100%> (+3.3%)	⬆️
controller/pause.go	95.83% <50%> (ø)	⬆️
controller/bluegreen.go	80.79% <62.5%> (-1.13%)	⬇️
utils/replicaset/replicaset.go	88.41% <71.42%> (-5.85%)	⬇️
controller/sync.go	76.34% <88.88%> (+1.57%)	⬆️
controller/service.go	87.75% <0%> (+0.68%)	⬆️
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f35cb0...9e26654. Read the comment docs.

[dthomson25]

LGTM