From a6178c135fb02729685ea2fe8d633d858ae5b6ed Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Wed, 19 May 2021 08:49:09 +0300 Subject: [PATCH] Fix starting page server in non-daemonize mode. Commit 746f667311 moved the "chdir" earlier in the startup sequence, before daemonizing. But it forgot to remove a corresponding chdir call later in the sequence when not in daemonize mode. As a result, if you tried to start the pageserver without the --daemonize option, it always failed with "No such file or directory" error. --- pageserver/src/bin/pageserver.rs | 8 -------- 1 file changed, 8 deletions(-) diff --git a/pageserver/src/bin/pageserver.rs b/pageserver/src/bin/pageserver.rs index e24f169657..68c48fd3ae 100644 --- a/pageserver/src/bin/pageserver.rs +++ b/pageserver/src/bin/pageserver.rs @@ -195,14 +195,6 @@ fn start_pageserver(conf: &PageServerConf) -> Result<()> { Ok(_) => info!("Success, daemonized"), Err(e) => error!("Error, {}", e), } - } else { - // change into the repository directory. In daemon mode, Daemonize - // does this for us. - std::env::set_current_dir(&conf.workdir)?; - info!( - "Changed current directory to repository in {:?}", - &conf.workdir - ); } let mut threads = Vec::new();