Skip to content

Commit 61560f5

Browse files
committed
Fix formatting issues
Signed-off-by: Guvenc Gulce <guevenc.guelce@sap.com>
1 parent 347aa9e commit 61560f5

File tree

40 files changed

+6800
-252
lines changed

40 files changed

+6800
-252
lines changed

cli/src/host_commands.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@ use anyhow::{Context, Result};
22
use clap::{Args, Subcommand};
33
use digest::Digest;
44
use proto_definitions::host_service::{
5-
host_service_client::HostServiceClient, upgrade_request, Empty, UpgradeMetadata,
6-
UpgradeRequest,
5+
host_service_client::HostServiceClient, upgrade_request, Empty, UpgradeMetadata, UpgradeRequest,
76
};
87
use sha2::Sha256;
98
use std::path::PathBuf;
@@ -79,7 +78,9 @@ async fn upgrade_feos(client: &mut HostServiceClient<Channel>, binary_path: Path
7978
let request_stream = ReceiverStream::new(rx);
8079

8180
let upload_task = tokio::spawn(async move {
82-
let metadata = UpgradeMetadata { sha256_sum: checksum };
81+
let metadata = UpgradeMetadata {
82+
sha256_sum: checksum,
83+
};
8384
let metadata_req = UpgradeRequest {
8485
payload: Some(upgrade_request::Payload::Metadata(metadata)),
8586
};
@@ -127,4 +128,4 @@ async fn upgrade_feos(client: &mut HostServiceClient<Channel>, binary_path: Path
127128
upload_task.await?;
128129

129130
Ok(())
130-
}
131+
}

cli/src/image_commands.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,4 +138,4 @@ async fn delete_image(client: &mut ImageServiceClient<Channel>, image_uuid: Stri
138138
client.delete_image(request).await?;
139139
println!("Successfully deleted image: {image_uuid}");
140140
Ok(())
141-
}
141+
}

cli/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,4 +35,4 @@ async fn main() -> Result<()> {
3535
}
3636

3737
Ok(())
38-
}
38+
}

cli/src/vm_commands.rs

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@ use crossterm::tty::IsTty;
55
use prost::Message;
66
use proto_definitions::vm_service::{
77
stream_vm_console_request as console_input, vm_service_client::VmServiceClient,
8-
AttachConsoleMessage, AttachDiskRequest, ConsoleData, CreateVmRequest, DeleteVmRequest,
9-
DiskConfig, GetVmRequest, ListVmsRequest, MemoryConfig, PauseVmRequest, PingVmRequest,
10-
RemoveDiskRequest, ResumeVmRequest, ShutdownVmRequest, StartVmRequest, StreamVmConsoleRequest,
11-
StreamVmEventsRequest, VmConfig, VmState, VmStateChangedEvent, CpuConfig,
8+
AttachConsoleMessage, AttachDiskRequest, ConsoleData, CpuConfig, CreateVmRequest,
9+
DeleteVmRequest, DiskConfig, GetVmRequest, ListVmsRequest, MemoryConfig, PauseVmRequest,
10+
PingVmRequest, RemoveDiskRequest, ResumeVmRequest, ShutdownVmRequest, StartVmRequest,
11+
StreamVmConsoleRequest, StreamVmEventsRequest, VmConfig, VmState, VmStateChangedEvent,
1212
};
1313
use tokio::io::{AsyncReadExt, AsyncWriteExt};
1414
use tokio::sync::mpsc;
@@ -149,10 +149,7 @@ async fn create_vm(
149149
"Use 'feos-cli vm events {}' to watch its progress.",
150150
response.vm_id
151151
);
152-
println!(
153-
"Then 'feos-cli vm start {}' to start it.",
154-
response.vm_id
155-
);
152+
println!("Then 'feos-cli vm start {}' to start it.", response.vm_id);
156153

