Unverified Commit 199ee2a7 authored by Aaron Todd's avatar Aaron Todd Committed by GitHub
Browse files

fix sdk benchmarks and add checks to ci (#4275) 

## Description
<!--- Describe your changes in detail -->

Yesterday's release failed internally due to sdk-perf benchmark failing
to build. This PR updates the lockfiles and adds building the benchmarks
to CI to close this gap between internal release pipeline and CI.

----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._
parent f982b05e

.github/workflows/ci.yml

+2 −0
Original line number Diff line number Diff line
@@ -189,6 +189,8 @@ jobs: 
          runner: smithy_ubuntu-latest_8-core

        - action: check-aws-sdk-standalone-integration-tests

          runner: ubuntu-latest

        - action: check-aws-sdk-benchmarks

          runner: ubuntu-latest

    steps:

    - uses: GitHubSecurityLab/actions-permissions/monitor@v1

    - uses: actions/checkout@v4

aws/sdk/benchmarks/previous-release-comparison/Cargo.lock

+856 −812

File changed.

Preview size limit exceeded, changes collapsed.

aws/sdk/benchmarks/s3-express/Cargo.lock

+577 −477

File changed.

Preview size limit exceeded, changes collapsed.

aws/sdk/benchmarks/sdk-perf/Cargo.lock

+468 −225

File changed.

Preview size limit exceeded, changes collapsed.

tools/ci-scripts/check-aws-sdk-benchmarks

0 → 100755
+25 −0
Original line number Diff line number Diff line 
#!/bin/bash

#

# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.

# SPDX-License-Identifier: Apache-2.0

#



C_YELLOW='\033[1;33m'

C_RESET='\033[0m'



set -eux

cd smithy-rs



./gradlew :aws:sdk:assemble



# Build all benchmark directories

pushd aws/sdk/benchmarks

for dir in */; do

    if [ -f "$dir/Cargo.toml" ]; then

        echo -e "${C_YELLOW}# Building benchmark $dir...${C_RESET}"

        pushd "$dir"

        cargo build

        popd

    fi

done

popd