netavark, dns: don't double-fork aardvark instead wait for process

Netavark now does not double-forks aardvark-dns's server instead it waits for aardvark-process to return back and success return means aardvark-dns is ready to serve requests and now forking happens at aardvark end. This needs: containers/aardvark-dns#148 Signed-off-by: Aditya R <[email protected]>
flouthoc · Jun 15, 2022 · 7cd459f · 7cd459f
1 parent 1d56470
commit 7cd459f
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/dns/aardvark.rs b/src/dns/aardvark.rs
@@ -106,14 +106,17 @@ impl Aardvark {
             false => Stdio::null(),
         };
 
+        // This command will block till aardvark-dns's parent process
+        // returns back.
         Command::new(&aardvark_args[0])
             .args(&aardvark_args[1..])
             .stdin(Stdio::null())
             .stdout(Stdio::null())
             .stderr(output)
             // set RUST_LOG for aardvark
             .env("RUST_LOG", log::max_level().as_str())
-            .spawn()?;
+            .output()
+            .expect("failed to start aardvark-dns process");
 
         Ok(())
     }