Skip to content

Commit b98c996

Browse files
fix: enable configuring log_dir from runtime config
Signed-off-by: Kate Goldenring <[email protected]>
1 parent 2a9bf7c commit b98c996

File tree

1 file changed

+13
-11
lines changed
  • crates/runtime-config/src

1 file changed

+13
-11
lines changed

crates/runtime-config/src/lib.rs

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -129,27 +129,29 @@ where
129129
let runtime_config_dir = runtime_config_path
130130
.and_then(Path::parent)
131131
.map(ToOwned::to_owned);
132+
let state_dir = toml_resolver.state_dir()?;
132133
let tls_resolver = runtime_config_dir.clone().map(SpinTlsRuntimeConfig::new);
133-
let key_value_config_resolver =
134-
key_value_config_resolver(runtime_config_dir, toml_resolver.state_dir()?);
135-
let sqlite_config_resolver = sqlite_config_resolver(toml_resolver.state_dir()?)
134+
let key_value_resolver = key_value_config_resolver(runtime_config_dir, state_dir.clone());
135+
let sqlite_resolver = sqlite_config_resolver(state_dir.clone())
136136
.context("failed to resolve sqlite runtime config")?;
137137

138+
let toml = toml_resolver.toml();
139+
let log_dir = toml_resolver.log_dir()?;
138140
let source = TomlRuntimeConfigSource::new(
139-
toml_resolver.clone(),
140-
&key_value_config_resolver,
141+
toml_resolver,
142+
&key_value_resolver,
141143
tls_resolver.as_ref(),
142-
&sqlite_config_resolver,
144+
&sqlite_resolver,
143145
);
144146
let runtime_config: T = source.try_into().map_err(Into::into)?;
145147

146148
Ok(Self {
147149
runtime_config,
148-
key_value_resolver: key_value_config_resolver,
149-
sqlite_resolver: sqlite_config_resolver,
150-
state_dir: toml_resolver.state_dir()?,
151-
log_dir: toml_resolver.log_dir()?,
152-
toml: toml_resolver.toml(),
150+
key_value_resolver,
151+
sqlite_resolver,
152+
state_dir,
153+
log_dir,
154+
toml,
153155
})
154156
}
155157

0 commit comments

Comments
 (0)