Kind of an RFC. Alex and I (and hopefully others?) have been using this flag locally to deprecate the CLI API and do everything over gRPC for some time and have fixed a number of issues that came up, it seems pretty stable at this point. Is it time to flip the switch and make it the default? We still have an opt-out switch if new surprising issues come up.
BK, launch runs in dagit and verify they are happening over gRPC
Would be nice to give ppl some hints about what to do next
do bools just default true? this is a little confusing to me
but this isn't the default anymore?
We weren't using the default value in the config here, changing it does nothing.
it is the default (managed grpc) - the other option is now deployed/static grpc server. Will rename the other option to make more clear
@alangenfeld re: telemetry, @rexledesma added telemetry for launch_scheduled_execution and is creating a doc to share (1) how to add telemetry to other functions and (2) how to query the collected data via sql. Are you thinking of adding telemetry for workspace_from_load_target, including the type of load_target -- should be similar to log_repo_stats unless we want to intercept the load_target within the telemetry_wrapper