Fix consolidated praefect server info handler
In #1809 (closed), we created a ServerInfo
handler that consolidates the ServerInfo
results from a praefect's internal gitaly nodes.
However, we left out ServerVersion
and GitVersion
.
If we can assume (enforce, rather) that all praefect gitaly nodes must have the same git version and be on the same gitaly version, then we can do the following in the praefect ServerInfo
handler:
- loop through all praefect gitaly nodes and get the git version. If something doesn't match, return an error for the RPC call. If they all match, then return that git version
- do the same for server version