Merge ring and ringDescriber

[sylwiaszunejko]

Previously we had two separate structs to store hosts data.

This PR merges both of them and additionally does some refactoring like removing unnecessary methods/structs.

[sylwiaszunejko]

@dkropachev This is my draft PR with change you suggested. I would also consider changing some names: e.g. GetHosts() to something like GetHostsFromSystem() to make it clear when we are querying database and when we are getting cached hosts info like in allHosts() and currentHosts() (btw these names are also pretty random, why not getHostsList/Map?). I don't know how much change can we do to be honest. And I am not sure now if this is a good idea to do such big refactoring. Do we plan on merging upstream regularly? Because it would be harder probably. Are we ok with that?

[sylwiaszunejko]

@dkropachev could you take a look, please?

[dkropachev]

@dkropachev This is my draft PR with change you suggested. I would also consider changing some names: e.g. GetHosts() to something like GetHostsFromSystem() to make it clear when we are querying database and when we are getting cached hosts info like in allHosts() and currentHosts() (btw these names are also pretty random, why not getHostsList/Map?). I don't know how much change can we do to be honest. And I am not sure now if this is a good idea to do such big refactoring. Do we plan on merging upstream regularly? Because it would be harder probably. Are we ok with that?

It looks great, changing names is not s problem.
If you want - do it.
We can try to merge it to upstream to test waters, they are pretty active these days.