157154
Ok(())
158155
}
@@ -429,8 +426,6 @@ async fn remove_disk(
429426
device_id: device_id.clone(),
430427
};
431428
client.remove_disk(request).await?;
432-
println!(
433-
"Disk remove request sent for device {device_id} on VM {vm_id}"
434-
);
429+
println!("Disk remove request sent for device {device_id} on VM {vm_id}");
435430
Ok(())
436-
}
431+
}

feos/src/lib.rs

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,24 +11,25 @@ use vm_service::VM_CONSOLE_DIR;
1111

1212
use log::{error, info, warn};
1313
use network::configure_network_devices;
14+
use nix::libc;
1415
use nix::unistd::Uid;
1516
use proto_definitions::{
1617
host_service::host_service_server::HostServiceServer,
1718
image_service::image_service_server::ImageServiceServer,
1819
vm_service::vm_service_server::VmServiceServer,
1920
};
20-
use std::path::Path;
21-
use tokio::fs::{self, File};
2221
use std::env;
23-
use nix::libc;
2422
use std::ffi::CString;
2523
use std::os::unix::ffi::OsStringExt;
2624
use std::os::unix::fs::PermissionsExt;
25+
use std::path::Path;
26+
use tokio::fs::{self, File};
2727
use tokio::{net::UnixListener, sync::mpsc};
2828
use tokio_stream::wrappers::UnixListenerStream;
2929
use tonic::transport::Server;
3030
use vm_service::{
31-
api::VmApiHandler, dispatcher::VmServiceDispatcher, Command as VmCommand, VM_API_SOCKET_DIR, DEFAULT_VM_DB_URL
31+
api::VmApiHandler, dispatcher::VmServiceDispatcher, Command as VmCommand, DEFAULT_VM_DB_URL,
32+
VM_API_SOCKET_DIR,
3233
};
3334

3435
pub async fn run_server(restarted_after_upgrade: bool) -> Result<()> {
@@ -76,11 +77,17 @@ pub async fn run_server(restarted_after_upgrade: bool) -> Result<()> {
7677
if let Some(db_path_str) = db_url.strip_prefix("sqlite:") {
7778
let db_path = Path::new(db_path_str);
7879
if let Some(db_dir) = db_path.parent() {
79-
info!("MAIN: Ensuring database directory '{}' exists...", db_dir.display());
80+
info!(
81+
"MAIN: Ensuring database directory '{}' exists...",
82+
db_dir.display()
83+
);
8084
fs::create_dir_all(db_dir).await?;
8185
}
8286
if !db_path.exists() {
83-
info!("MAIN: Database file does not exist, creating at '{}'...", db_path.display());
87+
info!(
88+
"MAIN: Database file does not exist, creating at '{}'...",
89+
db_path.display()
90+
);
8491
File::create(db_path).await?;
8592
}
8693
}
@@ -226,4 +233,4 @@ pub async fn run_server(restarted_after_upgrade: bool) -> Result<()> {
226233
};
227234

228235
Ok(())
229-
}
236+
}

feos/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,4 +41,4 @@ async fn main() -> Result<()> {
4141
.init();
4242

4343
run_server(args.restarted_after_upgrade).await
44-
}
44+
}

feos/tests/integration_tests.rs

