Skip to content

Commit

Permalink
Adding optional -device param to vmrt. Configurable self test. (#165)
Browse files Browse the repository at this point in the history
* Optional --pci-device param

* Support of env property YA_RUNTIME_VM_PCI_DEVICE for --pci-device param

* Self test --test-cpu-cores --test-mem-gib and --test-timeout cmdline params

* Reading self-test output dir from stdout

* Rust toolchain 1.70.0 and rustfmt config. MacOS CI bugfix.

* Logging self test stdout/err.

* Test default mem limit changed from 0.125 GiB to 0.5 GiB

---------

Co-authored-by: Norbert Mauger <[email protected]>
Co-authored-by: norbibi <[email protected]>
Co-authored-by: Przemyslaw Walski <[email protected]>
Co-authored-by: Kamil Koczurek <[email protected]>
Co-authored-by: nieznanysprawiciel <[email protected]>
  • Loading branch information
6 people authored Jul 20, 2023
1 parent 008489b commit 96c19ea
Show file tree
Hide file tree
Showing 12 changed files with 528 additions and 144 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ jobs:
env:
RUSTFLAGS: "-D warnings -C opt-level=z -C target-cpu=x86-64 -C debuginfo=1"
X86_64_PC_WINDOWS_MSVC_OPENSSL_DIR: c:/vcpkg/installed/x64-windows
rust_stable: 1.70.0
runs-on: ${{ matrix.os }}
strategy:
matrix:
Expand All @@ -26,10 +27,10 @@ jobs:
- name: Checkout
uses: actions/checkout@v1

- name: Install Last Stable Rust
- name: Install Rust ${{ env.rust_stable }}
uses: actions-rs/toolchain@v1
with:
toolchain: stable
toolchain: ${{ env.rust_stable }}

- name: Cache cargo registry
uses: actions/cache@v1
Expand Down
8 changes: 5 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,9 @@ on:
- pre-rel-*

env:
self-test-img_tag: pre-rel-v0.1.0
self-test-img_tag: v0.1.4
self-test-img_repository: golemfactory/ya-self-test-img
rust_stable: 1.70.0

jobs:
create-release:
Expand Down Expand Up @@ -112,9 +113,10 @@ jobs:
console.log(release.data.upload_url);
return release.data.upload_url
- uses: actions-rs/toolchain@v1
- name: Install Rust ${{ env.rust_stable }}
uses: actions-rs/toolchain@v1
with:
toolchain: stable
toolchain: ${{ env.rust_stable }}
target: x86_64-unknown-linux-musl
override: true
- name: Build
Expand Down
178 changes: 175 additions & 3 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Empty file modified qemu/Dockerfile
100644 → 100755
Empty file.
3 changes: 3 additions & 0 deletions runtime/Cargo.toml
100644 → 100755
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@ bollard-stubs = "1.40.2"
crc = "1.8"
futures = "0.3"
log = "0.4.8"
# "crossbeam-channel" and "macos_fsevent" are default features.
# Remove `macos_fsevent` if `macos` build will get dropped.
notify = { version = "6.0", features = ["crossbeam-channel", "macos_fsevent"] }
rand = "0.8"
raw-cpuid = "10.7"
serde = { version = "^1.0", features = ["derive"] }
Expand Down
1 change: 0 additions & 1 deletion runtime/src/deploy.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ impl Deployment {
input.take(json_len as u64).read_to_string(&mut buf).await?;
buf
};

if crc32::checksum_ieee(json.as_bytes()) != crc {
return Err(anyhow::anyhow!("Invalid ContainerConfig crc32 sum"));
}
Expand Down
Loading

0 comments on commit 96c19ea

Please sign in to comment.