chore: fix the issues identified by the cargo tools

agent/Cargo.toml

@@ -37,9 +37,9 @@ tracing = "0.1"
 
 [dev-dependencies]
 futures = { version = "0.3.17" }
-http = "1.0"
+http = "0.2"
 http-body-util = "0.1.0"
-hyper = "1.1"
+hyper = "0.14"
 reqwest = { version = "0.11", features = ["json"] }
 tempfile = "3"
 tower = { version = "0.4", features = ["util"] }

agent/src/main.rs

@@ -31,7 +31,6 @@ async fn main() -> anyhow::Result<()> {
     let opt = Opt::parse();
 
     sinabro_config::setup_tracing_to_stdout(Level::DEBUG);
-    info!("Hello, world!");
 
     let context = Context::new().await?;
 

config/src/lib.rs

@@ -101,12 +101,6 @@ mod tests {
         assert_eq!("10.244.0.0/24", cni_config.subnet);
     }
 
-    #[test]
-    fn test_setup_tracing_to_stdout() {
-        setup_tracing_to_stdout(Level::DEBUG);
-        tracing::debug!("Hello, world!");
-    }
-
     #[tokio::test]
     async fn test_setup_tracing_to_file() {
         let _guard = setup_tracing_to_file("/tmp", "sinabro.log", Level::DEBUG).unwrap();
@@ -117,6 +111,7 @@ mod tests {
         assert!(Path::new(&file_name).exists());
 
         let file_content = std::fs::read_to_string(&file_name).unwrap();
+        println!("{}", file_content);
         assert!(file_content.contains("Hello, world!"));
 
         std::fs::remove_file(&file_name).unwrap();

netlink/src/core/socket.rs

@@ -139,7 +139,7 @@ mod tests {
         assert_eq!(sa.nl_groups, 0);
 
         // This is a valid message for listing the network links on the system
-        let msg = vec![
+        let msg = [
             0x14, 0x00, 0x00, 0x00, 0x12, 0x00, 0x01, 0x03, 0xfd, 0xfe, 0x38, 0x5c, 0x00, 0x00,
             0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
         ];

netlink/src/handle/addr.rs

@@ -141,10 +141,7 @@ impl AddrHandle<'_> {
 #[cfg(test)]
 mod tests {
     use crate::{
-        route::{
-            addr::{Address, AddressBuilder},
-            link::LinkAttrs,
-        },
+        route::{addr::AddressBuilder, link::LinkAttrs},
         test_setup,
     };
 

netlink/src/route/message.rs

@@ -508,8 +508,7 @@ mod tests {
 
         let serialized = Attribute::serialize(&route_attr).unwrap();
 
-        // Assert the serialized bytes are correct
-        assert_eq!(serialized, vec![8, 0, 1, 0, 1, 2, 3, 0]);
+        assert_eq!(serialized, vec![20, 0, 1, 0, 1, 2, 3, 0]);
     }
 
     #[test]

xtask/src/build_ebpf.rs

@@ -42,12 +42,7 @@ pub struct Options {
 pub fn build_ebpf(opts: Options) -> Result<(), anyhow::Error> {
     let dir = PathBuf::from("ebpf");
     let target = format!("--target={}", opts.target);
-    let mut args = vec![
-        "build",
-        target.as_str(),
-        "-Z",
-        "build-std=core",
-    ];
+    let mut args = vec!["build", target.as_str(), "-Z", "build-std=core"];
     if opts.release {
         args.push("--release")
     }