Skip to content

Commit

Permalink
Remove unused packages as part of build dependencies
Browse files Browse the repository at this point in the history
The reproducible runtime test failed due to running out of space. If we have
multiple tests failing due to out of space, and all of our tests have these
unused, it makes sense just to always so uninstall.

Also extends the time limit of reproducible-runtime, as 2h has been hit a few
times before.
  • Loading branch information
kayabaNerve committed Oct 11, 2023
1 parent 9bf8c92 commit 985795e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 19 deletions.
3 changes: 3 additions & 0 deletions .github/actions/build-dependencies/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ inputs:
runs:
using: "composite"
steps:
- name: Remove unused packages
run: sudo apt remove -y *powershell* *bazel* *nodejs* *npm* *yarn* *terraform* *firefox* *chromium* *qemu* *texinfo* *sqlite3* *imagemagick* && sudo apt autoremove -y

- name: Install Protobuf
uses: arduino/setup-protoc@a8b67ba40b37d35169e222f3bb352603327985b6
with:
Expand Down
21 changes: 3 additions & 18 deletions .github/workflows/full-stack-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,25 +15,10 @@ jobs:
steps:
- uses: actions/checkout@3df4ab11eba7bda6032a0b82a6bb43b11571feac

# - name: Install Build Dependencies
# uses: ./.github/actions/build-dependencies
# with:
# github-token: ${{ inputs.github-token }}

# Inlined build-dependencies action to minimize disk usage
- name: Install Protobuf
uses: arduino/setup-protoc@a8b67ba40b37d35169e222f3bb352603327985b6
with:
repo-token: ${{ inputs.github-token }}

- name: Install Rust
uses: dtolnay/rust-toolchain@5cb429dd810e16ff67df78472fa81cf760f4d1c0
- name: Install Build Dependencies
uses: ./.github/actions/build-dependencies
with:
toolchain: stable
targets: wasm32-unknown-unknown

- name: Remove unused packages
run: sudo apt remove -y *powershell* *bazel* *nodejs* *npm* *yarn* *terraform* *firefox* *chromium* *texinfo* *sqlite3* *imagemagick* && sudo apt autoremove -y
github-token: ${{ inputs.github-token }}

- name: Run Full Stack Docker tests
run: cd tests/full-stack && GITHUB_CI=true RUST_BACKTRACE=1 cargo test
2 changes: 1 addition & 1 deletion tests/reproducible-runtime/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ pub fn reproducibly_builds() {
use dockertest::{PullPolicy, Image, Composition, DockerTest};

const RUNS: usize = 3;
const TIMEOUT: u16 = 120 * 60; // 120 minutes
const TIMEOUT: u16 = 180 * 60; // 3 hours

serai_docker_tests::build("runtime".to_string());

Expand Down

0 comments on commit 985795e

Please sign in to comment.