m/cli/metroctl: clean up RPC API
This removes a bunch of logic from the metroctl core RPC functions,
forcing users (currently only other metroctl code) to use grpc.Dial and
the metropolis RPC library directly.
We also make the core functions take ConnectOptions structures where
appropriate instead of passing around tons of arguments.
Change-Id: I4d7aa232a659097da35027dfb9b87c58cbb4ab84
Reviewed-on: https://review.monogon.dev/c/monogon/+/2742
Reviewed-by: Lorenz Brun <lorenz@monogon.tech>
Tested-by: Jenkins CI
diff --git a/metropolis/cli/metroctl/main.go b/metropolis/cli/metroctl/main.go
index a4ea438..698dbbc 100644
--- a/metropolis/cli/metroctl/main.go
+++ b/metropolis/cli/metroctl/main.go
@@ -65,8 +65,9 @@
// currently set.
func connectOptions() *core.ConnectOptions {
return &core.ConnectOptions{
- ProxyServer: flags.proxyAddr,
- Endpoints: flags.clusterEndpoints,
- ConfigPath: flags.configPath,
+ ConfigPath: flags.configPath,
+ ProxyServer: flags.proxyAddr,
+ Endpoints: flags.clusterEndpoints,
+ ResolverLogger: rpcLogger,
}
}