-
Notifications
You must be signed in to change notification settings - Fork 21
Add Arm machines support to benchamrks [MOD-8531] #600
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
Merged
Merged
Changes from all commits
Commits
Show all changes
95 commits
Select commit
Hold shift + click to select a range
1171d17
Add arm support
dor-forer 8102ad1
Changed the arm cpu info
dor-forer 0504e08
Add ip test
dor-forer ba931d0
Add to tests
dor-forer e0642c8
Added tests andbm
dor-forer 4b8c347
fix tests
dor-forer 3039eb8
Add github benchmakrs
dor-forer 9a67ee8
Check 1
dor-forer a9b87d4
only arm
dor-forer da3c880
change ami
dor-forer 1fdb6d5
Try ireland
dor-forer b4302e1
Try different image
dor-forer a83947a
try image
dor-forer a698070
back to old image
dor-forer 730d8ac
larger image
dor-forer 38371c5
Add option to change env
dor-forer 202a89d
back to default region
dor-forer 185703d
Created new image
dor-forer 90e885c
Try to add the x86 to check
dor-forer d61c358
Try different machine
dor-forer 4a88b1f
added include
dor-forer 3ceadaa
Try without opti on arm
dor-forer e89762c
Change to c6g
dor-forer ba1ea86
added matrix region
dor-forer 76b7132
change to west
dor-forer 55bb40f
try the i8
dor-forer 1b84ced
Try oregon
dor-forer 3f98c27
Change subnet id
dor-forer 66d96a1
Now subnet
dor-forer 0c5f16c
Change subnet
dor-forer b2af693
add subnet
dor-forer 20e596c
Try group id
dor-forer 0682472
Change to vpc id
dor-forer 9be3846
change subnet
dor-forer 125e30b
Change ami
dor-forer 6758753
Try without subnet
dor-forer 2a37fb3
add security group again
dor-forer 7d97821
Change the subnets
dor-forer 97e7249
Change to ids
dor-forer 4545554
Change sg
dor-forer 3a443d3
psubnet
dor-forer a472150
Try different
dor-forer bee1c27
different
dor-forer 4a891da
to a file
dor-forer 0341dd7
print
dor-forer f8f424a
p
dor-forer ee0458a
leave empty
dor-forer 26ff2cc
empty
dor-forer d3eaeeb
Try different account
dor-forer 55bc653
Run 2 arm machines
dor-forer 21de162
Move both to us-west-2
dor-forer 6f8e4d4
Try workflow
dor-forer eedc25c
Change name
dor-forer 578b88d
Changes
dor-forer 41e920f
Change the secrets
dor-forer 6218a9c
Add supprted arch
dor-forer 1533ba7
Add defaults
dor-forer a86d7ac
Support all
dor-forer 7652c9e
Change the jq
dor-forer c369125
Change machine to t4g
dor-forer 9d9a047
Change the name
dor-forer 14f8739
Change the machine
dor-forer 2f119ec
fix the stop
dor-forer 96d63af
only benchamrk
dor-forer 305aa0b
add the secrets
dor-forer 4e45109
region secret
dor-forer 1b4649a
benchmark region
dor-forer 797d1d6
Change timeout
dor-forer db9c63e
Added support for arch name in benchamrks
dor-forer 106fc5e
change th json
dor-forer a0d62fb
changed to v9.0
dor-forer b8075b1
Change the check
dor-forer 2007e33
add v9
dor-forer 606cea7
Check alt version of armv9
dor-forer 12bead0
added check
dor-forer 976c366
add arc_arch
dor-forer 8e23a2f
changed to CONCAT_WITH_UNDERSCORE_ARCH
dor-forer e81ce18
change the check
dor-forer f8f3d9e
Add full check
dor-forer f408017
fix the instruct
dor-forer 0af63d8
Added the cmake
dor-forer 38d563a
fix the support
dor-forer 87ac845
put it back to cmake
dor-forer 14bcd59
back
dor-forer b48d9c4
change the condition
dor-forer 47b9724
No armpl for now
dor-forer 1b35e30
cland format
dor-forer cafb30c
remove the opt
dor-forer bde60e4
Changed to one machine
dor-forer 421715c
Added BENCHMARK_ARCH
dor-forer 3c07da6
fix endif
dor-forer eabe27c
Remove secrets call
dor-forer 7beb70b
pr changes
dor-forer 69a2f24
suuport check for armv7
dor-forer fd6291e
Change or OR
dor-forer File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,112 @@ | ||
| on: | ||
| workflow_call: | ||
| inputs: | ||
| setup: | ||
| required: true | ||
| type: string | ||
| architecture: | ||
| required: true | ||
| type: string | ||
| instance-type: | ||
| required: true | ||
| type: string | ||
| ami-id: | ||
| required: true | ||
| type: string | ||
| github-runner-label: | ||
| required: true | ||
| type: string | ||
|
|
||
| jobs: | ||
| start-runner: | ||
| name: Start self-hosted EC2 runner | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| runner_label: ${{ steps.start-ec2-runner.outputs.label }} | ||
| ec2_instance_id: ${{ steps.start-ec2-runner.outputs.ec2-instance-id }} | ||
| steps: | ||
| - name: Configure AWS credentials | ||
| uses: aws-actions/configure-aws-credentials@v4 | ||
| with: | ||
| aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| aws-region: ${{ secrets.AWS_REGION_BENCHMARK }} | ||
| - name: Start EC2 runner | ||
| id: start-ec2-runner | ||
| uses: machulav/ec2-github-runner@v2 | ||
| with: | ||
| mode: start | ||
| github-token: ${{ secrets.GH_PERSONAL_ACCESS_TOKEN }} | ||
| ec2-image-id: ${{ inputs.ami-id }} | ||
| ec2-instance-type: ${{ inputs.instance-type }} | ||
| subnet-id: ${{ secrets.AWS_EC2_SUBNET_ID_BENCHMARK }} | ||
| security-group-id: ${{ secrets.AWS_EC2_SG_ID_BENCHMARK }} | ||
| label: ${{ inputs.github-runner-label }} | ||
|
|
||
| benchmark: | ||
| name: Run benchmarks on runner | ||
| needs: start-runner | ||
| runs-on: ${{ needs.start-runner.outputs.runner_label }} | ||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| ref: ${{ github.event.number && format('refs/pull/{0}/merge', github.event.number) || github.head_ref }} | ||
| - name: Print runner info | ||
| run: | | ||
| printf "Runner lscpu:\n$(lscpu)\n" | ||
| printf "Runner lsmem:\n$(lsmem)\n" | ||
| printf "Runner nproc:\n$(nproc)\n" | ||
| printf "Runner uname:\n$(uname -a)\n" | ||
| printf "Runner arch:\n$(arch)\n" | ||
| - name: Install benchmark dependencies | ||
| run: | | ||
| sudo .install/install_script.sh | ||
| sudo apt install python3-pip -y | ||
| pip3 install --upgrade pip PyYAML setuptools redisbench-admin | ||
| pip3 install -r requirements.txt | ||
| - name: Download pre-generated indices | ||
| timeout-minutes: 20 | ||
| run: ./tests/benchmark/bm_files.sh ${{ inputs.setup }} | ||
| - name: Run Benchmark | ||
| env: | ||
| ARCH: ${{ inputs.architecture }} | ||
| timeout-minutes: 300 | ||
| run: | | ||
| make benchmark BM_FILTER=${{ inputs.setup }} | ||
| - name: Collect results | ||
| run: | | ||
| ./tests/benchmark/benchmarks.sh ${{ inputs.setup }} | xargs -P 0 -I {} redisbench-admin export \ | ||
| --redistimeseries_host ${{ secrets.PERFORMANCE_RTS_HOST }} \ | ||
| --redistimeseries_port ${{ secrets.PERFORMANCE_RTS_PORT }} \ | ||
| --redistimeseries_user default \ | ||
| --redistimeseries_pass '${{ secrets.PERFORMANCE_RTS_AUTH }}' \ | ||
| --github_repo ${{ github.event.repository.name }} \ | ||
| --github_org ${{ github.repository_owner }} \ | ||
| --github_branch ${{ github.head_ref || github.ref_name }} \ | ||
| --github_actor ${{ github.triggering_actor }} \ | ||
| --results-format google.benchmark \ | ||
| --benchmark-result-file {}_results.json | ||
|
|
||
| stop-runner: | ||
| name: Stop self-hosted EC2 runner | ||
| needs: | ||
| - start-runner # required to get output from the start-runner job | ||
| - benchmark # required to wait when the main job is done | ||
| runs-on: ubuntu-latest | ||
| if: ${{ always() }} # required to stop the runner even if the error happened in the previous jobs | ||
| steps: | ||
| - name: Configure AWS credentials | ||
| uses: aws-actions/configure-aws-credentials@v4 | ||
| with: | ||
| aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| aws-region: ${{ secrets.AWS_REGION_BENCHMARK }} | ||
| - name: Stop EC2 runner | ||
| uses: machulav/ec2-github-runner@v2 | ||
| with: | ||
| mode: stop | ||
| github-token: ${{ secrets.GH_PERSONAL_ACCESS_TOKEN }} | ||
| label: ${{ inputs.github-runner-label }} | ||
| ec2-instance-id: ${{ needs.start-runner.outputs.ec2_instance_id }} | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| include(CheckCXXCompilerFlag) | ||
dor-forer marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
|
|
||
| message(STATUS "Building for ARM aarch64") | ||
|
|
||
| # Check what compiler flags are supported | ||
| CHECK_CXX_COMPILER_FLAG("-march=armv7-a+neon" CXX_ARMV7_NEON) | ||
| CHECK_CXX_COMPILER_FLAG("-march=armv8-a" CXX_ARMV8A) | ||
| CHECK_CXX_COMPILER_FLAG("-march=armv8-a+sve" CXX_SVE) | ||
| CHECK_CXX_COMPILER_FLAG("-march=armv9-a+sve2" CXX_ARMV9) | ||
|
|
||
| # Only use ARMv9 if both compiler and CPU support it | ||
| if(CXX_ARMV9) | ||
| message(STATUS "Using ARMv9.0-a with SVE2 (supported by CPU)") | ||
| add_compile_definitions(OPT_ARMV9) | ||
| endif() | ||
| if (CXX_ARMV8A OR CXX_ARMV7_NEON) | ||
| add_compile_definitions(OPT_NEON) | ||
| endif() | ||
| if (CXX_SVE) | ||
| add_compile_definitions(OPT_SVE) | ||
| endif() | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The job 'prepare_runner_configurations' does not declare outputs, yet its output 'matrix' is referenced in 'run_benchmarks'. Please add an outputs section in 'prepare_runner_configurations' to expose the 'matrix' from the 'set-matrix' step.