Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement async compression #22

Merged
merged 11 commits into from
Jun 9, 2024
Merged

Conversation

erikreinert
Copy link
Member

@erikreinert erikreinert commented Jun 9, 2024

Added

  • added async-compression dependencies in Cargo.toml

Removed

  • tar dependency in Cargo.toml
  • tempfile dependency in Cargo.toml

Changed

  • Updated flake.nix with new cargoSha256
  • Updated justfile with new build and start commands
  • Refactored vorpal.rs to use new dependencies
  • Refactored run_build.rs to use process-stream for streaming output
  • Refactored run_prepare.rs to use async file operations
  • Refactored proxy/mod.rs to handle different compression formats
  • Refactored store/mod.rs to use async-compression and tokio-tar

Fixed

  • Corrected key directory path in mod.rs
  • Improved error handling in run_build.rs and run_prepare.rs

erikreinert and others added 11 commits June 8, 2024 17:10
- Updated .gitignore to consolidate vorpal-related entries.
- Added new dependencies to Cargo.lock and Cargo.toml, including `futures`, `process-stream`, and `tap`.
- Modified example/rust/vorpal.rs to use `coreutils` instead of `busybox`.
- Updated flake.nix with a new `cargoSha256` value.
- Simplified justfile by removing redundant profiles and adding a `clean` task.
- Fixed key directory path in src/service/build/mod.rs.
- Replaced `tokio::process::Command` with `process_stream::Process` in src/service/build/run_build.rs for better process handling.
* feat: add tracing dependencies, update cargoSha256

- Add `tracing` and `tracing-subscriber` dependencies to `Cargo.toml`
and `Cargo.lock`
- Update `cargoSha256` in `flake.nix` to reflect new dependencies
- Implement tracing level configuration in `src/command/mod.rs`

* refactor: remove unused imports and replace println with logging

- Remove `tracing_subscriber::layer::SubscriberExt` import as it's unused.
- Replace `println!` with appropriate logging macros (`info`, `debug`, `error`) across various modules for better log management and consistency.
- Adjust log messages to follow a consistent format, enhancing readability and maintainability of log output.
Co-authored-by: Erik Reinert <4638629+erikreinert@users.noreply.github.com>
- Removed unused import `PathBuf` from `src/service/proxy/package/mod.rs`.
- Added `OsStr` and `info` imports in `src/store/mod.rs`.
- Refactored `compress_tar_gz` function to use generic parameters for better flexibility and type safety.
- Simplified the `append_path_with_name` and `append_dir_all` calls by removing unnecessary clones.
…tency

Replaced println! statements with info! macro to ensure consistent logging practices across the codebase. This change affects the logging of prepared gzip and bzip2 sources in the prepare function.
The cargoSha256 hash for the buildRustPackage has been updated from "sha256-I9yYNZEGJml2MyLK6BshNro7wFpn8MdZpUeGyMJs2o0=" to "sha256-VrAN1joJAs7puM76oyHIByNxovhDrp6t2Xw2FiG5tT4=". This change ensures the integrity and correctness of the package dependencies.
- Refactored error handling in `run_prepare.rs` to use `if let` for better readability and maintainability.
- Simplified the source kind preparation logic in `package/mod.rs` by replacing `match` with `if let` statements.
- Improved code clarity and reduced nesting by handling specific MIME types directly in conditional blocks.
@erikreinert erikreinert changed the title Implement sandbox dependencies Implement async compression Jun 9, 2024
@erikreinert erikreinert self-assigned this Jun 9, 2024
@erikreinert erikreinert merged commit f5c6acb into main Jun 9, 2024
2 checks passed
@erikreinert erikreinert deleted the 19-dependency-binaries-in-sandbox branch June 9, 2024 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants