Skip to content
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

fix: updated the smoke test pipeline #1506

Closed
wants to merge 17 commits into from
34 changes: 20 additions & 14 deletions .github/actions/install-dependencies/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,40 +24,40 @@ inputs:
description: "Skip Go installation"
required: false
default: "false"

runs:
using: "composite"
steps:
- uses: actions/checkout@v3

- uses: actions/setup-go@v4
-
uses: actions/checkout@v3
-
uses: actions/setup-go@v4
if: ${{ inputs.skip_go == 'false' }}
with:
go-version: '1.20'
cache: false

- uses: actions/setup-python@v4
-
uses: actions/setup-python@v4
if: ${{ inputs.skip_python == 'false' }}
with:
python-version: 'pypy3.9'

- uses: chrisdickinson/setup-yq@latest

- id: install-aws-cli
-
uses: chrisdickinson/setup-yq@latest
-
id: install-aws-cli
if: ${{ inputs.skip_aws_cli == 'false' }}
uses: unfor19/install-aws-cli-action@v1
with:
version: 2
arch: ${{ inputs.cpu_architecture }}

- name: Dependencies.
-
name: Dependencies.
shell: bash
if: ${{ inputs.skip_docker_compose == 'false' }}
run: |
sudo apt-get update
sudo apt-get install wget jq -y

- name: Install Docker Compose
-
name: Install Docker Compose
if: ${{ inputs.skip_docker_compose == 'false' }}
shell: bash
run: |
Expand All @@ -74,3 +74,9 @@ runs:
$(lsb_release -cs) stable" | sudo tee /etc/apt/sources.list.d/docker.list > /dev/null
sudo apt-get update || echo "Issue running apt-get update."
sudo apt-get install docker-compose-plugin -y || echo "Issue installing docker compose"
-
name: Set up QEMU
uses: docker/setup-qemu-action@v3
-
name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
19 changes: 14 additions & 5 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ env:

jobs:
build-and-test:
runs-on: buildjet-4vcpu-ubuntu-2004
runs-on: ubuntu-latest
timeout-minutes: 15
concurrency:
group: "build-and-test"
Expand Down Expand Up @@ -78,8 +78,8 @@ jobs:
run: rm -rf *

smoke-test:
runs-on: ["zeta-runners"] # 25 minutes
timeout-minutes: 35
runs-on: buildjet-4vcpu-ubuntu-2004
timeout-minutes: 60
steps:
- uses: actions/checkout@v3

Expand Down Expand Up @@ -107,20 +107,29 @@ jobs:
- name: Login to Docker Hub
uses: docker/login-action@v2
if: github.event.repository.full_name == 'zetachain-chain/node'
env:
DOCKER_BUILDKIT: 1
with:
username: ${{ secrets.DOCKER_HUB_USERNAME }}
password: ${{ secrets.DOCKER_HUB_READ_ONLY }}

- name: Start Private Network
- name: Start Private Network.
env:
DOCKER_BUILDKIT: 1
run: |
make zetanode
cd contrib/localnet/
docker compose up -d zetacore0 zetacore1 zetaclient0 zetaclient1 eth bitcoin
echo "Sleep for 1 minute longer to let network start"
sleep 60
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't be necessary, the orchestrator already wait for the chain to start


- name: Run Smoke Test
env:
DOCKER_BUILDKIT: 1
run: |
cd contrib/localnet
docker-compose up orchestrator --exit-code-from orchestrator
docker-compose up --exit-code-from orchestrator orchestrator || echo "error"
docker ps
if [ $? -ne 0 ]; then
echo "Smoke Test Failed"
exit 1
Expand Down
1 change: 1 addition & 0 deletions changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
* add check to verify new tss has been produced when triggering tss funds migration
* fix Athens-3 log print issue - avoid posting uncessary outtx confirmation
* fix docker build issues with version: golang:1.20-alpine3.18
* fix smoke-test build issue with docker buildkit.

### Refactoring

Expand Down
Loading