Skip to content

Commit 34a027a

Browse files
jarhodes314didier-wenzek
authored andcommitted
Rename file_async to file
1 parent 4749a5c commit 34a027a

File tree

23 files changed

+294
-1074
lines changed

23 files changed

+294
-1074
lines changed

.config/nextest.toml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,15 @@
22
retries = 0
33
test-threads = "num-cpus"
44

5+
[test-groups]
6+
serial = { max-threads = 1 }
7+
58
[[profile.default.overrides]]
69
# Remove once https://github.com/thin-edge/thin-edge.io/issues/3030 is resolved
710
filter = 'test(uploaded_file_can_be_downloaded_from_the_api)'
811
retries = 4
12+
13+
[[profile.default.overrides]]
14+
filter = 'package(c8y_firmware_manager)'
15+
test-group = 'serial'
16+

crates/common/download/src/download.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ use std::io::Write;
2424
use std::path::Path;
2525
use std::path::PathBuf;
2626
use std::time::Duration;
27-
use tedge_utils::file_async::FileError;
27+
use tedge_utils::file::FileError;
2828

2929
#[cfg(target_os = "linux")]
3030
use nix::fcntl::fallocate;

crates/common/download/src/error.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ pub enum DownloadError {
1010
FromIo { context: String, source: io::Error },
1111

1212
#[error("Error while performing a file operation")]
13-
FromFileError(#[from] tedge_utils::file_async::FileError),
13+
FromFileError(#[from] tedge_utils::file::FileError),
1414

1515
#[error("Not enough disk space")]
1616
InsufficientSpace,

crates/common/tedge_config/src/tedge_toml/tedge_config_location.rs

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,10 @@ use camino::Utf8Path;
1313
use camino::Utf8PathBuf;
1414
use serde::Serialize;
1515
use std::path::PathBuf;
16-
use tedge_utils::file::PermissionEntry;
16+
use tedge_utils::file::change_mode;
17+
use tedge_utils::file::change_mode_sync;
18+
use tedge_utils::file::change_user_and_group;
19+
use tedge_utils::file::change_user_and_group_sync;
1720
use tedge_utils::fs::atomically_write_file_async;
1821
use tedge_utils::fs::atomically_write_file_sync;
1922
use tracing::debug;
@@ -228,14 +231,14 @@ impl TEdgeConfigLocation {
228231

229232
atomically_write_file_async(toml_path, toml.as_bytes()).await?;
230233

231-
let entry = PermissionEntry {
232-
user: Some("tedge".to_string()),
233-
group: Some("tedge".to_string()),
234-
mode: Some(0o644),
235-
};
234+
if let Err(err) =
235+
change_user_and_group(toml_path.into(), "tedge".into(), "tedge".into()).await
236+
{
237+
warn!("failed to set file ownership for '{toml_path}': {err}");
238+
}
236239

237-
if let Err(err) = entry.apply(toml_path.as_std_path()) {
238-
warn!("failed to set file ownership and permissions for '{toml_path}': {err}");
240+
if let Err(err) = change_mode(toml_path.as_ref(), 0o644).await {
241+
warn!("failed to set file permissions for '{toml_path}': {err}");
239242
}
240243

241244
Ok(())
@@ -253,14 +256,12 @@ impl TEdgeConfigLocation {
253256

254257
atomically_write_file_sync(toml_path, toml.as_bytes())?;
255258

256-
let entry = PermissionEntry {
257-
user: Some("tedge".to_string()),
258-
group: Some("tedge".to_string()),
259-
mode: Some(0o644),
260-
};
259+
if let Err(err) = change_user_and_group_sync(toml_path.as_ref(), "tedge", "tedge") {
260+
warn!("failed to set file ownership for '{toml_path}': {err}");
261+
}
261262

262-
if let Err(err) = entry.apply(toml_path.as_std_path()) {
263-
warn!("failed to set file ownership and permissions for '{toml_path}': {err}");
263+
if let Err(err) = change_mode_sync(toml_path.as_ref(), 0o644) {
264+
warn!("failed to set file permissions for '{toml_path}': {err}");
264265
}
265266

266267
Ok(())

0 commit comments

Comments
 (0)