Lines changed: 10 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,19 @@ use image_service::{IMAGE_DIR, IMAGE_SERVICE_SOCKET};
33
use log::{error, info, warn};
44
use nix::sys::signal::{kill, Signal};
55
use nix::unistd::{self, Pid};
6+
use prost::Message;
67
use proto_definitions::{
78
host_service::{host_service_client::HostServiceClient, Empty},
89
image_service::{
910
image_service_client::ImageServiceClient, DeleteImageRequest, ImageState,
1011
ListImagesRequest, PullImageRequest, WatchImageStatusRequest,
1112
},
1213
vm_service::{
13-
vm_service_client::VmServiceClient, CreateVmRequest, DeleteVmRequest, GetVmRequest,
14-
PingVmRequest, StartVmRequest, StreamVmEventsRequest, CpuConfig, MemoryConfig, VmConfig,
14+
vm_service_client::VmServiceClient, CpuConfig, CreateVmRequest, DeleteVmRequest,
15+
GetVmRequest, MemoryConfig, PingVmRequest, StartVmRequest, StreamVmEventsRequest, VmConfig,
1516
VmEvent, VmState, VmStateChangedEvent,
1617
},
1718
};
18-
use prost::Message;
1919
use std::path::Path;
2020
use std::process::Command;
2121
use std::sync::Arc;
@@ -24,8 +24,8 @@ use tokio::net::UnixStream;
2424
use tokio::sync::OnceCell;
2525
use tokio::time::timeout;
2626
use tokio_stream::StreamExt;
27-
use tower::service_fn;
2827
use tonic::transport::{Channel, Endpoint, Uri};
28+
use tower::service_fn;
2929
use vm_service::{VM_API_SOCKET_DIR, VM_CH_BIN};
3030

3131
const PUBLIC_SERVER_ADDRESS: &str = "http://[::1]:1337";
@@ -73,7 +73,9 @@ async fn get_public_clients() -> Result<(VmServiceClient<Channel>, HostServiceCl
7373
async fn get_image_service_client() -> Result<ImageServiceClient<Channel>> {
7474
let endpoint = Endpoint::from_static("http://[::1]:50051");
7575
let channel = endpoint
76-
.connect_with_connector(service_fn(|_: Uri| UnixStream::connect(IMAGE_SERVICE_SOCKET)))
76+
.connect_with_connector(service_fn(|_: Uri| {
77+
UnixStream::connect(IMAGE_SERVICE_SOCKET)
78+
}))
7779
.await?;
7880
Ok(ImageServiceClient::new(channel))
7981
}
@@ -146,10 +148,7 @@ async fn wait_for_vm_state(
146148
}
147149

148150
if new_state == VmState::Crashed {
149-
let err_msg = format!(
150-
"VM entered Crashed state. Reason: {}",
151-
state_change.reason
152-
);
151+
let err_msg = format!("VM entered Crashed state. Reason: {}", state_change.reason);
153152
error!("{}", &err_msg);
154153
return Err(anyhow::anyhow!(err_msg));
155154
}
@@ -197,10 +196,7 @@ async fn test_create_and_start_vm() -> Result<()> {
197196
cleanup_disabled: false,
198197
};
199198

200-
info!(
201-
"Connecting to StreamVmEvents stream for vm_id: {}",
202-
&vm_id
203-
);
199+
info!("Connecting to StreamVmEvents stream for vm_id: {}", &vm_id);
204200
let events_req = StreamVmEventsRequest {
205201
vm_id: vm_id.clone(),
206202
..Default::default()
@@ -386,4 +382,4 @@ where
386382
}
387383
}
388384
anyhow::bail!("Stream ended before image became ready");
389-
}
385+
}

filesystem/src/fsmount.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,4 +45,4 @@ pub const FSCONFIG_CMD_CREATE: u32 = 6;
4545
pub fn fsmount(fd: BorrowedFd, flags: u32, mount_attr: u32) -> Result<RawFd> {
4646
let ret = unsafe { syscall(SYS_fsmount, fd.as_raw_fd(), flags, mount_attr) };
4747
Errno::result(ret.try_into().unwrap())
48-
}
48+
}

filesystem/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,4 @@ mod r#move; // Use r# to allow 'move' as a module name
44

55
// Re-export public functions
66
pub use mount::mount_virtual_filesystems;
7-
pub use r#move::{get_root_fstype, move_root};
7+
pub use r#move::{get_root_fstype, move_root};

filesystem/src/mount.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,4 +53,4 @@ pub fn mount_virtual_filesystems() {
5353
NONE,
5454
)
5555
.unwrap_or_else(|e| panic!("/sys/fs/cgroup mount failed: {e}"));
56-
}
56+
}

0 commit comments

Comments
 (0)