From e02403bb4590da6823eb956bcbc89e9a937d431d Mon Sep 17 00:00:00 2001 From: Bert Belder Date: Tue, 11 May 2021 22:06:58 +0000 Subject: [PATCH] ci: run Linux release builds in sysroot with Ubuntu 18.04 and LLVM (#10920) --- .github/workflows/ci.yml | 310 ++++++++++++++++++++++++---------- Cargo.toml | 29 +++- cli/bench/deno_http_native.js | 6 +- cli/bench/main.rs | 8 +- cli/tools/installer.rs | 3 + 5 files changed, 264 insertions(+), 92 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index da4c7a1816..77f33b96a8 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -20,39 +20,40 @@ jobs: - os: windows-2019 kind: test profile: release + - os: ${{ github.repository == 'denoland/deno' && 'ubuntu-latest-xl' || 'ubuntu-latest' }} + kind: test + profile: release + use_sysroot: true - os: ${{ github.repository == 'denoland/deno' && 'ubuntu-latest-xl' || 'ubuntu-latest' }} kind: bench profile: release + use_sysroot: true + - os: ${{ github.repository == 'denoland/deno' && 'ubuntu-latest-xl' || 'ubuntu-latest' }} + kind: test + profile: debug - os: ${{ github.repository == 'denoland/deno' && 'ubuntu-latest-xl' || 'ubuntu-latest' }} kind: lint profile: debug - - os: ${{ github.repository == 'denoland/deno' && 'ubuntu-latest-xl' || 'ubuntu-latest' }} - kind: test - profile: debug - # Warning: Do not upgrade test_release to newer version of ubuntu - # runners. We need to link against older version of glibc in order to - # run on older systems. glibc in 20.04 is not compatible with 18.04. - # See #9484. - - os: ubuntu-18.04 - kind: test - profile: release # Always run main branch builds to completion. This allows the cache to # stay mostly up-to-date in situations where a single job fails due to # e.g. a flaky test. # Don't fast-fail on tag build because publishing binaries shouldn't be # prevented if if any of the stages fails (which can be a false negative). - fail-fast: ${{ github.event_name == 'pull_request' || (github.ref != - 'refs/heads/main' && !startsWith(github.ref, 'refs/tags/')) }} + fail-fast: ${{ github.event_name == 'pull_request' || + (github.ref != 'refs/heads/main' && + !startsWith(github.ref, 'refs/tags/')) }} env: + CARGO_TERM_COLOR: always RUST_BACKTRACE: full RUSTC_FORCE_INCREMENTAL: 1 - CARGO_TERM_COLOR: always steps: - name: Configure git - run: git config --global core.symlinks true + run: | + git config --global core.symlinks true + git config --global fetch.parallel 32 - name: Clone repository uses: actions/checkout@v2 @@ -72,9 +73,10 @@ jobs: startsWith(github.ref, 'refs/tags/') run: | mkdir -p target/release - tar --exclude=.cargo_home --exclude=".git*" --exclude=target --exclude=third_party/prebuilt -czvf target/release/deno_src.tar.gz -C .. deno + tar --exclude=".git*" --exclude=target --exclude=third_party/prebuilt \ + -czvf target/release/deno_src.tar.gz -C .. deno - - name: Install rust + - name: Install Rust uses: hecrj/setup-rust-action@v1 with: rust-version: 1.53.0 @@ -86,32 +88,20 @@ jobs: rustup component add rustfmt - name: Install Deno - if: | - !startsWith(matrix.os, 'windows') - run: |- + if: matrix.kind == 'lint' + run: | curl -fsSL https://deno.land/x/install/install.sh | sh -s v1.7.2 echo "$HOME/.deno/bin" >> $GITHUB_PATH - - name: Error on Warning - run: echo "RUSTFLAGS=-D warnings" >> $GITHUB_ENV - - - name: Install Deno (Windows) - if: startsWith(matrix.os, 'windows') - run: |- - curl -fsSL https://deno.land/x/install/install.sh | sh -s v1.7.2 - echo "$HOME/.deno/bin" >> $env:GITHUB_PATH - - name: Install Python uses: actions/setup-python@v1 with: - python-version: "3.8" - architecture: x64 + python-version: 3.8 - name: Install Node uses: actions/setup-node@v2 with: - node-version: "16" - check-latest: true + node-version: 16 - name: Remove unused versions of Python if: startsWith(matrix.os, 'windows') @@ -151,23 +141,92 @@ jobs: service_account_key: ${{ secrets.GCP_SA_KEY }} export_default_credentials: true - - name: Configure canary build - if: | - matrix.kind == 'test' && - matrix.profile == 'release' && - github.repository == 'denoland/deno' && - github.ref == 'refs/heads/main' - shell: bash + - name: Error on warning + # TODO(piscisaureus): enable this on Windows again. + if: "!matrix.use_sysroot && !startsWith(matrix.os, 'windows')" + run: echo "RUSTFLAGS=-D warnings" >> $GITHUB_ENV + + - name: Set up Linux sysroot with Ubuntu 18.04 and LLVM + if: matrix.use_sysroot run: | - echo "DENO_CANARY=true" >> $GITHUB_ENV + sudo apt-get update + sudo apt-get install debootstrap + + # Note: git, nc, strace, and time, are needed to run the benchmarks. + sudo debootstrap \ + --include=ca-certificates,curl,git,netcat-openbsd,strace,time \ + --no-merged-usr --variant=minbase bionic /sysroot \ + http://azure.archive.ubuntu.com/ubuntu + sudo mount --rbind /dev /sysroot/dev + sudo mount --rbind /sys /sysroot/sys + sudo mount --rbind /home /sysroot/home + sudo mount -t proc /proc /sysroot/proc + + sudo ln --force --target /sysroot/etc \ + /etc/passwd /etc/shadow /etc/group /etc/gshadow + + # Install clang-12 and lld-12 into the chroot environment. + echo "deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-12 main" | + sudo dd of=/sysroot/etc/apt/sources.list.d/llvm-toolchain-bionic-12.list + curl https://apt.llvm.org/llvm-snapshot.gpg.key | + gpg --dearmor | + sudo dd of=/sysroot/etc/apt/trusted.gpg.d/llvm-snapshot.gpg + sudo chroot /sysroot apt update -y + sudo chroot /sysroot apt install --no-install-recommends -y \ + clang-12 lld-12 + + # Make rust available inside the chroot environment. + sudo mkdir -p /sysroot/usr/share/rust + sudo mount --rbind /usr/share/rust /sysroot/usr/share/rust + + # Make node (needed to run the benchmarks) available. + sudo ln --target /sysroot/usr/bin "$(which node)" + + cat >> ~/.bash_profile << ___ + cd "$(pwd)" + + # Add cargo, rustc, and deno to $PATH. + source /usr/share/rust/.cargo/env + export PATH="$PATH:$(pwd)/target/release" + + # Rust build configuration. + export CARGO_PROFILE_BENCH_INCREMENTAL=false + export CARGO_PROFILE_BENCH_LTO=false + export CARGO_PROFILE_RELEASE_INCREMENTAL=false + export CARGO_PROFILE_RELEASE_LTO=false + export RUSTFLAGS=" + -C linker-plugin-lto=true + -C linker=clang-12 + -C link-arg=-fuse-ld=lld-12 + -C link-arg=-Wl,--thinlto-cache-dir=$(pwd)/target/release/lto-cache + -C link-arg=-Wl,--thinlto-cache-policy,cache_size_bytes=700m + -D warnings + " + export RUSTDOCFLAGS="\$RUSTFLAGS" + unset RUSTC_FORCE_INCREMENTAL + + # C build configuration. + export CC=clang-12 # Compile c source files with clang. + export CCFLAGs=-flto=thin # Tell clang to produce llvm bitcode. + + # Miscellaneous flags. + export CARGO_TERM_COLOR=always + export CI=true + export DENO_CANARY=true + ___ - name: Log versions + shell: bash run: | node -v python --version rustc --version cargo --version - deno --version + # Deno is installed when linting. + if [ "${{ matrix.kind }}" == "lint" ] + then + deno --version + fi - name: Cache Cargo home uses: actions/cache@v2 @@ -177,21 +236,24 @@ jobs: ~/.cargo/registry/index ~/.cargo/registry/cache ~/.cargo/git/db - key: a-cargo-home-${{ matrix.os }}-${{ hashFiles('Cargo.lock') }} + key: z-cargo-home-${{ matrix.os }}-${{ hashFiles('Cargo.lock') }} # In main branch, always creates fresh cache - name: Cache build output (main) # TODO(kt3k): Change the version to the released version # when https://github.com/actions/cache/pull/489 (or 571) is merged. uses: actions/cache@03e00da99d75a2204924908e1cca7902cafce66b - if: github.ref == 'refs/heads/main' + if: matrix.profile == 'release' && github.ref == 'refs/heads/main' with: path: | ./target + !./target/*/gn_out + !./target/*/*.zip + !./target/*/*.tar.gz key: | - a-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ hashFiles('Cargo.lock') }}-${{ github.sha }} + z-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ github.sha }} - # Restores cache from the latest main branch's Cache + # Restore cache from the latest 'main' branch build. - name: Cache build output (PR) # TODO(kt3k): Change the version to the released version # when https://github.com/actions/cache/pull/489 (or 571) is merged. @@ -200,22 +262,31 @@ jobs: with: path: | ./target - key: | - s0mth1ng_rand0m # Cache never be created for this key. + !./target/*/gn_out + !./target/*/*.zip + !./target/*/*.tar.gz + key: never_saved restore-keys: | - a-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ hashFiles('Cargo.lock') }}- + z-cargo-target-${{ matrix.os }}-${{ matrix.profile }}- - # Skips saving cache in PR branches + # Don't save cache after building PRs or branches other than 'main'. - name: Skip save cache (PR) run: echo "CACHE_SKIP_SAVE=true" >> $GITHUB_ENV shell: bash if: github.ref != 'refs/heads/main' - name: Apply and update mtime cache + if: matrix.profile == 'release' uses: ./.github/mtime_cache with: cache-path: ./target + # Shallow the cloning the crates.io index makes CI faster because it + # obviates the need for Cargo to clone the index. If we don't do this + # Cargo will `git clone` the github repository that contains the entire + # history of the crates.io index from github. We don't believe the + # identifier '1ecc6299db9ec823' will ever change, but if it does then this + # command must be updated. - name: Shallow clone crates.io index shell: bash run: | @@ -226,6 +297,16 @@ jobs: ~/.cargo/registry/index/github.com-1ecc6299db9ec823 fi + - name: Configure canary build + if: | + matrix.kind == 'test' && + matrix.profile == 'release' && + !matrix.use_sysroot && + github.repository == 'denoland/deno' && + github.ref == 'refs/heads/main' + shell: bash + run: echo "DENO_CANARY=true" >> $GITHUB_ENV + - name: test_format.js if: matrix.kind == 'lint' run: deno run --unstable --allow-write --allow-read --allow-run ./tools/format.js --check @@ -235,14 +316,27 @@ jobs: # TODO(ry) assert matrix.profile == "debug" run: deno run --unstable --allow-write --allow-read --allow-run ./tools/lint.js - - name: Build release - if: (matrix.kind == 'test' || matrix.kind == 'bench') && matrix.profile == 'release' - run: cargo build --release --locked --all-targets -vv - - name: Build debug - if: (matrix.kind == 'test' || matrix.kind == 'bench') && matrix.profile == 'debug' + if: | + (matrix.kind == 'test' || matrix.kind == 'bench') && + matrix.profile == 'debug' && !matrix.use_sysroot run: cargo build --locked --all-targets + - name: Build release + if: | + (matrix.kind == 'test' || matrix.kind == 'bench') && + matrix.profile == 'release' && !matrix.use_sysroot + run: cargo build --release --locked --all-targets + + - name: Build release (in sysroot) + if: | + (matrix.kind == 'test' || matrix.kind == 'bench') && + matrix.profile == 'release' && matrix.use_sysroot + run: | + sudo chroot /sysroot \ + su -l "$(whoami)" \ + -c "cargo build --release --locked --all-targets" + - name: Pre-release (linux) if: | startsWith(matrix.os, 'ubuntu') && @@ -297,28 +391,66 @@ jobs: echo $(git rev-parse HEAD) > canary-latest.txt gsutil cp canary-latest.txt gs://dl.deno.land/canary-latest.txt - - name: Test release - if: matrix.kind == 'test' && matrix.profile == 'release' - run: cargo test --release --locked --all-targets - - name: Test debug - if: matrix.kind == 'test' && matrix.profile == 'debug' + if: | + matrix.kind == 'test' && matrix.profile == 'debug' && + !matrix.use_sysroot run: | cargo test --locked --doc - cargo test --locked --all-targets + cargo test --locked - # TODO(ry) Because CI is so slow on for OSX and Windows, we currently only run WPT on Linux. - - name: Configure hosts file for WPT (linux) - if: startsWith(matrix.os, 'ubuntu') + - name: Test release + if: | + matrix.kind == 'test' && matrix.profile == 'release' && + !matrix.use_sysroot + run: cargo test --release --locked + + - name: Test release (in sysroot) + if: | + matrix.kind == 'test' && matrix.profile == 'release' && + matrix.use_sysroot + run: | + sudo chroot /sysroot \ + su -l "$(whoami)" \ + -c "cargo test --release --locked" + + # TODO(ry): Because CI is so slow on for OSX and Windows, we currently + # run the Web Platform tests only on Linux. + - name: Configure hosts file for WPT + if: startsWith(matrix.os, 'ubuntu') && matrix.kind == 'test' run: ./wpt make-hosts-file | sudo tee -a /etc/hosts working-directory: test_util/wpt/ - - name: Run web platform tests (release) - if: startsWith(matrix.os, 'ubuntu') && matrix.kind == 'test' && matrix.profile == 'release' + - name: Run web platform tests (debug) + if: | + startsWith(matrix.os, 'ubuntu') && matrix.kind == 'test' && + matrix.profile == 'debug' + env: + DENO_BIN: ./target/debug/deno run: | - deno run --unstable --allow-write --allow-read --allow-net --allow-env --allow-run ./tools/wpt.ts setup - deno run --unstable --allow-write --allow-read --allow-net --allow-env --allow-run ./tools/wpt.ts run --quiet --release --json=wpt.json --wptreport=wptreport.json - gzip ./wptreport.json + "$DENO_BIN" run --allow-env --allow-net --allow-read --allow-run \ + --allow-write --unstable \ + ./tools/wpt.ts setup + "$DENO_BIN" run --allow-env --allow-net --allow-read --allow-run \ + --allow-write --unstable \ + ./tools/wpt.ts run --quiet --binary="$DENO_BIN" + + - name: Run web platform tests (release) + if: | + startsWith(matrix.os, 'ubuntu') && matrix.kind == 'test' && + matrix.profile == 'release' + env: + DENO_BIN: ./target/release/deno + run: | + "$DENO_BIN" run --allow-env --allow-net --allow-read --allow-run \ + --allow-write --unstable \ + ./tools/wpt.ts setup + "$DENO_BIN" run --allow-env --allow-net --allow-read --allow-run \ + --allow-write --unstable \ + ./tools/wpt.ts run --quiet --release \ + --binary="$DENO_BIN" \ + --json=wpt.json \ + --wptreport=wptreport.json - name: Upload wpt results to dl.deno.land if: | @@ -328,6 +460,7 @@ jobs: github.repository == 'denoland/deno' && github.ref == 'refs/heads/main' run: | + gzip ./wptreport.json gsutil cp ./wpt.json gs://dl.deno.land/wpt/$(git rev-parse HEAD).json gsutil cp ./wptreport.json.gz gs://dl.deno.land/wpt/$(git rev-parse HEAD)-wptreport.json.gz echo $(git rev-parse HEAD) > wpt-latest.txt @@ -345,17 +478,19 @@ jobs: WPT_FYI_STAGING_PW: ${{ secrets.WPT_FYI_STAGING_PW }} GITHUB_TOKEN: ${{ secrets.DENOBOT_PAT }} run: | - deno run -A ./tools/upload_wptfyi.js $(git rev-parse HEAD) --ghstatus + ./target/release/deno run --allow-all \ + ./tools/upload_wptfyi.js $(git rev-parse HEAD) --ghstatus - - name: Run web platform tests (debug) - if: startsWith(matrix.os, 'ubuntu') && matrix.kind == 'test' && matrix.profile == 'debug' + - name: Run benchmarks + if: matrix.kind == 'bench' && !matrix.use_sysroot + run: cargo bench --locked + + - name: Run benchmarks (in sysroot) + if: matrix.kind == 'bench' && matrix.use_sysroot run: | - deno run --unstable --allow-write --allow-read --allow-net --allow-env --allow-run ./tools/wpt.ts setup - deno run --unstable --allow-write --allow-read --allow-net --allow-env --allow-run ./tools/wpt.ts run --quiet - - - name: Run Benchmarks - if: matrix.kind == 'bench' - run: cargo bench + sudo chroot /sysroot \ + su -l "$(whoami)" \ + -c "cargo bench --locked" - name: Post Benchmarks if: | @@ -365,8 +500,11 @@ jobs: env: DENOBOT_PAT: ${{ secrets.DENOBOT_PAT }} run: | - git clone --depth 1 -b gh-pages https://${DENOBOT_PAT}@github.com/denoland/benchmark_data.git gh-pages - deno run --unstable -A ./tools/build_benchmark_jsons.js --release + git clone --depth 1 --branch gh_pages \ + https://${DENOBOT_PAT}@github.com/denoland/benchmark_data.git \ + gh-pages + ./target/release/deno run --allow-all --unstable \ + ./tools/build_benchmark_jsons.js --release cd gh-pages git config user.email "propelml@gmail.com" git config user.name "denobot" @@ -374,6 +512,12 @@ jobs: git commit --message "Update benchmarks" git push origin gh-pages + - name: Build product size info + if: matrix.kind != 'lint' + run: | + du -hd1 "./target/${{ matrix.profile }}" + du -ha "./target/${{ matrix.profile }}/deno" + - name: Worker info if: matrix.kind == 'bench' run: | @@ -424,11 +568,3 @@ jobs: target/release/deno_src.tar.gz target/release/lib.deno.d.ts draft: true - - - name: Clean before cache - shell: bash - run: | - rm -f target/*/deno target/*/test_server - rm -rf target/*/examples/ - rm -rf target/*/gn_out/ - rm -rf target/*/*.zip diff --git a/Cargo.toml b/Cargo.toml index 26ea136477..178d9de9cf 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -22,19 +22,46 @@ members = [ ] exclude = ["test_util/std/hash/_wasm"] +# NB: the `bench` and `release` profiles must remain EXACTLY the same. [profile.release] codegen-units = 1 incremental = true lto = true opt-level = 'z' # Optimize for size +# NB: the `bench` and `release` profiles must remain EXACTLY the same. [profile.bench] codegen-units = 1 incremental = true lto = true opt-level = 'z' # Optimize for size -# Optimize these packages for perf +# Optimize these packages for performance. +# NB: the `bench` and `release` profiles must remain EXACTLY the same. +[profile.bench.package.rand] +opt-level = 3 +[profile.bench.package.flate2] +opt-level = 3 +[profile.bench.package.brotli] +opt-level = 3 +[profile.bench.package.miniz_oxide] +opt-level = 3 +[profile.bench.package.async-compression] +opt-level = 3 +[profile.bench.package.brotli-decompressor] +opt-level = 3 +[profile.bench.package.deno_core] +opt-level = 3 +[profile.bench.package.deno_runtime] +opt-level = 3 +[profile.bench.package.rusty_v8] +opt-level = 3 +[profile.bench.package.serde_v8] +opt-level = 3 +[profile.bench.package.serde] +opt-level = 3 + +# NB: the `bench` and `release` profiles must remain EXACTLY the same. [profile.release.package.rand] opt-level = 3 [profile.release.package.flate2] diff --git a/cli/bench/deno_http_native.js b/cli/bench/deno_http_native.js index 4c06bd7c19..2b576255ee 100644 --- a/cli/bench/deno_http_native.js +++ b/cli/bench/deno_http_native.js @@ -12,7 +12,11 @@ for await (const conn of listener) { (async () => { const requests = Deno.serveHttp(conn); for await (const { respondWith } of requests) { - respondWith(new Response(body)); + try { + respondWith(new Response(body)); + } catch { + // Ignore. + } } })(); } diff --git a/cli/bench/main.rs b/cli/bench/main.rs index d19731faa9..c7a6a714e1 100644 --- a/cli/bench/main.rs +++ b/cli/bench/main.rs @@ -340,10 +340,10 @@ fn run_strace_benchmarks( let mut thread_count = HashMap::::new(); let mut syscall_count = HashMap::::new(); - for (name, args, _) in EXEC_TIME_BENCHMARKS { + for (name, args, expected_exit_code) in EXEC_TIME_BENCHMARKS { let mut file = tempfile::NamedTempFile::new()?; - Command::new("strace") + let exit_status = Command::new("strace") .args(&[ "-c", "-f", @@ -352,9 +352,11 @@ fn run_strace_benchmarks( deno_exe.to_str().unwrap(), ]) .args(args.iter()) - .stdout(Stdio::inherit()) + .stdout(Stdio::null()) .spawn()? .wait()?; + let expected_exit_code = expected_exit_code.unwrap_or(0); + assert_eq!(exit_status.code(), Some(expected_exit_code)); let mut output = String::new(); file.as_file_mut().read_to_string(&mut output)?; diff --git a/cli/tools/installer.rs b/cli/tools/installer.rs index 02dbd71ce4..5877494d13 100644 --- a/cli/tools/installer.rs +++ b/cli/tools/installer.rs @@ -814,7 +814,10 @@ mod tests { } } + // This test is disabled because it uses the `deno` binary found in `$PATH`. + // It should use the one located in `./target/{debug|release}/`. #[test] + #[ignore] fn install_unicode() { let temp_dir = TempDir::new().expect("tempdir fail"); let bin_dir = temp_dir.path().join("bin");