This is just a raw dict, it isn't validated against a config type -- we might want to do that, will push DagsterInstance closer to ConfigurableClass.
@alangenfeld any objections
only concerns around the complexity that future us will deal with when we figure out a better way to model what is currently the instance. This seems marginal - add a test that encodes the intended use case this supports
leave some comments at least