github/kata-containers
1
0
Fork 1
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-05-02 13:44:33 +00:00
Code Issues Projects Releases Wiki Activity

rustjail: Add anyhow context for D-Bus connections

Browse Source 
In cases where the D-Bus connection fails, add a little additional context about
the origin of the error.

Fixes: 6561

Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
Suggested-by: Archana Shinde <archana.m.shinde@intel.com>
Spell-checked-by: Greg Kurz <gkurz@redhat.com>
This commit is contained in:
Christophe de Dinechin 2023-03-31 10:49:24 +02:00
parent 7796e6ccc6
commit b661e0cf3f
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/agent/rustjail/src/cgroups/systemd/dbus_client.rs
View File

@ -36,8 +36,9 @@ pub struct DBusClient {}
impl DBusClient { impl DBusClient {
fn build_proxy(&self) -> Result<SystemManager<'static>> { fn build_proxy(&self) -> Result<SystemManager<'static>> {
let connection = zbus::blocking::Connection::system()?; let connection =
let proxy = SystemManager::new(&connection)?; zbus::blocking::Connection::system().context("Establishing a D-Bus connection")?;
let proxy = SystemManager::new(&connection).context("Building a D-Bus proxy manager")?;
Ok(proxy) Ok(proxy)
} }
} }
@ -109,7 +110,9 @@ impl SystemdInterface for DBusClient {
} }
fn unit_exists(&self, unit_name: &str) -> Result<bool> { fn unit_exists(&self, unit_name: &str) -> Result<bool> {
let proxy = self.build_proxy()?; let proxy = self
.build_proxy()
.with_context(|| format!("Checking if systemd unit {} exists", unit_name))?;
Ok(proxy.get_unit(unit_name).is_ok()) Ok(proxy.get_unit(unit_name).is_ok())
} }