Skip to content

Commit 26dae3a

Browse files
committed
chore: enable local_upgrade_test again
1 parent 799f998 commit 26dae3a

File tree

2 files changed

+49
-54
lines changed

2 files changed

+49
-54
lines changed

.github/workflows/ci.yml

+49-50
Original file line numberDiff line numberDiff line change
@@ -945,56 +945,55 @@ jobs:
945945
path: diagnostics*.gz
946946
retention-days: 1
947947

948-
# FIXME: This is disabled until we fix the problem with the test when creating a new topic
949-
# local_upgrade_test:
950-
# name: Upgrade local cluster test on (${{ matrix.run }})
951-
# needs: build_primary_binaries
952-
# runs-on: ${{ matrix.os }}
953-
# strategy:
954-
# # fail-fast: false
955-
# matrix:
956-
# os: [ubuntu-latest]
957-
# rust-target: [x86_64-unknown-linux-musl]
958-
# run: [r1]
959-
# steps:
960-
# - uses: actions/checkout@v4
961-
# # Download artifacts
962-
# - name: Download artifact - fluvio
963-
# uses: actions/download-artifact@v4
964-
# with:
965-
# name: fluvio-${{ matrix.rust-target }}
966-
# path: ~/bin
967-
# - name: Download artifact - fluvio-run
968-
# uses: actions/download-artifact@v4
969-
# with:
970-
# name: fluvio-run-${{ matrix.rust-target }}
971-
# path: ~/bin
972-
# - name: Mark executable
973-
# run: |
974-
# chmod +x ~/bin/fluvio-run
975-
# chmod +x ~/bin/fluvio && ~/bin/fluvio version
976-
# echo "${HOME}/bin" >> $GITHUB_PATH
977-
#
978-
# - name: Run upgrade test with CI artifacts
979-
# timeout-minutes: 10
980-
# env:
981-
# TEST_DATA_BYTES: 10000
982-
# run: |
983-
# date
984-
# make FLUVIO_MODE=local FLUVIO_BIN=~/bin/fluvio upgrade-test
985-
#
986-
# - name: Run diagnostics
987-
# if: ${{ !success() }}
988-
# timeout-minutes: 5
989-
# run: fluvio cluster diagnostics
990-
# - name: Upload logs
991-
# timeout-minutes: 5
992-
# if: ${{ !success() }}
993-
# uses: actions/upload-artifact@v4
994-
# with:
995-
# name: local_upgrade_${{ matrix.run }}_log
996-
# path: diagnostics*.gz
997-
# retention-days: 1
948+
local_upgrade_test:
949+
name: Upgrade local cluster test on (${{ matrix.run }})
950+
needs: build_primary_binaries
951+
runs-on: ${{ matrix.os }}
952+
strategy:
953+
# fail-fast: false
954+
matrix:
955+
os: [ubuntu-latest]
956+
rust-target: [x86_64-unknown-linux-musl]
957+
run: [r1]
958+
steps:
959+
- uses: actions/checkout@v4
960+
# Download artifacts
961+
- name: Download artifact - fluvio
962+
uses: actions/download-artifact@v4
963+
with:
964+
name: fluvio-${{ matrix.rust-target }}
965+
path: ~/bin
966+
- name: Download artifact - fluvio-run
967+
uses: actions/download-artifact@v4
968+
with:
969+
name: fluvio-run-${{ matrix.rust-target }}
970+
path: ~/bin
971+
- name: Mark executable
972+
run: |
973+
chmod +x ~/bin/fluvio-run
974+
chmod +x ~/bin/fluvio && ~/bin/fluvio version
975+
echo "${HOME}/bin" >> $GITHUB_PATH
976+
977+
- name: Run upgrade test with CI artifacts
978+
timeout-minutes: 10
979+
env:
980+
TEST_DATA_BYTES: 10000
981+
run: |
982+
date
983+
make FLUVIO_MODE=local FLUVIO_BIN=~/bin/fluvio upgrade-test
984+
985+
- name: Run diagnostics
986+
if: ${{ !success() }}
987+
timeout-minutes: 5
988+
run: fluvio cluster diagnostics
989+
- name: Upload logs
990+
timeout-minutes: 5
991+
if: ${{ !success() }}
992+
uses: actions/upload-artifact@v4
993+
with:
994+
name: local_upgrade_${{ matrix.run }}_log
995+
path: diagnostics*.gz
996+
retention-days: 1
998997

999998
# Smoke test across different version of fluvio
1000999
cli_smoke:

tests/upgrade-test.sh

-4
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,7 @@ function cleanup() {
5252
# If we're in CI, we want to slow down execution
5353
# to give CPU some time to rest, so we don't time out
5454
function ci_check() {
55-
if [[ "$FLUVIO_MODE" == "local" ]]; then
56-
sleep $CI_SLEEP
57-
else
5855
:
59-
fi
6056
}
6157

6258
# This function is intended to be run second after the Stable-1 validation

0 commit comments

Comments
 (0)