snapshot-topologies was written a long time ago and I haven't been practicing it in a while. The idea was to have some "git history" for the topologies; I think the feature needs to be revisited and get some serious testing.

Well Shlomi, you set it up here. It's getting testing... but to be honest I'm not sure if the content it generates is being used. Were you wanting to get rid of it that would be fine I think.

Having said that the PR was because I'm moving all access to orchestrator to use the http interface so that I can switch the orchestrator clusters I talk to more easily (rather than use the config file). So moving to orchestrator-client means that any existing scripts etc collecting data which used the orchestrator binary need to be adapted. This was missing.

This patch has been running since I made the PR and seems to be working fine.