Added extra checks into ThriftConverter #179
Workflow file for this run
This file contains 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
name: sc-client-server-deb10 | |
on: | |
pull_request: | |
types: [ opened, synchronize, reopened ] | |
branches: [ "**" ] | |
paths: | |
- '.github/workflows/sc-client-server-deb10.yml' | |
- 'dockerfiles/buster/Dockerfile.client' | |
- 'dockerfiles/buster/Dockerfile.server' | |
- 'dockerfiles/buster/Dockerfile.saithrift-server' | |
- 'npu/broadcom/BCM56850/saivs/Dockerfile.server' | |
- 'common/**' | |
- 'cli/**' | |
- 'scripts/**' | |
- 'configs/**' | |
- 'tests/**' | |
- 'setup.py' | |
- 'build.sh' | |
- 'run.sh' | |
- 'exec.sh' | |
- '.dockerignore' | |
- 'sai.env' | |
env: | |
DOCKER_CLIENT: 'dockerfiles/buster/Dockerfile.client' | |
DOCKER_SERVER_BASE: 'dockerfiles/buster/Dockerfile.server' | |
DOCKER_SERVER: 'npu/broadcom/BCM56850/saivs/Dockerfile.server' | |
DOCKER_THRIFT_SERVER: 'dockerfiles/buster/Dockerfile.saithrift-server' | |
REDIS_CLIENT: 0 | |
REDIS_SERVER: 0 | |
THRIFT_SERVER: 0 | |
jobs: | |
build-sc-server: | |
name: Build SAI Challenger server image | |
runs-on: ubuntu-20.04 | |
steps: | |
- uses: actions/checkout@v3 | |
with: | |
fetch-depth: 2 | |
- name: Update submodules | |
run: git submodule update --init | |
- name: Check what files were updated | |
id: check_changes | |
run: | | |
echo 'changed_files<<EOF' >> $GITHUB_OUTPUT | |
echo "$(git diff --name-only HEAD~1)" >> $GITHUB_OUTPUT | |
echo 'EOF' >> $GITHUB_OUTPUT | |
- name: Check what Docker images have to be rebuild | |
run: | | |
for file in "$DOCKER_SERVER_BASE" "$DOCKER_SERVER" "sai.env"; do | |
if [[ "${{ steps.check_changes.outputs.changed_files }}" == *"$file"* ]]; then | |
echo "REDIS_SERVER=1" >> $GITHUB_ENV | |
fi | |
done | |
for file in "$DOCKER_THRIFT_SERVER" "sai.env"; do | |
if [[ "${{ steps.check_changes.outputs.changed_files }}" == *"$file"* ]]; then | |
echo "THRIFT_SERVER=1" >> $GITHUB_ENV | |
fi | |
done | |
- name: Build server Docker image | |
run: ./build.sh -i server -o deb10 | |
if: ${{ env.REDIS_SERVER == '1' }} | |
- name: Pull SAI-C server | |
run: ./run.sh -i server -o deb10 | |
if: ${{ env.REDIS_SERVER == '0' }} | |
- name: Save server Docker image | |
run: docker save sc-server-trident2-saivs > sc-server.tar | |
- name: Upload server image | |
uses: actions/upload-artifact@v3 | |
with: | |
name: Server Image | |
path: sc-server.tar | |
- name: Build Thrift server Docker image | |
run: ./build.sh -i server -o deb10 -s thrift | |
if: ${{ env.THRIFT_SERVER == '1' }} | |
build-sc-client: | |
name: Build SAI Challenger client image | |
runs-on: ubuntu-20.04 | |
steps: | |
- uses: actions/checkout@v3 | |
with: | |
fetch-depth: 2 | |
- name: Update submodules | |
run: git submodule update --init | |
- name: Check what files were updated | |
id: check_changes | |
run: | | |
echo 'changed_files<<EOF' >> $GITHUB_OUTPUT | |
echo "$(git diff --name-only HEAD~1)" >> $GITHUB_OUTPUT | |
echo 'EOF' >> $GITHUB_OUTPUT | |
- name: Check what Docker images have to be rebuild | |
run: | | |
for file in "$DOCKER_CLIENT" "sai.env"; do | |
if [[ "${{ steps.check_changes.outputs.changed_files }}" == *"$file"* ]]; then | |
echo "REDIS_CLIENT=1" >> $GITHUB_ENV | |
fi | |
done | |
- name: Build client Docker image | |
run: ./build.sh -i client -o deb10 --nosnappi | |
if: ${{ env.REDIS_CLIENT == '1' }} | |
- name: Pull SAI-C client | |
run: ./run.sh -i client -o deb10 | |
if: ${{ env.REDIS_CLIENT == '0' }} | |
- name: Save client Docker image | |
run: docker save sc-client > sc-client.tar | |
- name: Upload client Docker image | |
uses: actions/upload-artifact@v3 | |
with: | |
name: Client Image | |
path: sc-client.tar | |
run-sc-tests: | |
name: Run SAI Challenger tests in client-server mode | |
needs: [build-sc-client, build-sc-server] | |
runs-on: ubuntu-20.04 | |
steps: | |
- uses: actions/checkout@v3 | |
- name: Update submodules. | |
run: git submodule update --init | |
- name: Download client Docker image | |
uses: actions/download-artifact@v3 | |
with: | |
name: Client Image | |
- name: Load client image | |
run: docker load < sc-client.tar | |
- name: Download server Docker image | |
uses: actions/download-artifact@v3 | |
with: | |
name: Server Image | |
- name: Load server image | |
run: docker load < sc-server.tar | |
- name: Delete client image artifact | |
uses: geekyeggo/delete-artifact@v2 | |
with: | |
name: Client Image | |
- name: Delete server image artifact | |
uses: geekyeggo/delete-artifact@v2 | |
with: | |
name: Server Image | |
- name: Start SAI-C client | |
run: ./run.sh -i client -o deb10 | |
- name: Wait for the client to get the IP address first | |
run: sleep 5s | |
- name: Start SAI-C server | |
run: ./run.sh -i server -o deb10 | |
- name: Update SAI-C server package | |
run: ./exec.sh -i server --no-tty pip3 install /sai-challenger/common /sai-challenger | |
if: ${{ env.REDIS_SERVER == '0' }} | |
- name: Update SAI-C client package | |
run: ./exec.sh -i client --no-tty pip3 install /sai-challenger/common /sai-challenger | |
if: ${{ env.REDIS_CLIENT == '0' }} | |
- name: Create veth links between client and server dockers | |
run: sudo ./veth-create-host.sh sc-server-trident2-saivs-run sc-client-run | |
- name: Run functional test cases | |
run: ./exec.sh --no-tty -i client pytest --testbed=saivs_client_server -v -k "test_l2_basic" | |
- name: Run unit tests | |
run: ./exec.sh --no-tty -i client pytest --testbed=saivs_client_server -v ut/test_acl_ut.py ut/test_bridge_ut.py ut/test_vrf_ut.py ut/test_port_ut.py ut/test_fdb_ut.py ut/test_lag_ut.py | |
- name: Run unit tests | |
run: ./exec.sh --no-tty -i client pytest --testbed=saivs_client_server -v -k \ | |
"test_switch_ut and not sai_map_list_t" | |
- name: Run thift data-driven tests | |
run: ./exec.sh --no-tty -i client pytest --testbed=saivs_client_server -v test_l2_basic_dd.py |