[PATCH proxmox-backup v3 09/15] pbs-datastore: use general helpers for file lock create options

[Christian Ebner <c.ebner@proxmox.com>]

Instead of using local implementations, use the helpers provided by
the proxmox-product-config crate for consistency. By this the backup
user lookup is avoided as well, using the globally cached information
instead.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
---
 pbs-datastore/src/move_journal.rs  | 11 ++---------
 pbs-datastore/src/task_tracking.rs |  7 +------
 2 files changed, 3 insertions(+), 15 deletions(-)

diff --git a/pbs-datastore/src/move_journal.rs b/pbs-datastore/src/move_journal.rs
index a31001541..a678115d4 100644
--- a/pbs-datastore/src/move_journal.rs
+++ b/pbs-datastore/src/move_journal.rs
@@ -48,11 +48,8 @@ use std::path::{Path, PathBuf};
 use std::time::Duration;
 
 use anyhow::{Context, Error, bail, format_err};
-use nix::sys::stat::Mode;
 
-use proxmox_sys::fs::{CreateOptions, open_file_locked};
-
-use pbs_config::backup_user;
+use proxmox_sys::fs::open_file_locked;
 
 use crate::backup_info::DATASTORE_LOCKS_DIR;
 
@@ -78,11 +75,7 @@ fn ensure_parent(path: &Path) -> Result<(), Error> {
 
 fn open_locked_exclusive(path: &Path, timeout: Duration) -> Result<File, Error> {
     ensure_parent(path)?;
-    let user = backup_user()?;
-    let options = CreateOptions::new()
-        .perm(Mode::from_bits_truncate(0o660))
-        .owner(user.uid)
-        .group(user.gid);
+    let options = proxmox_product_config::lockfile_create_options();
     open_file_locked(path, timeout, true, options)
         .with_context(|| format!("failed to acquire exclusive move-journal lock at {path:?}"))
 }
diff --git a/pbs-datastore/src/task_tracking.rs b/pbs-datastore/src/task_tracking.rs
index d4cc076aa..3d7bf3e44 100644
--- a/pbs-datastore/src/task_tracking.rs
+++ b/pbs-datastore/src/task_tracking.rs
@@ -35,14 +35,9 @@ struct TaskOperations {
 }
 
 fn open_lock_file(name: &str) -> Result<(std::fs::File, CreateOptions), Error> {
-    let user = pbs_config::backup_user()?;
-
     let lock_path = PathBuf::from(format!("{}/{}.lock", crate::ACTIVE_OPERATIONS_DIR, name));
 
-    let options = CreateOptions::new()
-        .group(user.gid)
-        .owner(user.uid)
-        .perm(nix::sys::stat::Mode::from_bits_truncate(0o660));
+    let options = proxmox_product_config::lockfile_create_options();
 
     let timeout = std::time::Duration::new(10, 0);
 
-- 
2.47.3