From: Dietmar Maurer <dietmar@proxmox.com>
Date: Thu, 17 Jan 2019 10:38:22 +0000 (+0100)
Subject: proxmox-backup-client: implement upload to server
X-Git-Tag: v0.1.3~1743
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=23bb8780d469daecae2369eac8a51519ea37ef4c;hp=597641fdd51ccdbd914e4de978ecc0a457bb49bd;p=proxmox-backup.git

proxmox-backup-client: implement upload to server
---

diff --git a/src/bin/proxmox-backup-client.rs b/src/bin/proxmox-backup-client.rs
index 63299052..a4b91e05 100644
--- a/src/bin/proxmox-backup-client.rs
+++ b/src/bin/proxmox-backup-client.rs
@@ -7,49 +7,38 @@ use proxmox_backup::tools;
 use proxmox_backup::cli::command::*;
 use proxmox_backup::api::schema::*;
 use proxmox_backup::api::router::*;
+use proxmox_backup::client::http_client::*;
+use proxmox_backup::client::catar_backup_stream::*;
 //use proxmox_backup::backup::chunk_store::*;
 //use proxmox_backup::backup::image_index::*;
 //use proxmox_backup::config::datastore;
-use proxmox_backup::catar::encoder::*;
+//use proxmox_backup::catar::encoder::*;
 use proxmox_backup::backup::datastore::*;
+
 use serde_json::{Value};
+use hyper::Body;
+
 
 fn required_string_param<'a>(param: &'a Value, name: &str) -> &'a str {
     param[name].as_str().expect(&format!("missing parameter '{}'", name))
 }
 
-fn backup_dir(
-    datastore: &DataStore,
-    path: &str,
-    dir: &mut nix::dir::Dir,
-    target: &str,
-    chunk_size: usize,
-) -> Result<(), Error> {
-
-    let mut target = std::path::PathBuf::from(target);
-
-    if let Some(ext) = target.extension() {
-        if ext != "aidx" {
-            bail!("got wrong file extension - expected '.aidx'");
-        }
-    } else {
-        target.set_extension("aidx");
-    }
 
-    let mut index = datastore.create_archive_writer(&target, chunk_size)?;
+fn backup_directory(body: Body, store: &str, archive_name: &str) -> Result<(), Error> {
 
-    let path = std::path::PathBuf::from(path);
+    let client = HttpClient::new("localhost");
 
-    CaTarEncoder::encode(path, dir, None, &mut index)?;
+    let path = format!("api3/json/admin/datastore/{}/upload_catar?archive_name={}", store, archive_name);
 
-    index.close()?; // commit changes
+    client.upload(body, &path)?;
 
     Ok(())
 }
 
+/****
 fn backup_image(datastore: &DataStore, file: &std::fs::File, size: usize, target: &str, chunk_size: usize) -> Result<(), Error> {
 
-    let mut target = std::path::PathBuf::from(target);
+    let mut target = PathBuf::from(target);
 
     if let Some(ext) = target.extension() {
         if ext != "iidx" {
@@ -70,6 +59,7 @@ fn backup_image(datastore: &DataStore, file: &std::fs::File, size: usize, target
 
     Ok(())
 }
+*/
 
 fn create_backup(param: Value, _info: &ApiMethod) -> Result<Value, Error> {
 
@@ -89,26 +79,29 @@ fn create_backup(param: Value, _info: &ApiMethod) -> Result<Value, Error> {
         }
     }
 
-    let datastore = DataStore::open(store)?;
-
-    let file = std::fs::File::open(filename)?;
-    let rawfd = file.as_raw_fd();
-    let stat = nix::sys::stat::fstat(rawfd)?;
+    let stat = match nix::sys::stat::stat(filename) {
+        Ok(s) => s,
+        Err(err) => bail!("unable to access '{}' - {}", filename, err),
+    };
 
     if (stat.st_mode & libc::S_IFDIR) != 0 {
         println!("Backup directory '{}' to '{}'", filename, store);
 
-        let mut dir = nix::dir::Dir::from_fd(rawfd)?;
+        let stream = CaTarBackupStream::open(filename)?;
+
+        let body = Body::wrap_stream(stream);
 
-        backup_dir(&datastore, &filename, &mut dir, &target, chunk_size)?;
+        backup_directory(body, store, target)?;
 
     } else if (stat.st_mode & (libc::S_IFREG|libc::S_IFBLK)) != 0 {
-        println!("Backup file '{}' to '{}'", filename, store);
+        println!("Backup image '{}' to '{}'", filename, store);
 
         if stat.st_size <= 0 { bail!("got strange file size '{}'", stat.st_size); }
         let size = stat.st_size as usize;
 
-        backup_image(&datastore, &file, size, &target, chunk_size)?;
+        panic!("implement me");
+
+        //backup_image(&datastore, &file, size, &target, chunk_size)?;
 
        // let idx = datastore.open_image_reader(target)?;
        // idx.print_info();
diff --git a/src/client/catar_backup_stream.rs b/src/client/catar_backup_stream.rs
index f61fad37..6f0932a5 100644
--- a/src/client/catar_backup_stream.rs
+++ b/src/client/catar_backup_stream.rs
@@ -2,12 +2,14 @@ use failure::*;
 
 use std::thread;
 use std::os::unix::io::FromRawFd;
+use std::path::PathBuf;
 
 use futures::{Async, Poll};
 use futures::stream::Stream;
 
 use nix::fcntl::OFlag;
 use nix::sys::stat::Mode;
+use nix::dir::Dir;
 
 use crate::catar::encoder::*;
 
@@ -20,22 +22,21 @@ pub struct CaTarBackupStream {
 impl Drop for CaTarBackupStream {
 
     fn drop(&mut self) {
+        println!("START DROP");
         drop(self.pipe.take());
         self.child.take().unwrap().join().unwrap();
+        println!("END DROP");
     }
 }
 
 impl CaTarBackupStream {
 
-    pub fn new(dirname: &str) -> Result<Self, Error> {
+    pub fn new(mut dir: Dir, path: PathBuf) -> Result<Self, Error> {
         let mut buffer = Vec::with_capacity(4096);
         unsafe { buffer.set_len(buffer.capacity()); }
 
         let (rx, tx) = nix::unistd::pipe()?;
 
-        let mut dir = nix::dir::Dir::open(dirname, OFlag::O_DIRECTORY, Mode::empty())?;
-        let path = std::path::PathBuf::from(dirname);
-
         let child = thread::spawn(move|| {
             let mut writer = unsafe { std::fs::File::from_raw_fd(tx) };
             if let Err(err) = CaTarEncoder::encode(path, &mut dir, None, &mut writer) {
@@ -47,6 +48,14 @@ impl CaTarBackupStream {
 
         Ok(Self { pipe: Some(pipe), buffer, child: Some(child) })
     }
+
+    pub fn open(dirname: &str) -> Result<Self, Error> {
+
+        let mut dir = nix::dir::Dir::open(dirname, OFlag::O_DIRECTORY, Mode::empty())?;
+        let path = std::path::PathBuf::from(dirname);
+
+        Self::new(dir, path)
+    }
 }
 
 impl Stream for CaTarBackupStream {
@@ -60,7 +69,7 @@ impl Stream for CaTarBackupStream {
 
         use std::io::Read;
 
-         loop {
+        loop {
             let pipe = match self.pipe {
                 Some(ref mut pipe) => pipe,
                 None => unreachable!(),