move deserialize into remote finder

[DanCech]

This addresses the issue outlined in #2352 by moving the deserialize functions into the remote finder and using them in find_nodes.

While this code passes all tests I have not actually tried it yet and would appreciate feedback from anyone who has time to experiment with it.

[codecov-io]

Codecov Report

Merging #2354 into master will decrease coverage by 0.01%.
The diff coverage is 82.92%.

@@            Coverage Diff             @@
##           master    #2354      +/-   ##
==========================================
- Coverage   79.54%   79.53%   -0.02%     
==========================================
  Files          85       85              
  Lines        8899     8887      -12     
  Branches     1901     1899       -2     
==========================================
- Hits         7079     7068      -11     
+ Misses       1558     1557       -1     
  Partials      262      262

Impacted Files	Coverage Δ
webapp/graphite/readers/remote.py	100% <100%> (+8.98%)	⬆️
webapp/graphite/finders/remote.py	95.62% <82.5%> (-3.72%)	⬇️

---

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 17e2e1f...6c275a9. Read the comment docs.

[deniszh]

just FYI: trying to create simple docker compose to test cluster from 3 nodes

[DanCech]

The tags stuff also uses the finder request method, so I'm not going to try and shoehorn the de-serialization in there also at the moment.

[DanCech]

As far as I can tell this is working correctly, any objections to merging it @deniszh ?

[deniszh]

No objections. :)